Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Linux JS Test CI Job Failure due to a change in JS dependency #268

Closed
RRosio opened this issue Dec 4, 2023 · 0 comments · Fixed by #269
Closed

Linux JS Test CI Job Failure due to a change in JS dependency #268

RRosio opened this issue Dec 4, 2023 · 0 comments · Fixed by #269
Labels
bug Something isn't working status:Needs Triage

Comments

@RRosio
Copy link
Collaborator

RRosio commented Dec 4, 2023

Description

The JS tests fail in the CI and locally, likely due to a change in a JS dependency, with the error:

Timeout for http://localhost:8888/a@b/
Is the notebook server running?
FAIL ".CodeMirror-code" still did not exist in 10000ms

Reproduce

Following guide to run the tests as in https://nbclassic.readthedocs.io/en/latest/contributing.html#javascript-tests.

Expected behavior

Expecting the tests to Pass/Fail beyond this point as this seems to indicate something has broken in the environment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working status:Needs Triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Linux JS Test CI job krassowski/nbclassic
1 participant
@RRosio