8000 Make tests work without installing cookiecutter by vincentbernat · Pull Request #550 · cookiecutter/cookiecutter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Make tests work without installing cookiecutter #550

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

Merged

Conversation

vincentbernat
Copy link
Contributor

When running tests without installing cookiecutter, some of them fail
because they try to invoke cookiecutter. Moreover, if cookiecutter is
installed on the system, the tests would use the installed version
instead of the one in the build tree.

To solve both issues, use python -m cookiecutter.cli instead of
cookiecutter.

I know that the official way of testing is using tox, but in some cases, tox is not usable because it requires a network access.

When running tests without installing cookiecutter, some of them fail
because they try to invoke cookiecutter. Moreover, if cookiecutter is
installed on the system, the tests would use the installed version
instead of the one in the build tree.

To solve both issues, use `python -m cookiecutter.cli` instead of
`cookiecutter`.
@codecov-io
Copy link

Current coverage is 99.58%

Merging #550 into master will not affect coverage as of 425ea11

@@            master    #550   diff @@
======================================
  Files           12      12       
  Stmts          482     482       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            480     480       
  Partial          0       0       
  Missed           2       2       

Review entire Coverage Diff as of 425ea11


Uncovered Suggestions

  1. +0.42% via ...ookiecutter/utils.py#27...28

Powered by Codecov. Updated on successful CI builds.

@hackebrot
Copy link
Member

👍

@hackebrot hackebrot added developers This issue/PR has request to other project developers needs-review PR Only: This PR require review from other developer labels Oct 20, 2015
pydanny added a commit that referenced this pull request Nov 5, 2015
Make tests work without installing cookiecutter
@pydanny pydanny merged commit 4ed5332 into cookiecutter:master Nov 5, 2015
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 needs-review PR Only: This PR require review from other developer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0