Skip to content

Commit

Permalink
Merge pull request #76 from creative-commoners/pulls/5/sapphire-test-…
Browse files Browse the repository at this point in the history
…nine

API phpunit 9 support
  • Loading branch information
Maxime Rainville authored Nov 9, 2021
2 parents 2a59d84 + 3a9466b commit db5c304
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 11 deletions.
9 changes: 2 additions & 7 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,24 +8,19 @@ import:
jobs:
fast_finish: true
include:
- php: 7.1
env:
- DB=MYSQL
- REQUIRE_INSTALLER="4.5.x-dev"
- PHPUNIT_TEST=1
- php: 7.3
env:
- DB=MYSQL
- PDO=1
- REQUIRE_INSTALLER="4.7.x-dev"
- REQUIRE_INSTALLER="4.x-dev"
- PHPUNIT_TEST=1
- PHPCS_TEST=1
- php: 7.4
env:
- DB=MYSQL
- REQUIRE_INSTALLER="4.x-dev"
- PHPUNIT_COVERAGE_TEST=1
- php: nightly
- php: 8.0
env:
- DB=MYSQL
- REQUIRE_INSTALLER="4.x-dev"
Expand Down
5 changes: 3 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,11 @@
}
],
"require": {
"silverstripe/framework": "~4.0"
"php": "^7.3 || ^8.0",
"silverstripe/framework": "^4.10"
},
"require-dev": {
"sminnee/phpunit": "^5.7",
"phpunit/phpunit": "^9.5",
"squizlabs/php_codesniffer": "^3"
},
"extra": {
Expand Down
4 changes: 2 additions & 2 deletions tests/KeyValueFieldTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public function testFieldContainsPlaceholder()
$field->setKeyFieldPlaceholder('Key Placeholder');
$field->setValueFieldPlaceholder('Value Placeholder');
$html = $field->Field();
$this->assertContains('placeholder="Key Placeholder"', $html);
$this->assertContains('placeholder="Value Placeholder"', $html);
$this->assertStringContainsString('placeholder="Key Placeholder"', $html);
$this->assertStringContainsString('placeholder="Value Placeholder"', $html);
}
}

0 comments on commit db5c304

Please sign in to comment.