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

First configuration for tests #11

Merged
merged 3 commits into from
Apr 3, 2020

Conversation

anaschwendler
Copy link
Contributor

Tries to solve #3.

What I did?
Using as the base this project I've created two files to serve as set up for the test environment.

setup.cfg to set up pytest django settings file.

tests/settings.py with a basic setup to serve as settings file for the test environment.

What is happening now?
I don't do python in a long time 😅 but it seems to throw and error related to __init__.py
image

Copy link
Owner

@cuducos cuducos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great progress ✨ With that we tick the 1st checkbox of #3 💜

Regarding the error message, I think it's a fair message: maybe the DummyUser expects public_apps and pulic_models to be passed as keyword arguments, not simple arguments.

tests/settings.py Outdated Show resolved Hide resolved
tests/settings.py Outdated Show resolved Hide resolved
tests/settings.py Outdated Show resolved Hide resolved
@cuducos cuducos mentioned this pull request Apr 2, 2020
@anaschwendler anaschwendler marked this pull request as ready for review April 2, 2020 19:17
@anaschwendler anaschwendler changed the title [WIP] First configuration for tests First configuration for tests Apr 2, 2020
@cuducos cuducos merged commit 05d40e1 into cuducos:master Apr 3, 2020
@cuducos cuducos mentioned this pull request Apr 3, 2020
2 tasks
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