)]}'
{"tempest/api/compute/admin/test_live_migration_negative.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"a51272e7233a5a6b8b0a79b1f9c84b38251b4037","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        server_details \u003d self.admin_servers_client.show_server(server[\u0027id\u0027])"},{"line_number":56,"context_line":"        target_host \u003d server_details[\u0027server\u0027][\u0027OS-EXT-SRV-ATTR:host\u0027]"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        self.assertRaises(lib_exc.BadRequest, self._migrate_server_to,"},{"line_number":59,"context_line":"                          server[\u0027id\u0027], target_host)"},{"line_number":60,"context_line":"        waiters.wait_for_server_status(self.servers_client, server[\u0027id\u0027],"},{"line_number":61,"context_line":"                                       \u0027ACTIVE\u0027)"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_479a8314","line":59,"range":{"start_line":58,"start_character":0,"end_line":59,"end_character":52},"updated":"2018-04-17 18:29:06.000000000","message":"We have already a negative test for this API on the same response code (HTTP400). And the guideline says:\n\nhttps://github.com/openstack/tempest/blob/master/HACKING.rst#negative-tests\n\n About BadRequest(HTTP400) case: We can add a single negative\n tests of BadRequest for each resource and method(POST, PUT).\n Please don\u0027t implement more negative tests on the same\n combination of resource and method even if API request\n parameters are different from the existing test.\n\nFYI: developers are working on nova side related to this test as https://review.openstack.org/#/c/542689/","commit_id":"4c541bced46fd058a03caf7c38eff7efff7b3b08"}]}
