Page MenuHomePhabricator

[Wilds] Make "Bundle" test cases pass
ClosedPublic

Authored by epriestley on Sep 27 2018, 5:41 PM.
Tags
None
Referenced Files
F18989258: D19713.id47107.diff
Tue, Nov 18, 11:24 AM
F18966245: D19713.id47107.diff
Fri, Nov 14, 6:58 PM
F18875806: D19713.diff
Wed, Nov 5, 7:29 PM
F18854954: D19713.diff
Sat, Nov 1, 1:37 AM
F18750746: D19713.diff
Oct 4 2025, 8:47 AM
F18741302: D19713.diff
Oct 2 2025, 11:42 AM
F18657771: D19713.diff
Sep 23 2025, 3:20 AM
F18095126: D19713.id.diff
Aug 7 2025, 10:32 PM
Subscribers
None

Details

Summary

Ref T13098. Continue to chip away at unit tests.

I stubbed out the read of configuration directly from RepositoryAPI and would like to remove it later. This is the only configuration value that RepositoryAPI reads directly. Instead, I suspect this will be cleaner if higher-level callers are responsible for reading and applying commit range rules and the actual RepositorAPI object does not know about them.

(In this particular case it's pretty moot anyway since unit tests shouldn't depend on .arcconfig settings.)

Test Plan

Ran unit tests, got slightly fewer failures.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 27 2018, 5:41 PM
Harbormaster failed remote builds in B20916: Diff 47107!
This revision is now accepted and ready to land.Sep 27 2018, 8:19 PM
This revision was automatically updated to reflect the committed changes.