From 201d195f291b311e724ba49b4a7eb804099aae45 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Sat, 3 Jan 2015 10:30:15 +1100 Subject: [PATCH] Fix variable name Summary: Fix a variable name to be consistent with naming conventions. Test Plan: `arc lint` Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: aurelijus, Korvin, epriestley Differential Revision: https://secure.phabricator.com/D11140 --- src/unit/engine/PhpunitResultParser.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/unit/engine/PhpunitResultParser.php b/src/unit/engine/PhpunitResultParser.php index b9d4dc3e..693d8bcc 100644 --- a/src/unit/engine/PhpunitResultParser.php +++ b/src/unit/engine/PhpunitResultParser.php @@ -39,7 +39,7 @@ final class PhpunitResultParser extends ArcanistTestResultParser { case 'test': break; case 'testStart': - $lastTestFinished = false; + $last_test_finished = false; // fall through default: continue 2; // switch + loop @@ -80,10 +80,10 @@ final class PhpunitResultParser extends ArcanistTestResultParser { $result->setUserData($user_data); $results[] = $result; - $lastTestFinished = true; + $last_test_finished = true; } - if (!$lastTestFinished) { + if (!$last_test_finished) { $results[] = id(new ArcanistUnitTestResult()) ->setName($event->test) // use last event ->setUserData($this->stderr)