This commit is contained in:
René Pfeuffer
2019-06-13 10:38:15 +02:00
parent ffa4625e36
commit 67a78fd3b2
4 changed files with 12 additions and 12 deletions

View File

@@ -142,8 +142,8 @@ public class BootstrapContextListener implements ServletContextListener {
Throwable startupError = SCMContext.getContext().getStartupError(); Throwable startupError = SCMContext.getContext().getStartupError();
if (startupError != null) { if (startupError != null) {
contextListener = SingleView.error(startupError); contextListener = SingleView.error(startupError);
} else if (Versions.isToOld()) { } else if (Versions.isTooOld()) {
contextListener = SingleView.view("/templates/to-old.mustache", HttpServletResponse.SC_CONFLICT); contextListener = SingleView.view("/templates/too-old.mustache", HttpServletResponse.SC_CONFLICT);
} else { } else {
createMigrationOrNormalContextListener(); createMigrationOrNormalContextListener();
Versions.writeNew(); Versions.writeNew();

View File

@@ -29,7 +29,7 @@ class Versions {
} }
@VisibleForTesting @VisibleForTesting
boolean isPreviousVersionToOld() { boolean isPreviousVersionTooOld() {
return readVersion().map(v -> v.isOlder(MIN_VERSION)).orElse(false); return readVersion().map(v -> v.isOlder(MIN_VERSION)).orElse(false);
} }
@@ -66,8 +66,8 @@ class Versions {
} }
} }
static boolean isToOld() { static boolean isTooOld() {
return new Versions(SCMContext.getContext()).isPreviousVersionToOld(); return new Versions(SCMContext.getContext()).isPreviousVersionTooOld();
} }
static void writeNew() { static void writeNew() {

View File

@@ -29,37 +29,37 @@ class VersionsTest {
@Test @Test
void shouldReturnTrueForVersionsPreviousTo160(@TempDirectory.TempDir Path directory) throws IOException { void shouldReturnTrueForVersionsPreviousTo160(@TempDirectory.TempDir Path directory) throws IOException {
setVersion(directory, "1.59"); setVersion(directory, "1.59");
assertThat(versions.isPreviousVersionToOld()).isTrue(); assertThat(versions.isPreviousVersionTooOld()).isTrue();
setVersion(directory, "1.12"); setVersion(directory, "1.12");
assertThat(versions.isPreviousVersionToOld()).isTrue(); assertThat(versions.isPreviousVersionTooOld()).isTrue();
} }
@Test @Test
void shouldReturnFalseForVersion160(@TempDirectory.TempDir Path directory) throws IOException { void shouldReturnFalseForVersion160(@TempDirectory.TempDir Path directory) throws IOException {
setVersion(directory, "1.60"); setVersion(directory, "1.60");
assertThat(versions.isPreviousVersionToOld()).isFalse(); assertThat(versions.isPreviousVersionTooOld()).isFalse();
} }
@Test @Test
void shouldNotFailIfVersionContainsLineBreak(@TempDirectory.TempDir Path directory) throws IOException { void shouldNotFailIfVersionContainsLineBreak(@TempDirectory.TempDir Path directory) throws IOException {
setVersion(directory, "1.59\n"); setVersion(directory, "1.59\n");
assertThat(versions.isPreviousVersionToOld()).isTrue(); assertThat(versions.isPreviousVersionTooOld()).isTrue();
} }
@Test @Test
void shouldReturnFalseForVersionsNewerAs160(@TempDirectory.TempDir Path directory) throws IOException { void shouldReturnFalseForVersionsNewerAs160(@TempDirectory.TempDir Path directory) throws IOException {
setVersion(directory, "1.61"); setVersion(directory, "1.61");
assertThat(versions.isPreviousVersionToOld()).isFalse(); assertThat(versions.isPreviousVersionTooOld()).isFalse();
setVersion(directory, "1.82"); setVersion(directory, "1.82");
assertThat(versions.isPreviousVersionToOld()).isFalse(); assertThat(versions.isPreviousVersionTooOld()).isFalse();
} }
@Test @Test
void shouldReturnFalseForNonExistingVersionFile(@TempDirectory.TempDir Path directory) { void shouldReturnFalseForNonExistingVersionFile(@TempDirectory.TempDir Path directory) {
setVersionFile(directory.resolve("version.txt")); setVersionFile(directory.resolve("version.txt"));
assertThat(versions.isPreviousVersionToOld()).isFalse(); assertThat(versions.isPreviousVersionTooOld()).isFalse();
} }
@Test @Test