Skip to content

Commit

Permalink
Fix all 2.3 tests
Browse files Browse the repository at this point in the history
  • Loading branch information
cstroe committed Sep 12, 2020
1 parent 716d9f7 commit b642492
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ GET / ${controllers()}${packageId}.Application.inde

def "can add additional imports"() {
// Play version 2.3 not supported
Assume.assumeTrue(playVersion.compareTo(VersionNumber.parse("2.4")) >= 0)
Assume.assumeTrue(playVersion > VersionNumber.parse("2.3.99"))
given:
withRoutesTemplate()
and:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,12 +55,8 @@ class Play24RoutesCompileIntegrationTest extends AbstractRoutesCompileIntegratio
}

def "can specify route compiler type as injected"() {
// Play version 2.3 does not support injected routers
Assume.assumeTrue(playVersion.compareTo(VersionNumber.parse("2.4")) >= 0)
// Play before version 2.6 does not support Java 1.9+
if (playVersion.compareTo(VersionNumber.parse("2.6")) < 0) {
Assume.assumeTrue(System.getProperty("java.version").startsWith("1.8"))
}
// Play version 2.3 not supported
Assume.assumeTrue(playVersion > VersionNumber.parse("2.3.99"))
given:
withRoutesTemplate()
withInjectedRoutesController()
Expand All @@ -79,7 +75,7 @@ play {

def "recompiles when route compiler type is changed"() {
// Play version 2.3 not supported
Assume.assumeTrue(playVersion.compareTo(VersionNumber.parse("2.4")) >= 0)
Assume.assumeTrue(playVersion > VersionNumber.parse("2.3.99"))
// Play 2.7+ only has a single route compiler type.
Assume.assumeTrue(playVersion < VersionNumber.parse("2.7"))
when:
Expand Down

0 comments on commit b642492

Please sign in to comment.