Skip to content

Commit

Permalink
Merge pull request #7 from Lakion/unused-option
Browse files Browse the repository at this point in the history
Removed unused option from FailedStepListener
  • Loading branch information
Paweł Jędrzejewski committed Sep 15, 2015
2 parents 16da28c + cf10435 commit e811211
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
9 changes: 1 addition & 8 deletions src/Listener/FailedStepListener.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,6 @@ class FailedStepListener implements EventSubscriberInterface
*/
private $screenshot;

/**
* @var boolean
*/
private $cleanStart;

/**
* Used to ensure that screenshot and log comes from the same failed step.
*
Expand All @@ -57,14 +52,12 @@ class FailedStepListener implements EventSubscriberInterface
* @param Mink $mink
* @param string $logDirectory
* @param boolean $screenshot
* @param boolean $cleanStart
*/
public function __construct(Mink $mink, $logDirectory, $screenshot, $cleanStart)
public function __construct(Mink $mink, $logDirectory, $screenshot)
{
$this->mink = $mink;
$this->logDirectory = $logDirectory;
$this->screenshot = $screenshot;
$this->cleanStart = $cleanStart;
}

/**
Expand Down
1 change: 0 additions & 1 deletion src/ServiceContainer/MinkDebugExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,6 @@ private function loadStepFailureListener(ContainerBuilder $container)
new Reference('mink'),
'%mink_debug.directory%',
'%mink_debug.screenshot%',
'%mink_debug.clean_start%',
]);

$definition->addTag(EventDispatcherExtension::SUBSCRIBER_TAG, ['priority' => 0]);
Expand Down

0 comments on commit e811211

Please sign in to comment.