HomePhabricator

Make the libphutil library map test failure more readable

Description

Make the libphutil library map test failure more readable

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

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10457

Differential Revision: https://secure.phabricator.com/D15370

Details

Provenance
epriestleyAuthored on Mar 1 2016, 2:15 PM
epriestleyPushed on Mar 1 2016, 2:51 PM
Reviewer
chad
Differential Revision
D15370: Make the libphutil library map test failure more readable
Parents
rPHU6ef9295e388d: Declare tokenCache on AASTNode
Branches
Unknown
Tags
Unknown
Tasks
T10457: Improve Harbomaster/Drydock UI/UX before release
Build Status
Buildable 10900
Build 13451: Run Core Tests