From b4a90a400f7f59de50d4ce7c46f666e8924982c0 Mon Sep 17 00:00:00 2001 From: Dave Marion Date: Fri, 23 Aug 2024 13:49:07 +0000 Subject: [PATCH] Resolve todo in ServerContextTest for upgrade test --- .../java/org/apache/accumulo/server/ServerContextTest.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java b/server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java index eac064f88ff..fe04b78eb1c 100644 --- a/server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java +++ b/server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java @@ -134,8 +134,7 @@ public void testSasl() throws Exception { @Test public void testCanRun() { final int oldestSupported = AccumuloDataVersion.oldestUpgradeableVersion(); - // ELASTICITY_TODO #4587 basically disable check until upgrade is working. Should be: - // assertEquals(10, oldestSupported); // make sure it hasn't changed accidentally + assertEquals(10, oldestSupported); // make sure it hasn't changed accidentally final int currentVersion = AccumuloDataVersion.get(); IntConsumer shouldPass = ServerContext::ensureDataVersionCompatible; IntConsumer shouldFail = v -> assertThrows(IllegalStateException.class,