You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ji...@apache.org on 2016/04/08 18:26:36 UTC

[18/36] incubator-geode git commit: GEODE-1171: Preventing accidental cron triggering in AutoBalancerJUnitTest

GEODE-1171: Preventing accidental cron triggering in AutoBalancerJUnitTest

This test creates an Autobalancer with a cron schedule. Upping the
the default scheduler to trigger after 30 minutes, to avoid
accidentally trigging the cron job.

Some tests do appear to exercise the scheduler, so changing this to an
integration test because it starts threads.


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

Branch: refs/heads/feature/GEODE-17-2
Commit: 566dc0b1c3756f958ea89af897b0b8da5872dea3
Parents: 36f3f8d
Author: Dan Smith <up...@apache.org>
Authored: Tue Apr 5 10:42:18 2016 -0700
Committer: Dan Smith <up...@apache.org>
Committed: Tue Apr 5 17:13:07 2016 -0700

----------------------------------------------------------------------
 .../com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/566dc0b1/geode-rebalancer/src/test/java/com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-rebalancer/src/test/java/com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java b/geode-rebalancer/src/test/java/com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java
index cb88022..4acd0b3 100644
--- a/geode-rebalancer/src/test/java/com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java
+++ b/geode-rebalancer/src/test/java/com/gemstone/gemfire/cache/util/AutoBalancerJUnitTest.java
@@ -58,12 +58,12 @@ import com.gemstone.gemfire.internal.cache.PartitionedRegion;
 import com.gemstone.gemfire.internal.cache.control.InternalResourceManager;
 import com.gemstone.gemfire.internal.cache.partitioned.InternalPRInfo;
 import com.gemstone.gemfire.internal.cache.partitioned.LoadProbe;
-import com.gemstone.gemfire.test.junit.categories.UnitTest;
+import com.gemstone.gemfire.test.junit.categories.IntegrationTest;
 
 /**
  * UnitTests for AutoBalancer. All collaborators should be mocked.
  */
-@Category(UnitTest.class)
+@Category(IntegrationTest.class)
 public class AutoBalancerJUnitTest {
   Mockery mockContext;
 
@@ -598,7 +598,7 @@ public class AutoBalancerJUnitTest {
   static Properties getBasicConfig() {
     Properties props = new Properties();
     // every second schedule
-    props.put(AutoBalancer.SCHEDULE, "* * * * * ?");
+    props.put(AutoBalancer.SCHEDULE, "* 0/30 * * * ?");
     return props;
   }
 }