From 43d33dc025d754e361a469fff022330e0c006c41 Mon Sep 17 00:00:00 2001 From: Saurabh Kumar Date: Mon, 20 Apr 2015 17:49:12 +0530 Subject: [PATCH] Add DjangoCookieTestCase --- .gitignore | 1 + .travis.yml | 6 +-- Makefile | 2 +- dev-requirements.txt | 3 -- requirements.txt | 13 +++++++ setup.cfg | 3 ++ tests/base.py | 51 +++++++++++++++++++++++++ tests/test_cookiecutter_substitution.py | 32 ++++++---------- 8 files changed, 84 insertions(+), 27 deletions(-) delete mode 100644 dev-requirements.txt create mode 100644 requirements.txt create mode 100644 setup.cfg create mode 100644 tests/base.py diff --git a/.gitignore b/.gitignore index 0aae0c9c..b9a2bb82 100644 --- a/.gitignore +++ b/.gitignore @@ -4,3 +4,4 @@ local_settings.py repo_name .idea +my_test_project/* diff --git a/.travis.yml b/.travis.yml index a0dc7921..2c6d2348 100644 --- a/.travis.yml +++ b/.travis.yml @@ -4,7 +4,7 @@ before_install: - time pip install pep8 install: -- pip install -r dev-requirements.txt +- pip install -r requirements.txt script: - pep8 --ignore E201,E202 --max-line-length=120 --exclude='migrations' . @@ -12,5 +12,5 @@ script: notifications: email: - on_success: never - on_failure: never + on_success: change + on_failure: always diff --git a/Makefile b/Makefile index ddb3a803..3b126041 100644 --- a/Makefile +++ b/Makefile @@ -1,4 +1,4 @@ .PHONY: test test: - py.test -q tests/*.py + py.test tests/*.py diff --git a/dev-requirements.txt b/dev-requirements.txt deleted file mode 100644 index fbcc80e1..00000000 --- a/dev-requirements.txt +++ /dev/null @@ -1,3 +0,0 @@ -cookiecutter -pep8 -pytest diff --git a/requirements.txt b/requirements.txt new file mode 100644 index 00000000..b9e49a29 --- /dev/null +++ b/requirements.txt @@ -0,0 +1,13 @@ +cookiecutter==1.0.0 +flake8==2.4.0 +sh + +# Debugging +# ------------------------------------- +ipdb==0.8 +ipython==3.1.0 + +# Testing +# ------------------------------------- +pytest==2.7.0 +git+git://github.com/mverteuil/pytest-ipdb.git diff --git a/setup.cfg b/setup.cfg new file mode 100644 index 00000000..1133f7a3 --- /dev/null +++ b/setup.cfg @@ -0,0 +1,3 @@ +[pytest] +python_paths = . +norecursedirs = .tox .git */migrations/* */static/* docs venv */{{cookiecutter.repo_name}}/* diff --git a/tests/base.py b/tests/base.py new file mode 100644 index 00000000..f6ba90fb --- /dev/null +++ b/tests/base.py @@ -0,0 +1,51 @@ +import os +import shutil +import unittest +from os.path import exists, dirname, join + +import sh + +from cookiecutter.main import cookiecutter + + +class DjangoCookieTestCase(unittest.TestCase): + + root_dir = dirname(dirname(__file__)) + ctx = {} + destpath = None + + def generate_project(self, extra_context=None): + ctx = { + "project_name": "My Test Project", + "repo_name": "my_test_project", + "author_name": "Test Author", + "email": "test@example.com", + "description": "A short description of the project.", + "domain_name": "example.com", + "version": "0.1.0", + "timezone": "UTC", + "now": "2015/01/13", + "year": "2015" + } + if extra_context: + assert isinstance(extra_context, dict) + ctx.update(extra_context) + + self.ctx = ctx + self.destpath = join(self.root_dir, self.ctx['repo_name']) + + cookiecutter(template='./', checkout=None, no_input=True, extra_context=ctx) + + # Build a list containing absolute paths to the generated files + paths = [os.path.join(dirpath, file_path) + for dirpath, subdirs, files in os.walk(self.destpath) + for file_path in files] + return paths + + def clean(self): + if exists(self.destpath): + shutil.rmtree(self.destpath) + sh.cd(self.root_dir) + + def tearDown(self): + self.clean() diff --git a/tests/test_cookiecutter_substitution.py b/tests/test_cookiecutter_substitution.py index 4cef245c..dd09daa8 100644 --- a/tests/test_cookiecutter_substitution.py +++ b/tests/test_cookiecutter_substitution.py @@ -1,28 +1,16 @@ -import os import re -import shutil -import unittest -from os.path import dirname, exists, join -from cookiecutter.main import cookiecutter +import sh + +from .base import DjangoCookieTestCase -class TestCookiecutterSubstitution(unittest.TestCase): +class TestCookiecutterSubstitution(DjangoCookieTestCase): """Test that all cookiecutter instances are substituted""" - cookiecutter(dirname(dirname(__file__)), no_input=True) - - destpath = join(dirname(dirname(__file__)), 'project_name') - - def tearDown(self): - if exists(self.destpath): - shutil.rmtree(self.destpath) - def test_all_cookiecutter_instances_are_substituted(self): # Build a list containing absolute paths to the generated files - paths = [os.path.join(dirpath, file_path) - for dirpath, subdirs, files in os.walk(self.destpath) - for file_path in files] + paths = self.generate_project() # Construct the cookiecutter search pattern pattern = "{{(\s?cookiecutter)[.](.*?)}}" @@ -36,6 +24,10 @@ class TestCookiecutterSubstitution(unittest.TestCase): match, "cookiecutter variable not replaced in {}".format(path)) - -if __name__ == '__main__': - unittest.main() + def test_flake8_complaince(self): + """generated project should pass flake8""" + self.generate_project() + try: + sh.flake8(self.destpath) + except sh.ErrorReturnCode as e: + raise AssertionError(e)