System Test Overhaul spec/system/application/timeout_warning_spec.rb

[FireLemons]

Description

ensure each test in spec/system/application/timeout_warning_spec.rb is checking for database updates after the page has finished loading after a submit

See: https://github.com/rubyforgood/casa/pull/6305 for an example

Other improvements to tests in the file are welcome

If there are none, leave a comment on this issue and we will close the issue

Login Emails for test environment:

• [email protected] view site as a volunteer
• [email protected] view site as a supervisor
• [email protected] view site as an admin
• [email protected] view site as an all casa admin
  • go to /all_casa_admins/sign_in

password for all users: 12345678

Questions? Join Slack!

We highly recommend that you join us in slack #casa channel to ask questions quickly. And discord for office hours (currently Tuesday 5-7pm Pacific), stakeholder news, and upcoming new issues.

[github-actions[bot]]

This issue has been open without changes for a long time! What's up?

[JenMcD-star]

Hi @FireLemons, I believe no updates should be needed for this spec, since the timeout warning functionality only renders a JavaScript variable (which what the spec is checking for) and doesn’t trigger any form submission or database updates. There's nothing to verify in the DB.