Run tests separately (#448)

Author: sverdlov93

Diff for: .github/workflows/test.yml

@@ -1,26 +1,64 @@
-name: "Test"
+name: "Go Tests"
 
 on:
   push:
   # Triggers the workflow on labeled PRs only.
   pull_request_target:
     types: [labeled]
 
+# Ensures that only the latest commit is running for each PR at a time.
+# Ignores this rule for push events.
+concurrency:
+  group: ${{ github.event.pull_request.number || github.sha }}
+  cancel-in-progress: true
 jobs:
-  tests:
+  Pretest:
     if: contains(github.event.pull_request.labels.*.name, 'safe to test') || github.event_name == 'push'
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+    runs-on: ubuntu-latest
     steps:
       - name: Unlabel 'safe to test'
         uses: actions-ecosystem/action-remove-labels@v1
-        if: ${{github.event_name != 'push' }} && runner.os == 'Linux'
+        if: ${{ github.event_name != 'push' }}
+        with:
+          labels: "safe to test"
+
+      - name: Checkout code
+        uses: actions/checkout@v3
         with:
-          labels: 'safe to test'
+          ref: ${{ github.event.pull_request.head.sha }}
+
+      - name: Install Go
+        uses: actions/setup-go@v3
+        with:
+          go-version: 1.20.x
+
+      - name: Go Cache
+        uses: actions/cache@v3
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: ${{ runner.os }}-go-
+
+      # Generate mocks
+      - name: Generate mocks
+        run: go generate ./...
+        if: ${{ matrix.suite != 'unit' }}
+
+      - name: Lint
+        run: go vet -v ./...
 
+  tests:
+    needs: Pretest
+    name: ${{ matrix.suite }} Tests (${{ matrix.os }})
+    runs-on: ${{ matrix.os }}-latest
+    env:
+      JFROG_CLI_LOG_LEVEL: "DEBUG"
+    strategy:
+      fail-fast: false
+      matrix:
+        suite: [ unit, scanrepository, scanpullrequest, packagehandlers ]
+        os: [ ubuntu, windows, macos ]
+    steps:
       # Configure prerequisites
       - uses: actions/checkout@v3
         with:
@@ -29,10 +67,6 @@ jobs:
         uses: actions/setup-go@v3
         with:
           go-version: 1.20.x
-      - name: Install npm
-        uses: actions/setup-node@v3
-        with:
-          node-version: "16"
 
       - name: Go Cache
         uses: actions/cache@v3
@@ -42,6 +76,11 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-go-
 
+      - name: Install npm
+        uses: actions/setup-node@v3
+        with:
+          node-version: "16"
+
       - name: Setup Python3
         uses: actions/setup-python@v4
         with:
@@ -53,26 +92,21 @@ jobs:
       # Generate mocks
       - name: Generate mocks
         run: go generate ./...
+        if: ${{ matrix.suite != 'unit' }}
 
-      # Lint
-      - name: Lint
-        run: go vet ./...
-
-      # Run tests
-      - name: Tests
-        run: go test -p 1 ./... -v -race -timeout 30m -cover -coverprofile=covprofile -covermode=atomic
+      - name: unit Tests
+        run: go test github.com/jfrog/frogbot -v -race -timeout 30m -cover
         env:
           JF_URL: ${{ secrets.PLATFORM_URL }}
           JF_ACCESS_TOKEN: ${{ secrets.PLATFORM_ADMIN_TOKEN }}
-          JFROG_CLI_LOG_LEVEL: "DEBUG"
-      # Generate code coverage
-      - name: Send coverage
-        run: |
-          go install github.com/mattn/goveralls@latest
-          goveralls -coverprofile=covprofile -service=github
+        if: ${{ matrix.suite == 'unit' }}
+
+      - name: ${{ matrix.test }} Tests
+        run: go test github.com/jfrog/frogbot/${{ matrix.suite }} -v -race -timeout 30m -cover
         env:
-          COVERALLS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        if: runner.os == 'Linux'
+          JF_URL: ${{ secrets.PLATFORM_URL }}
+          JF_ACCESS_TOKEN: ${{ secrets.PLATFORM_ADMIN_TOKEN }}
+        if: ${{ matrix.suite != 'unit' }}
 
   # Build and run frogbot current changes for visual sanity check.
   sanity-pull-request: