Repository: incubator-sentry
Updated Branches:
  refs/heads/master 654704120 -> dd9edc666


SENTRY-578: Print detail error for TestHDFSIntegration when test failed (Dapeng 
Sun, reviewed by Xiaomeng Huang)


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/dd9edc66
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/dd9edc66
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/dd9edc66

Branch: refs/heads/master
Commit: dd9edc666b9f013f3c98094d47b63bc0a896855e
Parents: 6547041
Author: Sun Dapeng <[email protected]>
Authored: Sat Feb 28 10:41:10 2015 +0800
Committer: Sun Dapeng <[email protected]>
Committed: Sat Feb 28 10:41:10 2015 +0800

----------------------------------------------------------------------
 .../org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/dd9edc66/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
----------------------------------------------------------------------
diff --git 
a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
 
b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
index 1a40bd6..4cb7275 100644
--- 
a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
+++ 
b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
@@ -996,9 +996,10 @@ public class TestHDFSIntegration {
     try {
       fStatus = miniDFS.getFileSystem().getFileStatus(p);
       if (groupShouldExist) {
-        Assert.assertEquals(fsAction, getAcls(p).get(group));
+        Assert.assertEquals("Error at verifying Path action : " + p + " ;", 
fsAction, getAcls(p).get(group));
       } else {
-        Assert.assertFalse(getAcls(p).containsKey(group));
+        Assert.assertFalse("Error at verifying Path : " + p + " ," +
+                " group : " + group + " ;", getAcls(p).containsKey(group));
       }
     } catch (Throwable th) {
       if (retry > 0) {

Reply via email to