)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"66fc2c12afead2cf582de14840103d094e42692d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3c2846c1_1680b954","updated":"2023-05-04 14:39:39.000000000","message":"Looks good, Thank you!!","commit_id":"e2190c4feb7b237c55a5d7df49e0db340a9d342f"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"14e518a9409139c9424a935d22862180850f7690","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b754b0eb_d7b463a5","updated":"2023-05-16 15:51:05.000000000","message":"This is a pretty ugly bug that the structure of our unit tests made difficult to detect.  I wonder how many more of these are lurking in the code.","commit_id":"e2190c4feb7b237c55a5d7df49e0db340a9d342f"}],"glanceclient/tests/unit/v2/test_shell_v2.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"14e518a9409139c9424a935d22862180850f7690","unresolved":true,"context_lines":[{"line_number":2240,"context_line":""},{"line_number":2241,"context_line":"    def test_image_import_glance_download(self):"},{"line_number":2242,"context_line":"        args \u003d self._make_args("},{"line_number":2243,"context_line":"            {\u0027id\u0027: \u0027IMG-01\u0027, \u0027uri\u0027: None, \u0027remote-region\u0027: \u0027REGION2\u0027,"},{"line_number":2244,"context_line":"             \u0027remote-image-id\u0027: \u0027IMG-02\u0027,"},{"line_number":2245,"context_line":"             \u0027import_method\u0027: \u0027glance-download\u0027,"},{"line_number":2246,"context_line":"             \u0027remote-service-interface\u0027: \u0027public\u0027})"},{"line_number":2247,"context_line":"        with mock.patch.object(self.gc.images, \u0027image_import\u0027) as mock_import:"},{"line_number":2248,"context_line":"            with mock.patch.object(self.gc.images, \u0027get\u0027) as mocked_get:"},{"line_number":2249,"context_line":"                with mock.patch.object(self.gc.images,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ab76f879_05c4579f","side":"PARENT","line":2246,"range":{"start_line":2243,"start_character":0,"end_line":2246,"end_character":51},"updated":"2023-05-16 15:51:05.000000000","message":"OK, this explains why this test was passing previously.  We created the args that argparse would pass in by hand ourselves right here, and we included the incorrect names that argparse would NOT put in the args object, and thus we had values instead of None at lines 2259-2261.","commit_id":"52fb6b29d84644c690c0b8566117c4bfb963b09b"}]}