)]}'
{"tempest/api/identity/admin/v3/test_version.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"39d0cda9817db92557749f9c2e03b4efbefc71d2","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def test_get(self):"},{"line_number":26,"context_line":"        resp, version \u003d self.client.get_version()"},{"line_number":27,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # Validate several of the more interesting fields in the response."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_b2cd0689","line":27,"updated":"2014-07-10 06:44:33.000000000","message":"Response code check should be moved to client side. As per BP- https://review.openstack.org/#/c/93037/","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"054f4e015303a1c8df7e36c4f195c9b01e82dbcb","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def test_get(self):"},{"line_number":26,"context_line":"        resp, version \u003d self.client.get_version()"},{"line_number":27,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # Validate several of the more interesting fields in the response."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_39eedded","line":27,"in_reply_to":"baada198_b2cd0689","updated":"2014-07-13 16:25:07.000000000","message":"Done","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"11a65ae5f6d275e04380323b8612a6a3a45ba6f2","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        is_self_link \u003d lambda x: x[\u0027rel\u0027] \u003d\u003d \u0027self\u0027"},{"line_number":37,"context_line":"        self_link \u003d filter(is_self_link, version[\u0027links\u0027])[0]"},{"line_number":38,"context_line":"        self.assertThat(self_link[\u0027href\u0027], matchers.StartsWith(\u0027http://\u0027))"},{"line_number":39,"context_line":"        self.assertThat(self_link[\u0027href\u0027], matchers.EndsWith(\u0027/v3/\u0027))"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_be1fe1b7","line":38,"updated":"2014-07-22 08:57:31.000000000","message":"I am using Tempest to test a OpenStack environment that sits behind a SSL webserver. Thus my OS_AUTH_URL starts with https://.\n\nCould you change this assertThat so that it also works with https please ?","commit_id":"f8a0507f3aa34b386830858375c5598c3b39350d"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696044574cd6ff3e93aa5e85366c32d053c01af6","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        is_self_link \u003d lambda x: x[\u0027rel\u0027] \u003d\u003d \u0027self\u0027"},{"line_number":37,"context_line":"        self_link \u003d filter(is_self_link, version[\u0027links\u0027])[0]"},{"line_number":38,"context_line":"        self.assertThat(self_link[\u0027href\u0027], matchers.StartsWith(\u0027http://\u0027))"},{"line_number":39,"context_line":"        self.assertThat(self_link[\u0027href\u0027], matchers.EndsWith(\u0027/v3/\u0027))"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_e9c8a123","line":38,"in_reply_to":"baada198_be1fe1b7","updated":"2014-07-29 21:10:00.000000000","message":"Done","commit_id":"f8a0507f3aa34b386830858375c5598c3b39350d"}],"tempest/api/identity/admin/v3/test_versions.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"e7f2cadba0b138da9b0cea71c127dc735e07bd39","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def test_get(self):"},{"line_number":26,"context_line":"        resp, body \u003d self.client.get_versions()"},{"line_number":27,"context_line":"        self.assertThat(resp[\u0027status\u0027], matchers.Equals(\u0027200\u0027))"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # Validate several of the more interesting fields in the response."},{"line_number":30,"context_line":"        version \u003d body[\u0027version\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_b26c21d2","line":27,"updated":"2014-06-24 18:09:40.000000000","message":"Is there any reason to use self.assertThat instead of self.assertEqual here? With self.assertEqual it looks like this\n\n    self.assertEqual(\u003cexpected result\u003e, \u003cactual result\u003e)","commit_id":"709fa59155cfad991e2d5a6814f6f41cfde37c62"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"98c5966fca8611f568f7050afb68b00aae792b52","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def test_get(self):"},{"line_number":26,"context_line":"        resp, body \u003d self.client.get_versions()"},{"line_number":27,"context_line":"        self.assertThat(resp[\u0027status\u0027], matchers.Equals(\u0027200\u0027))"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # Validate several of the more interesting fields in the response."},{"line_number":30,"context_line":"        version \u003d body[\u0027version\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_3a137bb8","line":27,"in_reply_to":"1ae5cdf2_b26c21d2","updated":"2014-06-25 01:46:43.000000000","message":"Done. I figured the assert* methods would be deprecated by now.","commit_id":"709fa59155cfad991e2d5a6814f6f41cfde37c62"}],"tempest/services/identity/v3/json/identity_client.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"0152c0b10980704191b75d6131f2c79dda8b55b0","unresolved":false,"context_lines":[{"line_number":453,"context_line":"    def get_versions(self):"},{"line_number":454,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":455,"context_line":"        body \u003d json.loads(body)"},{"line_number":456,"context_line":"        return resp, body"},{"line_number":457,"context_line":""},{"line_number":458,"context_line":""},{"line_number":459,"context_line":"class V3TokenClientJSON(rest_client.RestClient):"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_780408c0","line":456,"updated":"2014-07-03 07:18:37.000000000","message":"As function gets version, it\u0027s better to return version\u0027s items instead of body.\n\nreturn resp, body[\u0027version\u0027]","commit_id":"7779fafbd0a28285526bc43fb31f6ac6daaddc03"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"828faa17ace15ed0c81c98cc6fd7f1bca13136fa","unresolved":false,"context_lines":[{"line_number":453,"context_line":"    def get_versions(self):"},{"line_number":454,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":455,"context_line":"        body \u003d json.loads(body)"},{"line_number":456,"context_line":"        return resp, body"},{"line_number":457,"context_line":""},{"line_number":458,"context_line":""},{"line_number":459,"context_line":"class V3TokenClientJSON(rest_client.RestClient):"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_a4a9d225","line":456,"in_reply_to":"baada198_780408c0","updated":"2014-07-06 16:07:20.000000000","message":"Done","commit_id":"7779fafbd0a28285526bc43fb31f6ac6daaddc03"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"39d0cda9817db92557749f9c2e03b4efbefc71d2","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        return resp, body"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"    def get_version(self):"},{"line_number":454,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":455,"context_line":"        body \u003d json.loads(body)"},{"line_number":456,"context_line":"        return resp, body[\u0027version\u0027]"},{"line_number":457,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_12c31258","line":454,"updated":"2014-07-10 06:44:33.000000000","message":"Response code check needs to be done here. As per BP- https://review.openstack.org/#/c/93037/","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"054f4e015303a1c8df7e36c4f195c9b01e82dbcb","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        return resp, body"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"    def get_version(self):"},{"line_number":454,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":455,"context_line":"        body \u003d json.loads(body)"},{"line_number":456,"context_line":"        return resp, body[\u0027version\u0027]"},{"line_number":457,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_59eb91db","line":454,"in_reply_to":"baada198_12c31258","updated":"2014-07-13 16:25:07.000000000","message":"Done","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"}],"tempest/services/identity/v3/xml/identity_client.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"0152c0b10980704191b75d6131f2c79dda8b55b0","unresolved":false,"context_lines":[{"line_number":455,"context_line":"    def get_versions(self):"},{"line_number":456,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":457,"context_line":"        body \u003d self._parse_versions(etree.fromstring(body))"},{"line_number":458,"context_line":"        return resp, body"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"class V3TokenClientXML(rest_client.RestClient):"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_18e22c03","line":458,"updated":"2014-07-03 07:18:37.000000000","message":"same comment in JSON client","commit_id":"7779fafbd0a28285526bc43fb31f6ac6daaddc03"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"828faa17ace15ed0c81c98cc6fd7f1bca13136fa","unresolved":false,"context_lines":[{"line_number":455,"context_line":"    def get_versions(self):"},{"line_number":456,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":457,"context_line":"        body \u003d self._parse_versions(etree.fromstring(body))"},{"line_number":458,"context_line":"        return resp, body"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"class V3TokenClientXML(rest_client.RestClient):"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_c4a4064d","line":458,"in_reply_to":"baada198_18e22c03","updated":"2014-07-06 16:07:20.000000000","message":"Done","commit_id":"7779fafbd0a28285526bc43fb31f6ac6daaddc03"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"39d0cda9817db92557749f9c2e03b4efbefc71d2","unresolved":false,"context_lines":[{"line_number":453,"context_line":"        return resp, body"},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"    def get_version(self):"},{"line_number":456,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":457,"context_line":"        body \u003d self._parse_version(etree.fromstring(body))"},{"line_number":458,"context_line":"        return resp, body[\u0027version\u0027]"},{"line_number":459,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_92ae4297","line":456,"updated":"2014-07-10 06:44:33.000000000","message":"Response code check needs to be done here. As per BP- https://review.openstack.org/#/c/93037/","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"054f4e015303a1c8df7e36c4f195c9b01e82dbcb","unresolved":false,"context_lines":[{"line_number":453,"context_line":"        return resp, body"},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"    def get_version(self):"},{"line_number":456,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":457,"context_line":"        body \u003d self._parse_version(etree.fromstring(body))"},{"line_number":458,"context_line":"        return resp, body[\u0027version\u0027]"},{"line_number":459,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_79bbf5e4","line":456,"in_reply_to":"baada198_92ae4297","updated":"2014-07-13 16:25:07.000000000","message":"Done","commit_id":"0454de8f0c1e42e068c49b951100c77173fc96f8"}]}
