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;
}