Skip to content

Commit

Permalink
Merge pull request #45 from nexcess/fix/phpunit-6.x-compat
Browse files Browse the repository at this point in the history
Use assertContains() instead of assertStringContainsString()
  • Loading branch information
bswatson authored Jul 16, 2020
2 parents 2a1cae9 + 1f2e18c commit 6b0018c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/SettingsTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ public function a_notice_should_be_shown_if_there_have_been_orders_in_the_curren
$limiter = new OrderLimiter();
$limiter->init();

$this->assertStringContainsString(
$this->assertContains(
'<div class="notice notice-info">',
$this->get_setting_by_id( 'limit-orders-general', new Settings( $limiter ) )['desc'],
'Expected to see a notice about limits being recalculated.'
Expand All @@ -111,7 +111,7 @@ public function a_notice_should_not_be_shown_if_there_have_not_been_any_orders_i
$limiter = new OrderLimiter();
$limiter->init();

$this->assertStringNotContainsString(
$this->assertNotContains(
'<div class="notice notice-info">',
$this->get_setting_by_id( 'limit-orders-general', new Settings( $limiter ) )['desc'],
'Did not expect to see a notice about limits being recalculated.'
Expand Down

0 comments on commit 6b0018c

Please sign in to comment.