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.

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

Add abstract classes, remove empty conftest.py #55

Merged
merged 7 commits into from
Apr 4, 2024
Merged

Conversation

PavelErsh
Copy link
Collaborator

No description provided.

@PavelErsh
Copy link
Collaborator Author

на данный момент падают тесты

FAILED tests/functional/src/test_film.py::test_get_film_details_from_cache - assert 404 == 200
FAILED tests/functional/src/test_genre.py::test_get_genre_details_from_cache - assert 404 == 200
FAILED tests/functional/src/test_person.py::test_get_person_details_from_cache - assert 404 == 200

======================== 3 failed, 14 passed in 22.83s ========================

Screenshot 2024-04-03 at 23 31 53

посмотри пожалуйста код

async_api/src/services/person.py Outdated Show resolved Hide resolved
@a1d4r a1d4r merged commit 3b1c093 into master Apr 4, 2024
2 checks passed
@a1d4r a1d4r mentioned this pull request Apr 4, 2024
@a1d4r a1d4r changed the title correction of the fourth and first points of the review Add abstract classes, remove empty conftest.py Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants