[openstreetmap/openstreetmap-website] Reorder directions stubbing calls in tests (PR #7176)

Marwin Hochfelsner notifications at github.com
Wed Jun 24 19:56:43 UTC 2026


Of all the attempts since the merge of #7147, these map-layout system tests have failed:
- `DirectionsSystemTest#test removes popup on sidebar close`: 13 times
- `DirectionsSystemTest#test updates route output on mode change`: 6 times

In these cases, the stub calls don't wait for the module to load, as that wasn't needed before.
But after the first input, the module has to be loaded. So that's where the stub calls will have to be able to override the requests.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/7176

-- Commit Summary --

  * Reorder stubbing calls

-- File Changes --

    M test/system/directions_test.rb (8)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/7176.patch
https://github.com/openstreetmap/openstreetmap-website/pull/7176.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7176
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/7176 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260624/a5d198ee/attachment.htm>


More information about the rails-dev mailing list