X-Git-Url: https://plomlompom.com/repos/berlin_corona.txt?a=blobdiff_plain;f=tests%2Fprocesses.py;h=7d1d0f13776d06afe490e097c0b475d927283a05;hb=d31168fca24980f4e5f996bc187ed773eb504629;hp=9e769c1e3da6acafa81b3d529d040aba4a5c5251;hpb=80491fac3c476788d90010812c9ba0b95701e09b;p=plomtask diff --git a/tests/processes.py b/tests/processes.py index 9e769c1..7d1d0f1 100644 --- a/tests/processes.py +++ b/tests/processes.py @@ -9,31 +9,22 @@ from plomtask.todos import Todo class TestsSansDB(TestCaseSansDB): """Module tests not requiring DB setup.""" checked_class = Process - - def test_Process_id_setting(self) -> None: - """Test .id_ being set and its legal range being enforced.""" - self.check_id_setting() - - def test_Process_versioned_defaults(self) -> None: - """Test defaults of VersionedAttributes.""" - self.check_versioned_defaults({ - 'title': 'UNNAMED', - 'description': '', - 'effort': 1.0}) + do_id_test = True + versioned_defaults_to_test = {'title': 'UNNAMED', 'description': '', + 'effort': 1.0} class TestsSansDBProcessStep(TestCaseSansDB): """Module tests not requiring DB setup.""" checked_class = ProcessStep - - def test_ProcessStep_id_setting(self) -> None: - """Test .id_ being set and its legal range being enforced.""" - self.check_id_setting(2, 3, 4) + do_id_test = True + default_init_args = [2, 3, 4] class TestsWithDB(TestCaseWithDB): """Module tests requiring DB setup.""" checked_class = Process + test_versioneds = {'title': str, 'description': str, 'effort': float} def three_processes(self) -> tuple[Process, Process, Process]: """Return three saved processes.""" @@ -64,13 +55,8 @@ class TestsWithDB(TestCaseWithDB): p.save(self.db_conn) return p, set_1, set_2, set_3 - def test_Process_saving_and_caching(self) -> None: + def test_Process_conditions_saving(self) -> None: """Test .save/.save_core.""" - kwargs = {'id_': 1} - self.check_saving_and_caching(**kwargs) - self.check_saving_of_versioned('title', str) - self.check_saving_of_versioned('description', str) - self.check_saving_of_versioned('effort', float) p, set1, set2, set3 = self.p_of_conditions() p.uncache() r = Process.by_id(self.db_conn, p.id_) @@ -109,32 +95,49 @@ class TestsWithDB(TestCaseWithDB): assert isinstance(p2.id_, int) assert isinstance(p3.id_, int) steps_p1: list[tuple[int | None, int, int | None]] = [] + # add step of process p2 as first (top-level) step to p1 add_step(p1, steps_p1, (None, p2.id_, None), 1) p1_dict: dict[int, ProcessStepsNode] = {} p1_dict[1] = ProcessStepsNode(p2, None, True, {}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) + # add step of process p3 as second (top-level) step to p1 add_step(p1, steps_p1, (None, p3.id_, None), 2) step_2 = p1.explicit_steps[-1] p1_dict[2] = ProcessStepsNode(p3, None, True, {}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) + # add step of process p3 as first (top-level) step to p2, + # expect it as implicit sub-step of p1's second (p3) step steps_p2: list[tuple[int | None, int, int | None]] = [] add_step(p2, steps_p2, (None, p3.id_, None), 3) p1_dict[1].steps[3] = ProcessStepsNode(p3, None, False, {}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) + # add step of process p2 as explicit sub-step to p1's first sub-step add_step(p1, steps_p1, (None, p2.id_, step_2.id_), 4) step_3 = ProcessStepsNode(p3, None, False, {}, True) p1_dict[2].steps[4] = ProcessStepsNode(p2, step_2.id_, True, {3: step_3}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) + # add step of process p3 as explicit sub-step to non-existing p1 + # sub-step (of id=999), expect it to become another p1 top-level step add_step(p1, steps_p1, (None, p3.id_, 999), 5) p1_dict[5] = ProcessStepsNode(p3, None, True, {}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) + # add step of process p3 as explicit sub-step to p1's implicit p3 + # sub-step, expect it to become another p1 top-level step add_step(p1, steps_p1, (None, p3.id_, 3), 6) p1_dict[6] = ProcessStepsNode(p3, None, True, {}, False) self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) self.assertEqual(p1.used_as_step_by(self.db_conn), []) self.assertEqual(p2.used_as_step_by(self.db_conn), [p1]) self.assertEqual(p3.used_as_step_by(self.db_conn), [p1, p2]) + # add step of process p2 as explicit sub-step to p1's second (p3) + # top-level step + add_step(p1, steps_p1, (None, p3.id_, 2), 7) + p1_dict[2].steps[7] = ProcessStepsNode(p3, 2, True, {}, False) + # import pprint + # pprint.pp(p1.get_steps(self.db_conn, None)) + # pprint.pp(p1_dict) + self.assertEqual(p1.get_steps(self.db_conn, None), p1_dict) def test_Process_conditions(self) -> None: """Test setting Process.conditions/enables/disables.""" @@ -203,14 +206,8 @@ class TestsWithDB(TestCaseWithDB): class TestsWithDBForProcessStep(TestCaseWithDB): """Module tests requiring DB setup.""" checked_class = ProcessStep - - def test_ProcessStep_saving_and_caching(self) -> None: - """Test .save/.save_core.""" - kwargs = {'id_': 1, - 'owner_id': 2, - 'step_process_id': 3, - 'parent_step_id': 4} - self.check_saving_and_caching(**kwargs) + default_init_kwargs = {'owner_id': 2, 'step_process_id': 3, + 'parent_step_id': 4} def test_ProcessStep_from_table_row(self) -> None: """Test .from_table_row() properly reads in class from DB""" @@ -263,8 +260,125 @@ class TestsWithServer(TestCaseWithServer): self.check_post(form_data, '/process?id=6', 404) self.check_post(form_data, '/process?id=5', 302, '/processes') + def test_do_POST_process_steps(self) -> None: + """Test behavior of ProcessStep posting.""" + # pylint: disable=too-many-statements + form_data_1 = self.post_process(1) + self.post_process(2) + self.post_process(3) + # post first (top-level) step of process 2 to process 1 + form_data_1['new_top_step'] = [2] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 1) + retrieved_step = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step.step_process_id, 2) + self.assertEqual(retrieved_step.owner_id, 1) + self.assertEqual(retrieved_step.parent_step_id, None) + # post empty steps list to process, expect clean slate, and old step to + # completely disappear + form_data_1['new_top_step'] = [] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(retrieved_process.explicit_steps, []) + with self.assertRaises(NotFoundException): + ProcessStep.by_id(self.db_conn, retrieved_step.id_) + # post new first (top_level) step of process 3 to process 1 + form_data_1['new_top_step'] = [3] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + retrieved_step = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step.step_process_id, 3) + self.assertEqual(retrieved_step.owner_id, 1) + self.assertEqual(retrieved_step.parent_step_id, None) + # post to process steps list without keeps, expect clean slate + form_data_1['new_top_step'] = [] + form_data_1['steps'] = [retrieved_step.id_] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(retrieved_process.explicit_steps, []) + # post to process empty steps list but keep, expect 400 + form_data_1['steps'] = [] + form_data_1['keep_step'] = [retrieved_step.id_] + self.check_post(form_data_1, '/process?id=1', 400, '/process?id=1') + # post to process steps list with keep on non-created step, expect 400 + form_data_1['steps'] = [retrieved_step.id_] + form_data_1['keep_step'] = [retrieved_step.id_] + self.check_post(form_data_1, '/process?id=1', 400, '/process?id=1') + # post to process steps list with keep and process ID, expect 200 + form_data_1[f'step_{retrieved_step.id_}_process_id'] = [2] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 1) + retrieved_step = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step.step_process_id, 2) + self.assertEqual(retrieved_step.owner_id, 1) + self.assertEqual(retrieved_step.parent_step_id, None) + # post nonsensical new_top_step id and otherwise zero'd steps, expect + # 400 and preservation of previous state + form_data_1['new_top_step'] = ['foo'] + form_data_1['steps'] = [] + form_data_1['keep_step'] = [] + self.check_post(form_data_1, '/process?id=1', 400, '/process?id=1') + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 1) + retrieved_step = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step.step_process_id, 2) + self.assertEqual(retrieved_step.owner_id, 1) + self.assertEqual(retrieved_step.parent_step_id, None) + # post to process steps list with keep and process ID, expect 200 + form_data_1['new_top_step'] = [3] + form_data_1['steps'] = [retrieved_step.id_] + form_data_1['keep_step'] = [retrieved_step.id_] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 2) + retrieved_step_0 = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step_0.step_process_id, 2) + self.assertEqual(retrieved_step_0.owner_id, 1) + self.assertEqual(retrieved_step_0.parent_step_id, None) + retrieved_step_1 = retrieved_process.explicit_steps[1] + self.assertEqual(retrieved_step_1.step_process_id, 3) + self.assertEqual(retrieved_step_1.owner_id, 1) + self.assertEqual(retrieved_step_1.parent_step_id, None) + # post to process steps list with keeps etc., but trigger recursion + form_data_1['new_top_step'] = [] + form_data_1['steps'] = [retrieved_step_0.id_, retrieved_step_1.id_] + form_data_1['keep_step'] = [retrieved_step_0.id_, retrieved_step_1.id_] + form_data_1[f'step_{retrieved_step_0.id_}_process_id'] = [2] + form_data_1[f'step_{retrieved_step_1.id_}_process_id'] = [1] + self.check_post(form_data_1, '/process?id=1', 400, '/process?id=1') + # check previous status preserved despite failed steps setting + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 2) + retrieved_step_0 = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step_0.step_process_id, 2) + self.assertEqual(retrieved_step_0.owner_id, 1) + self.assertEqual(retrieved_step_0.parent_step_id, None) + retrieved_step_1 = retrieved_process.explicit_steps[1] + self.assertEqual(retrieved_step_1.step_process_id, 3) + self.assertEqual(retrieved_step_1.owner_id, 1) + self.assertEqual(retrieved_step_1.parent_step_id, None) + form_data_1[f'step_{retrieved_step_1.id_}_process_id'] = [3] + # post sub-step to step + form_data_1[f'new_step_to_{retrieved_step_1.id_}'] = [3] + self.post_process(1, form_data_1) + retrieved_process = Process.by_id(self.db_conn, 1) + self.assertEqual(len(retrieved_process.explicit_steps), 3) + retrieved_step_0 = retrieved_process.explicit_steps[0] + self.assertEqual(retrieved_step_0.step_process_id, 2) + self.assertEqual(retrieved_step_0.owner_id, 1) + self.assertEqual(retrieved_step_0.parent_step_id, None) + retrieved_step_1 = retrieved_process.explicit_steps[1] + self.assertEqual(retrieved_step_1.step_process_id, 3) + self.assertEqual(retrieved_step_1.owner_id, 1) + self.assertEqual(retrieved_step_1.parent_step_id, None) + retrieved_step_2 = retrieved_process.explicit_steps[2] + self.assertEqual(retrieved_step_2.step_process_id, 3) + self.assertEqual(retrieved_step_2.owner_id, 1) + self.assertEqual(retrieved_step_2.parent_step_id, retrieved_step_1.id_) + def test_do_GET(self) -> None: """Test /process and /processes response codes.""" - self.post_process() self.check_get_defaults('/process') self.check_get('/processes', 200)