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: Start integrating pytest #6827
Merged
Merged
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
76a5942
changed test_errors to pytest
me-diru 35d50fa
modified test_auth to pytest
me-diru a9bae16
added pytest fixture to test_auth
me-diru c7b8b65
removed hound errors
me-diru 43eb8c2
build fail removed in test_auth.py
me-diru 2b2ebb3
removed unused import
me-diru 80ebfad
changed travis code and added pytest
me-diru 0210840
removed unused commands
me-diru b51136c
changed test command in travis
me-diru b8fda86
changed to single command
me-diru e9ac3d7
Merge branch 'development' into pytest
iamareebjamal 55f5f45
finalize
iamareebjamal ff0b7a7
finalize
iamareebjamal fa5d264
finalize
iamareebjamal c70976b
finalize
iamareebjamal f1abace
omit
iamareebjamal File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,8 +29,9 @@ before_script: | |
- export TEST_DATABASE_URL=postgresql://postgres@localhost:5432/test | ||
- bash scripts/test_multiple_heads.sh | ||
|
||
script: | ||
- nosetests tests/ -v --with-coverage | ||
script: | | ||
nosetests tests/ -v --with-coverage -I tests/all/integration/api/helpers/test_auth.py | ||
pytest tests/all/integration/api/helpers/test_auth.py | ||
|
||
after_success: | ||
- 'bash <(curl -s https://codecov.io/bash)' | ||
|
@@ -40,3 +41,4 @@ branches: | |
only: | ||
- master | ||
- development | ||
- pytest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not cool |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,3 +2,5 @@ | |
|
||
coverage~=5.0 | ||
dredd_hooks~=0.2 | ||
pytest ~= 5.2.1 | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not cool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't know another way to do it.
Somehow I have to exclude test_auth from nostests and test it with pytest