Page MenuHomePhabricator

Move repository URI normalization out of PullLocalDaemon
ClosedPublic

Authored by epriestley on Jan 16 2014, 6:54 PM.
Tags
None
Referenced Files
F15509549: D7981.id18084.diff
Wed, Apr 16, 1:50 PM
F15485845: D7981.diff
Thu, Apr 10, 2:58 AM
F15437935: D7981.diff
Tue, Mar 25, 10:14 PM
F15429930: D7981.id18057.diff
Mon, Mar 24, 4:56 AM
F15422139: D7981.id18084.diff
Sat, Mar 22, 5:53 AM
F15413260: D7981.diff
Wed, Mar 19, 5:38 PM
F15410906: D7981.diff
Wed, Mar 19, 8:41 AM
F15410869: D7981.diff
Wed, Mar 19, 8:36 AM
Subscribers

Details

Summary

Ref T4327. Moves us one small step forward toward testable change parsers by separating out this unrelated logic from the PullLocal daemon. We will also probably want to run this logic so we can do remote path lookups to limit the role of Arcanist Projects in the future, which is why I made the URI type (here, only "git") a parameter rather than calling this a GitURINormalizer or something.

Test Plan
  • Ran unit tests.
  • Ran repository discover on a correctly-configured remote repository.
  • Ran repository discover on an incorrectly-configured remote, got an error message.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped