Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

refactor(tests): use factory-boy fixtures instead of importing factories #1045

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

refactor(tests): use factory-boy fixtures instead of importing factories #1045

wants to merge 1 commit into from

Conversation

c0rydoras
Copy link
Contributor

No description provided.

@c0rydoras c0rydoras requested a review from a team as a code owner April 16, 2024 08:43
@c0rydoras c0rydoras requested a review from hairmare April 16, 2024 08:44
@c0rydoras c0rydoras force-pushed the refactor/tests/factoryboy-fixtures-instead-of-import branch 2 times, most recently from 6b004a4 to 62585bc Compare April 16, 2024 08:53
@hairmare
Copy link
Contributor

Should this also update pytest-factoryboy given that the update seems possible with these changes? wdyt?

@c0rydoras
Copy link
Contributor Author

it still has the exact same problems :c

@c0rydoras c0rydoras force-pushed the refactor/tests/factoryboy-fixtures-instead-of-import branch from 62585bc to ce04c1d Compare April 16, 2024 09:14
@hairmare
Copy link
Contributor

it still has the exact same problems :c

meh

@c0rydoras c0rydoras force-pushed the refactor/tests/factoryboy-fixtures-instead-of-import branch 2 times, most recently from bb7d4c2 to f7e6ff7 Compare April 16, 2024 09:55
@c0rydoras c0rydoras force-pushed the refactor/tests/factoryboy-fixtures-instead-of-import branch from f7e6ff7 to 2ced1ea Compare April 16, 2024 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hairmare hairmare hairmare approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@c0rydoras @hairmare