-
Notifications
You must be signed in to change notification settings - Fork 88
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
Test failure: test_job_spark_python_task_linter_unhappy_path
#2845
Comments
❌ test_job_spark_python_task_linter_unhappy_path: assert 0 == 1 (10.091s)
Running from nightly #219 |
❌ test_job_spark_python_task_linter_unhappy_path: assert 0 == 1 (12.153s)
Running from nightly #220 |
1 task
1 task
❌ test_job_spark_python_task_linter_unhappy_path: assert 0 == 1 (13.413s)
Running from nightly #221 |
nfx
pushed a commit
that referenced
this issue
Oct 9, 2024
…2853) ## Changes This PR updates some linting related integration tests that weren't testing what they expected: - The tests were intending to verify linting of notebooks, and set up a notebook with code to trigger a linting record. - The linting code was failing prematurely due to the notebook extension, not due to the problem in source. Some usage of `greenlet` has also been replaced: it was previously unknown to the linter and various integration tests assumed this. Since #2830 these tests have no longer been working correctly. ### Related Issues Overlaps with #2852. Overlaps with #2867. Overlaps with #2868. Resolves #2843. Resolves #2844. Resolves #2845. Resolves #2846. ### Tests - [X] updated integration tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
❌ test_job_spark_python_task_linter_unhappy_path: assert 0 == 1 (9.81s)
Running from nightly #218
The text was updated successfully, but these errors were encountered: