)]}'
{"tempest/tests/cmd/test_verify_tempest_config.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"c0c84e85cb57790d01326f0bc9b8cae13b8b9b19","unresolved":false,"context_lines":[{"line_number":220,"context_line":"            verify_tempest_config.verify_cinder_api_versions(fake_os, True)"},{"line_number":221,"context_line":"        print_mock.assert_not_called()"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    @mock.patch(\u0027tempest.lib.common.http.ClosingHttp.request\u0027)"},{"line_number":224,"context_line":"    def test_verify_cinder_api_versions_no_v1(self, mock_request):"},{"line_number":225,"context_line":"        self.useFixture(fixtures.MockPatchObject("},{"line_number":226,"context_line":"            verify_tempest_config, \u0027_get_unversioned_endpoint\u0027,"},{"line_number":227,"context_line":"            return_value\u003d\u0027http://fake_endpoint:5000\u0027))"},{"line_number":228,"context_line":"        fake_resp \u003d {\u0027versions\u0027: [{\u0027id\u0027: \u0027v3.0\u0027}]}"},{"line_number":229,"context_line":"        fake_resp \u003d json.dumps(fake_resp)"},{"line_number":230,"context_line":"        mock_request.return_value \u003d (None, fake_resp)"},{"line_number":231,"context_line":"        fake_os \u003d mock.MagicMock()"},{"line_number":232,"context_line":"        with mock.patch.object(verify_tempest_config,"},{"line_number":233,"context_line":"                               \u0027print_and_or_update\u0027) as print_mock:"},{"line_number":234,"context_line":"            verify_tempest_config.verify_cinder_api_versions(fake_os, True)"},{"line_number":235,"context_line":"        print_mock.assert_not_called()"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_verify_glance_version_no_v2_with_v1_1(self):"},{"line_number":238,"context_line":"        # This test verifies that wrong config api_v2 \u003d True is detected"}],"source_content_type":"text/x-python","patch_set":3,"id":"df87a7cf_56bb5271","line":235,"range":{"start_line":223,"start_character":0,"end_line":235,"end_character":38},"updated":"2017-12-13 02:17:13.000000000","message":"This test now looks identical to the test above it so can be removed.\n\nHowever, why can\u0027t you set CONF.set_default(\u0027api_v2\u0027, True, group\u003d\u0027volume_feature_enabled\u0027) at the beginning of the test? Won\u0027t v2 continue to be supported in Tempest (but won\u0027t be run in the gates by default) so don\u0027t we want to continue to verify the case where v2/v3 are both supported but not v1?","commit_id":"81b27583c377116f6faab8c7eb1d8db527bb085d"}]}
