)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"59ec576187540f3bda3aca76ffe81c25abfb2d08","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Add secure-rbac tests for Quotas"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds basic secure-rbac testing for the Quotas resource"},{"line_number":10,"context_line":"for the reader, member, and admin personas."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I290b808bbb8be26da281cf8971661c44fede71b0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"32018bd8_dc52ddb2","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":42},"updated":"2021-04-01 20:13:38.000000000","message":"add: for project scope.  so we can keep track of when we add the system scope stuff.","commit_id":"14afff748de5727b355b365938a7b2c593aef806"}],"barbican_tempest_plugin/tests/rbac/v1/test_quotas.py":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"59ec576187540f3bda3aca76ffe81c25abfb2d08","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @abc.abstractmethod"},{"line_number":44,"context_line":"    def test_get_custom_quota_for_project(self):"},{"line_number":45,"context_line":"        \"\"\"Test setting a custom quota for a specific project"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        Testing: GET /v1/project-quotas/{project-id}"},{"line_number":48,"context_line":"        This test must check:"}],"source_content_type":"text/x-python","patch_set":2,"id":"e553ae1b_b261f2b1","line":45,"range":{"start_line":45,"start_character":16,"end_line":45,"end_character":24},"updated":"2021-04-01 20:13:38.000000000","message":"s/setting/getting","commit_id":"14afff748de5727b355b365938a7b2c593aef806"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"3ae4c1b0ab98e44fa4fba574e60e77c731a01dde","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @abc.abstractmethod"},{"line_number":44,"context_line":"    def test_get_custom_quota_for_project(self):"},{"line_number":45,"context_line":"        \"\"\"Test setting a custom quota for a specific project"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        Testing: GET /v1/project-quotas/{project-id}"},{"line_number":48,"context_line":"        This test must check:"}],"source_content_type":"text/x-python","patch_set":2,"id":"263f345a_4d3f4ff1","line":45,"range":{"start_line":45,"start_character":16,"end_line":45,"end_character":24},"in_reply_to":"e553ae1b_b261f2b1","updated":"2021-04-01 22:02:08.000000000","message":"Done","commit_id":"14afff748de5727b355b365938a7b2c593aef806"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"59ec576187540f3bda3aca76ffe81c25abfb2d08","unresolved":true,"context_lines":[{"line_number":52,"context_line":"        raise NotImplementedError"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    @abc.abstractmethod"},{"line_number":55,"context_line":"    def test_setting_new_quota_for_project(self):"},{"line_number":56,"context_line":"        \"\"\"Test setting a custom quota for a specific project"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        Testing: PUT /v1/project-quotas/{project-id}"}],"source_content_type":"text/x-python","patch_set":2,"id":"1c68a3ce_9aff9ace","line":55,"range":{"start_line":55,"start_character":13,"end_line":55,"end_character":20},"updated":"2021-04-01 20:13:38.000000000","message":"NIT: to be consistent with other method names - test_set_custom_quota_for_project()","commit_id":"14afff748de5727b355b365938a7b2c593aef806"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"3ae4c1b0ab98e44fa4fba574e60e77c731a01dde","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        raise NotImplementedError"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    @abc.abstractmethod"},{"line_number":55,"context_line":"    def test_setting_new_quota_for_project(self):"},{"line_number":56,"context_line":"        \"\"\"Test setting a custom quota for a specific project"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        Testing: PUT /v1/project-quotas/{project-id}"}],"source_content_type":"text/x-python","patch_set":2,"id":"30ef737d_37e8e574","line":55,"range":{"start_line":55,"start_character":13,"end_line":55,"end_character":20},"in_reply_to":"1c68a3ce_9aff9ace","updated":"2021-04-01 22:02:08.000000000","message":"Done","commit_id":"14afff748de5727b355b365938a7b2c593aef806"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"876b1a81231e7b91db2e3a7f875e65bd066ab3c5","unresolved":true,"context_lines":[{"line_number":88,"context_line":"    def test_list_project_quotas(self):"},{"line_number":89,"context_line":"        self.do_request(\u0027list_quotas\u0027, expected_status\u003dexceptions.Forbidden)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def test_get_custom_quota_for_project(self):"},{"line_number":92,"context_line":"        project_id \u003d self.client.tenant_id"},{"line_number":93,"context_line":"        self.do_request(\u0027get_project_quota\u0027,"},{"line_number":94,"context_line":"                        expected_status\u003dexceptions.Forbidden,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9051e784_417e0457","line":91,"updated":"2021-04-05 18:27:55.000000000","message":"Are we going to add system-scope tests for this in a separate patch? It doesn\u0027t looks like the ProjectAdminTests below exercise administrative APIs?","commit_id":"98b7523de340c2b411f4a705cfd9e415717c42b6"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"99d1855931290417c83d5c418e3666e523b65691","unresolved":true,"context_lines":[{"line_number":88,"context_line":"    def test_list_project_quotas(self):"},{"line_number":89,"context_line":"        self.do_request(\u0027list_quotas\u0027, expected_status\u003dexceptions.Forbidden)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def test_get_custom_quota_for_project(self):"},{"line_number":92,"context_line":"        project_id \u003d self.client.tenant_id"},{"line_number":93,"context_line":"        self.do_request(\u0027get_project_quota\u0027,"},{"line_number":94,"context_line":"                        expected_status\u003dexceptions.Forbidden,"}],"source_content_type":"text/x-python","patch_set":3,"id":"e64cf3c9_a95d1fd0","line":91,"in_reply_to":"9051e784_417e0457","updated":"2021-04-06 13:46:59.000000000","message":"Yeah, system-scope test are still pending.  As with the negative tests, I\u0027m trying to keep patches small by splitting different scopes and positive/negative tests in different patches.","commit_id":"98b7523de340c2b411f4a705cfd9e415717c42b6"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"464b5685134cae86c7ead6fa1cd40958f3576bcb","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def test_list_project_quotas(self):"},{"line_number":89,"context_line":"        self.do_request(\u0027list_quotas\u0027, expected_status\u003dexceptions.Forbidden)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def test_get_custom_quota_for_project(self):"},{"line_number":92,"context_line":"        project_id \u003d self.client.tenant_id"},{"line_number":93,"context_line":"        self.do_request(\u0027get_project_quota\u0027,"},{"line_number":94,"context_line":"                        expected_status\u003dexceptions.Forbidden,"}],"source_content_type":"text/x-python","patch_set":3,"id":"69c980e8_6d2225bb","line":91,"in_reply_to":"e64cf3c9_a95d1fd0","updated":"2021-04-09 21:17:30.000000000","message":"Ack - that makes sense, thanks!","commit_id":"98b7523de340c2b411f4a705cfd9e415717c42b6"}]}