Skip to content

Commit 0031301

Browse files
khvn26Zaimwa9
andauthored
fix(e2e): Missing RBAC tests (#5419)
Co-authored-by: Wadii Zaim <wadii.zaim@flagsmith.com>
1 parent 7f0035e commit 0031301

17 files changed

Lines changed: 73 additions & 212 deletions

.github/workflows/.reusable-docker-e2e-tests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,16 +12,16 @@ on:
1212
type: string
1313
description: Frontend Docker with E2E capabilities image to use, e.g., `ghcr.io/flagsmith/flagsmith-e2e:main`
1414
required: true
15-
tests:
15+
args:
1616
type: string
17-
description: Space-delimited list of E2E tests to be executed
17+
description: Additional arguments to testcafe
1818
required: false
1919
default: ''
2020
concurrency:
2121
type: number
2222
description: The concurrent number of browsers to be used on testing
2323
required: false
24-
default: 3
24+
default: 16
2525
runs-on:
2626
type: string
2727
description: The runner label to use. Defaults to `depot-ubuntu-latest`
@@ -77,7 +77,7 @@ jobs:
7777
retry_on: error
7878
timeout_minutes: 20
7979
env:
80-
opts: ${{ inputs.tests }}
80+
opts: ${{ inputs.args }}
8181
API_IMAGE: ${{ inputs.api-image }}
8282
E2E_IMAGE: ${{ inputs.e2e-image }}
8383
E2E_CONCURRENCY: ${{ inputs.concurrency }}

.github/workflows/manual-e2e-tests.yml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,15 @@
1-
name: 'Manual E2E tests'
1+
name: Run E2E tests against a deployed API instance
22

33
on:
44
workflow_dispatch:
55
inputs:
66
e2e-token:
7-
description: 'The authentication token used by the E2E process'
7+
description: The authentication token used by the E2E process
88
required: true
9-
e2e-concurrency:
10-
description: 'The concurrency value to use when running the E2E process'
11-
default: 3
12-
type: number
139
api-url:
14-
description: 'Which database service to use to run the API against'
10+
description: An API URL to run the E2E tests against
1511
default: 'https://api.flagsmith.com/api/v1/'
16-
12+
1713
jobs:
1814
run-e2e-tests:
1915
runs-on: depot-ubuntu-latest
@@ -31,7 +27,6 @@ jobs:
3127
env:
3228
E2E_TEST_AUTH_TOKEN: ${{ inputs.e2e-token }}
3329
FLAGSMITH_API_URL: ${{ inputs.api-url }}
34-
E2E_CONCURRENCY: ${{ inputs.e2e-concurrency }}
3530
run: |
3631
npm ci
3732
npm run env

.github/workflows/platform-docker-build-test-publish.yml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,18 +72,20 @@ jobs:
7272
with:
7373
runs-on: ${{ matrix.runs-on }}
7474
e2e-image: ${{ needs.docker-build-e2e.outputs.image }}
75-
api-image: ${{ matrix.api-image }}
76-
concurrency: 16
75+
api-image: ${{ matrix.args.api-image }}
76+
args: ${{ matrix.args.args }}
7777
secrets:
7878
GCR_TOKEN: ${{ secrets.GITHUB_TOKEN }}
7979
SLACK_TOKEN: ${{ secrets.SLACK_TOKEN }}
8080

8181
strategy:
8282
matrix:
8383
runs-on: [depot-ubuntu-latest-16, depot-ubuntu-latest-arm-16]
84-
api-image:
85-
- ${{ needs.docker-build-api.outputs.image }}
86-
- ${{ needs.docker-build-private-cloud-api.outputs.image }}
84+
args:
85+
- api-image: ${{ needs.docker-build-api.outputs.image }}
86+
args: --meta-filter category=oss
87+
- api-image: ${{ needs.docker-build-private-cloud-api.outputs.image }}
88+
args: --meta-filter category=oss,category=enterprise
8789

8890
# Publish to dockerhub
8991

.github/workflows/platform-pull-request.yml

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -137,17 +137,7 @@ jobs:
137137
runs-on: ${{ matrix.runs-on }}
138138
e2e-image: ${{ needs.docker-build-e2e.outputs.image }}
139139
api-image: ${{ needs.docker-build-api.outputs.image }}
140-
concurrency: 16
141-
tests: >-
142-
segment-part-1
143-
segment-part-2
144-
segment-part-3
145-
environment
146-
signup
147-
flag
148-
invite
149-
project
150-
versioning
140+
args: --meta-filter category=oss
151141
secrets:
152142
GCR_TOKEN: ${{ needs.permissions-check.outputs.can-write == 'true' && secrets.GITHUB_TOKEN || '' }}
153143
SLACK_TOKEN: ${{ needs.permissions-check.outputs.can-write == 'true' && secrets.SLACK_TOKEN || '' }}
@@ -164,12 +154,7 @@ jobs:
164154
runs-on: ${{ matrix.runs-on }}
165155
e2e-image: ${{ needs.docker-build-e2e.outputs.image }}
166156
api-image: ${{ needs.docker-build-private-cloud.outputs.image }}
167-
concurrency: 16
168-
tests: >-
169-
organisation-permission
170-
environment-permission
171-
project-permission
172-
roles
157+
args: --meta-filter category=enterprise
173158
secrets:
174159
GCR_TOKEN: ${{ needs.permissions-check.outputs.can-write == 'true' && secrets.GITHUB_TOKEN || '' }}
175160
SLACK_TOKEN: ${{ needs.permissions-check.outputs.can-write == 'true' && secrets.SLACK_TOKEN || '' }}

docker/common/e2e/.e2e-tests.yml

Lines changed: 0 additions & 42 deletions
This file was deleted.

frontend/docker-compose-e2e-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ services:
5050
SLACK_TOKEN: ${SLACK_TOKEN}
5151
GITHUB_ACTION_URL: ${GITHUB_ACTION_URL}
5252
ports:
53-
- 8080:8080
53+
- 3000:3000
5454
depends_on:
5555
flagsmith-api:
5656
condition: service_healthy

frontend/e2e/cafe/environment.cafe.js

Lines changed: 0 additions & 17 deletions
This file was deleted.

frontend/e2e/cafe/invite.test.cafe.js

Lines changed: 0 additions & 14 deletions
This file was deleted.

frontend/e2e/cafe/project.cafe.js

Lines changed: 0 additions & 16 deletions
This file was deleted.

frontend/e2e/cafe/segments.cafe.js

Lines changed: 0 additions & 16 deletions
This file was deleted.

0 commit comments

Comments
 (0)