tests: fixing tests for docker target

Related #407
MatrixAI · Jul 26, 2022 · efc9190 · efc9190
1 parent a3fb46d
commit efc9190
Show file tree

Hide file tree

Showing 10 changed files with 225 additions and 213 deletions.
diff --git a/scripts/docker-run.sh b/scripts/docker-run.sh
@@ -10,5 +10,6 @@ exec docker run -i \
     --env PK_NODE_PATH \
     --env PK_RECOVERY_CODE \
     --env PK_TOKEN \
+    --env PK_ROOT_KEY \
     "$PK_TEST_DOCKER_IMAGE" \
     polykey "$@"
diff --git a/tests/bin/agent/stop.test.ts b/tests/bin/agent/stop.test.ts
@@ -160,9 +160,11 @@ describe('stop', () => {
     },
     global.defaultTimeout * 2,
   );
-  runTestIfPlatforms('linux', 'docker')(
+  runTestIfPlatforms('linux')(
     'stopping starting agent results in error',
     async () => {
+      // This relies on fast execution of `agent stop` while agent is starting,
+      //  docker may not run this fast enough
       const password = 'abc123';
       const status = new Status({
         statusPath: path.join(dataDir, 'polykey', config.defaults.statusBase),

diff --git a/tests/bin/identities/allowDisallowPermissions.test.ts b/tests/bin/identities/allowDisallowPermissions.test.ts
@@ -198,9 +198,10 @@ describe('allow/disallow/permissions', () => {
       });
     },
   );
-  runTestIfPlatforms('linux', 'docker')(
+  runTestIfPlatforms('linux')(
     'allows/disallows/gets gestalt permissions by identity',
     async () => {
+      // Can't test with target executable due to mocking
       let exitCode, stdout;
       // Add the node to our node graph, otherwise we won't be able to contact it
       await testBinUtils.pkStdioSwitch(global.testCmd)(

diff --git a/tests/bin/identities/authenticateAuthenticated.test.ts b/tests/bin/identities/authenticateAuthenticated.test.ts
@@ -55,9 +55,10 @@ describe('authenticate/authenticated', () => {
       recursive: true,
     });
   });
-  runTestIfPlatforms('linux', 'docker')(
+  runTestIfPlatforms('linux')(
     'authenticates identity with a provider and gets authenticated identity',
     async () => {
+      // Can't test with target command due to mocking
       let exitCode, stdout;
       const mockedBrowser = jest
         .spyOn(identitiesUtils, 'browser')