)]}'
{"tempest/api/network/test_extensions.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"aa6ce13d5fb1af9ae38df0c67ac15513e19f7ab5","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        resp, extensions \u003d self.client.list_extensions()"},{"line_number":49,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":50,"context_line":"        list_extensions \u003d extensions[\u0027extensions\u0027]"},{"line_number":51,"context_line":"        self.assertEqual(len(expected_alias), len(list_extensions))"},{"line_number":52,"context_line":"        for e in list_extensions:"},{"line_number":53,"context_line":"            self.assertIn(e[\u0027alias\u0027], expected_alias)"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FL8U%3D","line":51,"updated":"2013-09-10 12:00:34.000000000","message":"I have doubts we really want verify the len here.\n\nIMHO additional extension should be allowed to appear without a tempest change.","commit_id":"f659c7af7f38f453fec299d8e4f559d9a228c14b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1666c6b93420badb137a1818f2af8a2de648a6af","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    _interface \u003d \u0027json\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    \"\"\""},{"line_number":27,"context_line":"    Tests the following operations in the Quantum API using the REST client for"},{"line_number":28,"context_line":"    Quantum:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        List all available extensions"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F%2Fww%3D","line":27,"updated":"2013-09-15 02:25:25.000000000","message":"s/Quantum/Neutron/\n\nor\n\ns/Quantum/Network/","commit_id":"63de1261c6b630abd01c302fb7e4fe00743b1b82"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1666c6b93420badb137a1818f2af8a2de648a6af","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    \"\"\""},{"line_number":27,"context_line":"    Tests the following operations in the Quantum API using the REST client for"},{"line_number":28,"context_line":"    Quantum:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        List all available extensions"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F%2Fwk%3D","line":28,"updated":"2013-09-15 02:25:25.000000000","message":"ditto","commit_id":"63de1261c6b630abd01c302fb7e4fe00743b1b82"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1666c6b93420badb137a1818f2af8a2de648a6af","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        List all available extensions"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    v2.0 of the Quantum API is assumed. It is also assumed that the following"},{"line_number":33,"context_line":"    options are defined in the [network] section of etc/tempest.conf:"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F%2Fwg%3D","line":32,"updated":"2013-09-15 02:25:25.000000000","message":"ditto","commit_id":"63de1261c6b630abd01c302fb7e4fe00743b1b82"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"c9f2b567671854d62b5676182af4c2dea06c51fc","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def setUpClass(cls):"},{"line_number":39,"context_line":"        super(ExtensionsTestJSON, cls).setUpClass()"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":42,"context_line":"    def test_list_show_extensions(self):"},{"line_number":43,"context_line":"        # List available extensions for the tenant"},{"line_number":44,"context_line":"        expected_alias \u003d [\u0027security-group\u0027, \u0027l3_agent_scheduler\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAS3%2F%2F%2BqM%3D","line":41,"updated":"2013-09-16 07:53:56.000000000","message":"Type should be changed to \"smoke\"","commit_id":"b521fd2cb8e11c9f57482823c59ca95dee30af29"},{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"001b2f1b8f1fc2a0153ad6b7a8d37bf50e238f01","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        # Verify if expected extensions are present in the actual list"},{"line_number":73,"context_line":"        # of extensions returned"},{"line_number":74,"context_line":"        for e in expected_alias:"},{"line_number":75,"context_line":"            self.assertIn(e, actual_alias)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"class ExtensionsTestXML(ExtensionsTestJSON):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2F7SQ%3D","line":75,"updated":"2013-09-17 04:51:23.000000000","message":"Maybe, we should verify if actual extensions are present in the expected list too.","commit_id":"1dd3cb35dd9164f2959a5f8afc83005ae1d6a1f9"},{"author":{"_account_id":2238,"name":"rajalakshmi-ganesan","email":"rajalakshmi.girish@hpe.com","username":"rajalakshmi-ganesan"},"change_message_id":"d48a667ce61a5f3d81535177a74c599710ed99bb","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        # Verify if expected extensions are present in the actual list"},{"line_number":73,"context_line":"        # of extensions returned"},{"line_number":74,"context_line":"        for e in expected_alias:"},{"line_number":75,"context_line":"            self.assertIn(e, actual_alias)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"class ExtensionsTestXML(ExtensionsTestJSON):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2F6kE%3D","line":75,"in_reply_to":"AAAAS3%2F%2F7SQ%3D","updated":"2013-09-17 09:07:25.000000000","message":"Hi YangLei\t,\n\nIn patch set-3 the logic was as u mentioned (ie, verifying if all actual extensions are present as per the expected list of extensions). But according to the following comment of Afazekas, I changed the logic:\n\n\"IMHO additional extension should be allowed to appear without a tempest change\"","commit_id":"1dd3cb35dd9164f2959a5f8afc83005ae1d6a1f9"}]}