This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 4e7a98f10915b959134e13808f3021401012c2d8
Author: Kirk Lund <[email protected]>
AuthorDate: Mon Jan 29 12:24:09 2018 -0800

    GEODE-4418: add FlakyTest category to 
testMemLRUEvictionNDestroyNNumOverflowOnDiskCount
---
 .../internal/cache/eviction/EvictionStatsDUnitTest.java | 17 +++++++----------
 .../cache/eviction/OffHeapEvictionStatsDUnitTest.java   |  5 ++---
 2 files changed, 9 insertions(+), 13 deletions(-)

diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/EvictionStatsDUnitTest.java
 
b/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/EvictionStatsDUnitTest.java
index 001b9d7..1a2d156 100755
--- 
a/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/EvictionStatsDUnitTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/EvictionStatsDUnitTest.java
@@ -14,7 +14,9 @@
  */
 package org.apache.geode.internal.cache.eviction;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
 
 import java.io.File;
 import java.util.Iterator;
@@ -39,7 +41,6 @@ import org.apache.geode.distributed.DistributedSystem;
 import org.apache.geode.internal.OSProcess;
 import org.apache.geode.internal.cache.BucketRegion;
 import org.apache.geode.internal.cache.PartitionedRegion;
-import org.apache.geode.internal.cache.RegionMap;
 import org.apache.geode.internal.cache.control.HeapMemoryMonitor;
 import org.apache.geode.internal.cache.control.InternalResourceManager;
 import org.apache.geode.test.dunit.Assert;
@@ -48,12 +49,12 @@ import org.apache.geode.test.dunit.IgnoredException;
 import org.apache.geode.test.dunit.LogWriterUtils;
 import org.apache.geode.test.dunit.SerializableRunnable;
 import org.apache.geode.test.dunit.VM;
-import org.apache.geode.test.dunit.cache.internal.JUnit4CacheTestCase;
-import org.apache.geode.test.dunit.internal.JUnit4DistributedTestCase;
+import org.apache.geode.test.dunit.cache.CacheTestCase;
 import org.apache.geode.test.junit.categories.DistributedTest;
+import org.apache.geode.test.junit.categories.FlakyTest;
 
 @Category(DistributedTest.class)
-public class EvictionStatsDUnitTest extends JUnit4CacheTestCase {
+public class EvictionStatsDUnitTest extends CacheTestCase {
 
   protected static Cache cache = null;
 
@@ -69,11 +70,6 @@ public class EvictionStatsDUnitTest extends 
JUnit4CacheTestCase {
 
   static int totalNoOfBuckets = 2;
 
-  public EvictionStatsDUnitTest() {
-    super();
-    // TODO Auto-generated constructor stub
-  }
-
   @Override
   public final void postSetUp() throws Exception {
     Host host = Host.getHost(0);
@@ -306,6 +302,7 @@ public class EvictionStatsDUnitTest extends 
JUnit4CacheTestCase {
   }
 
   @Test
+  @Category(FlakyTest.class)
   public void testMemLRUEvictionNDestroyNNumOverflowOnDiskCount() {
     // Ignore this excetion as this can happen if pool is shutting down
     IgnoredException
diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/OffHeapEvictionStatsDUnitTest.java
 
b/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/OffHeapEvictionStatsDUnitTest.java
index 278c774..9ccaed7 100644
--- 
a/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/OffHeapEvictionStatsDUnitTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/internal/cache/eviction/OffHeapEvictionStatsDUnitTest.java
@@ -14,8 +14,8 @@
  */
 package org.apache.geode.internal.cache.eviction;
 
-import static org.apache.geode.distributed.ConfigurationProperties.*;
-import static org.junit.Assert.*;
+import static 
org.apache.geode.distributed.ConfigurationProperties.OFF_HEAP_MEMORY_SIZE;
+import static org.junit.Assert.assertNotNull;
 
 import java.util.Properties;
 
@@ -24,7 +24,6 @@ import org.junit.experimental.categories.Category;
 import org.apache.geode.cache.CacheFactory;
 import org.apache.geode.distributed.DistributedSystem;
 import org.apache.geode.internal.cache.OffHeapTestUtil;
-import org.apache.geode.internal.cache.eviction.EvictionStatsDUnitTest;
 import org.apache.geode.test.dunit.Assert;
 import org.apache.geode.test.dunit.Invoke;
 import org.apache.geode.test.dunit.LogWriterUtils;

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to