)]}'
{"openstack_dashboard/dashboards/admin/instances/tests.py":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"e02db3e5a282b25e8f351a6c97914d40a15370fb","unresolved":false,"context_lines":[{"line_number":172,"context_line":"                        api.keystone: (\u0027tenant_list\u0027,)})"},{"line_number":173,"context_line":"    def test_index_options_after_migrate(self):"},{"line_number":174,"context_line":"        servers \u003d self.servers.list()"},{"line_number":175,"context_line":"        server1 \u003d servers[0]"},{"line_number":176,"context_line":"        server1.status \u003d \"VERIFY_RESIZE\""},{"line_number":177,"context_line":"        server2 \u003d servers[2]"},{"line_number":178,"context_line":"        server2.status \u003d \"VERIFY_RESIZE\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FuoY%3D","line":175,"updated":"2013-06-24 10:19:33.000000000","message":"why was this added here? It doesn\u0027t seem to be connected with the tenant-rename and also not connected with csv export.","commit_id":"0c0153a037594c6c98025f698e8593a01ed0ea2d"},{"author":{"_account_id":2455,"name":"Tihomir Trifonov","email":"t.trifonov@gmail.com","username":"ttrifonov"},"change_message_id":"3e9d5a9be57dbb9b42cece8638667349e5437649","unresolved":false,"context_lines":[{"line_number":172,"context_line":"                        api.keystone: (\u0027tenant_list\u0027,)})"},{"line_number":173,"context_line":"    def test_index_options_after_migrate(self):"},{"line_number":174,"context_line":"        servers \u003d self.servers.list()"},{"line_number":175,"context_line":"        server1 \u003d servers[0]"},{"line_number":176,"context_line":"        server1.status \u003d \"VERIFY_RESIZE\""},{"line_number":177,"context_line":"        server2 \u003d servers[2]"},{"line_number":178,"context_line":"        server2.status \u003d \"VERIFY_RESIZE\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FsIE%3D","line":175,"in_reply_to":"AAAAPn%2F%2FuoY%3D","updated":"2013-06-25 07:39:10.000000000","message":"That\u0027s because a new server(instance) was added to nova_data.py. Before the change there were 2 servers, one with status \u0027BUILD\u0027 and the other being \u0027ACTIVE\u0027. The latter is marked here as \u0027VERIFY_RESIZE\u0027 for this test, and the last assert here verifies that no instances is being migrated. As the new server added is also in status \u0027ACTIVE\u0027, the same logic needs to be applied to the new server for the test to run properly. And the new server is required for the CSV tests. So it is related.","commit_id":"0c0153a037594c6c98025f698e8593a01ed0ea2d"}],"openstack_dashboard/test/test_data/keystone_data.py":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"e02db3e5a282b25e8f351a6c97914d40a15370fb","unresolved":false,"context_lines":[{"line_number":181,"context_line":"                     \u0027name\u0027: \u0027disabled_tenant\u0027,"},{"line_number":182,"context_line":"                     \u0027description\u0027: \"a disabled test tenant.\","},{"line_number":183,"context_line":"                     \u0027enabled\u0027: False}"},{"line_number":184,"context_line":"    tenant_dict_3 \u003d {\u0027id\u0027: \"3\","},{"line_number":185,"context_line":"                     \u0027name\u0027: u\u0027\\u4e91\\u89c4\\u5219\u0027,"},{"line_number":186,"context_line":"                     \u0027description\u0027: \"an unicode-named tenant.\","},{"line_number":187,"context_line":"                     \u0027enabled\u0027: True}"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fung%3D","line":184,"updated":"2013-06-24 10:19:33.000000000","message":"as much as I like enhancing tests or improving coverage, I still disagree in adding that in other changesets.","commit_id":"0c0153a037594c6c98025f698e8593a01ed0ea2d"},{"author":{"_account_id":2455,"name":"Tihomir Trifonov","email":"t.trifonov@gmail.com","username":"ttrifonov"},"change_message_id":"3e9d5a9be57dbb9b42cece8638667349e5437649","unresolved":false,"context_lines":[{"line_number":181,"context_line":"                     \u0027name\u0027: \u0027disabled_tenant\u0027,"},{"line_number":182,"context_line":"                     \u0027description\u0027: \"a disabled test tenant.\","},{"line_number":183,"context_line":"                     \u0027enabled\u0027: False}"},{"line_number":184,"context_line":"    tenant_dict_3 \u003d {\u0027id\u0027: \"3\","},{"line_number":185,"context_line":"                     \u0027name\u0027: u\u0027\\u4e91\\u89c4\\u5219\u0027,"},{"line_number":186,"context_line":"                     \u0027description\u0027: \"an unicode-named tenant.\","},{"line_number":187,"context_line":"                     \u0027enabled\u0027: True}"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FsHs%3D","line":184,"in_reply_to":"AAAAPn%2F%2Fung%3D","updated":"2013-06-25 07:39:10.000000000","message":"What\u0027s the problem with adding new tenant for tests? Actually some of the tests were making wrong usage of volumes and quotas across tenants, as they assume that all existing volumes/quotas belong the the only one tenant in the test data. These were fixed, and also will help avoiding future errors in tests. Again - this is related to the CSV patch, as the patch applies the CSV rendering for usage data for project level and for admin level(for all projects). Thus an example with multiple tenants is needed for the tests there.","commit_id":"0c0153a037594c6c98025f698e8593a01ed0ea2d"}]}