You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2024/01/26 17:58:37 UTC

(accumulo) 01/02: Trivial - give variable name to test constant

This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git

commit b701918ec02dbb7465ce0f8e9b3fc1f9ae5a3ebb
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Fri Jan 26 12:48:31 2024 -0500

    Trivial - give variable name to test constant
    
    Give a variable name to make it clear the purpose of the hard-coded
    constant in MiniAccumuloClusteImplTest is to declare the expected number
    of tables before the test runs.
---
 .../apache/accumulo/miniclusterImpl/MiniAccumuloClusterImplTest.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/minicluster/src/test/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImplTest.java b/minicluster/src/test/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImplTest.java
index a60b9e5304..c8cf747fe9 100644
--- a/minicluster/src/test/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImplTest.java
+++ b/minicluster/src/test/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImplTest.java
@@ -106,9 +106,10 @@ public class MiniAccumuloClusterImplTest {
   @Timeout(60)
   public void saneMonitorInfo() throws Exception {
     ManagerMonitorInfo stats;
+    int expectedNumTables = 3;
     while (true) {
       stats = accumulo.getManagerMonitorInfo();
-      if (stats.tableMap.size() <= 2) {
+      if (stats.tableMap.size() < expectedNumTables) {
         continue;
       }