)]}'
{"cinderclient/tests/unit/test_api_versions.py":[{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"3925059aa8cf6eaf56a09fe35747c0c266869eb5","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        v4 \u003d api_versions.APIVersion(\"2.0\")"},{"line_number":54,"context_line":"        v_null \u003d api_versions.APIVersion()"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        self.assertLess(v1, v2)"},{"line_number":57,"context_line":"        self.assertGreater(v3, v2)"},{"line_number":58,"context_line":"        self.assertNotEqual(v1, v2)"},{"line_number":59,"context_line":"        self.assertEqual(v1, v4)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_d48a2663","line":56,"updated":"2016-08-24 02:38:08.000000000","message":"I think it is better to use assertTrue. It tests the operators of class APIVersion","commit_id":"654a09e53845a9b463ef560629c0e661d8b192c0"},{"author":{"_account_id":22059,"name":"zheng yin","email":"yin__zheng@163.com","username":"zy751713126"},"change_message_id":"0476bd870944592a94c1fb49cc24e23206560ba7","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        v4 \u003d api_versions.APIVersion(\"2.0\")"},{"line_number":54,"context_line":"        v_null \u003d api_versions.APIVersion()"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        self.assertLess(v1, v2)"},{"line_number":57,"context_line":"        self.assertGreater(v3, v2)"},{"line_number":58,"context_line":"        self.assertNotEqual(v1, v2)"},{"line_number":59,"context_line":"        self.assertEqual(v1, v4)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_f41f62b4","line":56,"in_reply_to":"1ac06dbe_d48a2663","updated":"2016-08-24 02:55:05.000000000","message":"thank you. I think assertLess is readable and it is unnecessary to use assertTrue all the time.","commit_id":"654a09e53845a9b463ef560629c0e661d8b192c0"}]}