You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@carbondata.apache.org by ra...@apache.org on 2019/08/01 13:47:38 UTC
[carbondata] branch branch-1.6 updated: [HOTFIX] CLI test case
failed during release because of space differences
This is an automated email from the ASF dual-hosted git repository.
ravipesala pushed a commit to branch branch-1.6
in repository https://gitbox.apache.org/repos/asf/carbondata.git
The following commit(s) were added to refs/heads/branch-1.6 by this push:
new 917e041 [HOTFIX] CLI test case failed during release because of space differences
917e041 is described below
commit 917e041439282985ec28ff89249db6088e6771df
Author: ravipesala <ra...@gmail.com>
AuthorDate: Thu Aug 1 18:12:50 2019 +0530
[HOTFIX] CLI test case failed during release because of space differences
CLI test case is failed if the release name is short and without snapshot, it adds more space.
That's why changed test check the individual contains instead of a batch of lines.
This closes #3344
---
.../src/test/java/org/apache/carbondata/tool/CarbonCliTest.java | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/tools/cli/src/test/java/org/apache/carbondata/tool/CarbonCliTest.java b/tools/cli/src/test/java/org/apache/carbondata/tool/CarbonCliTest.java
index 26901f8..af8d51d 100644
--- a/tools/cli/src/test/java/org/apache/carbondata/tool/CarbonCliTest.java
+++ b/tools/cli/src/test/java/org/apache/carbondata/tool/CarbonCliTest.java
@@ -241,12 +241,9 @@ public class CarbonCliTest {
"2 0 3.36KB 4.06MB false 0 0.0B 93.76KB 0.0 100.0 7 2999998 " ,
"2 1 2.04KB 1.49MB false 0 0.0B 89.62KB 0.0 100.0 9 2999999 ");
Assert.assertTrue(output.contains(expectedOutput));
-
- expectedOutput = buildLines(
- "## version Details",
- "written_by Version ",
- "TestUtil "+ CarbonVersionConstants.CARBONDATA_VERSION+" ");
- Assert.assertTrue(output.contains(expectedOutput));
+ Assert.assertTrue(output.contains("## version Details"));
+ Assert.assertTrue(output.contains("written_by Version"));
+ Assert.assertTrue(output.contains("TestUtil "+ CarbonVersionConstants.CARBONDATA_VERSION));
}
@Test