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.

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

Add "E2E" test that revises a sequence #3238

Open
corneliusroemer opened this issue Nov 19, 2024 · 1 comment
Open

Add "E2E" test that revises a sequence #3238

corneliusroemer opened this issue Nov 19, 2024 · 1 comment
Labels
e2e E2E test related feature Feature proposal tests Test related issues

Comments

@corneliusroemer
Copy link
Contributor

corneliusroemer commented Nov 19, 2024

We don't seem to have one, should be simple, would have caught #3224

(It's easy to write lots of tests that don't ever catch anything useful. So it's good to write tests when something was missed and that thing is easy to test - as in that case one definitely tests something that's relevant)

  1. After a sequence has been released by a user
  2. Navigate to the seq details page
  3. Click the "revise" button
  4. Make a small change
  5. Submit
  6. Check result

Not all of this might be possible with our current setup, that's not a problem, bug #3224 would have been caught when performing step 3/4.

@corneliusroemer corneliusroemer added tests Test related issues e2e E2E test related feature Feature proposal labels Nov 19, 2024
@theosanderson
Copy link
Member

Yeah I'm going to try to merge the new integration tests pretty soon to allow this sort of thing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e E2E test related feature Feature proposal tests Test related issues
Projects
None yet
Development

No branches or pull requests

2 participants