Hi Siddharth,

Thanks for the cleanup! Using `test_oid_to_path` here makes the helper cleaner and avoids the `sed` trick.

I tested this patch locally and the tests passed.

One small thought: maybe we could quote `$obj` in the `test_oid_to_path` call, like this:

          path="$repo/.git/objects/$(test_oid_to_path "$obj")" &&

Also, would it make sense to use `local` for `repo` and `obj` to avoid leaking variables outside the helper? Not a strong opinion, but it might make the helper a bit safer.

Thanks, Pushkar