)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"d62254b7908e8cd622ddaeb9fdb2e194adc566c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"198ecd8d_fbd7026d","updated":"2026-08-11 09:14:14.000000000","message":"The test itself is fine, but we might want to reduce duplicate code? Please see my inline comment","commit_id":"f91356b3fbdbc7866319545589f5cf7e9b3fb14d"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"5f7d16923bbbe20313a6bf7a0ee2750028e33a6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"734bdb8c_b0ab7fbe","updated":"2026-08-11 09:31:58.000000000","message":"LGTM, thx!","commit_id":"3c92bd107445c943609c3038d603705845a68406"}],"test/unit/common/middleware/test_object_versioning.py":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"d62254b7908e8cd622ddaeb9fdb2e194adc566c6","unresolved":true,"context_lines":[{"line_number":2436,"context_line":"            [(\u0027OPTIONS\u0027, \u0027/v1/a/c/o?version-id\u003d0000000060.00000\u0027)],"},{"line_number":2437,"context_line":"            self.app.calls)"},{"line_number":2438,"context_line":""},{"line_number":2439,"context_line":"    def test_OPTIONS_with_version_id_is_null(self):"},{"line_number":2440,"context_line":"        # the path is not re-written to the versions container"},{"line_number":2441,"context_line":"        self.app.register(\u0027OPTIONS\u0027, \u0027/v1/a/c/o\u0027, swob.HTTPOk, {"},{"line_number":2442,"context_line":"            \u0027Allow\u0027: \u0027HEAD, GET, PUT, POST, DELETE, OPTIONS\u0027}, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c1e237d3_9b641e27","line":2439,"updated":"2026-08-11 09:14:14.000000000","message":"This is basically identical to the test `test_OPTIONS_with_version_id` above, except for the the `version_id`. I\u0027m wondering if we should use a similar approach to `_do_test_PUT_overwrite_unversioned_object_timestamp` (line 1200) and use a thin wrapper, eg. `_do_test_OPTIONS_with_version_id(self, version_id)`?","commit_id":"f91356b3fbdbc7866319545589f5cf7e9b3fb14d"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"5f7d16923bbbe20313a6bf7a0ee2750028e33a6f","unresolved":false,"context_lines":[{"line_number":2436,"context_line":"            [(\u0027OPTIONS\u0027, \u0027/v1/a/c/o?version-id\u003d0000000060.00000\u0027)],"},{"line_number":2437,"context_line":"            self.app.calls)"},{"line_number":2438,"context_line":""},{"line_number":2439,"context_line":"    def test_OPTIONS_with_version_id_is_null(self):"},{"line_number":2440,"context_line":"        # the path is not re-written to the versions container"},{"line_number":2441,"context_line":"        self.app.register(\u0027OPTIONS\u0027, \u0027/v1/a/c/o\u0027, swob.HTTPOk, {"},{"line_number":2442,"context_line":"            \u0027Allow\u0027: \u0027HEAD, GET, PUT, POST, DELETE, OPTIONS\u0027}, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b7dfc950_067fa548","line":2439,"in_reply_to":"c1e237d3_9b641e27","updated":"2026-08-11 09:31:58.000000000","message":"Done","commit_id":"f91356b3fbdbc7866319545589f5cf7e9b3fb14d"}]}