Closing 91. Tests for IllegalStateException on getCacheNames when Cac…

…heManager closed.
jsr107 · May 15, 2016 · bc59f8d · bc59f8d
1 parent 6ed24d0
commit bc59f8d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/cache-tests/src/test/java/org/jsr107/tck/CacheManagerTest.java b/cache-tests/src/test/java/org/jsr107/tck/CacheManagerTest.java
@@ -386,7 +386,12 @@ public void close_cachesEmpty() {
     cacheManager.createCache("c2", new MutableConfiguration());
 
     cacheManager.close();
-    assertFalse(cacheManager.getCacheNames().iterator().hasNext());
+    try {
+      cacheManager.getCacheNames();
+      fail();
+    } catch (IllegalStateException e) {
+      //expected
+    }
   }
 
   @Test

diff --git a/cache-tests/src/test/java/org/jsr107/tck/management/CacheManagerManagementTest.java b/cache-tests/src/test/java/org/jsr107/tck/management/CacheManagerManagementTest.java
@@ -123,8 +123,10 @@ public void setUp() throws Exception {
   @After
   public void tearDown() throws MalformedObjectNameException {
     //assertEquals(0, mBeanServer.queryNames(new ObjectName("java.cache:*"), null).size());
-    for (String cacheName : cacheManager.getCacheNames()) {
-      cacheManager.destroyCache(cacheName);
+    if (!cacheManager.isClosed()) {
+      for (String cacheName : cacheManager.getCacheNames()) {
+        cacheManager.destroyCache(cacheName);
+      }
     }
     cacheManager.close();
     //All registered object names should be removed during shutdown