Splitting permission test (#1757)

minio · Mar 24, 2022 · f582d83 · f582d83
1 parent b12fa5e
commit f582d83
Show file tree

Hide file tree

Showing 25 changed files with 172 additions and 6 deletions.
diff --git a/.github/workflows/jobs.yaml b/.github/workflows/jobs.yaml
@@ -669,8 +669,8 @@ jobs:
         with:
           args: '"chrome:headless" portal-ui/tests/operator/ --skip-js-errors -c 3'
 
-  permissions:
-    name: Permissions Tests
+  permissions-1:
+    name: Permissions Tests Part 1
     needs:
       - lint-job
       - no-warnings-and-make-assets
@@ -741,7 +741,163 @@ jobs:
       - name: Run TestCafe Tests
         uses: DevExpress/testcafe-action@latest
         with:
-          args: '"chrome:headless" portal-ui/tests/permissions/ --skip-js-errors -c 3'
+          args: '"chrome:headless" portal-ui/tests/permissions-1/ --skip-js-errors -c 3'
+
+      - name: Clean up users & policies
+        run: |
+          make cleanup-permissions
+
+  permissions-2:
+    name: Permissions Tests Part 2
+    needs:
+      - lint-job
+      - no-warnings-and-make-assets
+      - reuse-golang-dependencies
+      - vulnerable-dependencies-checks
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        go-version: [ 1.17.x ]
+        os: [ ubuntu-latest ]
+    steps:
+      - name: Set up Go ${{ matrix.go-version }} on ${{ matrix.os }}
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-version }}
+        id: go
+      - uses: actions/setup-node@v2
+        with:
+          node-version: '16'
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v2
+
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
+
+      - uses: actions/cache@v2
+        id: yarn-cache
+        name: Yarn Cache
+        with:
+          path: |
+            ${{ steps.yarn-cache-dir-path.outputs.dir }}
+            ./portal-ui/node_modules/
+            ./portal-ui/build/
+          key: ${{ runner.os }}-yarn-${{ hashFiles('./portal-ui/yarn.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-yarn-
+
+      - uses: actions/cache@v2
+        id: assets-cache
+        name: Assets Cache
+        with:
+          path: |
+            ./portal-ui/build/
+          key: ${{ runner.os }}-assets-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-assets-
+
+      - uses: actions/cache@v2
+        name: Go Mod Cache
+        with:
+          path: |
+            ~/.cache/go-build
+            ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ github.run_id }}
+
+      - name: Build Console on ${{ matrix.os }}
+        env:
+          GO111MODULE: on
+          GOOS: linux
+        run: |
+          make console
+
+      - name: Start Console, front-end app and initialize users/policies
+        run: |
+          (./console server) & (make initialize-permissions)
+
+      - name: Run TestCafe Tests
+        uses: DevExpress/testcafe-action@latest
+        with:
+          args: '"chrome:headless" portal-ui/tests/permissions-2/ --skip-js-errors -c 3'
+
+      - name: Clean up users & policies
+        run: |
+          make cleanup-permissions
+
+  permissions-3:
+    name: Permissions Tests Part 3
+    needs:
+      - lint-job
+      - no-warnings-and-make-assets
+      - reuse-golang-dependencies
+      - vulnerable-dependencies-checks
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        go-version: [ 1.17.x ]
+        os: [ ubuntu-latest ]
+    steps:
+      - name: Set up Go ${{ matrix.go-version }} on ${{ matrix.os }}
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-version }}
+        id: go
+      - uses: actions/setup-node@v2
+        with:
+          node-version: '16'
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v2
+
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
+
+      - uses: actions/cache@v2
+        id: yarn-cache
+        name: Yarn Cache
+        with:
+          path: |
+            ${{ steps.yarn-cache-dir-path.outputs.dir }}
+            ./portal-ui/node_modules/
+            ./portal-ui/build/
+          key: ${{ runner.os }}-yarn-${{ hashFiles('./portal-ui/yarn.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-yarn-
+
+      - uses: actions/cache@v2
+        id: assets-cache
+        name: Assets Cache
+        with:
+          path: |
+            ./portal-ui/build/
+          key: ${{ runner.os }}-assets-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-assets-
+
+      - uses: actions/cache@v2
+        name: Go Mod Cache
+        with:
+          path: |
+            ~/.cache/go-build
+            ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ github.run_id }}
+
+      - name: Build Console on ${{ matrix.os }}
+        env:
+          GO111MODULE: on
+          GOOS: linux
+        run: |
+          make console
+
+      - name: Start Console, front-end app and initialize users/policies
+        run: |
+          (./console server) & (make initialize-permissions)
+
+      - name: Run TestCafe Tests
+        uses: DevExpress/testcafe-action@latest
+        with:
+          args: '"chrome:headless" portal-ui/tests/permissions-3/ --skip-js-errors -c 3'
 
       - name: Clean up users & policies
         run: |

diff --git a/Makefile b/Makefile
@@ -89,9 +89,19 @@ test-operator:
 	@(env bash $(PWD)/portal-ui/tests/scripts/operator.sh)
 	@(docker stop minio)
 
-test-permissions:
+test-permissions-1:
 	@(docker run -v /data1 -v /data2 -v /data3 -v /data4 -d --name minio --rm -p 9000:9000 quay.io/minio/minio:latest server /data{1...4})
-	@(env bash $(PWD)/portal-ui/tests/scripts/permissions.sh)
+	@(env bash $(PWD)/portal-ui/tests/scripts/permissions.sh "portal-ui/tests/permissions-1/")
+	@(docker stop minio)
+
+test-permissions-2:
+	@(docker run -v /data1 -v /data2 -v /data3 -v /data4 -d --name minio --rm -p 9000:9000 quay.io/minio/minio:latest server /data{1...4})
+	@(env bash $(PWD)/portal-ui/tests/scripts/permissions.sh "portal-ui/tests/permissions-2/")
+	@(docker stop minio)
+
+test-permissions-3:
+	@(docker run -v /data1 -v /data2 -v /data3 -v /data4 -d --name minio --rm -p 9000:9000 quay.io/minio/minio:latest server /data{1...4})
+	@(env bash $(PWD)/portal-ui/tests/scripts/permissions.sh "portal-ui/tests/permissions-3/")
 	@(docker stop minio)
 
 test-apply-permissions:

diff --git a/...ests/permissions/bucketWritePrefixOnly.ts → ...ts/permissions-1/bucketWritePrefixOnly.ts b/...ests/permissions/bucketWritePrefixOnly.ts → ...ts/permissions-1/bucketWritePrefixOnly.ts
diff --git a/portal-ui/tests/permissions/groups.ts → portal-ui/tests/permissions-1/groups.ts b/portal-ui/tests/permissions/groups.ts → portal-ui/tests/permissions-1/groups.ts
diff --git a/portal-ui/tests/permissions/iamPolicies.ts → portal-ui/tests/permissions-1/iamPolicies.ts b/portal-ui/tests/permissions/iamPolicies.ts → portal-ui/tests/permissions-1/iamPolicies.ts
diff --git a/...ests/permissions/notificationEndpoints.ts → ...ts/permissions-1/notificationEndpoints.ts b/...ests/permissions/notificationEndpoints.ts → ...ts/permissions-1/notificationEndpoints.ts
diff --git a/...rmissions/test-fix-ui-crash-for-policy.ts → ...issions-1/test-fix-ui-crash-for-policy.ts b/...rmissions/test-fix-ui-crash-for-policy.ts → ...issions-1/test-fix-ui-crash-for-policy.ts
diff --git a/portal-ui/tests/permissions/trace.ts → portal-ui/tests/permissions-1/trace.ts b/portal-ui/tests/permissions/trace.ts → portal-ui/tests/permissions-1/trace.ts
diff --git a/portal-ui/tests/permissions/users.ts → portal-ui/tests/permissions-1/users.ts b/portal-ui/tests/permissions/users.ts → portal-ui/tests/permissions-1/users.ts
diff --git a/portal-ui/tests/permissions/bucketWrite.ts → portal-ui/tests/permissions-2/bucketWrite.ts b/portal-ui/tests/permissions/bucketWrite.ts → portal-ui/tests/permissions-2/bucketWrite.ts
diff --git a/portal-ui/tests/permissions/dashboard.ts → portal-ui/tests/permissions-2/dashboard.ts b/portal-ui/tests/permissions/dashboard.ts → portal-ui/tests/permissions-2/dashboard.ts
diff --git a/portal-ui/tests/permissions/diagnostics.ts → portal-ui/tests/permissions-2/diagnostics.ts b/portal-ui/tests/permissions/diagnostics.ts → portal-ui/tests/permissions-2/diagnostics.ts
diff --git a/portal-ui/tests/permissions/heal.ts → portal-ui/tests/permissions-2/heal.ts b/portal-ui/tests/permissions/heal.ts → portal-ui/tests/permissions-2/heal.ts
diff --git a/portal-ui/tests/permissions/inspect.ts → portal-ui/tests/permissions-2/inspect.ts b/portal-ui/tests/permissions/inspect.ts → portal-ui/tests/permissions-2/inspect.ts
diff --git a/portal-ui/tests/permissions/settings.ts → portal-ui/tests/permissions-2/settings.ts b/portal-ui/tests/permissions/settings.ts → portal-ui/tests/permissions-2/settings.ts
diff --git a/portal-ui/tests/permissions/tiers.ts → portal-ui/tests/permissions-2/tiers.ts b/portal-ui/tests/permissions/tiers.ts → portal-ui/tests/permissions-2/tiers.ts
diff --git a/portal-ui/tests/permissions/watch.ts → portal-ui/tests/permissions-2/watch.ts b/portal-ui/tests/permissions/watch.ts → portal-ui/tests/permissions-2/watch.ts
diff --git a/portal-ui/tests/permissions/admin.ts → portal-ui/tests/permissions-3/admin.ts b/portal-ui/tests/permissions/admin.ts → portal-ui/tests/permissions-3/admin.ts
diff --git a/...i/tests/permissions/bucketAssignPolicy.ts → ...tests/permissions-3/bucketAssignPolicy.ts b/...i/tests/permissions/bucketAssignPolicy.ts → ...tests/permissions-3/bucketAssignPolicy.ts
diff --git a/...ts/permissions/bucketDeleteAllVersions.ts → .../permissions-3/bucketDeleteAllVersions.ts b/...ts/permissions/bucketDeleteAllVersions.ts → .../permissions-3/bucketDeleteAllVersions.ts
diff --git a/...-ui/tests/permissions/bucketObjectTags.ts → ...i/tests/permissions-3/bucketObjectTags.ts b/...-ui/tests/permissions/bucketObjectTags.ts → ...i/tests/permissions-3/bucketObjectTags.ts
diff --git a/portal-ui/tests/permissions/bucketRead.ts → portal-ui/tests/permissions-3/bucketRead.ts b/portal-ui/tests/permissions/bucketRead.ts → portal-ui/tests/permissions-3/bucketRead.ts
diff --git a/...al-ui/tests/permissions/bucketSpecific.ts → ...-ui/tests/permissions-3/bucketSpecific.ts b/...al-ui/tests/permissions/bucketSpecific.ts → ...-ui/tests/permissions-3/bucketSpecific.ts
diff --git a/portal-ui/tests/permissions/logs.ts → portal-ui/tests/permissions-3/logs.ts b/portal-ui/tests/permissions/logs.ts → portal-ui/tests/permissions-3/logs.ts
diff --git a/portal-ui/tests/scripts/permissions.sh b/portal-ui/tests/scripts/permissions.sh
@@ -86,7 +86,7 @@ __init__() {
 }
 
 main() {
-  (yarn start &> /dev/null) & (./console server &> /dev/null) & (testcafe "chrome:headless" portal-ui/tests/permissions/ -q --skip-js-errors -c 3)
+  (yarn start &> /dev/null) & (./console server &> /dev/null) & (testcafe "chrome:headless" "$1" -q --skip-js-errors -c 3)
   cleanup
 }