[openstreetmap/openstreetmap-website] Remove test_logout_removes_session_token (PR #5456)
Anton Khorev
notifications at github.com
Thu Jan 2 07:09:28 UTC 2025
This test was about tokens that don't exist since #4535. That PR modified the test by removing the assert that checked the number of tokens in https://github.com/openstreetmap/openstreetmap-website/commit/4dff06a6293971c3e17f8508859a1d80717a23f6. Yet the test was kept although it doesn't do what it was supposed to do.
It also has `post user_save_path` that used to happen after `post user_new_path` when the terms page was shown during signups. This is not happening after #4455, and `post user_save_path` was removed from other tests in https://github.com/openstreetmap/openstreetmap-website/commit/1276fb944a8024884f3121d21577ef892353dfd1.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5456
-- Commit Summary --
* Remove test_logout_removes_session_token
-- File Changes --
M test/controllers/sessions_controller_test.rb (8)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/5456.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5456.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5456
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5456 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250101/227d9b07/attachment-0001.htm>
More information about the rails-dev
mailing list