Page MenuHomePhorge

Arcanist Test Result Parser Updates
Open, Needs TriagePublic

Description

Following on from T15667,

  • Update ArcanistXUnitTestResultParser->parseTestResults() to take a second argument, $path, which, if provided and $test_results is empty, will mimic the ArcanistPhpunitTestResultParser->parseTestResults() behaviour in returning a RESULT_BROKEN ArcanistUnitTestResult. I find this preferable to throwing an exception.
  • Move the ArcanistPhpunitTestResultParser->readCoverage() into a new ArcanistCloverCoverageParser.php