)]}'
{"cyborg_tempest_plugin/services/cyborg_rest_client.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"f53a801de4502eb1de6e570b43d771cad35f76aa","unresolved":true,"context_lines":[{"line_number":59,"context_line":"        resp, body \u003d self.get(\"/devices\")"},{"line_number":60,"context_line":"        return self._response_helper(resp, body)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def get_device(self, device_uuid):"},{"line_number":63,"context_line":"        resp, body \u003d self.get(\"/devices/%s\" % device_uuid)"},{"line_number":64,"context_line":"        return self._response_helper(resp, body)"},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"39a78c7f_f5f06170","line":62,"updated":"2020-12-22 09:10:39.000000000","message":"It\u0027s better to add the active test for the this API.","commit_id":"99cb60fb3fd0ff883c4637786b4c38c1e27cd37d"}],"cyborg_tempest_plugin/tests/api/test_device_negative.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"f53a801de4502eb1de6e570b43d771cad35f76aa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dfd528c1_56bf5820","updated":"2020-12-22 09:10:39.000000000","message":"We dont need to add a new file to test the negative scenarios for the device APIs, you can mergethis test in https://github.com/openstack/cyborg-tempest-plugin/blob/master/cyborg_tempest_plugin/tests/api/test_device.py\n\nAs above said, it\u0027s also need to be add the active test to cover this API.","commit_id":"99cb60fb3fd0ff883c4637786b4c38c1e27cd37d"}]}
