You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by sk...@apache.org on 2020/01/16 04:51:26 UTC

[phoenix] branch 4.x-HBase-1.3 updated: PHOENIX-5675 addendum

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

skadam pushed a commit to branch 4.x-HBase-1.3
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/4.x-HBase-1.3 by this push:
     new da17a38  PHOENIX-5675 addendum
da17a38 is described below

commit da17a3864e4d7482f8b5d1c7bb784dac4b43d4ba
Author: s.kadam <s....@apache.org>
AuthorDate: Wed Jan 15 20:46:59 2020 -0800

    PHOENIX-5675 addendum
---
 .../test/java/org/apache/phoenix/index/IndexUpgradeToolTest.java | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/phoenix-core/src/test/java/org/apache/phoenix/index/IndexUpgradeToolTest.java b/phoenix-core/src/test/java/org/apache/phoenix/index/IndexUpgradeToolTest.java
index 87655a2..facd9ee 100644
--- a/phoenix-core/src/test/java/org/apache/phoenix/index/IndexUpgradeToolTest.java
+++ b/phoenix-core/src/test/java/org/apache/phoenix/index/IndexUpgradeToolTest.java
@@ -39,6 +39,7 @@ public class IndexUpgradeToolTest {
     private static final String INPUT_LIST = "TEST.MOCK1,TEST1.MOCK2,TEST.MOCK3";
     private final boolean upgrade;
     private static final String DUMMY_STRING_VALUE = "anyValue";
+    private static final String DUMMY_VERIFY_VALUE = "someVerifyValue";
     private IndexUpgradeTool indexUpgradeTool=null;
     private String outputFile;
 
@@ -50,7 +51,7 @@ public class IndexUpgradeToolTest {
     public void setup() {
         outputFile = "/tmp/index_upgrade_" + UUID.randomUUID().toString();
         String [] args = {"-o", upgrade ? UPGRADE_OP : ROLLBACK_OP, "-tb",
-                INPUT_LIST, "-lf", outputFile, "-d", "-v", DUMMY_STRING_VALUE};
+                INPUT_LIST, "-lf", outputFile, "-d", "-v", DUMMY_VERIFY_VALUE};
         indexUpgradeTool = new IndexUpgradeTool();
         CommandLine cmd = indexUpgradeTool.parseOptions(args);
         indexUpgradeTool.initializeTool(cmd);
@@ -70,14 +71,14 @@ public class IndexUpgradeToolTest {
             return;
         }
         Assert.assertEquals("value passed with verify option does not match with provided value",
-                DUMMY_STRING_VALUE, indexUpgradeTool.getVerify());
+                DUMMY_VERIFY_VALUE, indexUpgradeTool.getVerify());
         String [] values = indexUpgradeTool.getIndexToolArgValues(DUMMY_STRING_VALUE,
                 DUMMY_STRING_VALUE, DUMMY_STRING_VALUE, DUMMY_STRING_VALUE, DUMMY_STRING_VALUE);
         List<String> argList =  Arrays.asList(values);
-        Assert.assertTrue(argList.contains(DUMMY_STRING_VALUE));
+        Assert.assertTrue(argList.contains(DUMMY_VERIFY_VALUE));
         Assert.assertTrue(argList.contains("-v"));
         Assert.assertEquals("verify option and value are not passed consecutively", 1,
-                argList.indexOf(DUMMY_STRING_VALUE) - argList.indexOf("-v"));
+                argList.indexOf(DUMMY_VERIFY_VALUE) - argList.indexOf("-v"));
     }
 
     @Parameters(name ="IndexUpgradeToolTest_mutable={1}")