fix: parallelize unit and system tests in circleci (CN-940) (#775)

* fix: add separate test:unit and test:system steps

* fix: moved tests to system

* fix: updated import paths

* fix: update resource class to medium to fix OOM

Author: ividalATSnyk

.circleci/config.yml

@@ -132,17 +132,30 @@ jobs:
       - attach_workspace:
           at: ~/snyk-docker-plugin
       - run: npm run lint
-  test:
+  test_unit:
+    <<: *defaults
+    resource_class: medium
+    steps:
+      - checkout
+      - attach_workspace:
+          at: ~/snyk-docker-plugin
+      - run: npm run test:unit > test-unit-logs.txt 2>&1
+      - store_artifacts:
+          path: test-unit-logs.txt
+          destination: test-unit-logs
+  test_system:
     <<: *defaults
     steps:
       - checkout
       - setup_remote_docker
       - attach_workspace:
           at: ~/snyk-docker-plugin
-      - run: npm run test-jest > test-logs.txt 2>&1
+      - run:
+          command: npm run test:system > test-system-logs.txt 2>&1
+          no_output_timeout: 20m
       - store_artifacts:
-          path: test-logs.txt
-          destination: test-logs
+          path: test-system-logs.txt
+          destination: test-system-logs
   test_jest_windows_with_docker:
     <<: *windows_big
     steps:
@@ -252,8 +265,17 @@ workflows:
           context: infrasec_container
           post-steps:
             - *slack-fail-notify
-      - test:
-          name: Test
+      - test_unit:
+          name: Unit Test
+          context:
+            - nodejs-install
+            - snyk-bot-slack
+          requires:
+            - Build
+          post-steps:
+            - *slack-fail-notify
+      - test_system:
+          name: System Test
           context:
             - nodejs-install
             - snyk-bot-slack
@@ -303,7 +325,8 @@ workflows:
             - Lint
             - Build
             - Security Scans
-            - Test
+            - Unit Test
+            - System Test
             - Test Jest Windows with Docker
             - Test Jest Windows no Docker
           post-steps:

test/lib/docker.spec.ts test/system/docker.spec.tstest/lib/docker.spec.ts renamed to test/system/docker.spec.ts

@@ -4,11 +4,11 @@ import * as tmp from "tmp";
 import { v4 as uuidv4 } from "uuid";
 
 import { DockerPullResult } from "@snyk/snyk-docker-pull";
-import * as plugin from "../../../lib";
-import * as imageInspector from "../../../lib/analyzer/image-inspector";
-import { ArchiveResult } from "../../../lib/analyzer/types";
-import { Docker } from "../../../lib/docker";
-import * as subProcess from "../../../lib/sub-process";
+import * as plugin from "../../lib";
+import * as imageInspector from "../../lib/analyzer/image-inspector";
+import { ArchiveResult } from "../../lib/analyzer/types";
+import { Docker } from "../../lib/docker";
+import * as subProcess from "../../lib/sub-process";
 
 function rmdirRecursive(customPath: string[]): void {
   if (customPath.length < 2) {
@@ -118,7 +118,7 @@ describe("getImageArchive", () => {
       const dockerPullSpy = jest.spyOn(Docker.prototype, "pull");
       const loadImage = path.join(
         __dirname,
-        "../../fixtures/docker-archives",
+        "../fixtures/docker-archives",
         "docker-save/hello-world.tar",
       );
       await subProcess.execute("docker", ["load", "--input", loadImage]);