Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

Commit 5ee2b82

Browse files
committed
Update phpstan
1 parent 8b64b7a commit 5ee2b82

File tree

3 files changed

+19
-14
lines changed

3 files changed

+19
-14
lines changed

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ matrix:
2626
- phpenv config-rm xdebug.ini
2727
before_script: ~
2828
script:
29-
- composer require phpstan/phpstan-shim # Not part of require-dev, because it won't install on PHP 5.6
29+
- composer require phpstan/phpstan # Not part of require-dev, because it won't install on PHP 5.6
3030
- composer analyze
3131
- composer codestyle:check
3232
after_script: ~

lib/WebDriverExpectedCondition.php

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -522,21 +522,23 @@ function () use ($element_or_by, $selected) {
522522
return $element_or_by->isSelected() === $selected;
523523
}
524524
);
525-
} else {
526-
if ($element_or_by instanceof WebDriverBy) {
527-
return new static(
528-
function (WebDriver $driver) use ($element_or_by, $selected) {
529-
try {
530-
$element = $driver->findElement($element_or_by);
531-
532-
return $element->isSelected() === $selected;
533-
} catch (StaleElementReferenceException $e) {
534-
return null;
535-
}
525+
}
526+
527+
if ($element_or_by instanceof WebDriverBy) {
528+
return new static(
529+
function (WebDriver $driver) use ($element_or_by, $selected) {
530+
try {
531+
$element = $driver->findElement($element_or_by);
532+
533+
return $element->isSelected() === $selected;
534+
} catch (StaleElementReferenceException $e) {
535+
return null;
536536
}
537-
);
538-
}
537+
}
538+
);
539539
}
540+
541+
throw new \InvalidArgumentException('Instance of either WebDriverElement or WebDriverBy must be given');
540542
}
541543

542544
/**

phpstan.neon

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,6 @@ parameters:
88
- '#Call to an undefined method Facebook\\WebDriver\\WebDriver::getTouch\(\)#'
99
- '#Call to an undefined method Facebook\\WebDriver\\WebDriverElement::getCoordinates\(\)#'
1010
- '#Call to an undefined method Facebook\\WebDriver\\WebDriverElement::equals\(\)#'
11+
- '#Unsafe usage of new static\(\)#'
12+
13+
inferPrivatePropertyTypeFromConstructor: true

0 commit comments

Comments
 (0)