Page MenuHomePhabricator

tests: also test spaces in conflict marker labels
ClosedPublic

Authored by martinvonz on Jan 21 2022, 6:31 PM.

Details

Summary

Our fake merge tool printed the arguments using $1 etc. without
quotes, which means that the shell collapsed repeated spaces within
the argument. That resulted in tests that did not pick up changes to
the number of spaces.

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

martinvonz created this revision.Jan 21 2022, 6:31 PM
Alphare accepted this revision.Jan 24 2022, 4:33 AM
This revision is now accepted and ready to land.Jan 24 2022, 4:33 AM
This revision was automatically updated to reflect the committed changes.