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

fix flaky test in casa cases show spec #4904

Merged

Conversation

fchatterji
Copy link
Contributor

What github issue is this PR for, if any?

Resolves #4894

What changed, and why?

  • removed the creation and login as volunteer1, as there is already a volunteer set up at the beginning of the tests. This should fix the flakiness of the previous test
  • added a test that, if the hide old cases option is not checked, all cases are displayed. I felt the previous test was incomplete, it could've worked for example only because we use 2 different volunteers.
  • tried to inmprove readability of the test

How will this affect user permissions?

  • Volunteer permissions: /
  • Supervisor permissions: /
  • Admin permissions: /

How is this tested? (please write tests!) 💖💪

ran the test 100 times locally, no failure

Screenshots please :)

Feelings gif (optional)

What gif best describes your feeling working on this issue? https://giphy.com/
How to embed:
![alt text](https://media.giphy.com/media/1nP7ThJFes5pgXKUNf/giphy.gif)

Feedback please? (optional)

We are very interested in your feedback! Please give us some :) https://forms.gle/1D5ACNgTs2u9gSdh9

@github-actions github-actions bot added ruby Pull requests that update Ruby code Tests! 🎉💖👏 labels Jun 24, 2023
@FireLemons FireLemons merged commit c70382e into rubyforgood:main Jun 26, 2023
@FireLemons
Copy link
Collaborator

🥇

@fchatterji fchatterji deleted the fix_flaky_casa_cases_show_spec branch June 28, 2023 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ruby Pull requests that update Ruby code Tests! 🎉💖👏
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Flaky test in spec/system/casa_cases/show_spec.rb
2 participants