feat(ci): implement new single-script e2e test workflow by seershan · Pull Request #972 · OWASP/wrongsecrets-ctf-party

seershan · 2025-09-10T14:19:25Z

Thank you for submitting a pull request to the WrongSecrets Party!

What kind of changes does this PR include?

Fixes or refactors
A new feature

Checklist:

All the contributions made are solely the work of me and my co-authors
I tested the changes in this PR (if applicable)
I added tests to ensure my change works (if applicable)
The PR passes pre-commit hooks and automated tests

This pull request adds a suite of Cypress End-to-End tests and a new GitHub Action to run them, fulfilling the requirements of issue #284. It also includes the necessary fixes to the deployment scripts and configuration to get the local and CI environments working.

This is a clean replacement for the previous pull request (#970).

Closes #284

commjoen · 2025-09-12T09:11:07Z

+      - name: Install yq
+        run: sudo wget https://github.com/mikefarah/yq/releases/latest/download/yq_linux_amd64 -O /usr/bin/yq && sudo chmod +x /usr/bin/yq
+      - name: Run Deployment Script
+        run: ./build-and-deploy-minikube.sh


it seems it is still waiting here as the script will never end. Instead it might be better to run this in the background (e.g. ./build-and-deploy-minikube.sh & and then do a poll:

Suggested change

run: ./build-and-deploy-minikube.sh

run: |

./build-and-deploy-minikube.sh &

pause 10

kubectl wait --for=condition=ready pod -l app=wrongsecrets-balancer --timeout=2s

commjoen · 2025-10-04T05:55:23Z

Hi @seershan , it's been a while! Hope you are doing great!
Can you please have a look at the feedback and the failing checks? Feel free to DM me over slack if you are having trouble resolving the issues.

seershan · 2025-10-04T07:14:43Z

Hello @commjoen actually im out of station currently, I will start again by Tuesday.
Sorry for the delay.

commjoen · 2025-10-04T08:54:28Z

all good! Have a good time and will see the commits later 👍

commjoen · 2026-03-25T05:44:54Z

any updates on this good sir ;) ?

commjoen · 2026-06-12T06:17:08Z

Any updates on this @seershan ?

seershan · 2026-06-14T13:58:50Z

Hi @commjoen,

Sorry for the long delay on this. I've been quite busy with personal commitments and only recently had the chance to get back to this PR.

I started debugging the workflow again and reproduced deployment-related issues locally. I found that the workflow was starting the deployment script in the background while simultaneously waiting for pods, so I've pushed a change to run the deployment script synchronously and am waiting for the CI checks to complete.

I'll investigate further based on the results of the new workflow run. Thanks for your patience.

…ixes Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

…cript 5.9.2 compatibility Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

…nd enforce engines requirements Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Co-authored-by: Jeroen Willemsen <jeroenwillemsen2001@gmail.com>

Bumps [axios](https://github.com/axios/axios) from 1.11.0 to 1.12.0. - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v1.11.0...v1.12.0) --- updated-dependencies: - dependency-name: axios dependency-version: 1.12.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>

…41.114.0

…43.102.8

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.23 to 4.18.1. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.23...4.18.1) --- updated-dependencies: - dependency-name: lodash dependency-version: 4.18.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

Bumps [axios](https://github.com/axios/axios) from 1.14.0 to 1.15.0. - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v1.14.0...v1.15.0) --- updated-dependencies: - dependency-name: axios dependency-version: 1.15.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>

…ller to v3.2.2

Bumps [protobufjs](https://github.com/protobufjs/protobuf.js) from 7.5.4 to 7.5.5. - [Release notes](https://github.com/protobufjs/protobuf.js/releases) - [Changelog](https://github.com/protobufjs/protobuf.js/blob/master/CHANGELOG.md) - [Commits](protobufjs/protobuf.js@protobufjs-v7.5.4...protobufjs-v7.5.5) --- updated-dependencies: - dependency-name: protobufjs dependency-version: 7.5.5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.11 to 1.16.0. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.11...v1.16.0) --- updated-dependencies: - dependency-name: follow-redirects dependency-version: 1.16.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) from 5.5.7 to 5.7.1. - [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases) - [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md) - [Commits](NaturalIntelligence/fast-xml-parser@v5.5.7...v5.7.1) --- updated-dependencies: - dependency-name: fast-xml-parser dependency-version: 5.7.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

…umentation Agent-Logs-Url: https://github.com/OWASP/wrongsecrets-ctf-party/sessions/f2f56e28-d430-4d39-a97b-78daa1c1407e Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

commjoen · 2026-06-29T20:49:18Z

hi @seershan , it looks like the branch has quite a few merge conflicts, are you sure these need to be updated?

feat(ci): implement new single-script e2e test workflow

67b18ac

seershan requested review from bendehaan and commjoen as code owners September 10, 2025 14:19

seershan mentioned this pull request Sep 10, 2025

feat(testing): add final cypress e2e tests and all required fixes #970

Closed

6 tasks

commjoen reviewed Sep 12, 2025

View reviewed changes

Copilot AI and others added 19 commits June 15, 2026 12:39

Initial plan

4f01706

feat: upgrade to Node.js 24 and update Jest to v30 with deprecation f…

3c9ade0

…ixes Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

feat: update dependencies and fix selectUnit function logic

57acaa5

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

feat: complete Node.js 24 upgrade and dependency modernization

25d3d63

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Initial assessment of failing tests and precommit issues

811f14a

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Fix Jest 30 test isolation issue and resolve lockfile synchronization

16c230a

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Upgrade TypeScript to latest version (5.9.2)

cf53edc

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Fix GitHub Actions and Docker build with --legacy-peer-deps for TypeS…

7e21b94

…cript 5.9.2 compatibility Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Fix GitHub Actions compatibility: add Node.js 24 setup to workflows a…

1a3feca

…nd enforce engines requirements Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

Fix precommit trailing whitespace issue in AdminPage.js

3624613

Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

chore(config): migrate config renovate.json

9b3ad2d

fix(ci): reduce cpu and memory request for minikube in ci

99d1f36

ci: run deploy script in background and wait for pods

c915211

docs: add detailed windows setup guide

d2be5fa

Update guides/WINDOWS_SETUP.md

fdb5da5

Co-authored-by: Jeroen Willemsen <jeroenwillemsen2001@gmail.com>

chore(deps): update terraform azurerm to v4.44.0

06f613b

chore(deps): update pre-commit hook renovatebot/pre-commit-hooks to v…

76d1758

…41.114.0

chore(deps): update dependency jest to v30.1.3

1a16e19

renovate Bot and others added 28 commits June 15, 2026 12:39

chore(deps): update actions/configure-pages action to v6

f99f835

chore(deps): update terraform-gcp to ~> 7.26.0

2054082

chore(deps): update pre-commit hook renovatebot/pre-commit-hooks to v…

2fcd541

…43.102.8

chore(deps): update terraform azurerm to v4.66.0

085c577

chore(deps): update terraform aws to v6.39.0

31a2859

chore(deps): update dependency minimatch to v10.2.5

fc5628c

chore(deps): update dependency lodash to v4.18.1

7b83c27

chore(deps): update actions/deploy-pages action to v5

94ffb5d

chore(deps): update dependency axios to v1.15.0 [security]

c86ba5c

chore(deps): update dependency kubernetes-sigs/aws-alb-ingress-contro…

3ecbd0f

…ller to v3.2.2

chore(deps): update dependency eslint to v10.2.1

e62703c

chore(deps): update dependency prettier to v3.8.3

3f5c17a

chore(deps): update react monorepo to v19.2.5

bedca1a

chore(deps): update dependency globals to v17.5.0

9ead893

Add reference to CHALLENGE62_GOOGLE_DRIVE_SETUP.md in challenge62 doc…

3e9b7fd

…umentation Agent-Logs-Url: https://github.com/OWASP/wrongsecrets-ctf-party/sessions/f2f56e28-d430-4d39-a97b-78daa1c1407e Co-authored-by: commjoen <1457214+commjoen@users.noreply.github.com>

chore(deps): update helm release kube-prometheus-stack to v82.18.0

f42aee4

fix(ci): run deployment script synchronously

6cd6402

fix(ci): improve deployment readiness checks

795cdb0

fix(ci): improve minikube deployment workflow

3032e30

fix(ci): avoid deployment race condition

928ae5e

fix(ci): add Cypress dependency and Node 24 setup

f4e969b

fix(ci): regenerate lockfile

93cff67

seershan force-pushed the fix/ci-e2e-script branch from 319dd9f to 93cff67 Compare June 15, 2026 07:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

feat(ci): implement new single-script e2e test workflow#972

feat(ci): implement new single-script e2e test workflow#972
seershan wants to merge 233 commits into
OWASP:mainfrom
seershan:fix/ci-e2e-script

seershan commented Sep 10, 2025

Uh oh!

commjoen Sep 12, 2025 •

edited

Loading

Uh oh!

commjoen commented Oct 4, 2025

Uh oh!

seershan commented Oct 4, 2025

Uh oh!

commjoen commented Oct 4, 2025

Uh oh!

commjoen commented Mar 25, 2026

Uh oh!

commjoen commented Jun 12, 2026

Uh oh!

seershan commented Jun 14, 2026

Uh oh!

commjoen commented Jun 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

-        run: ./build-and-deploy-minikube.sh
+        run: |
+        ./build-and-deploy-minikube.sh &
+       pause 10
+       kubectl wait --for=condition=ready pod -l app=wrongsecrets-balancer --timeout=2s

Uh oh!

Uh oh!

Conversation

seershan commented Sep 10, 2025

Uh oh!

commjoen Sep 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

commjoen commented Oct 4, 2025

Uh oh!

seershan commented Oct 4, 2025

Uh oh!

commjoen commented Oct 4, 2025

Uh oh!

commjoen commented Mar 25, 2026

Uh oh!

commjoen commented Jun 12, 2026

Uh oh!

seershan commented Jun 14, 2026

Uh oh!

commjoen commented Jun 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

commjoen Sep 12, 2025 •

edited

Loading