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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement cli_runner fixture for test_cli.py #790

Merged

Conversation

hackebrot
Copy link
Member

This makes it even easier to write CLI tests. 馃槀

Currently:

result = runner.invoke(main, ['--verbose', template])

With this PR:

result = cli_runner('--verbose', template)

@hackebrot hackebrot added enhancement This issue/PR relates to a feature request. developers This issue/PR has request to other project developers needs-review PR Only: This PR require review from other developer labels Jul 31, 2016
@codecov-io
Copy link

codecov-io commented Jul 31, 2016

Current coverage is 100% (diff: 100%)

Merging #790 into master will not change coverage

@@           master   #790   diff @@
====================================
  Files          15     15          
  Lines         601    601          
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
  Hits          601    601          
  Misses          0      0          
  Partials        0      0          

Sunburst

Powered by Codecov. Last update 0301bf6...9de7f66

@michaeljoseph
Copy link
Contributor

馃憣

@michaeljoseph michaeljoseph merged commit 3128651 into cookiecutter:master Aug 1, 2016
michaeljoseph added a commit to michaeljoseph/cookiecutter that referenced this pull request Aug 1, 2016
hackebrot added a commit to hackebrot/cookiecutter that referenced this pull request Aug 1, 2016
@hackebrot hackebrot deleted the improve-cli-testing-setup branch August 1, 2016 19:52
@hackebrot
Copy link
Member Author

Thanks man! 馃檱

terryjbates pushed a commit to terryjbates/cookiecutter that referenced this pull request Aug 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developers This issue/PR has request to other project developers enhancement This issue/PR relates to a feature request. needs-review PR Only: This PR require review from other developer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants