accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [15/17] git commit: ACCUMULO-3055 Fix the 1.6 test to verify what the current state is.
Date Sun, 10 Aug 2014 06:21:08 GMT
ACCUMULO-3055 Fix the 1.6 test to verify what the current state is.


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

Branch: refs/heads/master
Commit: 0277521daa753ac2168b7ebabc57f01baecfc02e
Parents: a6a8784
Author: Josh Elser <elserj@apache.org>
Authored: Sun Aug 10 02:14:13 2014 -0400
Committer: Josh Elser <elserj@apache.org>
Committed: Sun Aug 10 02:14:13 2014 -0400

----------------------------------------------------------------------
 .../minicluster/MiniAccumuloClusterStartStopTest.java    | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/0277521d/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterStartStopTest.java
----------------------------------------------------------------------
diff --git a/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterStartStopTest.java
b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterStartStopTest.java
index 246632c..9e38d09 100644
--- a/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterStartStopTest.java
+++ b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterStartStopTest.java
@@ -41,17 +41,14 @@ public class MiniAccumuloClusterStartStopTest {
     folder.delete();
   }
   
-  // Multiple start()'s failed in 1.5, but apparently is successful in 1.6.0
-  //  @Test
+  @Test
   public void multipleStartsThrowsAnException() throws Exception {
     MiniAccumuloCluster accumulo = new MiniAccumuloCluster(folder.getRoot(), "superSecret");
-    accumulo.start();
-    
+
+    // In 1.6.0, multiple start's did not throw an exception as advertised
     try {
       accumulo.start();
-      Assert.fail("Invoking start() while already started is an error");
-    } catch (IllegalStateException e) {
-      // pass
+      accumulo.start();
     } finally {
       accumulo.stop();
     }


Mime
View raw message