http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/redis/HashesJUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/redis/HashesJUnitTest.java b/geode-core/src/test/java/org/apache/geode/redis/HashesJUnitTest.java index ddfac6a..2db84b8 100755 --- a/geode-core/src/test/java/org/apache/geode/redis/HashesJUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/redis/HashesJUnitTest.java @@ -19,6 +19,7 @@ package org.apache.geode.redis; import org.apache.geode.cache.CacheFactory; import org.apache.geode.cache.GemFireCache; import org.apache.geode.internal.AvailablePortHelper; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.IntegrationTest; import org.apache.geode.redis.GeodeRedisServer; @@ -131,6 +132,7 @@ public class HashesJUnitTest { assertTrue(retSet.containsAll(keys)); } + @Category(FlakyTest.class) // GEODE-1942 @Test public void testHIncrBy() { String key = randString();
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java b/geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java index 4e42b1f..165e269 100644 --- a/geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java @@ -204,6 +204,7 @@ public class RedisDistDUnitTest extends JUnit4DistributedTestCase { /** * Just make sure there are no unexpected server crashes */ + @Category(FlakyTest.class) // GEODE-1697 @Test public void testConcOps() throws Exception { http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/ClientAuthenticationDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/ClientAuthenticationDUnitTest.java b/geode-core/src/test/java/org/apache/geode/security/ClientAuthenticationDUnitTest.java index 94d1526..053bb2e 100644 --- a/geode-core/src/test/java/org/apache/geode/security/ClientAuthenticationDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/ClientAuthenticationDUnitTest.java @@ -51,6 +51,7 @@ public class ClientAuthenticationDUnitTest extends ClientAuthenticationTestCase doTestInvalidCredentials(false); } + @Category(FlakyTest.class) // GEODE-1683 @Test public void testInvalidAuthInit() throws Exception { doTestInvalidAuthInit(false); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/DeltaClientPostAuthorizationDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/DeltaClientPostAuthorizationDUnitTest.java b/geode-core/src/test/java/org/apache/geode/security/DeltaClientPostAuthorizationDUnitTest.java index ba27840..61708c8 100644 --- a/geode-core/src/test/java/org/apache/geode/security/DeltaClientPostAuthorizationDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/DeltaClientPostAuthorizationDUnitTest.java @@ -40,6 +40,7 @@ import org.apache.geode.security.generator.AuthzCredentialGenerator; import org.apache.geode.security.generator.CredentialGenerator; import org.apache.geode.test.dunit.VM; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.SecurityTest; /** @@ -64,6 +65,7 @@ public class DeltaClientPostAuthorizationDUnitTest extends ClientAuthorizationTe closeCache(); } + @Category(FlakyTest.class) // GEODE-1502 @Test public void testPutPostOpNotifications() throws Exception { OperationWithAction[] allOps = allOps(); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientAuthDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/IntegratedClientAuthDUnitTest.java b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientAuthDUnitTest.java index 6809942..2aa633c 100644 --- a/geode-core/src/test/java/org/apache/geode/security/IntegratedClientAuthDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientAuthDUnitTest.java @@ -28,11 +28,13 @@ import org.apache.geode.cache.client.ClientRegionFactory; import org.apache.geode.cache.client.ClientRegionShortcut; import org.apache.geode.test.dunit.IgnoredException; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.SecurityTest; @Category({ DistributedTest.class, SecurityTest.class }) public class IntegratedClientAuthDUnitTest extends AbstractSecureServerDUnitTest { + @Category(FlakyTest.class) // GEODE-1877 @Test public void authWithCorrectPasswordShouldPass() { client1.invoke("logging in super-user with correct password", () -> { @@ -46,6 +48,7 @@ public class IntegratedClientAuthDUnitTest extends AbstractSecureServerDUnitTest }); } + @Category(FlakyTest.class) // GEODE-1875 @Test public void authWithIncorrectPasswordShouldFail() { IgnoredException.addIgnoredException(AuthenticationFailedException.class.getName()); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientRegionClearAuthDistributedTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/IntegratedClientRegionClearAuthDistributedTest.java b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientRegionClearAuthDistributedTest.java index eda16b7..4c4112d 100644 --- a/geode-core/src/test/java/org/apache/geode/security/IntegratedClientRegionClearAuthDistributedTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/IntegratedClientRegionClearAuthDistributedTest.java @@ -25,11 +25,13 @@ import org.apache.geode.cache.client.ClientCacheFactory; import org.apache.geode.cache.client.ClientRegionShortcut; import org.apache.geode.test.dunit.SerializableRunnable; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.SecurityTest; @Category({ DistributedTest.class, SecurityTest.class }) public class IntegratedClientRegionClearAuthDistributedTest extends AbstractSecureServerDUnitTest { + @Category(FlakyTest.class) // GEODE-1876 @Test public void testRegionClear() throws InterruptedException { // Verify that an unauthorized user can't clear the region http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java index 040bbf0..dc2ab8f 100644 --- a/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java @@ -34,6 +34,7 @@ import org.apache.geode.test.dunit.NetworkUtils; import org.apache.geode.test.dunit.VM; import org.apache.geode.test.dunit.cache.internal.JUnit4CacheTestCase; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.SecurityTest; import org.junit.Ignore; import org.junit.Test; @@ -78,6 +79,7 @@ public class IntegratedSecurityCacheLifecycleDistributedTest extends JUnit4Cache }); } + @Category(FlakyTest.class) // GEODE-1662 @Test public void initAndCloseTest() throws Exception { connect(); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleIntegrationTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleIntegrationTest.java b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleIntegrationTest.java index 0411580..2295552 100644 --- a/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleIntegrationTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleIntegrationTest.java @@ -16,9 +16,10 @@ */ package org.apache.geode.security; -import static org.apache.geode.distributed.ConfigurationProperties.*; -import static org.assertj.core.api.Assertions.*; -import static org.mockito.Mockito.*; +import static org.apache.geode.distributed.ConfigurationProperties.LOCATORS; +import static org.apache.geode.distributed.ConfigurationProperties.MCAST_PORT; +import static org.apache.geode.distributed.ConfigurationProperties.SECURITY_MANAGER; +import static org.assertj.core.api.Assertions.assertThat; import java.util.Properties; @@ -26,9 +27,9 @@ import org.apache.geode.cache.Cache; import org.apache.geode.cache.CacheFactory; import org.apache.geode.internal.security.IntegratedSecurityService; import org.apache.geode.internal.security.SecurityService; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.IntegrationTest; import org.apache.geode.test.junit.categories.SecurityTest; - import org.junit.After; import org.junit.Before; import org.junit.Test; @@ -63,6 +64,7 @@ public class IntegratedSecurityCacheLifecycleIntegrationTest { } } + @Category(FlakyTest.class) // GEODE-1661 @Test public void initAndCloseTest () { SpySecurityManager ssm = (SpySecurityManager)securityService.getSecurityManager(); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java b/geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java index db9f717..cf0df1b 100644 --- a/geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java @@ -50,6 +50,7 @@ import org.apache.geode.management.internal.cli.result.CommandResult; import org.apache.geode.management.internal.cli.util.CommandStringBuilder; import org.apache.geode.pdx.SimpleClass; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import org.apache.geode.test.junit.categories.SecurityTest; import org.apache.geode.test.junit.runners.CategoryWithParameterizedRunnerFactory; @@ -179,6 +180,7 @@ public class PDXPostProcessorDUnitTest extends AbstractSecureServerDUnitTest { assertEquals(pp.getCount(), 2); } + @Category(FlakyTest.class) // GEODE-1719 @Test public void testGfshCommand(){ // have client2 input some domain data into the region http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqResultSetUsingPoolDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqResultSetUsingPoolDUnitTest.java b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqResultSetUsingPoolDUnitTest.java index 6e2fc74..d939de7 100644 --- a/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqResultSetUsingPoolDUnitTest.java +++ b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqResultSetUsingPoolDUnitTest.java @@ -24,6 +24,7 @@ import static org.junit.Assert.*; import org.apache.geode.test.dunit.cache.internal.JUnit4CacheTestCase; import org.apache.geode.test.dunit.internal.JUnit4DistributedTestCase; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; import java.util.Collection; import java.util.HashSet; @@ -961,6 +962,7 @@ public class CqResultSetUsingPoolDUnitTest extends JUnit4CacheTestCase { * * @throws Exception */ + @Category(FlakyTest.class) // GEODE-1251 @Test public void testCqResultsCachingWithFailOver() throws Exception { http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/PrCqUsingPoolDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/PrCqUsingPoolDUnitTest.java b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/PrCqUsingPoolDUnitTest.java index 45a43af..c6d3cdf 100644 --- a/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/PrCqUsingPoolDUnitTest.java +++ b/geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/PrCqUsingPoolDUnitTest.java @@ -1274,7 +1274,7 @@ public class PrCqUsingPoolDUnitTest extends JUnit4CacheTestCase { * thus making the query data and region data inconsistent. * @throws Exception */ - @Category(FlakyTest.class) // GEODE-1181: random ports, eats exceptions (fixed some), async behavior + @Category(FlakyTest.class) // GEODE-1181, 1253: random ports, eats exceptions (fixed some), async behavior @Test public void testEventsDuringQueryExecution() throws Exception { final Host host = Host.getHost(0); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/8929e93b/geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java b/geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java index ef82cd5..9d0ed09 100755 --- a/geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java +++ b/geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java @@ -93,6 +93,7 @@ import org.apache.geode.test.dunit.VM; import org.apache.geode.test.dunit.Wait; import org.apache.geode.test.dunit.WaitCriterion; import org.apache.geode.test.junit.categories.DistributedTest; +import org.apache.geode.test.junit.categories.FlakyTest; /** * Tests putAll for c/s. Also tests removeAll @@ -3508,6 +3509,7 @@ public class PutAllCSDUnitTest extends ClientServerTestCase { /** * basically same test as testVersionsOnClientsWithNotificationsOnly but also do a removeAll */ + @Category(FlakyTest.class) // GEODE-1419 @Test public void testRAVersionsOnClientsWithNotificationsOnly() { final String title = "testRAVersionsInClients";
