This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push:
new 847487b164 Resolve todo in ServerContextTest for upgrade test (#4822)
847487b164 is described below
commit 847487b16493e055e1e619764d892734baa50b32
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Fri Aug 23 11:29:30 2024 -0400
Resolve todo in ServerContextTest for upgrade test (#4822)
---
.../src/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 eac064f88f..fe04b78eb1 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 class ServerContextTest {
@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,
