benchmark/comparability._repo_key picks a per-repo row's identity from repo_path/url/repo/name, then falls back to freeze_commit[:10], and finally to repr(sorted(entry.keys())) for a row with none of those. That final fallback (line 123) is never exercised by the test suite, so a row with no identity field — which must still get a stable, deterministic key rather than collide or raise — is unverified.
Ask
Add a _repo_key test asserting the identity-field precedence, the freeze_commit fallback, and the sorted-keys fallback for an identity-less row. Tests only.
benchmark/comparability._repo_keypicks a per-repo row's identity fromrepo_path/url/repo/name, then falls back tofreeze_commit[:10], and finally torepr(sorted(entry.keys()))for a row with none of those. That final fallback (line 123) is never exercised by the test suite, so a row with no identity field — which must still get a stable, deterministic key rather than collide or raise — is unverified.Ask
Add a
_repo_keytest asserting the identity-field precedence, thefreeze_commitfallback, and the sorted-keys fallback for an identity-less row. Tests only.