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

Fix broken tests #476

Open
stepan-anokhin opened this issue Mar 24, 2022 · 0 comments
Open

Fix broken tests #476

stepan-anokhin opened this issue Mar 24, 2022 · 0 comments
Assignees
@stepan-anokhin
Labels
tech debt

Comments

@stepan-anokhin
Copy link
Collaborator

stepan-anokhin commented Mar 24, 2022 •
edited
Loading

The following tests are broken after #454 and should be fixed:

FAILED tests/winnow/storage/test_remote_signatures_dao.py::test_save_matches_db[DBRemoteSignaturesDAO] - AttributeError: 'DBRemoteSignaturesDAO' object has no attribute 'save_matches'
FAILED tests/winnow/storage/test_remote_signatures_dao.py::test_save_matches_db[ReprRemoteSignaturesDAO] - AttributeError: 'ReprRemoteSignaturesDAO' object has no attribute 'save_matches'
FAILED tests/general_tests.py::test_extract_frame_level_features - AttributeE...
FAILED tests/general_tests.py::test_extract_video_level_features - AttributeE...
FAILED tests/general_tests.py::test_extract_video_signatures - AttributeError...
FAILED tests/general_tests.py::test_signatures_are_saved - AttributeError: 'P...
@stepan-anokhin stepan-anokhin self-assigned this Mar 24, 2022
stepan-anokhin added a commit to stepan-anokhin/VideoDeduplication that referenced this issue Mar 24, 2022
@stepan-anokhin
Temporary disable broken tests (benetech#476)
faa41ee
stepan-anokhin added a commit to stepan-anokhin/VideoDeduplication that referenced this issue Mar 24, 2022
@stepan-anokhin
Add dummy test to temporary fix integrationtests (benetech#476)
8d4709f
stepan-anokhin added a commit to stepan-anokhin/VideoDeduplication that referenced this issue Mar 25, 2022
@stepan-anokhin
Temporary disable integration tests (benetech#476)
99525cb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stepan-anokhin stepan-anokhin

Labels
tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stepan-anokhin