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

Test updates #209

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Test updates #209

wants to merge 3 commits into from

Conversation

hovi
Copy link

@hovi hovi commented Dec 10, 2024

Following #207 , I started playing with the project and getting tests running:

  • Updates tox file so that it works with version 4
    https://tox.wiki/en/4.0.3/faq.html#tox-4-changed-ini-rules
  • Updates test_reqs.txt to be consistent with tox.ini (versions and packages in test_reqs.txt were outdated)
  • Adds django manage.py file pointing to test tests.test_settings config file.

First 2 were breaking my test runs.
3rd one allows running tests simply by running python manage.py test in console - so "django way" of running tests. That means I can run every single test method by simply clicking icon in PyCharm, yay!

I haven't dived deeper into runtests.py and post_processor_runtests, but from first glance it seems like these could be fully removed - 1st by just doing python manage.py test and the second by running it with --testrunner tests.post_processor.discover_tests.DiscoverPostProcessorRunner.

@hovi
Copy link
Author

hovi commented Dec 10, 2024

No idea why the coverage upload isn't passing. I noticed that in job, there is tox>=3,<4, otherwise I don't see anything that could affect it.

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.

1 participant