)]}'
{"neutron/tests/unit/common/test_utils.py":[{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"3248ca32e9c61316e055b751e85ed76067800f8b","unresolved":false,"context_lines":[{"line_number":542,"context_line":"    def test__getattr(self):"},{"line_number":543,"context_line":"        with utils.Timer() as timer:"},{"line_number":544,"context_line":"            time.sleep(1)"},{"line_number":545,"context_line":"        self.assertIn(round(timer.total_seconds()), [1, 2])"},{"line_number":546,"context_line":"        self.assertEqual(1, timer.delta.seconds)"},{"line_number":547,"context_line":""},{"line_number":548,"context_line":"    def test__enter_with_timeout(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_ed8d2d2b","line":545,"range":{"start_line":545,"start_character":22,"end_line":545,"end_character":27},"updated":"2020-08-13 15:31:13.000000000","message":"nitty nit: math.floor() may be less surprising","commit_id":"c3baba82924650e40e54e4d17fe58475b6347e65"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"aef739fd019c12bd98956ff0af05f8b03bc7ea14","unresolved":false,"context_lines":[{"line_number":542,"context_line":"    def test__getattr(self):"},{"line_number":543,"context_line":"        with utils.Timer() as timer:"},{"line_number":544,"context_line":"            time.sleep(1)"},{"line_number":545,"context_line":"        self.assertIn(round(timer.total_seconds()), [1, 2])"},{"line_number":546,"context_line":"        self.assertEqual(1, timer.delta.seconds)"},{"line_number":547,"context_line":""},{"line_number":548,"context_line":"    def test__enter_with_timeout(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_66a74e0d","line":545,"range":{"start_line":545,"start_character":22,"end_line":545,"end_character":27},"in_reply_to":"9f560f44_ed8d2d2b","updated":"2020-08-18 09:42:42.000000000","message":"I agree with Bence. This may be confusing for future readers of the code why we allows \"2\" here also.\nSo solution proposed by Bence or maybe something like:\n\n    assertLess(2, timer.total_seconds())\n\nmay be easier to understand later.","commit_id":"c3baba82924650e40e54e4d17fe58475b6347e65"}]}