[1] in test_bugresolver setUp:
Not that it'll take a long time to run setUp, but we should move self.now to the top and use self.now when creating all the self.milestone_* attributes so they are all based on the same now.
247 + self.now = datetime.utcnow()
[2]test_run_with_set_milestone docstr is unhelpful with the expectations of what "correctly" is. Maybe a bit more to explain why 1 milestone is left None, one added and the 3rd left unchanged.
+ """Test plug-in with the set_milestone setting runs correctly."""
[3] test_run_with_set_milestone can use self.assertIsNone()
277 + self.assertEqual(self.bugs['0'].bug_tasks[1].milestone, None)
[4] In test__find_target_milestone_between can we have an assert to prove that the date is between milestone_past and milestone_future? Something like:
[1] in test_bugresolver setUp:
Not that it'll take a long time to run setUp, but we should move self.now to the top and use self.now when creating all the self.milestone_* attributes so they are all based on the same now.
247 + self.now = datetime.utcnow()
[2]test_ run_with_ set_milestone docstr is unhelpful with the expectations of what "correctly" is. Maybe a bit more to explain why 1 milestone is left None, one added and the 3rd left unchanged.
+ """Test plug-in with the set_milestone setting runs correctly."""
[3] test_run_ with_set_ milestone can use self.assertIsNone() l(self. bugs['0' ].bug_tasks[ 1].milestone, None)
277 + self.assertEqua
[4] In test__find_ target_ milestone_ between can we have an assert to prove that the date is between milestone_past and milestone_future? Something like:
now = self.milestone_ past.date_ targeted + timedelta(weeks=1) future) _find_target_ milestone( projects[ 1], now)
self.assertTrue(now , self.milestone_
milestone = self.plugin.
self.
[5] variable naming for in _get_and_ parse_config( ) clarity default -> default_milestone config( "default_ milestone" , None, *args)
96 + default = self.get_
97 + if default == "":
98 + default = None
[6] docstring fix in _find_target_ milestone
3) the last milestone in the list (covers len()==1 case).
"the last" -> "the most recent"