Page MenuHomePhabricator

Disregard `src/extensions/` when running library map unit tests
ClosedPublic

Authored by epriestley on Jul 5 2014, 9:40 PM.
Tags
None
Referenced Files
F14009296: D9834.id23596.diff
Wed, Oct 30, 2:17 PM
F14009295: D9834.id.diff
Wed, Oct 30, 2:17 PM
F14009294: D9834.id23613.diff
Wed, Oct 30, 2:17 PM
F14009281: D9834.diff
Wed, Oct 30, 2:06 PM
F13977497: D9834.id.diff
Oct 18 2024, 6:51 PM
F13976244: D9834.id23613.diff
Oct 18 2024, 1:23 PM
F13973733: D9834.diff
Oct 18 2024, 2:18 AM
F13967762: D9834.id23613.diff
Oct 16 2024, 3:46 PM
Subscribers

Details

Summary

The runtime map will include anything loaded via src/extensions/, which means the test will always fail if you have anything in that directory.

Instead, allow the base (no extensions) map to be retrieved, and use that for comparisions.

Test Plan
  • Ran unit tests with stuff in extensions.
  • Verified stuff in extensions still works fine.

Diff Detail

Repository
rPHU libphutil
Branch
noext
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1535
Build 1535: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Disregard `src/extensions/` when running library map unit tests.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.
This revision is now accepted and ready to land.Jul 5 2014, 10:04 PM
epriestley updated this revision to Diff 23613.

Closed by commit rPHU175694903cee (authored by @epriestley).