)]}'
{"tempest/api/compute/servers/test_server_actions.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"2f058aaf070de01f00d53d0718e89dfb4b955194","unresolved":false,"context_lines":[{"line_number":225,"context_line":"            resp, output \u003d self.servers_client.get_console_output("},{"line_number":226,"context_line":"                self.server_id, 10)"},{"line_number":227,"context_line":"            self.assertEqual(200, resp.status)"},{"line_number":228,"context_line":"            assert output, \"Console output was empty.\""},{"line_number":229,"context_line":"            lines \u003d len(output.split(\u0027\\n\u0027))"},{"line_number":230,"context_line":"            self.assertEqual(lines, 10)"},{"line_number":231,"context_line":"        self.wait_for(get_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2FwFE%3D","line":228,"updated":"2013-08-22 09:58:12.000000000","message":"Please don\u0027t use assert. This will cause wrong failure handling in tempest.\nYou can use self.assertTrue instead.","commit_id":"e5e32f3e140c57393170312c54b16e09befa4b6f"},{"author":{"_account_id":2340,"name":"Ionut Artarisi","email":"iartarisi@suse.cz","username":"mapleoin"},"change_message_id":"1d79193335d0ebcc96a8762d8533f298f8aa91b4","unresolved":false,"context_lines":[{"line_number":225,"context_line":"            resp, output \u003d self.servers_client.get_console_output("},{"line_number":226,"context_line":"                self.server_id, 10)"},{"line_number":227,"context_line":"            self.assertEqual(200, resp.status)"},{"line_number":228,"context_line":"            assert output, \"Console output was empty.\""},{"line_number":229,"context_line":"            lines \u003d len(output.split(\u0027\\n\u0027))"},{"line_number":230,"context_line":"            self.assertEqual(lines, 10)"},{"line_number":231,"context_line":"        self.wait_for(get_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2FwEY%3D","line":228,"in_reply_to":"AAAARn%2F%2FwFE%3D","updated":"2013-08-22 09:59:51.000000000","message":"Thanks, agreed. Resubmitting.","commit_id":"e5e32f3e140c57393170312c54b16e09befa4b6f"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"29ba5a1e83ad0228fce104d3d5cb36bf5d2931c9","unresolved":false,"context_lines":[{"line_number":225,"context_line":"            resp, output \u003d self.servers_client.get_console_output("},{"line_number":226,"context_line":"                self.server_id, 10)"},{"line_number":227,"context_line":"            self.assertEqual(200, resp.status)"},{"line_number":228,"context_line":"            self.assertTrue(output, \"Console output was empty.\")"},{"line_number":229,"context_line":"            lines \u003d len(output.split(\u0027\\n\u0027))"},{"line_number":230,"context_line":"            self.assertEqual(lines, 10)"},{"line_number":231,"context_line":"        self.wait_for(get_output)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAARn%2F%2FwBI%3D","line":228,"updated":"2013-08-22 10:20:39.000000000","message":"I might be missing some context here but why shall we assert this?\n\nI think we can use assertNotIn and put in a list all the \"unwanted\" outputs","commit_id":"8457299fc6dad664eb1f99a494f88ceeaed3eda0"}]}