pretenders / ployst

The ployst core repo
1 stars 0 forks source link

Add front-end tests #46

Closed txels closed 10 years ago

txels commented 10 years ago

I've been willing to get some front-end testing in for a while. This adds the first JS unit tests and the karma test runner is executed as part of the tox build. As a bonus I've also added jshint to the build.

alexcouper commented 10 years ago

When I run the tests using tox I get 3 errors and 2 failures in the python tests.

Am I doing something wrong?

alexcouper commented 10 years ago
diff --git a/tox.ini b/tox.ini
index 308f304..493c6b1 100644
--- a/tox.ini
+++ b/tox.ini
@@ -2,13 +2,17 @@
 envlist = py27

 [testenv]
-deps =
+setenv =
+    CFLAGS=-Qunused-arguments
+    CPPFLAGS=-Qunused-arguments
+deps =
     -r{toxinidir}/requirements/dev.txt
-commands =
+commands =
     flake8 ployst
     jshint ployst
     karma start --single-run
-    nosetests -s  # --with-coverage --cover-package=ployst --cover-html
+    python manage.py test
     sphinx-build -b html docs/source docs/build

 [flake8]

This is the output I get from tox after I make the above changes:

screen shot 2014-04-08 at 10 16 03 pm

Any idea how to remove the jshint/karma warnings?

txels commented 10 years ago

Nice to see Travis reporting green builds, cheers!

txels commented 10 years ago

I think this can be merged.

alexcouper commented 10 years ago

I'll merge once travis confirms life is good