)]}'
{"cinder/tests/unit/objects/test_group.py":[{"author":{"_account_id":15961,"name":"lisali","email":"xiaoyan.li@intel.com","username":"lisali"},"change_message_id":"c006cc925723916d9d72381f835167900578d461","unresolved":false,"context_lines":[{"line_number":187,"context_line":"        # group.is_replicated) to self.assertEqual(True, group.is_replicated)"},{"line_number":188,"context_line":"        # to address a review comment. This way this test will still pass"},{"line_number":189,"context_line":"        # even if is_replicated is a method and not a property."},{"line_number":190,"context_line":"        self.assertEqual(True, group.is_replicated)"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    @ddt.data(\u0027\u003cis\u003e False\u0027, None, \u0027notASpecValueWeCareAbout\u0027)"},{"line_number":193,"context_line":"    def test_is_replicated_false(self, spec_value):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_1ce3443b","line":190,"updated":"2018-12-18 07:11:55.000000000","message":"Shoud be self.assertTrue(group.is_replicated)","commit_id":"064bfc1517f82654488a14368d571ad85f944904"},{"author":{"_account_id":20191,"name":"Dinesh Bhor","email":"bhordinesh07@gmail.com","username":"dineshbhor"},"change_message_id":"50e3de010e89d700bb8a83f293433be988608fc0","unresolved":false,"context_lines":[{"line_number":187,"context_line":"        # group.is_replicated) to self.assertEqual(True, group.is_replicated)"},{"line_number":188,"context_line":"        # to address a review comment. This way this test will still pass"},{"line_number":189,"context_line":"        # even if is_replicated is a method and not a property."},{"line_number":190,"context_line":"        self.assertEqual(True, group.is_replicated)"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    @ddt.data(\u0027\u003cis\u003e False\u0027, None, \u0027notASpecValueWeCareAbout\u0027)"},{"line_number":193,"context_line":"    def test_is_replicated_false(self, spec_value):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_bc31f00b","line":190,"in_reply_to":"3f79a3b5_1ce3443b","updated":"2018-12-18 07:59:37.000000000","message":"LisaLi: There is a comment given above the code which makes the code reasonable.","commit_id":"064bfc1517f82654488a14368d571ad85f944904"},{"author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"change_message_id":"88cafc706a6eb5951c411061a3e8c8a3e616e43d","unresolved":false,"context_lines":[{"line_number":197,"context_line":"            # group.is_replicated) to address a review comment. This way this"},{"line_number":198,"context_line":"            # test will still pass even if is_replicated is a method and not"},{"line_number":199,"context_line":"            # a property."},{"line_number":200,"context_line":"            self.assertEqual(False, group.is_replicated)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_b0549624","line":200,"range":{"start_line":200,"start_character":17,"end_line":200,"end_character":28},"updated":"2018-12-19 09:18:54.000000000","message":"Agree with Eric comment so just addition: here we can use assertFalse","commit_id":"052b6278b99033d0fd041760e987fc8c5bf687e0"}]}