[Discussion] t1006: testing %(rest) with object names containing whitespace

Hi,

While exploring t/t1006-cat-file.sh, I came across the test for the %(rest) format atom in git cat-file --batch-check.

The test currently contains a FIXME explaining that %(rest) is incompatible with object names containing whitespace, such as HEAD:path/to/a/file with spaces. It uses test_expect_failure for that case and mentions using the resolved object ID as input instead of the original object name.

I investigated this test and noticed that the run_tests() helper already has access to the resolved object ID through $oid. I tried using $oid as the input to the %(rest) test rather than $object_name.

With this change, the focused t1006-cat-file.sh test passed all 423 tests locally.

Before preparing a patch, I wanted to ask whether this is the intended approach for testing %(rest) here, and whether removing the conditional expected-failure logic and using the resolved object ID would be a worthwhile improvement.

I would appreciate your guidance on whether this is worth pursuing or if there is a reason to retain the current test behavior.

Thanks,