8000 Replace nosetest with pytest by KostyaEsmukov · Pull Request #266 · geopy/geopy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace nosetest with pytest #266

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Apr 14, 2018
Merged

Replace nosetest with pytest #266

merged 4 commits into from
Apr 14, 2018

Conversation

KostyaEsmukov
Copy link
Member
@KostyaEsmukov KostyaEsmukov commented Mar 17, 2018 •
edited
Loading

@KostyaEsmukov KostyaEsmukov mentioned this pull request Mar 17, 2018
Closed
25 tasks
@KostyaEsmukov KostyaEsmukov force-pushed the pytest branch 3 times, most recently from 919fde3 to d458a85 Compare March 24, 2018 17:30
This was referenced Mar 26, 2018
Merged
Closed
@KostyaEsmukov KostyaEsmukov added this to the 1.14 milestone Apr 8, 2018
@KostyaEsmukov
Copy link
Member Author

OK, I believe it's ready. Let's throw this in and see how it goes.

As for the migration from nose, I'd say the following:

  • There's no need to import tests in __init__.py files anymore. pytest discovers them automatically.
  • There's no need to add doc strings to each test method anymore. They're not printed anyway.
  • To start using pytest, just follow the steps at CONTRIBUTING.md.

@KostyaEsmukov KostyaEsmukov merged commit 99be6fb into master Apr 14, 2018
@KostyaEsmukov KostyaEsmukov deleted the pytest branch April 14, 2018 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.14
Development

Successfully merging this pull request may close these issues.
1 participant
@KostyaEsmukov
0