Skip to content
This repository was archived by the owner on Sep 19, 2018. It is now read-only.

Fix pytest South migrations, removed workaround for ambassadors test #275

Merged
merged 4 commits into from
Aug 19, 2014

Conversation

jurecuhalev
Copy link
Contributor

As kindly debugged in pytest-dev/pytest-django#158 - renaming tests.py to test_{app_name}.py and removing python_files=*.py fixes the South migration issue.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.92%) when pulling 73ea4fc on gandalfar:fix_pytest into 22ef953 on codeeu:master.

@ercchy
Copy link
Contributor

ercchy commented Aug 19, 2014

Great!

What is the downfall of using this version, if any?

@jurecuhalev
Copy link
Contributor Author

You mean master of pytest_django? It's still unreleased, as such it might have more bugs, but we'd replace it with stable pypi release once it's out anyway. It's just that current release doesn't support Suth migrations.

@goranche
Copy link
Contributor

is there a good reason to introduce something that doesn't (yet) support what we need now?

@ercchy
Copy link
Contributor

ercchy commented Aug 19, 2014

Ok, we are on the look for when the stable one comes out!

Merging this awesome pull request now...

Thanks!

ercchy added a commit that referenced this pull request Aug 19, 2014
Fix pytest South migrations, removed workaround for ambassadors test
@ercchy ercchy merged commit 541d735 into codeeu:master Aug 19, 2014
@goranche
Copy link
Contributor

obviously ... there might be... or not 😄

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

Successfully merging this pull request may close these issues.

4 participants