)]}'
{"tempest/api/compute/images/test_image_metadata_negative.py":[{"author":{"_account_id":8625,"name":"Anju Tiwari","username":"Anju5","inactive":true},"change_message_id":"90821e4ff258e9bea5d8dea437435b4030767dc5","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":59,"context_line":"    def test_set_nonexistant_image_metadata_item(self):"},{"line_number":60,"context_line":"        # Negative test: Metadata item should not be set to a"},{"line_number":61,"context_line":"        # nonexistant image"},{"line_number":62,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt\u0027}"},{"line_number":63,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":64,"context_line":"                          self.client.set_image_metadata_item, 999, \u0027key1\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8oE0%3D","line":61,"updated":"2013-12-24 05:41:07.000000000","message":"I think this test is same as above:-\n\nNegative test: Metadata should not be set to a non-existent image\n\nI think we can check this in only a single test as above.","commit_id":"72ccf9093d6fc04e49c63bf6c6482c382e4178b9"},{"author":{"_account_id":7884,"name":"Noboru Arai","email":"arai@mxa.nes.nec.co.jp","username":"NoboruArai"},"change_message_id":"5c2cf1586276308e3dcb20962a3a9b0afac7aa88","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt1\u0027, \u0027key2\u0027: \u0027alt2\u0027}"},{"line_number":42,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":43,"context_line":"                          self.client.update_image_metadata,"},{"line_number":44,"context_line":"                          random.randint(1024, 4096), meta)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":47,"context_line":"    def test_get_nonexistent_image_metadata_item(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F8n60%3D","line":44,"updated":"2013-12-24 06:10:54.000000000","message":"is \"random\" standard library \"random\"?\n\nIn my understand, \"import random\" is nessesary.","commit_id":"67af52dc0c7825b810c1dd6b47029f1e3fa7cc34"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e45a2bab9b40eac3c0c811aeb60697e1033b01bf","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        # Negative test: List on nonexistent image"},{"line_number":36,"context_line":"        # metadata should not happen"},{"line_number":37,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.list_image_metadata,"},{"line_number":38,"context_line":"                          random.randint(1024, 4096))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":41,"context_line":"    def test_update_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F%2Fuk%3D","line":38,"updated":"2013-12-25 07:10:55.000000000","message":"This integer seems invalid because image-id must be uuid.\n\nhttp://api.openstack.org/api-ref-compute.html\n\n v2/images/?{image_id}?\n {\n    \"image\": {\n        \"created\": \"2011-01-01T01:02:03Z\",\n        \"id\": \"70a599e0-31e7-49b7-b260-868f441e862b\",\n        \"links\": [\n [..]\n\nHow about using data_utils.rand_uuid() insted of random.randint() like the other negative tests?","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"3f64652f01fc7c58796052948223bcbeb73bbf9c","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        # Negative test: List on nonexistent image"},{"line_number":36,"context_line":"        # metadata should not happen"},{"line_number":37,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.list_image_metadata,"},{"line_number":38,"context_line":"                          random.randint(1024, 4096))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":41,"context_line":"    def test_update_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F918%3D","line":38,"in_reply_to":"AAAAT3%2F%2F%2Fuk%3D","updated":"2013-12-26 06:14:46.000000000","message":"In original code, image id was 999(please see test_image_metadata.py), do you think this should be uuid?","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"20d1d34e989bd99f6deb0d6ef0fd1dc4eca570e9","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        # Negative test: List on nonexistent image"},{"line_number":36,"context_line":"        # metadata should not happen"},{"line_number":37,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.list_image_metadata,"},{"line_number":38,"context_line":"                          random.randint(1024, 4096))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":41,"context_line":"    def test_update_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F91A%3D","line":38,"in_reply_to":"AAAAT3%2F%2F918%3D","updated":"2013-12-26 06:21:55.000000000","message":"You are right, this is not due to your change.\nIt is an original problem.\n\nAs I said on the above comment, image_id is uuid according to API specification. in addition, we can see uuid as image-id if running \"nova image-list\" command:\n\n $ nova image-list\n +--------------------------------------+---------------------------------+--------+--------+\n | ID                                   | Name                            | Status | Server |\n +--------------------------------------+---------------------------------+--------+--------+\n | d0dc2f9d-047c-4b18-bd65-f8a456a64395 | cirros-0.3.1-x86_64-uec         | ACTIVE |        |\n | c5af5073-021f-4b1e-9907-a53f02509fb4 | cirros-0.3.1-x86_64-uec-kernel  | ACTIVE |        |\n | f55eea10-977f-421a-a19c-434786bd172e | cirros-0.3.1-x86_64-uec-ramdisk | ACTIVE |        |\n +--------------------------------------+---------------------------------+--------+--------+\n\nThis is a good chance for fitting negative tests to API specification.","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"f16478ada8ab5481ddd10301be65d58edb7d15ac","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        # Negative test: List on nonexistent image"},{"line_number":36,"context_line":"        # metadata should not happen"},{"line_number":37,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.list_image_metadata,"},{"line_number":38,"context_line":"                          random.randint(1024, 4096))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":41,"context_line":"    def test_update_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F9zA%3D","line":38,"in_reply_to":"AAAAT3%2F%2F91A%3D","updated":"2013-12-26 06:39:39.000000000","message":"Done","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e45a2bab9b40eac3c0c811aeb60697e1033b01bf","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        # Negative test: Get on non-existent image should not happen"},{"line_number":51,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":52,"context_line":"                          self.client.get_image_metadata_item,"},{"line_number":53,"context_line":"                          random.randint(1024, 4096), \u0027key2\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":56,"context_line":"    def test_set_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F%2Fug%3D","line":53,"updated":"2013-12-25 07:10:55.000000000","message":"ditto","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"f16478ada8ab5481ddd10301be65d58edb7d15ac","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        # Negative test: Get on non-existent image should not happen"},{"line_number":51,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":52,"context_line":"                          self.client.get_image_metadata_item,"},{"line_number":53,"context_line":"                          random.randint(1024, 4096), \u0027key2\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":56,"context_line":"    def test_set_nonexistent_image_metadata(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F9y8%3D","line":53,"in_reply_to":"AAAAT3%2F%2F%2Fug%3D","updated":"2013-12-26 06:39:39.000000000","message":"Done","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e45a2bab9b40eac3c0c811aeb60697e1033b01bf","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        # Negative test: Metadata should not be set to a non-existent image"},{"line_number":58,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt1\u0027, \u0027key2\u0027: \u0027alt2\u0027}"},{"line_number":59,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.set_image_metadata,"},{"line_number":60,"context_line":"                          random.randint(1024, 4096), meta)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":63,"context_line":"    def test_set_nonexistent_image_metadata_item(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F%2Fuc%3D","line":60,"updated":"2013-12-25 07:10:55.000000000","message":"ditto","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"f16478ada8ab5481ddd10301be65d58edb7d15ac","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        # Negative test: Metadata should not be set to a non-existent image"},{"line_number":58,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt1\u0027, \u0027key2\u0027: \u0027alt2\u0027}"},{"line_number":59,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.set_image_metadata,"},{"line_number":60,"context_line":"                          random.randint(1024, 4096), meta)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":63,"context_line":"    def test_set_nonexistent_image_metadata_item(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F9y4%3D","line":60,"in_reply_to":"AAAAT3%2F%2F%2Fuc%3D","updated":"2013-12-26 06:39:39.000000000","message":"Done","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e45a2bab9b40eac3c0c811aeb60697e1033b01bf","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt\u0027}"},{"line_number":67,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":68,"context_line":"                          self.client.set_image_metadata_item,"},{"line_number":69,"context_line":"                          random.randint(1024, 4096), \u0027key1\u0027,"},{"line_number":70,"context_line":"                          meta)"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F%2FuY%3D","line":69,"updated":"2013-12-25 07:10:55.000000000","message":"ditto","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"f16478ada8ab5481ddd10301be65d58edb7d15ac","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        meta \u003d {\u0027key1\u0027: \u0027alt\u0027}"},{"line_number":67,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":68,"context_line":"                          self.client.set_image_metadata_item,"},{"line_number":69,"context_line":"                          random.randint(1024, 4096), \u0027key1\u0027,"},{"line_number":70,"context_line":"                          meta)"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F9y0%3D","line":69,"in_reply_to":"AAAAT3%2F%2F%2FuY%3D","updated":"2013-12-26 06:39:39.000000000","message":"Done","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e45a2bab9b40eac3c0c811aeb60697e1033b01bf","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        # item from non-existent image"},{"line_number":76,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":77,"context_line":"                          self.client.delete_image_metadata_item,"},{"line_number":78,"context_line":"                          random.randint(1024, 4096), \u0027key1\u0027)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"class ImagesMetadataTestXML(ImagesMetadataTestJSON):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F%2FuU%3D","line":78,"updated":"2013-12-25 07:10:55.000000000","message":"ditto","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"f16478ada8ab5481ddd10301be65d58edb7d15ac","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        # item from non-existent image"},{"line_number":76,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":77,"context_line":"                          self.client.delete_image_metadata_item,"},{"line_number":78,"context_line":"                          random.randint(1024, 4096), \u0027key1\u0027)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"class ImagesMetadataTestXML(ImagesMetadataTestJSON):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAT3%2F%2F9yw%3D","line":78,"in_reply_to":"AAAAT3%2F%2F%2FuU%3D","updated":"2013-12-26 06:39:39.000000000","message":"Done","commit_id":"a24ed70db720edb4242606d1a36b6e90decdf028"}]}