Page MenuHomePhabricator

tests: add explicit coverage for update_hash_refs from rewriteutil
ClosedPublic

Authored by durin42 on Jul 29 2021, 4:39 PM.

Details

Summary

I couldn't find any evidence this is covered by tests in core (but there's a
good chance I missed it). We've seen a cute bug in that code, but first
let's just cover the cases that work correctly.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

durin42 created this revision.Jul 29 2021, 4:39 PM

I couldn't find any evidence this is covered by tests in core

I think the only coverage might be in 1a5d3e555c70, but clearly that's not thorough enough.

This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.