Issue 6892: workerManager.registerWorker: treat test-provisioner-id s…

…pecially

changelog/issue-6892.md

@@ -0,0 +1,8 @@
+audience: developers
+level: minor
+reference: issue 6892
+---
+Worker Manager now special-cases `registerWorker` API calls with provisionerId
+`test-provisioner-id`. It responds as if the call was successful with fake
+`credentials`, `expiry`, and `secret`, but the real `workerConfig`, if it
+exists.

services/worker-manager/schemas/v1/worker-pool-full.yml

@@ -9,7 +9,9 @@ properties:
     type:           string
     pattern:        {$const: workerpoolid-pattern}
     description: |
-      The ID of this worker pool (of the form `providerId/workerType` for compatibility)
+      The ID of this worker pool (of the form `provisionerId/workerType` for
+      compatibility). Worker Pool IDs beginning `test-provisioner-id/` are
+      reserved for integration testing.
   providerId:
     title: Provider
     type:           string

services/worker-manager/src/api.js

@@ -784,6 +784,11 @@ builder.declare({
     'This call both marks the worker as running and returns the credentials',
     'the worker will require to perform its work.  The worker must provide',
     'some proof of its identity, and that proof varies by provider type.',
+    '',
+    'This method may also be called by integration tests by passing a `workerPoolId`',
+    'beginning with `test-provisioner-id/`. Such integration tests do not require',
+    'a valid `workerIdentityProof`, `workerId` or `workerGroup`. The response',
+    'will include fake `credentials`, `expires` and `secret`, but a valid `workerConfig`.',
   ].join('\n'),
 }, async function(req, res) {
   const { workerPoolId, providerId, workerGroup, workerId, workerIdentityProof } = req.body;
@@ -820,6 +825,19 @@ builder.declare({
       `Worker pool ${workerPoolId} not associated with provider ${providerId}`, {});
   }
 
+  if (workerPoolId.startsWith("test-provisioner-id/")) {
+    return res.reply({
+      expires: "2000-00-00T00:00:00Z",
+      credentials: {
+        accessToken: "test-access-token",
+        certificate: "test-certificate",
+        clientId: "test-client-id",
+	  },
+      workerConfig,
+      secret: "test-secret",
+    });
+  }
+
   const worker = await Worker.get(this.db, { workerPoolId, workerGroup, workerId });
   if (!worker) {
     this.monitor.debug({