Merge lp:~dholbach/help-app/functional-test-template-improvements into lp:~nskaggs/help-app/functional-test-template-improvements

Proposed by Daniel Holbach
Status: Merged
Merged at revision: 161
Proposed branch: lp:~dholbach/help-app/functional-test-template-improvements
Merge into: lp:~nskaggs/help-app/functional-test-template-improvements
Diff against target: 63 lines (+11/-2)
4 files modified
HACKING (+1/-1)
internals/tests/build_utils.py (+4/-1)
internals/tests/functional/help_app/tests/test_app.py (+3/-0)
internals/tests/functional/help_app/tests/test_web.py (+3/-0)
To merge this branch: bzr merge lp:~dholbach/help-app/functional-test-template-improvements
Reviewer Review Type Date Requested Status
Nicholas Skaggs Pending
Review via email: mp+261402@code.launchpad.net
To post a comment you must log in.
162. By Daniel Holbach

move assertion if build worked out into test_app and test_web

163. By Daniel Holbach

move test into setUp of TestApp/TestWeb

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'HACKING'
--- HACKING 2015-05-28 19:34:12 +0000
+++ HACKING 2015-06-08 15:15:41 +0000
@@ -171,7 +171,7 @@
171 sudo apt-get install oxideqt-chromedriver171 sudo apt-get install oxideqt-chromedriver
172 sudo apt-add-repository ppa:canonical-platform-qa/selenium172 sudo apt-add-repository ppa:canonical-platform-qa/selenium
173 sudo apt-get update173 sudo apt-get update
174 sudo apt-get install python3-selenium174 sudo apt-get install python3-selenium xvfb
175175
176Run the tests:176Run the tests:
177177
178178
=== modified file 'internals/tests/build_utils.py'
--- internals/tests/build_utils.py 2015-06-05 15:03:41 +0000
+++ internals/tests/build_utils.py 2015-06-08 15:15:41 +0000
@@ -7,6 +7,7 @@
7sys.path.insert(0, '../..')7sys.path.insert(0, '../..')
88
9from pelicanconf import TOP_LEVEL_DIR9from pelicanconf import TOP_LEVEL_DIR
10import pelicanconf
1011
1112
12def clean_tempdir(tempdir):13def clean_tempdir(tempdir):
@@ -26,7 +27,9 @@
26 if self.build == 'web':27 if self.build == 'web':
27 self.env = {'OUTPUTDIR_WEB': self.tempdir}28 self.env = {'OUTPUTDIR_WEB': self.tempdir}
28 self.pwd = os.getcwd()29 self.pwd = os.getcwd()
29 os.chdir(TOP_LEVEL_DIR)30 top_level_dir = os.path.join(os.path.dirname(pelicanconf.__file__),
31 TOP_LEVEL_DIR)
32 os.chdir(top_level_dir)
30 self.run_build()33 self.run_build()
31 self.find_html_files()34 self.find_html_files()
32 self.html_path = os.path.commonprefix(self.html_files)35 self.html_path = os.path.commonprefix(self.html_files)
3336
=== modified file 'internals/tests/functional/help_app/tests/test_app.py'
--- internals/tests/functional/help_app/tests/test_app.py 2015-06-05 15:14:52 +0000
+++ internals/tests/functional/help_app/tests/test_app.py 2015-06-08 15:15:41 +0000
@@ -5,6 +5,9 @@
55
66
7class TestApp(HelpAppTest):7class TestApp(HelpAppTest):
8 def setUp(self):
9 super(TestApp, self).setUp()
10 self.assertEquals(self.app.build_runner.rc, 0)
811
9 def test_basic_launch(self):12 def test_basic_launch(self):
10 self.assertThat(self.app, NotEquals(None))13 self.assertThat(self.app, NotEquals(None))
1114
=== modified file 'internals/tests/functional/help_app/tests/test_web.py'
--- internals/tests/functional/help_app/tests/test_web.py 2015-06-05 15:14:52 +0000
+++ internals/tests/functional/help_app/tests/test_web.py 2015-06-08 15:15:41 +0000
@@ -5,6 +5,9 @@
55
66
7class TestWeb(HelpWebTest):7class TestWeb(HelpWebTest):
8 def setUp(self):
9 super(TestWeb, self).setUp()
10 self.assertEquals(self.app.build_runner.rc, 0)
811
9 def test_basic_launch(self):12 def test_basic_launch(self):
10 self.assertThat(self.app, NotEquals(None))13 self.assertThat(self.app, NotEquals(None))

Subscribers

People subscribed via source and target branches