Skip to content

Fix route_info test import#344

Open
nickschot wants to merge 1 commit intoemberjs:masterfrom
mainmatter:fix-route-info-test-import
Open

Fix route_info test import#344
nickschot wants to merge 1 commit intoemberjs:masterfrom
mainmatter:fix-route-info-test-import

Conversation

@nickschot
Copy link

@nickschot nickschot commented Jan 28, 2026

Seems this test file was renamed at some point without the import being updated, meaning these tests haven't run in CI for quite a while.

Note: these tests may fail, a bunch of them does locally for me anyway. Not a surprise if they haven't run for so long I guess. It seems to be related to the use of async/await within the tests. Probably something to do with the way backburner is integrated in to the test infra. Easiest solution appears to be to rewrite to not use async await.

Seems this test file was renamed at some point without the import being updated, meaning these tests haven't run in CI for quite a while.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant