)]}'
{"keystone/tests/unit/test_v3_assignment.py":[{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"71c73aaaf528d98d1f46b70774d1b97ff61207cc","unresolved":false,"context_lines":[{"line_number":719,"context_line":"        # projects[0] has no parents, parents_as_ids must be None"},{"line_number":720,"context_line":"        self.assertIsNone(parents_as_ids)"},{"line_number":721,"context_line":""},{"line_number":722,"context_line":"    def test_get_project_with_parents_as_list_with_full_access(self):"},{"line_number":723,"context_line":"        \"\"\"``GET /projects/{project_id}?parents_as_list`` with full access."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"        Test plan:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_212354bf","side":"PARENT","line":722,"updated":"2015-12-04 04:35:03.000000000","message":"this testcase looks to me should be kept in AssignmentTestCase.","commit_id":"d120187f53ffbd2183c8b06ae38fcc6d23256d07"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"83327adf5d8625cc133f9837177170869b5e5098","unresolved":false,"context_lines":[{"line_number":719,"context_line":"        # projects[0] has no parents, parents_as_ids must be None"},{"line_number":720,"context_line":"        self.assertIsNone(parents_as_ids)"},{"line_number":721,"context_line":""},{"line_number":722,"context_line":"    def test_get_project_with_parents_as_list_with_full_access(self):"},{"line_number":723,"context_line":"        \"\"\"``GET /projects/{project_id}?parents_as_list`` with full access."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"        Test plan:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_b488d623","side":"PARENT","line":722,"in_reply_to":"7a740942_212354bf","updated":"2015-12-04 11:26:05.000000000","message":"These tests check the output of /projects/ . It\u0027s true that, in their setups we use role assignements, but they are tests on resource (projects).\n\nI think it\u0027s fine to keep them at test_v3_resource.","commit_id":"d120187f53ffbd2183c8b06ae38fcc6d23256d07"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"71c73aaaf528d98d1f46b70774d1b97ff61207cc","unresolved":false,"context_lines":[{"line_number":749,"context_line":"        self.assertIn(parent, r.result[\u0027project\u0027][\u0027parents\u0027])"},{"line_number":750,"context_line":"        self.assertEqual(2, len(r.result[\u0027project\u0027][\u0027parents\u0027]))"},{"line_number":751,"context_line":""},{"line_number":752,"context_line":"    def test_get_project_with_parents_as_list_with_partial_access(self):"},{"line_number":753,"context_line":"        \"\"\"``GET /projects/{project_id}?parents_as_list`` with partial access."},{"line_number":754,"context_line":""},{"line_number":755,"context_line":"        Test plan:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_4c09f72b","side":"PARENT","line":752,"updated":"2015-12-04 04:35:03.000000000","message":"this one too.","commit_id":"d120187f53ffbd2183c8b06ae38fcc6d23256d07"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"71c73aaaf528d98d1f46b70774d1b97ff61207cc","unresolved":false,"context_lines":[{"line_number":884,"context_line":"        # projects[3] has no subtree, subtree_as_ids must be None"},{"line_number":885,"context_line":"        self.assertIsNone(subtree_as_ids)"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"    def test_get_project_with_subtree_as_list_with_full_access(self):"},{"line_number":888,"context_line":"        \"\"\"``GET /projects/{project_id}?subtree_as_list`` with full access."},{"line_number":889,"context_line":""},{"line_number":890,"context_line":"        Test plan:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_0c144f42","side":"PARENT","line":887,"updated":"2015-12-04 04:35:03.000000000","message":"and this one.","commit_id":"d120187f53ffbd2183c8b06ae38fcc6d23256d07"}],"keystone/tests/unit/test_v3_resource.py":[{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"71c73aaaf528d98d1f46b70774d1b97ff61207cc","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class ResourceTestCase(test_v3.RestfulTestCase,"},{"line_number":29,"context_line":"                       test_v3.AssignmentTestMixin):"},{"line_number":30,"context_line":"    \"\"\"Test domains and projects.\"\"\""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_2c490b32","line":29,"updated":"2015-12-04 04:35:03.000000000","message":"now that they has been splited, pls don\u0027t inherit from `test_v3.AssignmentTestMixin`\n - if there are some testcases still need this, then they should be belong to `test_v3_assignment.py` rather than here. \n - if they do belong to this test module, pls provide a private a method to do that.","commit_id":"8e1c106a53680d669d5dd76e136cc256dd15590d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"83327adf5d8625cc133f9837177170869b5e5098","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class ResourceTestCase(test_v3.RestfulTestCase,"},{"line_number":29,"context_line":"                       test_v3.AssignmentTestMixin):"},{"line_number":30,"context_line":"    \"\"\"Test domains and projects.\"\"\""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_14c20a34","line":29,"in_reply_to":"7a740942_2c490b32","updated":"2015-12-04 11:26:05.000000000","message":"The superclass provides help assignments functions that are needed in this class too.\n\nWhy should a test belong there, just because of the fact it needs a new role assignment in its setup? Examples are the tests you commented in the previous file.\n\nI don\u0027t think either we should duplicate the code.","commit_id":"8e1c106a53680d669d5dd76e136cc256dd15590d"}]}