)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Negative tests for testing chassis-create command"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Negative tests for the Ironic CLI commands which checks actions with"},{"line_number":10,"context_line":"chassis-create command like create chassis without arguments or with"},{"line_number":11,"context_line":"incorrect arguments and check that correct error message rised"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7aa08908_ec288035","line":9,"range":{"start_line":9,"start_character":49,"end_line":9,"end_character":55},"updated":"2016-06-08 07:32:58.000000000","message":"nit: s/checks/check/","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Negative tests for the Ironic CLI commands which checks actions with"},{"line_number":10,"context_line":"chassis-create command like create chassis without arguments or with"},{"line_number":11,"context_line":"incorrect arguments and check that correct error message rised"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I01541c2cefafc4612b804646da07bf8dd5884661"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7aa08908_aca798bb","line":11,"range":{"start_line":11,"start_character":57,"end_line":11,"end_character":62},"updated":"2016-06-08 07:32:58.000000000","message":"nit: s/rised/raised/","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"}],"ironicclient/tests/functional/test_chassis_create.py":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        2) check that chassis-create command without description using -d"},{"line_number":141,"context_line":"        argument triggers an exception"},{"line_number":142,"context_line":"        \"\"\""},{"line_number":143,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":144,"context_line":"            \u0027expected one argument\u0027"},{"line_number":145,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":146,"context_line":"                                self.create_chassis, \u0027-d\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_2c6728ea","line":143,"range":{"start_line":143,"start_character":77,"end_line":143,"end_character":79},"updated":"2016-06-08 07:32:58.000000000","message":"It is preferred to wrap long lines in parentheses and not a backslash for line continuation.\n\nOpenStack Style Guidelines\nhttp://docs.openstack.org/developer/hacking/","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        2) check that chassis-create command without description using -d"},{"line_number":141,"context_line":"        argument triggers an exception"},{"line_number":142,"context_line":"        \"\"\""},{"line_number":143,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":144,"context_line":"            \u0027expected one argument\u0027"},{"line_number":145,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":146,"context_line":"                                self.create_chassis, \u0027-d\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_dc7a8087","line":143,"range":{"start_line":143,"start_character":77,"end_line":143,"end_character":79},"in_reply_to":"7aa08908_2c6728ea","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        2) check that chassis-create command without description using"},{"line_number":153,"context_line":"        --description arguments triggers an exception"},{"line_number":154,"context_line":"        \"\"\""},{"line_number":155,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":156,"context_line":"            \u0027expected one argument\u0027"},{"line_number":157,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":158,"context_line":"                                self.create_chassis, \u0027--description\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_eca7e008","line":155,"range":{"start_line":155,"start_character":77,"end_line":155,"end_character":79},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        2) check that chassis-create command without description using"},{"line_number":153,"context_line":"        --description arguments triggers an exception"},{"line_number":154,"context_line":"        \"\"\""},{"line_number":155,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":156,"context_line":"            \u0027expected one argument\u0027"},{"line_number":157,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":158,"context_line":"                                self.create_chassis, \u0027--description\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_9c46384d","line":155,"range":{"start_line":155,"start_character":77,"end_line":155,"end_character":79},"in_reply_to":"7aa08908_eca7e008","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        2) check that chassis-create command without metadata using -e"},{"line_number":165,"context_line":"        arguments triggers an exception"},{"line_number":166,"context_line":"        \"\"\""},{"line_number":167,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -e/--extra: \u0027\\"},{"line_number":168,"context_line":"            \u0027expected one argument\u0027"},{"line_number":169,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":170,"context_line":"                                self.create_chassis, \u0027-e\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_4cb1d446","line":167,"range":{"start_line":167,"start_character":72,"end_line":167,"end_character":73},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        2) check that chassis-create command without metadata using -e"},{"line_number":165,"context_line":"        arguments triggers an exception"},{"line_number":166,"context_line":"        \"\"\""},{"line_number":167,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -e/--extra: \u0027\\"},{"line_number":168,"context_line":"            \u0027expected one argument\u0027"},{"line_number":169,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":170,"context_line":"                                self.create_chassis, \u0027-e\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_5c405034","line":167,"range":{"start_line":167,"start_character":72,"end_line":167,"end_character":73},"in_reply_to":"7aa08908_4cb1d446","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":176,"context_line":"        2) check that chassis-create command without metadata using --extra"},{"line_number":177,"context_line":"        arguments triggers an exception"},{"line_number":178,"context_line":"        \"\"\""},{"line_number":179,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -e/--extra: \u0027\\"},{"line_number":180,"context_line":"            \u0027expected one argument\u0027"},{"line_number":181,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":182,"context_line":"                                self.create_chassis, \u0027--extra\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_cc830455","line":179,"range":{"start_line":179,"start_character":71,"end_line":179,"end_character":73},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":176,"context_line":"        2) check that chassis-create command without metadata using --extra"},{"line_number":177,"context_line":"        arguments triggers an exception"},{"line_number":178,"context_line":"        \"\"\""},{"line_number":179,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -e/--extra: \u0027\\"},{"line_number":180,"context_line":"            \u0027expected one argument\u0027"},{"line_number":181,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":182,"context_line":"                                self.create_chassis, \u0027--extra\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_fc4e5c6d","line":179,"range":{"start_line":179,"start_character":71,"end_line":179,"end_character":73},"in_reply_to":"7aa08908_cc830455","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":188,"context_line":"        2) check that chassis-create command without specific using -u"},{"line_number":189,"context_line":"        arguments triggers an exception"},{"line_number":190,"context_line":"        \"\"\""},{"line_number":191,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027\\"},{"line_number":192,"context_line":"            \u0027expected one argument\u0027"},{"line_number":193,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":194,"context_line":"                                self.create_chassis, \u0027-u\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_ec7ec068","line":191,"range":{"start_line":191,"start_character":70,"end_line":191,"end_character":72},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":188,"context_line":"        2) check that chassis-create command without specific using -u"},{"line_number":189,"context_line":"        arguments triggers an exception"},{"line_number":190,"context_line":"        \"\"\""},{"line_number":191,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027\\"},{"line_number":192,"context_line":"            \u0027expected one argument\u0027"},{"line_number":193,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":194,"context_line":"                                self.create_chassis, \u0027-u\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_9c2f5887","line":191,"range":{"start_line":191,"start_character":70,"end_line":191,"end_character":72},"in_reply_to":"7aa08908_ec7ec068","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        2) check that chassis-create command without specific using -uuid"},{"line_number":201,"context_line":"        arguments triggers an exception"},{"line_number":202,"context_line":"        \"\"\""},{"line_number":203,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027\\"},{"line_number":204,"context_line":"            \u0027expected one argument\u0027"},{"line_number":205,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":206,"context_line":"                                self.create_chassis, \u0027--uuid\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_0cf18c01","line":203,"range":{"start_line":203,"start_character":71,"end_line":203,"end_character":72},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        2) check that chassis-create command without specific using -uuid"},{"line_number":201,"context_line":"        arguments triggers an exception"},{"line_number":202,"context_line":"        \"\"\""},{"line_number":203,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027\\"},{"line_number":204,"context_line":"            \u0027expected one argument\u0027"},{"line_number":205,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":206,"context_line":"                                self.create_chassis, \u0027--uuid\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_bc3214f0","line":203,"range":{"start_line":203,"start_character":71,"end_line":203,"end_character":72},"in_reply_to":"7aa08908_0cf18c01","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":213,"context_line":"        using -d argument triggers an exception"},{"line_number":214,"context_line":"        \"\"\""},{"line_number":215,"context_line":"        description \u003d \u0027--\u0027"},{"line_number":216,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":217,"context_line":"            \u0027expected one argument\u0027"},{"line_number":218,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":219,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_2cdc4872","line":216,"range":{"start_line":216,"start_character":77,"end_line":216,"end_character":79},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":213,"context_line":"        using -d argument triggers an exception"},{"line_number":214,"context_line":"        \"\"\""},{"line_number":215,"context_line":"        description \u003d \u0027--\u0027"},{"line_number":216,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":217,"context_line":"            \u0027expected one argument\u0027"},{"line_number":218,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":219,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_7c3c2cc8","line":216,"range":{"start_line":216,"start_character":77,"end_line":216,"end_character":79},"in_reply_to":"7aa08908_2cdc4872","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        using --description argument triggers an exception"},{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        description \u003d \u0027--\u0027"},{"line_number":231,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":232,"context_line":"            \u0027expected one argument\u0027"},{"line_number":233,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":234,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_6ce2d030","line":231,"range":{"start_line":231,"start_character":77,"end_line":231,"end_character":79},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        using --description argument triggers an exception"},{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        description \u003d \u0027--\u0027"},{"line_number":231,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: argument -d/--description: \u0027\\"},{"line_number":232,"context_line":"            \u0027expected one argument\u0027"},{"line_number":233,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":234,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_3c1e245f","line":231,"range":{"start_line":231,"start_character":77,"end_line":231,"end_character":79},"in_reply_to":"7aa08908_6ce2d030","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        using -e arguments triggers an exception"},{"line_number":243,"context_line":"        \"\"\""},{"line_number":244,"context_line":"        extra \u003d \"HelloWorld\""},{"line_number":245,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: \u0027\\"},{"line_number":246,"context_line":"            \u0027Attributes must be a list of PATH\u003dVALUE\u0027"},{"line_number":247,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":248,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_2cc3a88a","line":245,"range":{"start_line":245,"start_character":50,"end_line":245,"end_character":52},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        using -e arguments triggers an exception"},{"line_number":243,"context_line":"        \"\"\""},{"line_number":244,"context_line":"        extra \u003d \"HelloWorld\""},{"line_number":245,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: \u0027\\"},{"line_number":246,"context_line":"            \u0027Attributes must be a list of PATH\u003dVALUE\u0027"},{"line_number":247,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":248,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_dc2c409b","line":245,"range":{"start_line":245,"start_character":50,"end_line":245,"end_character":52},"in_reply_to":"7aa08908_2cc3a88a","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e77940f6b90e8a580cbd7d8f739519f4f99a4f08","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        using --extra arguments triggers an exception"},{"line_number":257,"context_line":"        \"\"\""},{"line_number":258,"context_line":"        extra \u003d \"HelloWorld\""},{"line_number":259,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: \u0027\\"},{"line_number":260,"context_line":"            \u0027Attributes must be a list of PATH\u003dVALUE\u0027"},{"line_number":261,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":262,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_6cb93015","line":259,"range":{"start_line":259,"start_character":50,"end_line":259,"end_character":52},"updated":"2016-06-08 07:32:58.000000000","message":"ditto","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c095c353be58f9fe81ca2d7e18881b97cec87949","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        using --extra arguments triggers an exception"},{"line_number":257,"context_line":"        \"\"\""},{"line_number":258,"context_line":"        extra \u003d \"HelloWorld\""},{"line_number":259,"context_line":"        ex_text \u003d r\u0027ironic chassis-create: error: \u0027\\"},{"line_number":260,"context_line":"            \u0027Attributes must be a list of PATH\u003dVALUE\u0027"},{"line_number":261,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":262,"context_line":"                                self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_fc277cb3","line":259,"range":{"start_line":259,"start_character":50,"end_line":259,"end_character":52},"in_reply_to":"7aa08908_6cb93015","updated":"2016-06-08 11:31:11.000000000","message":"Done","commit_id":"6513e8d4d5f6a1247f4473818c7337d5f42980cb"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"3a777de42910e514b85de098012cb11ecae86894","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"# under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from ironicclient.tests.functional import base"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":18,"context_line":"from tempest.lib import exceptions"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class NegativeChassisCreateTestsIronicClient(base.FunctionalTestBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_1aa56cb4","line":18,"range":{"start_line":15,"start_character":0,"end_line":18,"end_character":34},"updated":"2016-06-23 13:38:20.000000000","message":"wrong import order, 3d party lib first","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"22410390e5f312d0878ce3116e91c25ff2f21b5c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"# under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from ironicclient.tests.functional import base"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":18,"context_line":"from tempest.lib import exceptions"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class NegativeChassisCreateTestsIronicClient(base.FunctionalTestBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_63dc8b73","line":18,"range":{"start_line":15,"start_character":0,"end_line":18,"end_character":34},"in_reply_to":"3aaa91ec_1aa56cb4","updated":"2016-06-24 14:48:49.000000000","message":"Done","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"3a777de42910e514b85de098012cb11ecae86894","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \"\"\""},{"line_number":36,"context_line":"        ex_text \u003d (r\u0027ironic chassis-create: error: argument -d/--description: \u0027"},{"line_number":37,"context_line":"                   \u0027expected one argument\u0027)"},{"line_number":38,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":39,"context_line":"                                self.create_chassis, \u0027-d\u0027)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_chassis_create_without_description_description_arg(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_7a83d004","line":38,"range":{"start_line":38,"start_character":13,"end_line":38,"end_character":31},"updated":"2016-06-23 13:38:20.000000000","message":"same as here https://review.openstack.org/#/c/301757/","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"22410390e5f312d0878ce3116e91c25ff2f21b5c","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \"\"\""},{"line_number":36,"context_line":"        ex_text \u003d (r\u0027ironic chassis-create: error: argument -d/--description: \u0027"},{"line_number":37,"context_line":"                   \u0027expected one argument\u0027)"},{"line_number":38,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":39,"context_line":"                                self.create_chassis, \u0027-d\u0027)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_chassis_create_without_description_description_arg(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_7ec6e09a","line":38,"range":{"start_line":38,"start_character":13,"end_line":38,"end_character":31},"in_reply_to":"3aaa91ec_7a83d004","updated":"2016-06-24 14:48:49.000000000","message":"Done","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"3a777de42910e514b85de098012cb11ecae86894","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        2) check that chassis-create command without specific using -u"},{"line_number":82,"context_line":"        arguments triggers an exception"},{"line_number":83,"context_line":"        \"\"\""},{"line_number":84,"context_line":"        ex_text \u003d (r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027"},{"line_number":85,"context_line":"                   \u0027expected one argument\u0027)"},{"line_number":86,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":87,"context_line":"                                self.create_chassis, \u0027-u\u0027)"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_9a215cb7","line":85,"range":{"start_line":84,"start_character":20,"end_line":85,"end_character":42},"updated":"2016-06-23 13:38:20.000000000","message":"nit: a lot of test cases with equal template, you can define it one time and and substitute expected arg","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"a5b475757f66045a49b9dff5d27a560bada60302","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        2) check that chassis-create command without specific using -u"},{"line_number":82,"context_line":"        arguments triggers an exception"},{"line_number":83,"context_line":"        \"\"\""},{"line_number":84,"context_line":"        ex_text \u003d (r\u0027ironic chassis-create: error: argument -u/--uuid: \u0027"},{"line_number":85,"context_line":"                   \u0027expected one argument\u0027)"},{"line_number":86,"context_line":"        self.assertRaisesRegexp(exceptions.CommandFailed, ex_text,"},{"line_number":87,"context_line":"                                self.create_chassis, \u0027-u\u0027)"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_793d622a","line":85,"range":{"start_line":84,"start_character":20,"end_line":85,"end_character":42},"in_reply_to":"3aaa91ec_9a215cb7","updated":"2016-06-30 10:55:04.000000000","message":"Done","commit_id":"e5a8e97965341db92958d5394270dc089e6b5a03"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"f7554154902a01c0d0e1bc5a068b0fe00380b93f","unresolved":false,"context_lines":[{"line_number":167,"context_line":"        using -u arguments triggers an exception"},{"line_number":168,"context_line":"        \"\"\""},{"line_number":169,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received\u0027"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"},{"line_number":173,"context_line":"                              params\u003d\u0027-u {0}\u0027.format(*uuid))"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_d851be0f","line":170,"range":{"start_line":170,"start_character":18,"end_line":170,"end_character":49},"updated":"2016-07-01 07:01:01.000000000","message":"It would be better to verify that the wrong \"uuid\" is included too.\n\nhttps://github.com/openstack/ironic/blob/master/ironic/common/exception.py#L175-L176\n\nclass InvalidUUID(Invalid):\n  _msg_fmt \u003d _(\"Expected a uuid but received %(uuid)s.\")","commit_id":"73543bebbaf0107ad910519e1af262baabe98249"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"90f8b8d7b62aba111f11a4f0257d343a59ff0bfb","unresolved":false,"context_lines":[{"line_number":167,"context_line":"        using -u arguments triggers an exception"},{"line_number":168,"context_line":"        \"\"\""},{"line_number":169,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received\u0027"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"},{"line_number":173,"context_line":"                              params\u003d\u0027-u {0}\u0027.format(*uuid))"}],"source_content_type":"text/x-python","patch_set":8,"id":"1aa78d24_6cdcf7e4","line":170,"range":{"start_line":170,"start_character":18,"end_line":170,"end_character":49},"in_reply_to":"3aaa91ec_d851be0f","updated":"2016-07-04 20:53:09.000000000","message":"Naohiro, thank you for good example!","commit_id":"73543bebbaf0107ad910519e1af262baabe98249"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"f7554154902a01c0d0e1bc5a068b0fe00380b93f","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        using --uuid arguments triggers an exception"},{"line_number":181,"context_line":"        \"\"\""},{"line_number":182,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":183,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received\u0027"},{"line_number":184,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":185,"context_line":"                              self.create_chassis,"},{"line_number":186,"context_line":"                              params\u003d\u0027--uuid {0}\u0027.format(*uuid))"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_78436a42","line":183,"range":{"start_line":183,"start_character":18,"end_line":183,"end_character":49},"updated":"2016-07-01 07:01:01.000000000","message":"ditto","commit_id":"73543bebbaf0107ad910519e1af262baabe98249"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"90f8b8d7b62aba111f11a4f0257d343a59ff0bfb","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        using --uuid arguments triggers an exception"},{"line_number":181,"context_line":"        \"\"\""},{"line_number":182,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":183,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received\u0027"},{"line_number":184,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":185,"context_line":"                              self.create_chassis,"},{"line_number":186,"context_line":"                              params\u003d\u0027--uuid {0}\u0027.format(*uuid))"}],"source_content_type":"text/x-python","patch_set":8,"id":"1aa78d24_4cac9b35","line":183,"range":{"start_line":183,"start_character":18,"end_line":183,"end_character":49},"in_reply_to":"3aaa91ec_78436a42","updated":"2016-07-04 20:53:09.000000000","message":"Done","commit_id":"73543bebbaf0107ad910519e1af262baabe98249"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2f37851eeaf3639de667c82beb725125702225bd","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        2) check that chassis-create command with incorrect specific"},{"line_number":167,"context_line":"        using -u arguments triggers an exception"},{"line_number":168,"context_line":"        \"\"\""},{"line_number":169,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_30aa2271","line":169,"updated":"2016-07-05 13:19:03.000000000","message":"from oslo_utils import uuidutils\n\nuuidutils.generate_uuid()","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"f1ab2eb34069d1f63b79eea110c3810eca81d5d8","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        2) check that chassis-create command with incorrect specific"},{"line_number":167,"context_line":"        using -u arguments triggers an exception"},{"line_number":168,"context_line":"        \"\"\""},{"line_number":169,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_8a060c1b","line":169,"in_reply_to":"1aa78d24_30aa2271","updated":"2016-07-06 11:22:13.000000000","message":"Hello, Lucas. We have already merged two patches of tests which use rand_uuid() from tempest. Actually uuid-generating code there is the same as in oslo_utils.","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6936ae0fbb9bed3f2584e097073573597182463c","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        2) check that chassis-create command with incorrect specific"},{"line_number":167,"context_line":"        using -u arguments triggers an exception"},{"line_number":168,"context_line":"        \"\"\""},{"line_number":169,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_f364e422","line":169,"in_reply_to":"1aa78d24_8a060c1b","updated":"2016-07-06 13:52:27.000000000","message":"Right, it\u0027s more about being consistent with the ironcclient code base and having only 1 way to generate uuids for the tests.","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2f37851eeaf3639de667c82beb725125702225bd","unresolved":false,"context_lines":[{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"},{"line_number":173,"context_line":"                              params\u003d\u0027-u {0}\u0027.format(*uuid))"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    def test_chassis_create_with_incorrect_specific_uuid_full_arg(self):"},{"line_number":176,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_b010d2a9","line":173,"updated":"2016-07-05 13:19:03.000000000","message":"The \"*\" prefixing the uuid variable is not needed\n\nhttp://logs.openstack.org/88/322788/9/check/gate-ironicclient-dsvm-functional/2dcb231/console.html#_2016-07-04_21_09_42_837639","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"f1ab2eb34069d1f63b79eea110c3810eca81d5d8","unresolved":false,"context_lines":[{"line_number":170,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":171,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":172,"context_line":"                              self.create_chassis,"},{"line_number":173,"context_line":"                              params\u003d\u0027-u {0}\u0027.format(*uuid))"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    def test_chassis_create_with_incorrect_specific_uuid_full_arg(self):"},{"line_number":176,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_6a1c806a","line":173,"in_reply_to":"1aa78d24_b010d2a9","updated":"2016-07-06 11:22:13.000000000","message":"Done","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2f37851eeaf3639de667c82beb725125702225bd","unresolved":false,"context_lines":[{"line_number":179,"context_line":"        2) check that chassis-create command with incorrect specific"},{"line_number":180,"context_line":"        using --uuid arguments triggers an exception"},{"line_number":181,"context_line":"        \"\"\""},{"line_number":182,"context_line":"        uuid \u003d data_utils.rand_uuid()"},{"line_number":183,"context_line":"        ex_text \u003d r\u0027Expected a uuid but received {0}\u0027.format(uuid)"},{"line_number":184,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":185,"context_line":"                              self.create_chassis,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1aa78d24_10cc4669","line":182,"updated":"2016-07-05 13:19:03.000000000","message":"ditto","commit_id":"3e6f4865ce246d9ab17ba10bfdcc43410a0d78dd"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"efed0fad5ea9e6abb9f2d733fcb419debcc51b09","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    error_msg \u003d r\u0027ironic chassis-create: error:\u0027"},{"line_number":31,"context_line":"    expected_msg \u003d r\u0027expected one argument\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def test_chassis_create_without_description_short_arg(self):"},{"line_number":34,"context_line":"        \"\"\"Test steps:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        1) create chassis without description using -d argument"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_bc984d6b","line":33,"updated":"2016-07-06 23:45:07.000000000","message":"these test names are quite long. Have you thought about shortening them, eg, all this is testing chassis create (mentioned in the class), so it should be ok for the test methods to be something like \u0027test_description_short_arg_no_value()\u0027.","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"733050fc762e72191d130ba0e328f84b4391ab51","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    error_msg \u003d r\u0027ironic chassis-create: error:\u0027"},{"line_number":31,"context_line":"    expected_msg \u003d r\u0027expected one argument\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def test_chassis_create_without_description_short_arg(self):"},{"line_number":34,"context_line":"        \"\"\"Test steps:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        1) create chassis without description using -d argument"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_ceb04d8e","line":33,"in_reply_to":"1aa78d24_bc984d6b","updated":"2016-07-11 14:48:56.000000000","message":"Thanks for hint, i have shortened names of tests.","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"efed0fad5ea9e6abb9f2d733fcb419debcc51b09","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    def test_chassis_create_without_description_short_arg(self):"},{"line_number":34,"context_line":"        \"\"\"Test steps:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        1) create chassis without description using -d argument"},{"line_number":37,"context_line":"        2) check that chassis-create command without description using -d"},{"line_number":38,"context_line":"        argument triggers an exception"},{"line_number":39,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_1cbfe1aa","line":36,"updated":"2016-07-06 23:45:07.000000000","message":"might just be me, but I think of this test as \u0027create chassis using -d argument without the value\u0027. (Don\u0027t change this if you don\u0027t want, took me a few seconds to understand what you were doing)","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"733050fc762e72191d130ba0e328f84b4391ab51","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    def test_chassis_create_without_description_short_arg(self):"},{"line_number":34,"context_line":"        \"\"\"Test steps:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        1) create chassis without description using -d argument"},{"line_number":37,"context_line":"        2) check that chassis-create command without description using -d"},{"line_number":38,"context_line":"        argument triggers an exception"},{"line_number":39,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_aeab411d","line":36,"in_reply_to":"1aa78d24_1cbfe1aa","updated":"2016-07-11 14:48:56.000000000","message":"Done","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"efed0fad5ea9e6abb9f2d733fcb419debcc51b09","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        ex_text \u003d (r\u0027{0} argument -e/--extra: {1}\u0027"},{"line_number":77,"context_line":"                   .format(self.error_msg, self.expected_msg))"},{"line_number":78,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":79,"context_line":"                              self.create_chassis, \u0027--extra\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    def test_chassis_create_specific_uuid_without_short_argument(self):"},{"line_number":82,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_3cc8dd73","line":79,"updated":"2016-07-06 23:45:07.000000000","message":"why not just combine this with the test above? Everything is the same except for using -e vs --extra. Seems excessive to separate them into two different tests.","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"733050fc762e72191d130ba0e328f84b4391ab51","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        ex_text \u003d (r\u0027{0} argument -e/--extra: {1}\u0027"},{"line_number":77,"context_line":"                   .format(self.error_msg, self.expected_msg))"},{"line_number":78,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":79,"context_line":"                              self.create_chassis, \u0027--extra\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    def test_chassis_create_specific_uuid_without_short_argument(self):"},{"line_number":82,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_6e7039dd","line":79,"in_reply_to":"1aa78d24_3cc8dd73","updated":"2016-07-11 14:48:56.000000000","message":"Thanks for good idea! Done.","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"efed0fad5ea9e6abb9f2d733fcb419debcc51b09","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        ex_text \u003d (r\u0027{0} argument -u/--uuid: {1}\u0027"},{"line_number":101,"context_line":"                   .format(self.error_msg, self.expected_msg))"},{"line_number":102,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":103,"context_line":"                              self.create_chassis, \u0027--uuid\u0027)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    def test_chassis_create_with_incorrect_description_short_arg(self):"},{"line_number":106,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_bcaa0d83","line":103,"updated":"2016-07-06 23:45:07.000000000","message":"ditto","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"733050fc762e72191d130ba0e328f84b4391ab51","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        ex_text \u003d (r\u0027{0} argument -u/--uuid: {1}\u0027"},{"line_number":101,"context_line":"                   .format(self.error_msg, self.expected_msg))"},{"line_number":102,"context_line":"        six.assertRaisesRegex(self, exceptions.CommandFailed, ex_text,"},{"line_number":103,"context_line":"                              self.create_chassis, \u0027--uuid\u0027)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    def test_chassis_create_with_incorrect_description_short_arg(self):"},{"line_number":106,"context_line":"        \"\"\"Test steps:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_0e825594","line":103,"in_reply_to":"1aa78d24_bcaa0d83","updated":"2016-07-11 14:48:56.000000000","message":"Done","commit_id":"cbbc332dc4ebd8b3e7c92ff0a4bed7647cb95587"}]}