Add unit test for updateApiConfig hook

Author: emteknetnz

README.md

@@ -364,7 +364,7 @@ fetch(
     .then(responseJson => console.log(responseJson));
 ```
 
-When working with non-public API endpoints you may wish to disable the csrf token check so that you can quickly test `GET` queries in your browsers location bar. You can do this by calling `SecurityToken::disable()` in `app/_config.php`, though if you do this be very careful this isn't then disabled in production too. To be safe, wrap this in a check to an environment variable which you then define if your personal .env file, for example:
+When working with non-public API endpoints you may wish to disable the csrf token check so that you can quickly test `GET` queries in your browsers location bar. You can do this by calling `SecurityToken::disable()` in `app/_config.php`, though if you do this be very careful this isn't then disabled in production too. To be safe, wrap this in a check to an environment variable of your choosing that you set in your local .env file, for example:
 
 ```php
 use SilverStripe\Core\Environment;
@@ -433,4 +433,4 @@ Notes:
 | `onBeforeAction(DataObject $obj, string $action)` | Called during `ACTION` requests before running the action |
 | `onAfterAction(DataObject $obj, string $action)` | Called during `ACTION` requests after running the action |
 | `onBeforeSendResponse(HTTPResponse $response)` | Called during all requests before sending HTTP response |
-| `updateApiConfig(array &$apiConfig)` | Called during all requests before any processing. Allows late updates to `private static array $api_config`, though modifying the `RestApiEndpoint::PATH` key has no effect at this stage |
+| `updateApiConfig(array &$apiConfig)` | Called during all requests before any processing. Allows late updates to `private static array $api_config`. Note that modifying the `RestApiEndpoint::PATH` key will have no effect at this stage |

tests/Controllers/RestApiEndpointTest.php

@@ -2299,6 +2299,7 @@ public function testExtensionHooks(): void
         $this->assertTrue(TestApiEndpoint::$hooksCalled['onDeleteAfterDelete']);
         // BeforeSendResponse
         $this->assertTrue(TestApiEndpoint::$hooksCalled['onBeforeSendResponse']);
+        $this->assertTrue(TestApiEndpoint::$hooksCalled['updateApiConfig']);
     }
 
     private function login(int $authLevel)

tests/Controllers/RestApiEndpointTest/TestApiEndpoint.php

@@ -121,6 +121,7 @@ class TestApiEndpoint extends RestApiEndpoint implements PermissionProvider, Tes
         'onBeforeAction' => false,
         'onAfterAction' => false,
         'onBeforeSendResponse' => false,
+        'updateApiConfig' => false,
     ];
 
     public function providePermissions()
@@ -191,4 +192,9 @@ protected function onBeforeSendResponse(HTTPResponse $response): void
     {
         self::$hooksCalled['onBeforeSendResponse'] = true;
     }
+
+    protected function updateApiConfig(array $apiConfig): void
+    {
+        self::$hooksCalled['updateApiConfig'] = true;
+    }
 }