Merge lp:~bryce/quickly/template_tests into lp:quickly

[Bryce Harrington]

Stubs in a test directory, run script, and example test.

Opportunistic programmers may not normally think to write tests, but we all know how important they can be for ongoing maintenance of an application. In case they don't know how to set up a unittest framework, this removes that roadblock from them.

Obviously this could be improved upon a lot, but my thinking is that by keeping the test code simple, it won't look intimidating to newbs.

It might make sense to include a test that tests some of the generated classes in the project_root/python/ directory, both as working examples and to ensure all quickly projects begin life with 100% test coverage.

[Michael Terry (mterry)]

It'd be nice to have 'run-tests' exit with a non-zero value if any test failed for ease of integration into an automated test run.

And do you know if python-distutils (or -extra) has support for finding and running a test suite? Would be good to make this suite match distutils' expectations.

Also... Maybe a good first test would be to call pylint on the project? There's an example of good pylint arguments in my branch that adds a pylint test to quickly itself: https://code.launchpad.net/~mterry/quickly/pylint/+merge/42501

review: Needs Fixing

[Michael Terry (mterry)]

Oh, but thanks! This is a great idea. :)

[Tony Byrne (tony-badwolf)]

Nice idea. The supplied bash script generates an warning when you quickly package so I pushed a new branch that uses "quickly test" instead. The supplied unittest did not refer to the user's project so I upgraded to actually test a trivial project property to "document" the linkage. Uses nosetest to automatically create the test suite.

https://code.launchpad.net/~tony-badwolf/quickly/project_tests

[Michael Terry (mterry)]

I'll mark this as rejected since we merged a different but similar branch. Thanks, Bryce!

Unmerged revisions

583. By Bryce Harrington on 2011-01-22

Add a simple stubbed-in unit test framework to the quickly templates

=== added file 'data/templates/ubuntu-application/project_root/run-tests'
--- data/templates/ubuntu-application/project_root/run-tests	1970-01-01 00:00:00 +0000
+++ data/templates/ubuntu-application/project_root/run-tests	2011-01-22 22:53:57 +0000
@@ -0,0 +1,7 @@
+#!/bin/bash
+
+for t in tests/test*; do
+    echo $t
+    python $t
+    echo
+done

=== added directory 'data/templates/ubuntu-application/project_root/tests'
=== added file 'data/templates/ubuntu-application/project_root/tests/test_example'
--- data/templates/ubuntu-application/project_root/tests/test_example	1970-01-01 00:00:00 +0000
+++ data/templates/ubuntu-application/project_root/tests/test_example	2011-01-22 22:53:57 +0000
@@ -0,0 +1,19 @@
+#!/usr/bin/python
+
+import sys
+import os.path
+import unittest
+sys.path.insert(0, os.path.realpath(os.path.join(os.path.dirname(__file__), "..")))
+
+
+class TestExample(unittest.TestCase):
+    def setUp(self):
+        self.something = 42
+
+    def test_something(self):
+        my_value = 2 * 3 * 7
+        self.assertEqual(self.something, my_value)
+
+if __name__ == '__main__':
+    unittest.main()
+