You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/04/15 19:44:07 UTC

incubator-geode git commit: Intial testing of FlakyTest

Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-1233 [created] 40de500e4


Intial testing of FlakyTest


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

Branch: refs/heads/feature/GEODE-1233
Commit: 40de500e40720b6467b7b14dc7ed2bf4ed23e398
Parents: f702bcf
Author: Kirk Lund <kl...@apache.org>
Authored: Fri Apr 15 10:40:26 2016 -0700
Committer: Kirk Lund <kl...@apache.org>
Committed: Fri Apr 15 10:40:26 2016 -0700

----------------------------------------------------------------------
 .../gemfire/distributed/LocatorLauncherRemoteJUnitTest.java   | 4 +++-
 .../partitioned/PersistentPartitionedRegionDUnitTest.java     | 5 ++++-
 .../UniversalMembershipListenerAdapterDUnitTest.java          | 4 ++++
 .../internal/cli/commands/ClientCommandsDUnitTest.java        | 7 +++++--
 gradle/test.gradle                                            | 1 +
 5 files changed, 17 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/40de500e/geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteJUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteJUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteJUnitTest.java
index 52ebe24..1be5ea4 100755
--- a/geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteJUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteJUnitTest.java
@@ -48,6 +48,7 @@ import com.gemstone.gemfire.internal.process.ProcessControllerFactory;
 import com.gemstone.gemfire.internal.process.ProcessStreamReader;
 import com.gemstone.gemfire.internal.process.ProcessType;
 import com.gemstone.gemfire.internal.process.ProcessUtils;
+import com.gemstone.gemfire.test.junit.categories.FlakyTest;
 import com.gemstone.gemfire.test.junit.categories.IntegrationTest;
 
 /**
@@ -272,7 +273,8 @@ public class LocatorLauncherRemoteJUnitTest extends AbstractLocatorLauncherJUnit
       this.errorCollector.addError(e);
     }
   }
-  
+
+  @Category(FlakyTest.class)
   @Test
   public void testStartOverwritesStalePidFile() throws Throwable {
     // create existing pid file

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/40de500e/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
index ba1ddd2..c7cc33f 100644
--- a/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
@@ -32,6 +32,7 @@ import java.util.Set;
 import java.util.concurrent.CountDownLatch;
 
 import org.junit.Ignore;
+import org.junit.experimental.categories.Category;
 
 import com.gemstone.gemfire.DataSerializable;
 import com.gemstone.gemfire.cache.AttributesFactory;
@@ -88,6 +89,7 @@ import com.gemstone.gemfire.test.dunit.SerializableRunnable;
 import com.gemstone.gemfire.test.dunit.VM;
 import com.gemstone.gemfire.test.dunit.Wait;
 import com.gemstone.gemfire.test.dunit.WaitCriterion;
+import com.gemstone.gemfire.test.junit.categories.FlakyTest;
 
 /**
  * Tests the basic use cases for PR persistence.
@@ -1628,7 +1630,8 @@ public class PersistentPartitionedRegionDUnitTest extends PersistentPartitionedR
    * 5. Member A recovers, and gets stuck waiting for member B.
    * @throws Throwable 
   */
-  public void testBug42226() throws Throwable {
+  @Category(FlakyTest.class)
+  public void testBug42226() throws Exception {
     Host host = Host.getHost(0);
     VM vm0 = host.getVM(0);
     VM vm1 = host.getVM(1);

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/40de500e/geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java
index 5851878..cd2788e 100644
--- a/geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java
@@ -23,6 +23,8 @@ import java.util.List;
 import java.util.Properties;
 import java.util.Set;
 
+import org.junit.experimental.categories.Category;
+
 import com.gemstone.gemfire.InternalGemFireException;
 import com.gemstone.gemfire.LogWriter;
 import com.gemstone.gemfire.cache.AttributesFactory;
@@ -54,6 +56,7 @@ import com.gemstone.gemfire.test.dunit.SerializableCallable;
 import com.gemstone.gemfire.test.dunit.SerializableRunnable;
 import com.gemstone.gemfire.test.dunit.VM;
 import com.gemstone.gemfire.test.dunit.Wait;
+import com.gemstone.gemfire.test.junit.categories.FlakyTest;
 
 /**
  * Tests the UniversalMembershipListenerAdapter.
@@ -699,6 +702,7 @@ public class UniversalMembershipListenerAdapterDUnitTest extends ClientServerTes
   /**
    * Tests notification of events for loner bridge clients in server process.
    */
+  //@Category(FlakyTest.class) // GEODE-1174
   public void testSystemClientEventsInServer() throws Exception {
      try {
        doTestSystemClientEventsInServer();

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/40de500e/geode-cq/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ClientCommandsDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-cq/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ClientCommandsDUnitTest.java b/geode-cq/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ClientCommandsDUnitTest.java
index 6fef000..ad1d530 100644
--- a/geode-cq/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ClientCommandsDUnitTest.java
+++ b/geode-cq/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ClientCommandsDUnitTest.java
@@ -83,7 +83,7 @@ import com.gemstone.gemfire.test.junit.categories.FlakyTest;
  * Dunit class for testing gemfire Client commands : list client , describe client 
  * @since 8.0
  */
-@Category({ DistributedTest.class, FlakyTest.class }) // see GEODE-1034
+@Category(DistributedTest.class) // see GEODE-1034
 public class ClientCommandsDUnitTest extends CliCommandTestBase {
 
   private static final long serialVersionUID = 1L;
@@ -595,6 +595,7 @@ public void waitForListClientMbean2(){
   
   }
 
+  @Category(FlakyTest.class) // GEODE-908
   @Test
   public void testListClient() throws Exception {
     setupSystemForListClient();
@@ -1154,7 +1155,8 @@ public void waitForMixedClients(){
     }); 
     
   }
-  
+
+  @Category(FlakyTest.class) // GEODE-910
   @Test
   public void testDescribeClientForNonSubscribedClient() throws Exception {
     setUpNonSubscribedClient();
@@ -1230,6 +1232,7 @@ public void waitForMixedClients(){
     
   }
 
+  @Category(FlakyTest.class) // GEODE-1227
   @Test
   public void testDescribeMixClientWithServers() throws Exception {
     String[] clientIds = setupSystemWithSubAndNonSubClient();    

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/40de500e/gradle/test.gradle
----------------------------------------------------------------------
diff --git a/gradle/test.gradle b/gradle/test.gradle
index 2dd7521..08238cb 100644
--- a/gradle/test.gradle
+++ b/gradle/test.gradle
@@ -154,6 +154,7 @@ subprojects {
       excludeCategories 'com.gemstone.gemfire.test.junit.categories.UnitTest'
       excludeCategories 'com.gemstone.gemfire.test.junit.categories.IntegrationTest'
       includeCategories 'com.gemstone.gemfire.test.junit.categories.DistributedTest'
+      excludeCategories 'com.gemstone.gemfire.test.junit.categories.FlakyTest'
       excludeCategories 'com.gemstone.gemfire.test.junit.categories.PerformanceTest'
       excludeCategories 'com.gemstone.gemfire.test.junit.categories.HydraTest'
       excludeCategories 'com.gemstone.gemfire.test.junit.categories.ContainerTest'