accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [1/3] git commit: ACCUMULO-3292 Some extra information on flat out incorrect assertion failures.
Date Tue, 04 Nov 2014 21:09:27 GMT
Repository: accumulo
Updated Branches:
  refs/heads/1.6 2805630ef -> eb424221b
  refs/heads/master 569c6bcd2 -> 78187c136


ACCUMULO-3292 Some extra information on flat out incorrect assertion failures.


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

Branch: refs/heads/1.6
Commit: eb424221bafe583a37ad9c6a61b72d2820c37e46
Parents: 2805630
Author: Josh Elser <elserj@apache.org>
Authored: Tue Nov 4 16:06:21 2014 -0500
Committer: Josh Elser <elserj@apache.org>
Committed: Tue Nov 4 16:06:21 2014 -0500

----------------------------------------------------------------------
 .../accumulo/test/ConditionalWriterIT.java      | 26 ++++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/eb424221/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
----------------------------------------------------------------------
diff --git a/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java b/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
index 1a7fe20..516cd46 100644
--- a/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
@@ -357,8 +357,8 @@ public class ConditionalWriterIT extends SimpleMacIT {
     cm8.put("tx", "seq", cva, "1");
 
     try {
-      cw2.write(cm8).getStatus();
-      Assert.fail("Writing mutation with Authorizations the user doesn't have should fail");
+      Status status = cw2.write(cm8).getStatus();
+      Assert.fail("Writing mutation with Authorizations the user doesn't have should fail.
Got status: " + status);
     } catch (AccumuloSecurityException ase) {
       // expected, check specific failure?
     } finally {
@@ -1050,15 +1050,15 @@ public class ConditionalWriterIT extends SimpleMacIT {
     Assert.assertEquals(Status.ACCEPTED, cw3.write(cm1).getStatus());
 
     try {
-      cw1.write(cm1).getStatus();
-      Assert.fail("Expected exception writing conditional mutation to table the user doesn't
have write access to");
+      Status status = cw1.write(cm1).getStatus();
+      Assert.fail("Expected exception writing conditional mutation to table the user doesn't
have write access to, Got status: " + status);
     } catch (AccumuloSecurityException ase) {
 
     }
 
     try {
-      cw2.write(cm1).getStatus();
-      Assert.fail("Expected exception writing conditional mutation to table the user doesn't
have read access to");
+      Status status = cw2.write(cm1).getStatus();
+      Assert.fail("Expected exception writing conditional mutation to table the user doesn't
have read access to. Got status: " + status);
     } catch (AccumuloSecurityException ase) {
 
     }
@@ -1101,7 +1101,7 @@ public class ConditionalWriterIT extends SimpleMacIT {
       else if (cf.equals("data") && cq.equals("x"))
         Assert.assertEquals("Unexpected value in data:x", "a", val);
       else
-        Assert.fail("Saw unexpected column family and qualifier");
+        Assert.fail("Saw unexpected column family and qualifier: " + entry);
     }
 
     ConditionalMutation cm3 = new ConditionalMutation("r1", new Condition("tx", "seq").setValue("1"));
@@ -1136,8 +1136,8 @@ public class ConditionalWriterIT extends SimpleMacIT {
     Result result = cw.write(cm1);
 
     try {
-      result.getStatus();
-      Assert.fail("Expected exception writing conditional mutation to deleted table");
+      Status status = result.getStatus();
+      Assert.fail("Expected exception writing conditional mutation to deleted table. Got
status: " + status);
     } catch (AccumuloException ae) {
       Assert.assertEquals(TableDeletedException.class, ae.getCause().getClass());
     }
@@ -1161,8 +1161,8 @@ public class ConditionalWriterIT extends SimpleMacIT {
     Result result = cw.write(cm1);
 
     try {
-      result.getStatus();
-      Assert.fail("Expected exception writing conditional mutation to offline table");
+      Status status = result.getStatus();
+      Assert.fail("Expected exception writing conditional mutation to offline table. Got
status: " + status);
     } catch (AccumuloException ae) {
       Assert.assertEquals(TableOfflineException.class, ae.getCause().getClass());
     }
@@ -1193,8 +1193,8 @@ public class ConditionalWriterIT extends SimpleMacIT {
     Result result = cw.write(cm1);
 
     try {
-      result.getStatus();
-      Assert.fail("Expected exception using iterator which throws an error");
+      Status status = result.getStatus();
+      Assert.fail("Expected exception using iterator which throws an error, Got status: "
+ status);
     } catch (AccumuloException ae) {
 
     }


Mime
View raw message