)]}'
{"keystoneauth1/tests/unit/identity/test_identity_common.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"88404e69e3da603cc98feffbbe3a65a4a8c7c6d8","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        # First grab a version that can be returned without doing discovery"},{"line_number":437,"context_line":"        version \u003d s.get_api_major_version("},{"line_number":438,"context_line":"            service_type\u003d\u0027volumev2\u0027, interface\u003d\u0027admin\u0027)"},{"line_number":439,"context_line":"        self.assertEqual((2, 0), version)"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"        # Then ask for one that requires discovery"},{"line_number":442,"context_line":"        v2_compute \u003d self.TEST_COMPUTE_ADMIN + \u0027/v2.0\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f287b81_2679a32e","line":439,"range":{"start_line":439,"start_character":0,"end_line":439,"end_character":41},"updated":"2017-08-24 12:58:02.000000000","message":"Just so I\u0027m clear: we\u0027ve proven that discovery wasn\u0027t done by virtue of not having created a request mock?","commit_id":"dbcd880ce0d209a2b9868d7eb70d659ca2b537be"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"06bb0d64f62e4b0ac1461044e7426e9d899426a5","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        # First grab a version that can be returned without doing discovery"},{"line_number":437,"context_line":"        version \u003d s.get_api_major_version("},{"line_number":438,"context_line":"            service_type\u003d\u0027volumev2\u0027, interface\u003d\u0027admin\u0027)"},{"line_number":439,"context_line":"        self.assertEqual((2, 0), version)"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"        # Then ask for one that requires discovery"},{"line_number":442,"context_line":"        v2_compute \u003d self.TEST_COMPUTE_ADMIN + \u0027/v2.0\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f287b81_c12be97f","line":439,"range":{"start_line":439,"start_character":0,"end_line":439,"end_character":41},"in_reply_to":"7f287b81_2679a32e","updated":"2017-08-24 13:09:20.000000000","message":"Yes. If it tries to do discovery it\u0027ll blow up with an exception.","commit_id":"dbcd880ce0d209a2b9868d7eb70d659ca2b537be"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"88404e69e3da603cc98feffbbe3a65a4a8c7c6d8","unresolved":false,"context_lines":[{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        version \u003d s.get_api_major_version("},{"line_number":452,"context_line":"            service_type\u003d\u0027compute\u0027, interface\u003d\u0027admin\u0027)"},{"line_number":453,"context_line":"        self.assertEqual((3, 0), version)"},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"    def test_direct_discovering_with_relative_link(self):"},{"line_number":456,"context_line":"        # need to construct list this way for relative"},{"line_number":457,"context_line":"        disc \u003d fixture.DiscoveryList(v2\u003dFalse, v3\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f287b81_86005769","line":454,"range":{"start_line":453,"start_character":0,"end_line":454,"end_character":0},"updated":"2017-08-24 12:58:02.000000000","message":"And here, how have we proved that discovery happened?\n\nIn both these cases, it would be nice to explain with code comments so folks coming after can more easily understand it.  I, for one, find all the test cases in this file to be pretty tough to follow.","commit_id":"dbcd880ce0d209a2b9868d7eb70d659ca2b537be"}]}
