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

Dead fixtures check before run tests #377

Merged
merged 9 commits into from
Nov 1, 2021
Merged

Conversation

nvo87
Copy link
Contributor

@nvo87 nvo87 commented Oct 23, 2021

Добавил запуск --dead-fixtures. Данная опция не запускает тесты, а только делает проверку на фикстуры.
Закрывает #352

@nvo87
Copy link
Contributor Author

nvo87 commented Oct 23, 2021

Hey there, I believe the following fixture(s) are not being used:
Fixture name: as_staff, location: app/fixtures/api.py:17
Fixture name: staff, location: users/fixtures.py:15
Fixture name: anon, location: users/fixtures.py:5

Эта штука стреляет нам в ногу из-за данных прекрасных фикстур. На то чтобы поставить их в игнор заведен уже issue как пару лет.

Есть идеи как это обойти?

Co-authored-by: Renovate Bot <bot@renovateapp.com>
@f213
Copy link
Member

f213 commented Oct 26, 2021

Дропнуть нерабочие фикстуры из дистрибутива :-)

@nvo87
Copy link
Contributor Author

nvo87 commented Oct 26, 2021

Дропнуть нерабочие фикстуры из дистрибутива :-)

Я наоборот воспринимал их как крутые заготовки, что бы сразу тестить апи.

Может просто заюзать их в сепульках например?

@f213
Copy link
Member

f213 commented Oct 26, 2021

Тоже неплохой вариант

@nvo87
Copy link
Contributor Author

nvo87 commented Oct 31, 2021

Блин, я мельком взглянул и перепутал их с вот этими заготовками.

Fixture name: as_staff, location: app/fixtures/api.py:17
Fixture name: staff, location: users/fixtures.py:15
Fixture name: anon, location: users/fixtures.py:5

Согласен тогда, чтобы их грохнуть + останется фактори же если что.

@nvo87
Copy link
Contributor Author

nvo87 commented Oct 31, 2021

@f213 думаю, что готово к мержу

@f213 f213 merged commit be569c2 into fandsdev:master Nov 1, 2021
@f213
Copy link
Member

f213 commented Nov 1, 2021

Спасибо!

@nvo87
Copy link
Contributor Author

nvo87 commented Nov 1, 2021

Мой первый опенсорс! :))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants