)]}'
{"tempest/cmd/verify_tempest_config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"a831895c865147882ca874d4712d3807246f3306","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            continue"},{"line_number":351,"context_line":"        results \u003d verify_extensions(os, service, results)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"    for service in services + [\u0027keystone\u0027]:"},{"line_number":354,"context_line":"        verify_api_versions(os, service, update)"},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"    display_results(results, update, replace)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_69cde2b6","line":353,"updated":"2014-10-07 20:01:40.000000000","message":"Not the biggest fan of how this looks. But, it works fine so whatever.","commit_id":"bc0f9e3f7fdcdd72ca085847388c2b29185b9807"}],"tempest/tests/cmd/test_verify_tempest_config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"06dd421c6e1cfd835c42f4a43b49e837559a4654","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            m \u003d \u0027verify_%s_api_versions\u0027 % svc"},{"line_number":103,"context_line":"            with mock.patch.object(verify_tempest_config, m) as verify_mock:"},{"line_number":104,"context_line":"                verify_tempest_config.verify_api_versions(fake_os, \u0027foo\u0027, True)"},{"line_number":105,"context_line":"                verify_mock.assert_not_called()"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    def test_verify_keystone_api_versions_no_v3(self):"},{"line_number":108,"context_line":"        self.useFixture(mockpatch.PatchObject("}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_d0d90a58","line":105,"updated":"2014-10-07 22:35:48.000000000","message":"I don\u0027t think this is a real method, I remember reading this was a common mistake. This doesn\u0027t error out because MagicMocks are magic.\n\n\u003e\u003e\u003e test \u003d mock.MagicMock()\n\n\u003e\u003e\u003e print dir(test)\n[\u0027assert_any_call\u0027, \u0027assert_called_once_with\u0027, \u0027assert_called_with\u0027, \u0027assert_has_calls\u0027, \u0027attach_mock\u0027, \u0027call_args\u0027, \u0027call_args_list\u0027, \u0027call_count\u0027, \u0027called\u0027, \u0027configure_mock\u0027, \u0027method_calls\u0027, \u0027mock_add_spec\u0027, \u0027mock_calls\u0027, \u0027reset_mock\u0027, \u0027return_value\u0027, \u0027side_effect\u0027]\n\nI normally work around this by just asserting the call_count is 0. Although there is a called attr, that might just be a bool you can assertFalse.","commit_id":"89f1dfe7b5e593fd6a121a784075dfea7bae5599"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"299e0c987a2a0891d9c1de02afc6566cd0a193dd","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            m \u003d \u0027verify_%s_api_versions\u0027 % svc"},{"line_number":103,"context_line":"            with mock.patch.object(verify_tempest_config, m) as verify_mock:"},{"line_number":104,"context_line":"                verify_tempest_config.verify_api_versions(fake_os, \u0027foo\u0027, True)"},{"line_number":105,"context_line":"                self.assertFalse(verify_mock.called)"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    def test_verify_keystone_api_versions_no_v3(self):"},{"line_number":108,"context_line":"        self.useFixture(mockpatch.PatchObject("}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_5324440c","line":105,"updated":"2014-10-07 23:58:35.000000000","message":"A failure msg here might be helpful, but I don\u0027t think it\u0027s a big deal.","commit_id":"03af556aedd3c7c24812cb5c456c423083115a05"}]}