You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lens.apache.org by am...@apache.org on 2015/04/15 21:50:23 UTC
[49/50] [abbrv] incubator-lens git commit: LENS-441 : Fix checkstyle
issues in tests (Rajat Khandelwal via amareshwari)
LENS-441 : Fix checkstyle issues in tests (Rajat Khandelwal via amareshwari)
Project: http://git-wip-us.apache.org/repos/asf/incubator-lens/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-lens/commit/28dacf6b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-lens/tree/28dacf6b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-lens/diff/28dacf6b
Branch: refs/heads/current-release-line
Commit: 28dacf6b840fd1e3bedf936c5aa96c7091f4f6f7
Parents: 309f62c
Author: Rajat Khandelwal <pr...@apache.org>
Authored: Wed Apr 15 07:10:22 2015 -0500
Committer: Amareshwari Sriramadasu <am...@apache.org>
Committed: Wed Apr 15 07:10:22 2015 -0500
----------------------------------------------------------------------
.../java/org/apache/lens/cli/TestLensDimensionTableCommands.java | 2 ++
.../src/test/java/org/apache/lens/cli/TestLensFactCommands.java | 2 ++
2 files changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-lens/blob/28dacf6b/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
----------------------------------------------------------------------
diff --git a/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java b/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
index 5d8d453..3cbeed1 100644
--- a/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
+++ b/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
@@ -125,11 +125,13 @@ public class TestLensDimensionTableCommands extends LensCliApplicationTest {
Assert.assertEquals(command.showDimensionTables("blah"), dimList);
Assert.fail();
} catch (NotFoundException e) {
+ LOG.info("blah is not a table", e);
}
try {
Assert.assertEquals(command.showDimensionTables("dim_table2"), dimList);
Assert.fail();
} catch (NotFoundException e) {
+ LOG.info("dim_table2 is a table, but not a dimension", e);
}
Assert.assertTrue(dimList.contains(tableName), "dim_table table should be found");
}
http://git-wip-us.apache.org/repos/asf/incubator-lens/blob/28dacf6b/lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java
----------------------------------------------------------------------
diff --git a/lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java b/lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java
index b906f18..02f5ac7 100644
--- a/lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java
+++ b/lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java
@@ -117,11 +117,13 @@ public class TestLensFactCommands extends LensCliApplicationTest {
Assert.assertEquals(command.showFacts("blah"), factList);
Assert.fail();
} catch (NotFoundException e) {
+ LOG.info("blah is not a table", e);
}
try {
Assert.assertEquals(command.showFacts("fact1"), factList);
Assert.fail();
} catch (NotFoundException e) {
+ LOG.info("fact1 is a table, but not a cube table", e);
}
Assert.assertEquals("fact1", factList, "Fact1 table should be found");
}