Skip to content

Conversation

seershan
Copy link

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

- 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
Copy link
Collaborator

@commjoen commjoen Sep 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Collaborator

commjoen commented Oct 4, 2025

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
Copy link
Author

seershan commented Oct 4, 2025

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

@commjoen
Copy link
Collaborator

commjoen commented Oct 4, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create cypres test for full round trip (webtop &wrongsecrets)
2 participants