diff options
Diffstat (limited to 'ick2/build_tests.py')
-rw-r--r-- | ick2/build_tests.py | 72 |
1 files changed, 48 insertions, 24 deletions
diff --git a/ick2/build_tests.py b/ick2/build_tests.py index 47b2db8..9d17168 100644 --- a/ick2/build_tests.py +++ b/ick2/build_tests.py @@ -24,51 +24,75 @@ class BuildTests(unittest.TestCase): def setUp(self): as_dict = { - 'status': 'triggered', + 'status': ick2.BUILD_TRIGGERED, } self.resource = ick2.resource_from_dict(as_dict) + def test_is_finished_when_done(self): + build = ick2.Build(self.resource) + + self.assertEqual(build.get_status(), ick2.BUILD_TRIGGERED) + self.assertFalse(build.is_finished()) + + build.set_status(ick2.BUILD_BUILDING) + self.assertFalse(build.is_finished()) + + build.set_status(ick2.BUILD_DONE) + self.assertTrue(build.is_finished()) + + def test_is_finished_when_failed(self): + build = ick2.Build(self.resource) + + self.assertEqual(build.get_status(), ick2.BUILD_TRIGGERED) + self.assertFalse(build.is_finished()) + + build.set_status(ick2.BUILD_BUILDING) + self.assertFalse(build.is_finished()) + + build.set_status(ick2.BUILD_FAILED) + self.assertTrue(build.is_finished()) + def test_sets_status_from_triggered_only_when_acceptable(self): build = ick2.Build(self.resource) with self.assertRaises(ick2.WrongBuildStatusChange): - build.set_status('triggered') - self.assertEqual(build.get_status(), 'triggered') + build.set_status(ick2.BUILD_TRIGGERED) + self.assertEqual(build.get_status(), ick2.BUILD_TRIGGERED) with self.assertRaises(ick2.WrongBuildStatusChange): - build.set_status('done') - self.assertEqual(build.get_status(), 'triggered') + build.set_status(ick2.BUILD_DONE) + self.assertEqual(build.get_status(), ick2.BUILD_TRIGGERED) with self.assertRaises(ick2.WrongBuildStatusChange): - build.set_status('failed') - self.assertEqual(build.get_status(), 'triggered') + build.set_status(ick2.BUILD_FAILED) + self.assertEqual(build.get_status(), ick2.BUILD_TRIGGERED) - build.set_status('building') - self.assertEqual(build.get_status(), 'building') + build.set_status(ick2.BUILD_BUILDING) + self.assertEqual(build.get_status(), ick2.BUILD_BUILDING) def test_refuses_changing_status_from_building_when_unacceptable(self): build = ick2.Build(self.resource) - build.set_status('building') + build.set_status(ick2.BUILD_BUILDING) with self.assertRaises(ick2.WrongBuildStatusChange): - build.set_status('triggered') - self.assertEqual(build.get_status(), 'building') + build.set_status(ick2.BUILD_TRIGGERED) + self.assertEqual(build.get_status(), ick2.BUILD_BUILDING) with self.assertRaises(ick2.WrongBuildStatusChange): - build.set_status('building') - self.assertEqual(build.get_status(), 'building') + build.set_status(ick2.BUILD_BUILDING) + self.assertEqual(build.get_status(), ick2.BUILD_BUILDING) def test_changes_status_from_building_to_done(self): build = ick2.Build(self.resource) - build.set_status('building') - build.set_status('done') - self.assertEqual(build.get_status(), 'done') + build.set_status(ick2.BUILD_BUILDING) + build.set_status(ick2.BUILD_DONE) + self.assertEqual(build.get_status(), ick2.BUILD_DONE) def test_changes_status_from_building_to_failed(self): build = ick2.Build(self.resource) - build.set_status('building') - build.set_status('failed') - self.assertEqual(build.get_status(), 'failed') + build.set_status(ick2.BUILD_BUILDING) + build.set_status(ick2.BUILD_FAILED) + self.assertEqual(build.get_status(), ick2.BUILD_FAILED) def test_has_empty_build_graph_initially(self): build = ick2.Build(self.resource) @@ -80,7 +104,7 @@ class BuildTests(unittest.TestCase): self.resource['graph'] = { 1: { 'action': {'action': 'foo'}, - 'status': 'ready', + 'status': ick2.ACTION_READY, 'depends': [], }, } @@ -97,7 +121,7 @@ class BuildTests(unittest.TestCase): { action_id: { 'action': {'action': 'foo'}, - 'status': 'ready', + 'status': ick2.ACTION_READY, 'depends': [], }, } @@ -107,13 +131,13 @@ class BuildTests(unittest.TestCase): build = ick2.Build(self.resource) graph = build.get_graph() action_id = graph.append_action({'action': 'foo'}) - graph.set_action_status(action_id, 'building') + graph.set_action_status(action_id, ick2.BUILD_BUILDING) self.assertEqual( self.resource['graph'], { action_id: { 'action': {'action': 'foo'}, - 'status': 'building', + 'status': ick2.BUILD_BUILDING, 'depends': [], }, } |