Skip to content

Commit b066a8d

Browse files
committed
[SonarCloud] Component.php & FileUploadHelper.php
1 parent dd2318c commit b066a8d

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/lib/Browser/Component/Component.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@
2323
use OAndreyev\Mink\Driver\WebDriver;
2424
use RuntimeException;
2525

26+
class DevToolsDriverUnavailableException extends \RuntimeException {}
27+
2628
abstract class Component implements ComponentInterface
2729
{
2830
/** @var \Ibexa\Behat\Browser\Locator\LocatorCollection */
@@ -79,11 +81,11 @@ protected function getDevToolsDriver(): ChromeDevToolsDriver
7981
$webDriver = $driver->getWebDriver();
8082

8183
if (null === $webDriver) {
82-
throw new RuntimeException('Error happened when accessing the WebDriver');
84+
throw new DevToolsDriverUnavailableException('Error happened when accessing the WebDriver');
8385
}
8486

8587
if (!($webDriver instanceof \Facebook\WebDriver\Remote\RemoteWebDriver)) {
86-
throw new RuntimeException('Expected instance of Facebook\\WebDriver\\Remote\\RemoteWebDriver, got ' . (is_object($webDriver) ? get_class($webDriver) : gettype($webDriver)));
88+
throw new DevToolsDriverUnavailableException('Expected instance of Facebook\\WebDriver\\Remote\\RemoteWebDriver, got ' . (is_object($webDriver) ? get_class($webDriver) : gettype($webDriver)));
8789
}
8890

8991
return new ChromeDevToolsDriver($webDriver);

src/lib/Browser/FileUpload/FileUploadHelper.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
use Behat\Mink\Session;
1313
use FriendsOfBehat\SymfonyExtension\Mink\MinkParameters;
1414

15+
class FileReadException extends \RuntimeException {}
16+
1517
class FileUploadHelper
1618
{
1719
/** @var \Behat\Mink\Session */
@@ -32,13 +34,13 @@ public function getRemoteFileUploadPath($filename)
3234
$driver = $this->session->getDriver();
3335

3436
if ($driver instanceof Selenium2Driver) {
35-
if (!preg_match('#[\w\\/\.]*\.zip$#', $filename)) {
37+
if (!preg_match('#[\w/.]*\.zip$#', $filename)) {
3638
throw new \InvalidArgumentException('Zip archive required to upload to remote browser machine.');
3739
}
3840

3941
$fileContents = file_get_contents($localFile);
4042
if ($fileContents === false) {
41-
throw new \RuntimeException("Failed to read file: $localFile");
43+
throw new FileReadException("Failed to read file: $localFile");
4244
}
4345

4446
return $driver->getWebDriverSession()->file([

0 commit comments

Comments
 (0)