)]}'
{"config_tempest/tests/services/test_services.py":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"d14f1f449dc4b0622e6f9330750563dbefff69ed","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        self.assertEqual(services.service_catalog, \u0027catalog\u0027)"},{"line_number":84,"context_line":"        self.assertEqual(services.public_url, \u0027url\u0027)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_parse_endpoints(self):"},{"line_number":87,"context_line":"        services \u003d self._create_services_instance()"},{"line_number":88,"context_line":"        services.public_url \u003d \"url\""},{"line_number":89,"context_line":"        # empty endpoint list"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_c33234a8","line":86,"updated":"2018-05-03 15:50:49.000000000","message":"I think that the name of this test should be more test_parse_endpoints_empty \n\nand maybe there should be two fornon-empty one: one with \u0027identity\u0027 in the url, the other without. That should cover all the branches of the function.","commit_id":"e1e0911bf482c330187f95663b98f61a85c20c21"}]}