)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b0ef321fd208f46c102db3b34601d490246a33fd","unresolved":false,"context_lines":[{"line_number":7,"context_line":"make tests of endpoint_filter check endpoints num"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, because of a mistake, the tests of endpoint_filter check"},{"line_number":10,"context_line":"the number of services ranther than endpoints. This patch fixes this"},{"line_number":11,"context_line":"and adds a test to check this."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I03739a8ab5dd83bca8c49d152f979c3a80ad0033"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da86d52c_dd51dd64","line":10,"updated":"2015-02-13 16:00:40.000000000","message":"rather*","commit_id":"a16cf61311ca313aff6a462c9ac74a6be4aa6b61"}],"keystone/tests/test_associate_project_endpoint_extension.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b0ef321fd208f46c102db3b34601d490246a33fd","unresolved":false,"context_lines":[{"line_number":551,"context_line":"        endpoint_id2 \u003d r.result[\u0027endpoint\u0027][\u0027id\u0027]"},{"line_number":552,"context_line":""},{"line_number":553,"context_line":"        endpoint_ref \u003d self.new_endpoint_ref(service_id\u003dself.service_id)"},{"line_number":554,"context_line":"        self.post(\u0027/endpoints\u0027, body\u003d{\u0027endpoint\u0027: endpoint_ref})"},{"line_number":555,"context_line":""},{"line_number":556,"context_line":"        # only associate two endpoints with project"},{"line_number":557,"context_line":"        self.put(\u0027/OS-EP-FILTER/projects/%(project_id)s\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_1d6a9581","line":554,"updated":"2015-02-13 16:00:40.000000000","message":"a method could be used here for the duplicate logic.","commit_id":"a16cf61311ca313aff6a462c9ac74a6be4aa6b61"}],"keystone/tests/unit/test_associate_project_endpoint_extension.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cd60999cc5bc4b11363569be721e8ec5c126f5ce","unresolved":false,"context_lines":[{"line_number":542,"context_line":""},{"line_number":543,"context_line":"    def test_multiple_endpoint_project_associations(self):"},{"line_number":544,"context_line":""},{"line_number":545,"context_line":"        def _create_an_endpoint():"},{"line_number":546,"context_line":"            endpoint_ref \u003d self.new_endpoint_ref(service_id\u003dself.service_id)"},{"line_number":547,"context_line":"            r \u003d self.post(\u0027/endpoints\u0027, body\u003d{\u0027endpoint\u0027: endpoint_ref})"},{"line_number":548,"context_line":"            return r.result[\u0027endpoint\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_2f3e78da","line":545,"updated":"2015-02-26 02:27:21.000000000","message":"suggest remove \"an\", it just makes the name longer.","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"1957a1dc295a39db75078002eb2345c39e949426","unresolved":false,"context_lines":[{"line_number":574,"context_line":"            r,"},{"line_number":575,"context_line":"            require_catalog\u003dTrue,"},{"line_number":576,"context_line":"            endpoint_filter\u003dTrue,"},{"line_number":577,"context_line":"            ep_filter_assoc\u003d2)"},{"line_number":578,"context_line":""},{"line_number":579,"context_line":""},{"line_number":580,"context_line":"class JsonHomeTests(TestExtensionCase, test_v3.JsonHomeTestMixin):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_f4b48fd2","line":577,"updated":"2015-02-24 10:52:59.000000000","message":"we could make 2 a constant at the beginning of the test function.","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cd60999cc5bc4b11363569be721e8ec5c126f5ce","unresolved":false,"context_lines":[{"line_number":574,"context_line":"            r,"},{"line_number":575,"context_line":"            require_catalog\u003dTrue,"},{"line_number":576,"context_line":"            endpoint_filter\u003dTrue,"},{"line_number":577,"context_line":"            ep_filter_assoc\u003d2)"},{"line_number":578,"context_line":""},{"line_number":579,"context_line":""},{"line_number":580,"context_line":"class JsonHomeTests(TestExtensionCase, test_v3.JsonHomeTestMixin):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_4fb3ac97","line":577,"in_reply_to":"ba7be1f8_f4b48fd2","updated":"2015-02-26 02:27:21.000000000","message":"y, it is confusing since you wonder where the 2 came from.","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"}],"keystone/tests/unit/test_v3.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cd60999cc5bc4b11363569be721e8ec5c126f5ce","unresolved":false,"context_lines":[{"line_number":664,"context_line":"        token \u003d self.assertValidTokenResponse(r, *args, **kwargs)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        if require_catalog:"},{"line_number":667,"context_line":"            endpoint_num \u003d 0"},{"line_number":668,"context_line":"            self.assertIn(\u0027catalog\u0027, token)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"            if isinstance(token[\u0027catalog\u0027], list):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_cf66bc29","line":667,"updated":"2015-02-26 02:27:21.000000000","message":"this should be \"total_endpoint_count\", otherwise I think it\u0027s referring to the number of an endpoint.","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cd60999cc5bc4b11363569be721e8ec5c126f5ce","unresolved":false,"context_lines":[{"line_number":667,"context_line":"            endpoint_num \u003d 0"},{"line_number":668,"context_line":"            self.assertIn(\u0027catalog\u0027, token)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"            if isinstance(token[\u0027catalog\u0027], list):"},{"line_number":671,"context_line":"                # only test JSON"},{"line_number":672,"context_line":"                for service in token[\u0027catalog\u0027]:"},{"line_number":673,"context_line":"                    for endpoint in service[\u0027endpoints\u0027]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_8f80448c","line":670,"updated":"2015-02-26 02:27:21.000000000","message":"why wouldn\u0027t catalog be a list?","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cd60999cc5bc4b11363569be721e8ec5c126f5ce","unresolved":false,"context_lines":[{"line_number":674,"context_line":"                        self.assertNotIn(\u0027enabled\u0027, endpoint)"},{"line_number":675,"context_line":"                        self.assertNotIn(\u0027legacy_endpoint_id\u0027, endpoint)"},{"line_number":676,"context_line":"                        self.assertNotIn(\u0027service_id\u0027, endpoint)"},{"line_number":677,"context_line":"                        endpoint_num +\u003d 1"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"            # sub test for the OS-EP-FILTER extension enabled"},{"line_number":680,"context_line":"            if endpoint_filter:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_0f445472","line":677,"updated":"2015-02-26 02:27:21.000000000","message":"suggest use\n\n endpoint_num +\u003d len(service[\u0027endpoints\u0027])\n\nafter line 672","commit_id":"04592cc15e86e1b5af333d7222f6744c14465951"}]}