)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4fefcb43676c98747b69afe0c8c27a2fda5d67df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c31dfc98_57ba9d23","updated":"2022-06-20 14:02:09.000000000","message":"Thanks for fixing something I stupidly wrote.","commit_id":"4cf4af7ff4742cf67d2621560ba296e030d25676"}],"nova/tests/unit/cmd/test_manage.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a66afb214d9d4ff178c4ab5251f755a77edf9036","unresolved":true,"context_lines":[{"line_number":2907,"context_line":"            200, content\u003djsonutils.dumps({\"resource_providers\": rps}))"},{"line_number":2908,"context_line":""},{"line_number":2909,"context_line":"        # we found one orphaned allocation per RP and we had no faults"},{"line_number":2910,"context_line":"        check_orphaned_allocs.side_effect \u003d ((1, 0), (1, 0))"},{"line_number":2911,"context_line":""},{"line_number":2912,"context_line":"        ret \u003d self.cli.audit("},{"line_number":2913,"context_line":"            verbose\u003dTrue, delete\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"24425ebe_a9fedb20","line":2910,"range":{"start_line":2910,"start_character":52,"end_line":2910,"end_character":59},"updated":"2021-07-29 23:34:36.000000000","message":"The second item is not required because this is called only once.","commit_id":"84e18194720d59cfe0ebb7e5b4d621e91d38464d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8b06229d9bf9b3f481e727ede347b39645557c81","unresolved":false,"context_lines":[{"line_number":2907,"context_line":"            200, content\u003djsonutils.dumps({\"resource_providers\": rps}))"},{"line_number":2908,"context_line":""},{"line_number":2909,"context_line":"        # we found one orphaned allocation per RP and we had no faults"},{"line_number":2910,"context_line":"        check_orphaned_allocs.side_effect \u003d ((1, 0), (1, 0))"},{"line_number":2911,"context_line":""},{"line_number":2912,"context_line":"        ret \u003d self.cli.audit("},{"line_number":2913,"context_line":"            verbose\u003dTrue, delete\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"84222812_a94816ae","line":2910,"range":{"start_line":2910,"start_character":52,"end_line":2910,"end_character":59},"in_reply_to":"24425ebe_a9fedb20","updated":"2022-03-22 12:58:23.000000000","message":"Done","commit_id":"84e18194720d59cfe0ebb7e5b4d621e91d38464d"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e8892429a6a808060cd67bbb673b5bd7f5b4acde","unresolved":true,"context_lines":[{"line_number":2917,"context_line":"        self.assertEqual(3, ret)"},{"line_number":2918,"context_line":""},{"line_number":2919,"context_line":"        get_resource_providers.assert_called_once_with("},{"line_number":2920,"context_line":"            f\u0027/resource_providers?uuid\u003d{uuidsentinel.fake_uuid}\u0027,"},{"line_number":2921,"context_line":"            global_request_id\u003dmock.ANY,"},{"line_number":2922,"context_line":"            version\u003d\u00271.14\u0027)"},{"line_number":2923,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"40c15ed5_1c3af694","line":2920,"range":{"start_line":2920,"start_character":10,"end_line":2920,"end_character":65},"updated":"2022-06-20 09:38:40.000000000","message":"so this is where you\u0027re actully asserting the fixed behaviour\n\nhttps://review.opendev.org/c/openstack/nova/+/800855/1/nova/cmd/manage.py#2565\n\ni.e. that we use a \u0027?\u0027 not \u0027\u0026\u0027 as the UUID is the first/only query arg, not a subsequent one.\n\nthe rest is just the scaffolding to allow you to test that.","commit_id":"4cf4af7ff4742cf67d2621560ba296e030d25676"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e8892429a6a808060cd67bbb673b5bd7f5b4acde","unresolved":true,"context_lines":[{"line_number":2934,"context_line":"            self.cli, \u0027_get_resource_provider\u0027,"},{"line_number":2935,"context_line":"            side_effect\u003dexception.ResourceProviderNotFound("},{"line_number":2936,"context_line":"                name_or_uuid\u003duuidsentinel.fake_uuid),"},{"line_number":2937,"context_line":"        ):"},{"line_number":2938,"context_line":"            ret \u003d self.cli.audit("},{"line_number":2939,"context_line":"                provider_uuid\u003duuidsentinel.fake_uuid)"},{"line_number":2940,"context_line":"        self.assertEqual(127, ret)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8bc82701_bc388d6d","line":2937,"updated":"2022-06-20 09:38:40.000000000","message":"nit unrelated but im ok with this its more readable\n\nim guess ing you just copy pasted the test and updated it to write the new one which is what i would have done too.","commit_id":"4cf4af7ff4742cf67d2621560ba296e030d25676"}]}