Skip to content

Commit

Permalink
Use assertStringContainsString when needed
Browse files Browse the repository at this point in the history
  • Loading branch information
jderusse committed Aug 6, 2019
1 parent 9708054 commit 348bfd4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Tests/PhpProcessTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,10 @@ public function testCommandLine()
$commandLine = $process->getCommandLine();

$process->start();
$this->assertContains($commandLine, $process->getCommandLine(), '::getCommandLine() returns the command line of PHP after start');
$this->assertStringContainsString($commandLine, $process->getCommandLine(), '::getCommandLine() returns the command line of PHP after start');

$process->wait();
$this->assertContains($commandLine, $process->getCommandLine(), '::getCommandLine() returns the command line of PHP after wait');
$this->assertStringContainsString($commandLine, $process->getCommandLine(), '::getCommandLine() returns the command line of PHP after wait');

$this->assertSame(PHP_VERSION.\PHP_SAPI, $process->getOutput());
}
Expand Down
2 changes: 1 addition & 1 deletion Tests/ProcessBuilderTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ public function testShouldSetArguments()

$proc = $pb->getProcess();

$this->assertContains('second', $proc->getCommandLine());
$this->assertStringContainsString('second', $proc->getCommandLine());
}

public function testPrefixIsPrependedToAllGeneratedProcess()
Expand Down

0 comments on commit 348bfd4

Please sign in to comment.