From cee338cd75997802b711b933ac803fbb2db24015 Mon Sep 17 00:00:00 2001 From: Adam Prager Date: Fri, 28 Aug 2015 15:58:43 +0200 Subject: [PATCH] Fixed phantomjs options associative config Fixes #5 --- ConfigHelper.php | 2 +- Tests/ConfigHelperTest.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ConfigHelper.php b/ConfigHelper.php index a661e78..9779eb6 100644 --- a/ConfigHelper.php +++ b/ConfigHelper.php @@ -217,7 +217,7 @@ protected function processPhantomjsOptions() if (is_bool($value)) { $value = ($value) ? 'true' : 'false'; } - $options[] = sprintf('%s="%s"', $name, $value); + $options[] = sprintf('%s=%s', $name, $value); } } diff --git a/Tests/ConfigHelperTest.php b/Tests/ConfigHelperTest.php index 8622877..855ca17 100644 --- a/Tests/ConfigHelperTest.php +++ b/Tests/ConfigHelperTest.php @@ -126,7 +126,7 @@ public function arePhantomjsOptionsHandled() $process = $this->configHelper->buildProcess($url, 'e4e5k2'); $this->assertContains( - ProcessUtils::escapeArgument(sprintf('%s="%s"', '--ignore-ssl-errors', 'true')), + ProcessUtils::escapeArgument(sprintf('%s=%s', '--ignore-ssl-errors', 'true')), $process->getCommandLine() );