Page MenuHomePhabricator

Fix broken ArcanistPhpunitTestResultParser
ClosedPublic

Authored by sascha-egerer on May 7 2015, 3:27 PM.
Tags
None
Referenced Files
F15434037: D12751.id30638.diff
Tue, Mar 25, 1:39 AM
F15433460: D12751.id30640.diff
Mon, Mar 24, 11:19 PM
F15427359: D12751.id30640.diff
Sun, Mar 23, 2:00 PM
F15426114: D12751.id.diff
Sun, Mar 23, 7:13 AM
F15423108: D12751.id30638.diff
Sat, Mar 22, 1:03 PM
F15422679: D12751.diff
Sat, Mar 22, 10:10 AM
F15392235: D12751.id30640.diff
Sat, Mar 15, 1:59 PM
F15294836: D12751.diff
Wed, Mar 5, 11:02 AM

Details

Summary

The ArcanistPhpunitTestResultParser has been changed
to use phutil_json_decode instead of json_decode in
rARCa4d33ef117aa8702181154b60ce1ce52bcfc119b
That has broken the functionality as json_decode has returned
an object before but phutil_json_decode does return an array.
The code has now been adopted to work with the array result
instead of an object.

Test Plan

Run a phpunit test case

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sascha-egerer retitled this revision from to Fix broken ArcanistPhpunitTestResultParser.
sascha-egerer updated this object.
sascha-egerer edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.May 7 2015, 3:40 PM
Closed by commit rARC3e36b3fc430d: Fix broken ArcanistPhpunitTestResultParser (authored by Sascha Egerer <sascha.egerer@dkd.de>, committed by epriestley). · Explain WhyMay 7 2015, 3:41 PM
This revision was automatically updated to reflect the committed changes.