Page MenuHomePhabricator

Make the libphutil library map test failure more readable
ClosedPublic

Authored by epriestley on Mar 1 2016, 2:17 PM.
Tags
None
Referenced Files
F14005370: D15370.diff
Sun, Oct 27, 12:21 PM
F13981201: D15370.id37059.diff
Sat, Oct 19, 2:52 PM
F13972395: D15370.id.diff
Thu, Oct 17, 6:44 PM
F13963966: D15370.diff
Tue, Oct 15, 6:57 PM
F13956719: D15370.diff
Mon, Oct 14, 6:51 AM
Unknown Object (File)
Sep 5 2024, 6:46 PM
Unknown Object (File)
Sep 5 2024, 6:46 PM
Unknown Object (File)
Sep 5 2024, 6:46 PM
Subscribers
None

Details

Summary

Ref T10457. This is an upstream quality-of-life fix.

Currently, if the map rebuild test fails it generates about 9,000 lines of output in a huge diff of the two values. This is technically fine, but pretty hard to read.

Instead, compute a smaller difference showing only changed keys, since we never care about the values when this failure occurs (and barely care about the keys).

Test Plan
   FAIL  PhabricatorLibraryTestCase::testLibraryMap
Assertion failed, expected 'true' (at PhutilLibraryTestCase.php:51): The library map is out of date. Rebuild it with `arc liberate`.
These entries differ: class.NewExampleClass.

ACTUAL VALUE
false

Diff Detail

Repository
rPHU libphutil
Branch
dry14
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10899
Build 13450: Run Core Tests
Build 13449: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Make the libphutil library map test failure more readable.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 1 2016, 2:44 PM
This revision was automatically updated to reflect the committed changes.