-
Notifications
You must be signed in to change notification settings - Fork 0
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
failing tests after #127 -- some fixable now; some have to wait #131
Labels
Comments
martinfleis
pushed a commit
that referenced
this issue
Aug 1, 2024
This was referenced Aug 7, 2024
Merged
3 remaining
|
Merged
@anastassiavybornova Following #201, would it now be time to review/update the skipped over failing tests? |
yesss @jGaboardi i can try to do that tomorrow, will let you know if i need help with it |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Following #127, we have failures due to data being updated. Some we can fix now, some we can't
test_protocol.py::test_generate_case[original] - assert 25 == 17
- deleted service roads #161test_protocol.py::test_process_case_original[png]
- deleted service roads #161test_utils.py::test_read_original
- 72b5229[Aleppo-86101] - assert 78908 == 86101
[Auckland-109976] - assert 60364 == 109976
[Bucaramanga-110074] - assert 79317 == 110074
[Salt Lake City-110211] - assert 50917 == 110211
[Douala-88746] - assert 84819 == 88746
[Li\xe8ge-105174] - assert 79907 == 105174
[Wuhan-107551] - assert 92667 == 107551
test_utils.py::test_read_no_degree_2
- 603d01d[Bucaramanga-23127] - assert 16302 == 23127
[Aleppo-46179] - assert 43233 == 46179
[Douala-32020] - assert 30552 == 32020
[Auckland-40177] - assert 12439 == 40177
[Li\xe8ge-28597] - assert 16554 == 28597
[Salt Lake City-46362] - assert 13468 == 46362
test_common.py::TestContinuity::test_basic
- 1b7816bThe text was updated successfully, but these errors were encountered: