-
Notifications
You must be signed in to change notification settings - Fork 3
Description
We need a test that can read all routes registered in the express so we can check if our expected routes exist. The file for this test is mount.test.js.
Tests functional under Express 4.20.0, but we have upgraded to Express 5.1.0 and the test started failing. We think it is because the way routes are registered have changed and can no longer be detected as the code is written. We do not want to spin up and call a route to check if it exists, we just want to check to see if the Express app knows it after all the routes are registered in app.js.
Update this test file so the tests are compatible with Express 5.1 and can check that the routes are registered by checking the Express app settings. Use the existing 10-2-package-updates branch. There is already an open Draft Pull Request at #104.
You can find Express 5 documentation at https://expressjs.com/en/5x/api.html. The Express 5 GitHub repository is https://github.com/expressjs/express