Skip to content

Commit

Permalink
Fix issues identified by Psalm
Browse files Browse the repository at this point in the history
  • Loading branch information
sebastianbergmann committed Sep 9, 2019
1 parent 2400d79 commit d434b33
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/Util/Log/TeamCity.php
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ public function startTestSuite(TestSuite $suite): void
} else {
$split = \explode('::', $suiteName);

if (\count($split) === 2 && \method_exists($split[0], $split[1])) {
if (\count($split) === 2 && \class_exists($split[0]) && \method_exists($split[0], $split[1])) {
$fileName = self::getFileName($split[0]);
$parameters['locationHint'] = "php_qn://$fileName::\\$suiteName";
$parameters['name'] = $split[1];
Expand All @@ -238,7 +238,7 @@ public function endTestSuite(TestSuite $suite): void
if (!\class_exists($suiteName, false)) {
$split = \explode('::', $suiteName);

if (\count($split) === 2 && \method_exists($split[0], $split[1])) {
if (\count($split) === 2 && \class_exists($split[0]) && \method_exists($split[0], $split[1])) {
$parameters['name'] = $split[1];
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/Util/Test.php
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ public static function getMissingRequirements(string $className, string $methodN
foreach ($required['functions'] as $function) {
$pieces = \explode('::', $function);

if (\count($pieces) === 2 && \method_exists($pieces[0], $pieces[1])) {
if (\count($pieces) === 2 && \class_exists($pieces[0]) && \method_exists($pieces[0], $pieces[1])) {
continue;
}

Expand Down

0 comments on commit d434b33

Please sign in to comment.