New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Jupyter] Remove dev_server from tests #9713
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #9713 +/- ##
==========================================
- Coverage 58.98% 58.96% -0.02%
==========================================
Files 583 583
Lines 71133 71133
==========================================
- Hits 41956 41946 -10
- Misses 28550 28555 +5
- Partials 627 632 +5 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, a few minor comments
I converted all the tests away from using the dev_server fixture and removed the fixture. Additionally: * I added back in the python tests that were skipped because of the missing chunk errors. Doing so required a couple of changes to the names of the resources within the tests. * I updated the download code a bit to download to the jupyter's `root_dir` as opposed to the cwd of the server. I think the "correct" approach anyways, since you can configure which directory the jupyter server serves files from, but it also has the added benefit of allowing us to configure this value during the test so that we can write files to a temp directory. The way this test was written, you needed to delete the downloaded files before running the test again. * I also split up the config and pps/publish tests to their own files.
- Lumino tabs (#9692) - Uninstantiated python client bugfix (#9705) - Prevent bad config from being stored (#9711) - Remove dev_server (#9713) - UNKNOWN cluster status state (#9719) - Prevent error popups bugfix (#9721) - Infinite scroll in file explorer (#9715) - Fix local cypress tests (#9728) - Infinite scroll w/ scrollbar (#9746) - Clear mounted datums before new datums (#9716) - Make input spec non-resizeable (#9720)
I converted all the tests away from using the dev_server fixture and removed the fixture.
Additionally:
root_dir
as opposed to the cwd of the server. I think the "correct" approach anyways, since you can configure which directory the jupyter server serves files from, but it also has the added benefit of allowing us to configure this value during the test so that we can write files to a temp directory. The way this test was written, you needed to delete the downloaded files before running the test again.