Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add test for happy path #91

Merged
merged 20 commits into from
Jan 9, 2025
Merged

Add test for happy path #91

merged 20 commits into from
Jan 9, 2025

Conversation

jxjj
Copy link
Collaborator

@jxjj jxjj commented Jan 8, 2025

  • Add cypress
  • Add a simple "happy path" test for creating a deck with a card that flips.
  • Configure Github Actions to run test

jxjj added 20 commits December 27, 2024 13:08
@jxjj
use 127.0.0.1 instead of localhost for vite proxy host
9a2a57a 
Cypress will white-page when trying to load vite assets using localhost, but works fine with `127.0.0.1`.

Maybe this?
cypress-io/cypress#25397
@jxjj
laravel cypress config
bc1a932
@jxjj
fix cypress type errors
a6c52ea
@jxjj
add happy path test
3e12671
@jxjj
add test workflow
828f6ce
@jxjj
add workflow_dispatch
43821df
@jxjj
can I just fork the action?
6e86f45
@jxjj
use gh secrets to generate auth.json for Nova
cec32e5
@jxjj
fix: invalid group id
9ffff02
@jxjj
de-dockerize for ci
72dc696 
probably easier?
@jxjj
use mysql:8.0 and modify health check
659ab63
@jxjj
set db host to 127.0.0.1
e9f0d8b
@jxjj
start on port 80
733db23
@jxjj
fix: spawnsync npm error?
c497f21 
ci fails at `npm ci` step with some weird spawnSync error. Trying to add the setup-node action.
@jxjj
use non-privileged port
f746545
@jxjj
fix(tests): cypress fails with `MESA: error: ZINK: vkCreateInstance f…
d4c8462 
…ailed (VK_ERROR_INCOMPATIBLE_DRIVER)`

Trying electron instead of chrome.
@jxjj
reportedly this issue is with 24.04?
2b9c471
@jxjj
with chrome
27055e2
@jxjj
oops. maybe this
2589a9f
@jxjj
simplifying
28ed0c4
@jxjj jxjj requested a review from cmcfadden January 8, 2025 23:32
@jxjj jxjj self-assigned this Jan 8, 2025
cmcfadden
cmcfadden approved these changes Jan 9, 2025
View reviewed changes
Copy link

@cmcfadden cmcfadden left a comment

Choose a reason for hiding this comment

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

woooo

@jxjj jxjj merged commit f180253 into develop Jan 9, 2025
2 checks passed
@jxjj jxjj deleted the feature/85-Add-some-tests branch January 9, 2025 15:29
@jxjj jxjj mentioned this pull request Jan 9, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmcfadden cmcfadden cmcfadden approved these changes

Assignees

@jxjj jxjj

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jxjj @cmcfadden