You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@usergrid.apache.org by gr...@apache.org on 2015/11/18 00:47:16 UTC

[01/21] usergrid git commit: Add references to JIRA issues in @Ignore annotations

Repository: usergrid
Updated Branches:
  refs/heads/master fdcae67fa -> 6f3541be6


Add references to JIRA issues in @Ignore annotations


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/f1aa4506
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/f1aa4506
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/f1aa4506

Branch: refs/heads/master
Commit: f1aa4506da471ad01ce0fb0c57af9cdc33ea1a70
Parents: 27de65c
Author: Dave Johnson <sn...@apache.org>
Authored: Fri Nov 13 10:51:26 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Fri Nov 13 10:51:26 2015 -0500

----------------------------------------------------------------------
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java    | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java    | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
index 3d85c0e..380a683 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
@@ -34,7 +34,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes.
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime1IT extends AbstractSchedulerRuntimeIT {
 	private static final Logger logger = LoggerFactory.getLogger(SchedulerRuntime1IT.class.getName());
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
index 88f5311..2c2aa40 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
@@ -34,7 +34,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime2IT extends AbstractSchedulerRuntimeIT {
 	private static final Logger logger = LoggerFactory.getLogger(SchedulerRuntime2IT.class.getName());
     /** Test the scheduler ramps up correctly when there are more jobs to be read after a pause */

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
index dfe4f52..5fc6398 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
@@ -33,7 +33,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime3IT extends AbstractSchedulerRuntimeIT {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
index 2cf1c42..fbd49f2 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
@@ -31,7 +31,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime4IT extends AbstractSchedulerRuntimeIT {
     /**
      * Test the scheduler ramps up correctly when there are more jobs to be read after a pause when the job specifies

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
index aa4f018..1b5cbab 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
@@ -31,7 +31,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime5IT extends AbstractSchedulerRuntimeIT {
     /**
      * Test the scheduler ramps up correctly when there are more jobs to be read after a pause when the job specifies

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
index d4cb657..239101c 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
@@ -35,7 +35,7 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime6IT extends AbstractSchedulerRuntimeIT {
 
     /**

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
index 0cbe151..091c875 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
@@ -32,7 +32,7 @@ import static org.junit.Assert.assertTrue;
  */
 
 //@org.junit.Ignore( "Todd you need to take a look at this since it's not clear to me what was intended in this test." )
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime7IT extends AbstractSchedulerRuntimeIT {
 
     /** Test that we're only running once, even when a job exceeds the heartbeat time */

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f1aa4506/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
index a8c88ff..65b5e67 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
@@ -35,7 +35,7 @@ import static org.junit.Assert.assertFalse;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared spring context. Need to re-evaluate")
+@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
 public class SchedulerRuntime8IT extends AbstractSchedulerRuntimeIT {
 
     /**


[15/21] usergrid git commit: Further categorization of tests.

Posted by gr...@apache.org.
Further categorization of tests.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/a9c1e840
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/a9c1e840
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/a9c1e840

Branch: refs/heads/master
Commit: a9c1e840172fb399a1b4212b4d9f9237b271ff61
Parents: 6af43dc
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 09:31:30 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 09:31:30 2015 -0500

----------------------------------------------------------------------
 .../src/test/java/org/apache/usergrid/batch/BulkTestUtils.java  | 1 -
 .../org/apache/usergrid/count/BatchCountParallelismTest.java    | 5 ++++-
 .../src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java  | 4 +++-
 .../apache/usergrid/security/providers/FacebookProviderIT.java  | 4 +++-
 4 files changed, 10 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/a9c1e840/stack/core/src/test/java/org/apache/usergrid/batch/BulkTestUtils.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/BulkTestUtils.java b/stack/core/src/test/java/org/apache/usergrid/batch/BulkTestUtils.java
index c88175b..d711094 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/BulkTestUtils.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/BulkTestUtils.java
@@ -23,7 +23,6 @@ import org.apache.usergrid.batch.repository.JobDescriptor;
 
 
 /** @author zznate */
-@Ignore("Not a test")
 public class BulkTestUtils {
 
     public static JobFactory getBulkJobFactory() {

http://git-wip-us.apache.org/repos/asf/usergrid/blob/a9c1e840/stack/core/src/test/java/org/apache/usergrid/count/BatchCountParallelismTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/count/BatchCountParallelismTest.java b/stack/core/src/test/java/org/apache/usergrid/count/BatchCountParallelismTest.java
index 37beeac..78c58a4 100644
--- a/stack/core/src/test/java/org/apache/usergrid/count/BatchCountParallelismTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/count/BatchCountParallelismTest.java
@@ -28,9 +28,11 @@ import java.util.concurrent.Future;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicLong;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -62,7 +64,8 @@ public class BatchCountParallelismTest {
 
 
     @Test
-    @Ignore("This test causes the build to hang when all stack tests are run")
+    @Category(ExperimentalTest.class)
+    // "This test causes the build to hang when all stack tests are run"
     public void verifyConcurrentAdd() throws Exception {
 
         final long startCount = batcher.invocationCounter.count();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/a9c1e840/stack/core/src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java b/stack/core/src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java
index 0493d31..524c5ce 100644
--- a/stack/core/src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/utils/UUIDUtilsTest.java
@@ -27,8 +27,10 @@ import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 import java.util.concurrent.TimeUnit;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -142,7 +144,7 @@ public class UUIDUtilsTest {
 
 
     @Test
-    @Ignore("This test is timing dependent.")
+    @Category(ExperimentalTest.class)
     @SuppressWarnings("unchecked")
     public void verifyOrderingTsOnlyAndUnique() {
         int count = 500;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/a9c1e840/stack/services/src/test/java/org/apache/usergrid/security/providers/FacebookProviderIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/security/providers/FacebookProviderIT.java b/stack/services/src/test/java/org/apache/usergrid/security/providers/FacebookProviderIT.java
index d22647f..cde0255 100644
--- a/stack/services/src/test/java/org/apache/usergrid/security/providers/FacebookProviderIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/security/providers/FacebookProviderIT.java
@@ -20,6 +20,7 @@ package org.apache.usergrid.security.providers;
 import java.util.Map;
 import java.util.UUID;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.junit.BeforeClass;
 import org.junit.ClassRule;
 import org.junit.Ignore;
@@ -37,6 +38,7 @@ import org.apache.usergrid.persistence.entities.Application;
 import org.apache.usergrid.persistence.entities.User;
 import org.apache.usergrid.persistence.index.impl.ElasticSearchResource;
 import org.apache.usergrid.utils.MapUtils;
+import org.junit.experimental.categories.Category;
 
 import static org.apache.usergrid.TestHelper.newUUIDString;
 import static org.apache.usergrid.TestHelper.uniqueOrg;
@@ -71,7 +73,7 @@ public class FacebookProviderIT {
 
 
     @Test
-    @Ignore("Requires Facebook credentials")
+    @Category(ExperimentalTest.class)
     public void verifyGetOrCreateOk() throws Exception {
         Application application = setup.getEmf().getEntityManager( applicationId ).getApplication();
         Map fb_user = MapUtils.hashMap( "id", "12345678" ).map( "name", "Facebook User" ).map( "username", "fb.user" );


[06/21] usergrid git commit: Correct Jacoco profile command arg

Posted by gr...@apache.org.
Correct Jacoco profile command arg


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/bba77f0a
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/bba77f0a
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/bba77f0a

Branch: refs/heads/master
Commit: bba77f0af590a634415694d8b1f59e65bb85b2ff
Parents: 40cd379
Author: Dave Johnson <sn...@apache.org>
Authored: Mon Nov 16 13:10:44 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Mon Nov 16 13:10:44 2015 -0500

----------------------------------------------------------------------
 stack/Coverage.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/bba77f0a/stack/Coverage.md
----------------------------------------------------------------------
diff --git a/stack/Coverage.md b/stack/Coverage.md
index e09d951..0f6e0a8 100644
--- a/stack/Coverage.md
+++ b/stack/Coverage.md
@@ -5,7 +5,7 @@ To generate Jacoco coverage reports for Usergrid, do a build that runs the tests
 
 For example:
 
-    mvn install
+    mvn install -Pjacoco
 
 
 Note that all tests must pass, or some Jacoco files will be missing and report generation is likely to fail complete.


[05/21] usergrid git commit: Unignoring and fixing tests more tests.

Posted by gr...@apache.org.
Unignoring and fixing tests more tests.


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

Branch: refs/heads/master
Commit: 40cd37988b72ceaabe7d265eb7c4e5c6e0dd1e95
Parents: 5c85b75
Author: Dave Johnson <sn...@apache.org>
Authored: Mon Nov 16 09:18:05 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Mon Nov 16 09:18:05 2015 -0500

----------------------------------------------------------------------
 .../org/apache/usergrid/persistence/CounterIT.java     |  1 -
 .../usergrid/persistence/EntityConnectionsIT.java      | 10 ++++------
 .../org/apache/usergrid/persistence/EntityTest.java    |  2 --
 .../usergrid/persistence/GeoQueryBooleanTest.java      |  1 -
 .../graph/GraphManagerShardConsistencyIT.java          |  1 -
 .../collection/paging/PagingResourceIT.java            | 13 +++++--------
 6 files changed, 9 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java b/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
index 54302ad..e558bad 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
@@ -179,7 +179,6 @@ public class CounterIT extends AbstractCoreIT {
 
 
     @Test
-    @Ignore()
     public void testCommunityCounters() throws Exception {
 
         EntityManager em = setup.getEmf().getEntityManager( setup.getEmf().getManagementAppId() );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/core/src/test/java/org/apache/usergrid/persistence/EntityConnectionsIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityConnectionsIT.java b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityConnectionsIT.java
index 47b61e9..f5d3ce4 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityConnectionsIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityConnectionsIT.java
@@ -341,7 +341,6 @@ public class EntityConnectionsIT extends AbstractCoreIT {
 
 
     @Test
-    @Ignore("This is broken, and needs fixed after the refactor")
     public void testConnectionsIterable() throws Exception {
         EntityManager em = app.getEntityManager();
         assertNotNull( em );
@@ -356,7 +355,7 @@ public class EntityConnectionsIT extends AbstractCoreIT {
 
 
         final int connectionCount = 100;
-        final List<Entity> things = new ArrayList<>( connectionCount );
+        final Map<UUID, Entity> things = new HashMap<>();
 
         for(int i = 0; i < connectionCount; i ++){
             Map<String, Object> data = new HashMap<String, Object>();
@@ -366,7 +365,7 @@ public class EntityConnectionsIT extends AbstractCoreIT {
 
             em.createConnection( firstUserEntity, "likes", entity );
 
-            things.add( entity );
+            things.put( entity.getUuid(), entity );
         }
 
 
@@ -380,15 +379,14 @@ public class EntityConnectionsIT extends AbstractCoreIT {
         int checkedIndex = 0;
         for(; checkedIndex < connectionCount && itr.hasNext(); checkedIndex ++){
             final Entity returned = ( Entity ) itr.next();
-            final Entity expected = things.get( checkedIndex );
+            final Entity expected = things.get( returned.getUuid() );
 
             assertEquals("Entity expected", expected, returned);
         }
 
         assertEquals("Checked all entities", connectionCount, checkedIndex  );
-
-
     }
+
 //
 //
 //    @Test

http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/core/src/test/java/org/apache/usergrid/persistence/EntityTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityTest.java b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityTest.java
index e9e6163..1e8ded5 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityTest.java
@@ -39,7 +39,6 @@ public class EntityTest {
 
 
     @Test
-    // @Ignore( "Fix this then enable EntityTest.testEntityClasses:45 » ConcurrentModification" )
     public void testEntityClasses() throws Exception {
         logger.info( "testEntityClasses" );
 
@@ -75,7 +74,6 @@ public class EntityTest {
 
     @SuppressWarnings("unchecked")
     @Test
-    // @Ignore( "Fix this and enable: EntityTest.testJson:83 » ConcurrentModification" )
     public void testJson() throws Exception {
 
         User user = new User();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/core/src/test/java/org/apache/usergrid/persistence/GeoQueryBooleanTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/GeoQueryBooleanTest.java b/stack/core/src/test/java/org/apache/usergrid/persistence/GeoQueryBooleanTest.java
index d780f85..3c04ce6 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/GeoQueryBooleanTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/GeoQueryBooleanTest.java
@@ -108,7 +108,6 @@ public class GeoQueryBooleanTest extends AbstractCoreIT {
 
 
     @Test
-    //@Ignore // work in progress
     public void testGeoQueryWithNot() throws Exception {
 
         log.info( "GeoQueryBooleanTest.testGeoQueryWithOr" );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
index a9f9cb7..9000c75 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
@@ -80,7 +80,6 @@ import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.fail;
 
 
-//@Ignore( "Kills cassandra, needs to be part of functional testing" )
 public class GraphManagerShardConsistencyIT {
     private static final Logger log = LoggerFactory.getLogger( GraphManagerShardConsistencyIT.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/40cd3798/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/paging/PagingResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/paging/PagingResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/paging/PagingResourceIT.java
index 2e3a163..e3ec07c 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/paging/PagingResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/paging/PagingResourceIT.java
@@ -223,12 +223,9 @@ public class PagingResourceIT extends AbstractRestIT {
      * Pages through entities that are connected to each other
      * @throws IOException
      */
-    @Ignore("This does not return a page for any entities. It just keeps returning them in bulk."
-            + " Not sure about intended functionality")
     @Test
     public void pageThroughConnectedEntities() throws IOException {
 
-
         long created = 0;
         int numOfEntities = 100;
         int numOfPages = 10;
@@ -238,7 +235,7 @@ public class PagingResourceIT extends AbstractRestIT {
 
         for ( created = 1; created <= numOfEntities; created++ ) {
 
-            entityPayload.put( "name", "value" + created );
+            entityPayload.put( "name", created );
             Entity entity = new Entity( entityPayload );
             entity = this.app().collection( collectionName ).post( entity );
             refreshIndex();
@@ -252,11 +249,11 @@ public class PagingResourceIT extends AbstractRestIT {
 
         refreshIndex();
 
-        Collection colConnection =  this.app().collection( collectionName ).entity(connectedEntity).connection("likes").get();
-        assertNotNull(colConnection);
-        assertNotNull( colConnection.getCursor() );
-        pageAndVerifyEntities(collectionName, null, numOfPages, numOfEntities);
+        QueryParameters qp = new QueryParameters();
+        qp.setQuery("select * order by created asc");
+        qp.setLimit(10);
 
+        pageAndVerifyEntities(collectionName, qp, numOfPages, numOfEntities);
     }
 
 


[08/21] usergrid git commit: Merge branch 'master' into ignores

Posted by gr...@apache.org.
Merge branch 'master' into ignores


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/13853662
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/13853662
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/13853662

Branch: refs/heads/master
Commit: 13853662ee07ab4ec5b812fd3d5fe42ef8b99d9a
Parents: f2cbf1a fdcae67
Author: Dave Johnson <sn...@apache.org>
Authored: Mon Nov 16 13:53:02 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Mon Nov 16 13:53:02 2015 -0500

----------------------------------------------------------------------
 .../usergrid/corepersistence/CoreModule.java    |  2 +-
 .../asyncevents/AsyncEventsSchedulerFig.java    | 18 ------------
 .../pipeline/PipelineModule.java                |  2 +-
 .../pipeline/PipelineOperations.java            | 30 --------------------
 .../read/search/SearchCollectionFilter.java     |  2 --
 5 files changed, 2 insertions(+), 52 deletions(-)
----------------------------------------------------------------------



[12/21] usergrid git commit: Remove unnecessary ignores in some places, add new ignores for broken tests that were once excluded via moms.

Posted by gr...@apache.org.
Remove unnecessary ignores in some places, add new ignores for broken tests that were once excluded via moms.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/41dde128
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/41dde128
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/41dde128

Branch: refs/heads/master
Commit: 41dde128cc04d211a29bf5af13520a7755246325
Parents: 99b932a
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 07:55:32 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 07:55:32 2015 -0500

----------------------------------------------------------------------
 .../usergrid/batch/job/CountdownLatchJob.java   |  1 -
 .../usergrid/batch/job/DelayExecution.java      |  1 -
 .../usergrid/batch/job/DelayHeartbeat.java      |  1 -
 .../usergrid/batch/job/FailureJobExecution.java |  1 -
 .../usergrid/persistence/CoreSchemaManager.java |  1 -
 .../mvcc/stage/delete/UniqueCleanupTest.java    |  1 -
 .../apache/usergrid/rest/NotificationsIT.java   |  2 +
 .../applications/ApplicationResourceIT.java     | 61 ----------------
 .../users/extensions/TestResource.java          |  1 -
 .../rest/applications/utils/TestUtils.java      |  1 -
 .../usergrid/rest/management/AdminUsersIT.java  |  1 -
 .../rest/management/ExportResourceIT.java       | 22 +-----
 .../rest/management/ImportResourceIT.java       | 11 +--
 .../management/export/ExportServiceIT.java      | 75 --------------------
 .../management/importer/ImportServiceIT.java    |  1 -
 15 files changed, 4 insertions(+), 177 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/core/src/test/java/org/apache/usergrid/batch/job/CountdownLatchJob.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/CountdownLatchJob.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/CountdownLatchJob.java
index 53996a2..4149150 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/CountdownLatchJob.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/CountdownLatchJob.java
@@ -33,7 +33,6 @@ import org.apache.usergrid.batch.JobExecution;
  * @author tnine
  */
 @Component("countdownLatch")
-@Ignore("Not a test")
 public class CountdownLatchJob implements Job {
 
     private CountDownLatch latch = null;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayExecution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayExecution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayExecution.java
index aaa7071..4c8bda6 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayExecution.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayExecution.java
@@ -35,7 +35,6 @@ import org.apache.usergrid.batch.JobExecution;
  * @author tnine
  */
 @Component("delayExecution")
-@Ignore("Not a test")
 public class DelayExecution implements Job {
 
     private static final Logger logger = LoggerFactory.getLogger( DelayExecution.class );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayHeartbeat.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayHeartbeat.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayHeartbeat.java
index 0fc104b..391b18b 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayHeartbeat.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/DelayHeartbeat.java
@@ -35,7 +35,6 @@ import org.apache.usergrid.batch.JobExecution;
  * @author tnine
  */
 @Component("delayHeartbeat")
-@Ignore("Not a test")
 public class DelayHeartbeat implements Job {
 
     private static final Logger logger = LoggerFactory.getLogger( DelayHeartbeat.class );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/core/src/test/java/org/apache/usergrid/batch/job/FailureJobExecution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/FailureJobExecution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/FailureJobExecution.java
index b31b520..5072e4a 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/FailureJobExecution.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/FailureJobExecution.java
@@ -33,7 +33,6 @@ import org.apache.usergrid.batch.JobExecution;
  * @author tnine
  */
 @Component("failureJobExceuction")
-@Ignore("Not a test")
 public class FailureJobExecution implements Job {
 
     private CountDownLatch latch = null;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/core/src/test/java/org/apache/usergrid/persistence/CoreSchemaManager.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/CoreSchemaManager.java b/stack/core/src/test/java/org/apache/usergrid/persistence/CoreSchemaManager.java
index adecc40..e9de76e 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/CoreSchemaManager.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/CoreSchemaManager.java
@@ -33,7 +33,6 @@ import me.prettyprint.hector.api.Cluster;
 
 
 /** @author zznate */
-@Ignore( "Not a test" )
 public class CoreSchemaManager implements SchemaManager {
     private static final Logger LOG = LoggerFactory.getLogger( CoreSchemaManager.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/delete/UniqueCleanupTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/delete/UniqueCleanupTest.java b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/delete/UniqueCleanupTest.java
index c1f76f2..e361f42 100644
--- a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/delete/UniqueCleanupTest.java
+++ b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/delete/UniqueCleanupTest.java
@@ -430,7 +430,6 @@ public class UniqueCleanupTest {
 //    /**
 //     * Tests what happens when our listeners are VERY slow
 //     */
-////    @Ignore( "Test is a work in progress" )
 //    @Test( timeout = 10000 )
 //    public void multipleListenerMultipleVersionsNoThreadsToRun()
 //            throws ExecutionException, InterruptedException, ConnectionException {

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
index 22785dc..63e7b49 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
@@ -35,6 +35,7 @@ import org.junit.After;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -43,6 +44,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Test creating, sending and paging through Notifications via the REST API.
  */
+@Ignore
 public class NotificationsIT extends org.apache.usergrid.rest.test.resource.AbstractRestIT {
     private static final Logger logger = LoggerFactory.getLogger( NotificationsIT.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
index cd42262..5915eca 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
@@ -225,40 +225,6 @@ public class ApplicationResourceIT extends AbstractRestIT {
 
     }
 
-    /**
-     * Verifies that we return JSON even when text/html is requested.
-     * (for backwards compatibility)
-     */
-    @Test
-    @Ignore("no longer relevant - this type of backwards compatibility no longer needed")
-    public void jsonForAcceptsTextHtml() throws Exception {
-
-        //Create the organization resource
-        OrganizationResource orgResource = clientSetup.getRestClient()
-            .management().orgs().org( clientSetup.getOrganizationName() );
-
-        //retrieve the credentials
-        Credentials orgCredentials = new Credentials( orgResource.credentials().get(ApiResponse.class));
-        String clientId = orgCredentials.getClientId();
-        String clientSecret = orgCredentials.getClientSecret();
-
-        //retrieve the users collection, setting the "Accept" header to text/html
-        Invocation.Builder builder = this.app().collection( "users" ).getTarget()
-            //Add the org credentials to the query
-            .queryParam( "grant_type", "client_credentials" )
-            .queryParam( "client_id", clientId )
-            .queryParam( "client_secret", clientSecret )
-            .request();
-
-        ApiResponse apiResponse = builder
-            .accept(MediaType.TEXT_HTML)
-            .get(ApiResponse.class);
-
-        Collection users = new Collection(apiResponse);
-        //make sure that a valid response is returned without error
-        assertNotNull(users);
-        assertNull(users.getResponse().getError());
-    }
 
     /**
      * Retrieve an application using password credentials
@@ -762,33 +728,6 @@ public class ApplicationResourceIT extends AbstractRestIT {
         assertNotNull("It has expires_in.", apiResponse.getExpirationDate());
     }
 
-    /**
-     * Ensure that the Apigee Mobile Analytics config returns valid JSON
-     */
-    @Test
-    @Ignore("No longer relevant - Apigee internal")
-    public void validateApigeeApmConfigAPP() throws IOException {
-        String orgName = clientSetup.getOrganizationName().toLowerCase();
-        String appName = clientSetup.getAppName().toLowerCase();
-
-        try {
-            //GET the APM endpoint
-            String response = target().path( String.format( "/%s/%s/apm/apigeeMobileConfig", orgName, appName ) )
-                .request()
-                .accept( MediaType.APPLICATION_JSON )
-                .get(String.class);
-            //Parse the response
-            JsonNode node = mapper.readTree(response);
-
-            //if things are kosher then JSON should have value for instaOpsApplicationId
-            assertTrue("it's valid json for APM", node.has("instaOpsApplicationId"));
-        } catch (ClientErrorException uie) {
-            //Validate that APM config exists
-            assertNotEquals("APM Config API exists", Response.Status.NOT_FOUND,
-                uie.getResponse().getStatus()); //i.e It should not be "Not Found"
-        }
-    }
-
 
     /**
      * Retrieve an application token using organization credentials

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/users/extensions/TestResource.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/users/extensions/TestResource.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/users/extensions/TestResource.java
index b74a162..969430e 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/users/extensions/TestResource.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/users/extensions/TestResource.java
@@ -29,7 +29,6 @@ import org.springframework.stereotype.Component;
 import org.apache.usergrid.rest.applications.users.AbstractUserExtensionResource;
 
 
-@Ignore("Not a test")
 @Component("TestResource")
 @Scope("prototype")
 @Produces(MediaType.APPLICATION_JSON)

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/applications/utils/TestUtils.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/utils/TestUtils.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/utils/TestUtils.java
index fa8356d..9a7d71c 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/utils/TestUtils.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/utils/TestUtils.java
@@ -27,7 +27,6 @@ import org.apache.usergrid.utils.UUIDUtils;
 
 
 /** @author tnine */
-@Ignore("Not a test")
 public class TestUtils {
 
     /** Get the uuid at the given index for the root node.  If it doesn't exist, null is returned */

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
index 3aed997..fba0c43 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
@@ -437,7 +437,6 @@ public class AdminUsersIT extends AbstractRestIT {
 
 //    // TODO: will work once resetpw viewables work
 //    @Test
-//    @Ignore( "causes problems in build" )
 //    public void passwordResetIncorrectUserName() throws Exception {
 //
 //        String email = "test2@usergrid.com";

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
index 939363f..79590b8 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
@@ -32,6 +32,7 @@ import java.util.UUID;
 import static org.junit.Assert.*;
 
 
+@Ignore
 public class ExportResourceIT extends AbstractRestIT {
 
 
@@ -40,7 +41,6 @@ public class ExportResourceIT extends AbstractRestIT {
 
     }
 
-    @Ignore
     @Test
     public void exportApplicationUUIDRetTest() throws Exception {
 
@@ -54,7 +54,6 @@ public class ExportResourceIT extends AbstractRestIT {
 
     }
 
-    @Ignore
     @Test
     public void exportCollectionUUIDRetTest() throws Exception {
 
@@ -81,7 +80,6 @@ public class ExportResourceIT extends AbstractRestIT {
      * Check that you can get the org export uuid returned and that you can check the status of the job using that uuid.
      * @throws Exception
      */
-    @Ignore
     @Test
     public void exportGetOrganizationJobStatTest() throws Exception {
 
@@ -123,7 +121,6 @@ public class ExportResourceIT extends AbstractRestIT {
      * @throws Exception
      */
 
-    @Ignore
     @Test
     public void exportGetApplicationJobStatTest() throws Exception {
 
@@ -160,7 +157,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void exportGetCollectionJobStatTest() throws Exception {
 
@@ -192,7 +188,6 @@ public class ExportResourceIT extends AbstractRestIT {
 //
 //
 //    //    //do an unauthorized test for both post and get
-@Ignore
 @Test
     public void exportGetWrongUUID() throws Exception {
         UUID fake = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -209,7 +204,6 @@ public class ExportResourceIT extends AbstractRestIT {
 
 
     //
-    @Ignore
     @Test
     public void exportPostApplicationNullPointerProperties() throws Exception {
         try {
@@ -225,7 +219,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void exportPostOrganizationNullPointerProperties() throws Exception {
         try {
@@ -239,7 +232,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
     //
-    @Ignore
     @Test
     public void exportPostCollectionNullPointer() throws Exception {
         try {
@@ -255,7 +247,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportGetCollectionUnauthorized() throws Exception {
         UUID fake = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -272,7 +263,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportGetApplicationUnauthorized() throws Exception {
         UUID fake = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -288,7 +278,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportGetOrganizationUnauthorized() throws Exception {
         UUID fake = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -303,7 +292,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostOrganizationNullPointerStorageInfo() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -323,7 +311,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void exportPostApplicationNullPointerStorageInfo() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -344,7 +331,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostCollectionNullPointerStorageInfo() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -366,7 +352,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostOrganizationNullPointerStorageProvider() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -386,7 +371,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void exportPostApplicationNullPointerStorageProvider() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -407,7 +391,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostCollectionNullPointerStorageProvider() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -430,7 +413,6 @@ public class ExportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void exportPostOrganizationNullPointerStorageVerification() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -482,7 +464,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostApplicationNullPointerStorageVerification() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();
@@ -537,7 +518,6 @@ public class ExportResourceIT extends AbstractRestIT {
         }
     }
 
-    @Ignore
     @Test
     public void exportPostCollectionNullPointerStorageVerification() throws Exception {
         HashMap<String, Object> payload = payloadBuilder();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
index 784ff67..ee6ed8e 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
@@ -60,6 +60,7 @@ import java.util.Properties;
 import static org.junit.Assert.*;
 
 
+@Ignore
 public class ImportResourceIT extends AbstractRestIT {
 
     private static final Logger logger = LoggerFactory.getLogger(ImportResourceIT.class);
@@ -127,7 +128,6 @@ public class ImportResourceIT extends AbstractRestIT {
      *
      * @throws Exception
      */
-    @Ignore
     @Test
     public void importGetCollectionJobStatTest() throws Exception {
 
@@ -162,7 +162,6 @@ public class ImportResourceIT extends AbstractRestIT {
      * Verify that import job can only be read with an authorized token and cannot be read
      * with an invalid/notAllowed token.
      */
-    @Ignore
     @Test
     public void importTokenAuthorizationTest() throws Exception {
 
@@ -218,7 +217,6 @@ public class ImportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void importPostApplicationNullPointerProperties() throws Exception {
         String org = clientSetup.getOrganizationName();
@@ -235,7 +233,6 @@ public class ImportResourceIT extends AbstractRestIT {
         assertEquals(Response.Status.BAD_REQUEST, responseStatus);
     }
 
-    @Ignore
     @Test
     public void importPostApplicationNullPointerStorageInfo() throws Exception {
         String org = clientSetup.getOrganizationName();
@@ -256,7 +253,6 @@ public class ImportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void importPostApplicationNullPointerStorageProvider() throws Exception {
         String org = clientSetup.getOrganizationName();
@@ -278,7 +274,6 @@ public class ImportResourceIT extends AbstractRestIT {
     }
 
 
-    @Ignore
     @Test
     public void importPostApplicationNullPointerStorageVerification() throws Exception {
         String org = clientSetup.getOrganizationName();
@@ -412,7 +407,6 @@ public class ImportResourceIT extends AbstractRestIT {
     /**
      * TODO: Test that importing bad JSON will result in an informative error message.
      */
-    @Ignore
     @Test
     public void testImportGoodJson() throws Exception {
         // import from a bad JSON file
@@ -484,7 +478,6 @@ public class ImportResourceIT extends AbstractRestIT {
     /**
      * TODO: Test that importing bad JSON will result in an informative error message.
      */
-    @Ignore
     @Test
     public void testImportOneGoodOneBad() throws Exception {
 
@@ -532,7 +525,6 @@ public class ImportResourceIT extends AbstractRestIT {
     /**
      * TODO: Test that importing bad JSON will result in an informative error message.
      */
-    @Ignore
     @Test
     public void testImportOneBadFile() throws Exception {
         // import from a bad JSON file
@@ -581,7 +573,6 @@ public class ImportResourceIT extends AbstractRestIT {
     /**
      * TODO: Test that importing bad JSON will result in an informative error message.
      */
-    @Ignore
     @Test
     public void testImportBadJson() throws Exception {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
index 8f67e85..4f78b92 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
@@ -663,81 +663,6 @@ public class ExportServiceIT {
 
 
     @Test
-    @Ignore("this is a meaningless test because our export format does not support export of organizations")
-    public void testExportOneOrganization() throws Exception {
-
-        // create a bunch of organizations, each with applications and collections of entities
-
-        int maxOrgs = 3;
-        int maxApps = 3;
-        int maxEntities = 20;
-
-        List<ApplicationInfo> appsMade = new ArrayList<>();
-        List<OrganizationInfo> orgsMade = new ArrayList<>();
-
-        for ( int orgIndex = 0; orgIndex < maxOrgs; orgIndex++ ) {
-
-
-            String orgName = "org_" + RandomStringUtils.randomAlphanumeric(10);
-            OrganizationInfo orgMade = setup.getMgmtSvc().createOrganization( orgName, adminUser, true );
-            orgsMade.add( orgMade );
-            logger.debug("Created org {}", orgName);
-
-            for ( int appIndex = 0; appIndex < maxApps; appIndex++ ) {
-
-                String appName =  "app_" + RandomStringUtils.randomAlphanumeric(10);
-                ApplicationInfo appMade = setup.getMgmtSvc().createApplication( orgMade.getUuid(), appName );
-                appsMade.add( appMade );
-                logger.debug("Created app {}", appName);
-
-                for (int entityIndex = 0; entityIndex < maxEntities; entityIndex++) {
-
-                    EntityManager appEm = setup.getEmf().getEntityManager( appMade.getId() );
-                    appEm.create( appName + "_type", new HashMap<String, Object>() {{
-                        put("property1", "value1");
-                        put("property2", "value2");
-                    }});
-                }
-            }
-        }
-
-        // export one of the organizations only, using mock S3 export that writes to local disk
-
-        String exportFileName = "exportOneOrganization.json";
-        S3Export s3Export = new MockS3ExportImpl( exportFileName );
-
-        HashMap<String, Object> payload = payloadBuilder(appsMade.get(0).getName());
-        payload.put("organizationId", orgsMade.get(0).getUuid() );
-        payload.put( "applicationId", appsMade.get(0).getId() );
-
-        ExportService exportService = setup.getExportService();
-        UUID exportUUID = exportService.schedule( payload );
-
-        JobData jobData = jobDataCreator( payload, exportUUID, s3Export );
-        JobExecution jobExecution = mock( JobExecution.class );
-        when( jobExecution.getJobData() ).thenReturn(jobData);
-
-        exportService.doExport( jobExecution );
-
-        // finally, we check that file was created and contains the right number of entities in the array
-
-        File exportedFile = new File( exportFileName );
-        exportedFile.deleteOnExit();
-
-        TypeReference<HashMap<String,Object>> typeRef
-            = new TypeReference<HashMap<String,Object>>() {};
-
-        ObjectMapper mapper = new ObjectMapper();
-        Map<String,Object> jsonMap = mapper.readValue(new FileReader( exportedFile ), typeRef);
-        Map collectionsMap = (Map)jsonMap.get("collections");
-
-        List collectionList = (List)collectionsMap.get("users");
-
-        assertEquals( 3, collectionList.size() );
-    }
-
-
-    @Test
     public void testExportDoJob() throws Exception {
 
         String appName = newOrgAppAdminRule.getApplicationInfo().getName();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/41dde128/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
index e8d6081..d7d7028 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
@@ -64,7 +64,6 @@ import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
 
-//@Concurrent
 public class ImportServiceIT {
 
     private static final Logger logger = LoggerFactory.getLogger(ImportServiceIT.class);


[19/21] usergrid git commit: Adding "Pending" comments and JIRA links.

Posted by gr...@apache.org.
Adding "Pending" comments and JIRA links.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/78f3da6f
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/78f3da6f
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/78f3da6f

Branch: refs/heads/master
Commit: 78f3da6fcb194190af1d67151bb8d419f620c22e
Parents: 43f7ee0
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 10:53:38 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 10:53:38 2015 -0500

----------------------------------------------------------------------
 .../apache/usergrid/persistence/CounterIT.java  | 29 ++++++++++----------
 .../persistence/core/rx/OrderedMergeTest.java   |  5 ++--
 .../shard/impl/ShardGroupDeletionImplTest.java  |  2 +-
 .../collection/groups/GroupResourceIT.java      |  2 +-
 .../applications/events/EventsResourceIT.java   |  2 +-
 .../organizations/AdminEmailEncodingIT.java     |  3 +-
 6 files changed, 22 insertions(+), 21 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java b/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
index e558bad..77dacf2 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/CounterIT.java
@@ -18,17 +18,7 @@
 package org.apache.usergrid.persistence;
 
 
-import java.util.HashMap;
-import java.util.LinkedHashMap;
-import java.util.Map;
-import java.util.UUID;
-
-import org.junit.Ignore;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
+import net.jcip.annotations.NotThreadSafe;
 import org.apache.usergrid.AbstractCoreIT;
 import org.apache.usergrid.corepersistence.util.CpNamingUtils;
 import org.apache.usergrid.persistence.entities.Event;
@@ -39,12 +29,19 @@ import org.apache.usergrid.persistence.model.util.UUIDGenerator;
 import org.apache.usergrid.utils.ImmediateCounterRule;
 import org.apache.usergrid.utils.JsonUtils;
 import org.apache.usergrid.utils.UUIDUtils;
+import org.junit.Ignore;
+import org.junit.Rule;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
-import net.jcip.annotations.NotThreadSafe;
+import java.util.HashMap;
+import java.util.LinkedHashMap;
+import java.util.Map;
+import java.util.UUID;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.fail;
 
 
 @NotThreadSafe
@@ -63,7 +60,8 @@ public class CounterIT extends AbstractCoreIT {
     }
 
 
-    @Ignore( "needs to have elasticsearch refreshes implemented" )
+    @Ignore( "Pending https://issues.apache.org/jira/browse/USERGRID-1120. ")
+    // needs to have elasticsearch refreshes implemented
     @Test
     public void testIncrementAndDecrement() throws Exception {
 
@@ -89,12 +87,15 @@ public class CounterIT extends AbstractCoreIT {
 
         LOG.debug( "user={}", user );
 
+        em.refreshIndex();
 
         counters = em.getEntityCounters( applicationId );
         assertEquals( new Long( 1 ), counters.get( "application.collection.users" ) );
 
         em.delete( user );
 
+        em.refreshIndex();
+
         counters = em.getEntityCounters( applicationId );
         assertEquals( new Long( 0 ), counters.get( "application.collection.users" ) );
     }

http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/corepersistence/common/src/test/java/org/apache/usergrid/persistence/core/rx/OrderedMergeTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/common/src/test/java/org/apache/usergrid/persistence/core/rx/OrderedMergeTest.java b/stack/corepersistence/common/src/test/java/org/apache/usergrid/persistence/core/rx/OrderedMergeTest.java
index 6366de1..465e22d 100644
--- a/stack/corepersistence/common/src/test/java/org/apache/usergrid/persistence/core/rx/OrderedMergeTest.java
+++ b/stack/corepersistence/common/src/test/java/org/apache/usergrid/persistence/core/rx/OrderedMergeTest.java
@@ -148,7 +148,7 @@ public class OrderedMergeTest {
 
 
     @Test
-    @Ignore( "Doesn't work until backpressure is implemented" )
+    @Ignore( "Pending implementation of backpressure" )
     public void multipleOperatorSingleThreadSizeException() throws InterruptedException {
 
         List<Integer> expected1List = Arrays.asList( 5, 3, 2, 0 );
@@ -269,8 +269,7 @@ public class OrderedMergeTest {
 
 
     @Test
-    @Ignore("Shouldn't throw an exception, should work with current impl.  "
-            + "Needs to be changed when backpressure is introduced" )
+    @Ignore("Pending implementation of backpressure")
     public void multipleOperatorMultipleThreadSizeException() throws InterruptedException {
 
         List<Integer> expected1List = Arrays.asList( 10, 4, 3, 2, 1 );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/impl/shard/impl/ShardGroupDeletionImplTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/impl/shard/impl/ShardGroupDeletionImplTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/impl/shard/impl/ShardGroupDeletionImplTest.java
index 1cf6d54..fa6a3fd 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/impl/shard/impl/ShardGroupDeletionImplTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/impl/shard/impl/ShardGroupDeletionImplTest.java
@@ -56,7 +56,7 @@ import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 
-@Ignore("Temporarily removing until delete is re-enabled")
+@Ignore("Pending re-enable of delete functionality")
 public class ShardGroupDeletionImplTest {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/groups/GroupResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/groups/GroupResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/groups/GroupResourceIT.java
index abf2be5..bd75adf 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/groups/GroupResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/groups/GroupResourceIT.java
@@ -393,7 +393,7 @@ public class GroupResourceIT extends AbstractRestIT {
      * Post a group activity and make sure it can be read back only by group members
      *
      */
-    @Ignore("Fails. See todo in the test itself.")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1119. Fails. See todo in the test itself.")
     @Test
     public void postGroupActivity() throws IOException {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/rest/src/test/java/org/apache/usergrid/rest/applications/events/EventsResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/events/EventsResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/events/EventsResourceIT.java
index 2777967..185a37b 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/events/EventsResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/events/EventsResourceIT.java
@@ -45,7 +45,7 @@ public class EventsResourceIT extends AbstractRestIT {
 
 
     @Test
-    @Ignore("Events not working yet")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1118. Events not working yet")
     public void testEventPostandGet() throws IOException {
 
         Map<String, Object> payload = new LinkedHashMap<String, Object>();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/78f3da6f/stack/rest/src/test/java/org/apache/usergrid/rest/management/organizations/AdminEmailEncodingIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/organizations/AdminEmailEncodingIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/organizations/AdminEmailEncodingIT.java
index c615b60..bb5593e 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/organizations/AdminEmailEncodingIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/organizations/AdminEmailEncodingIT.java
@@ -77,7 +77,8 @@ public class AdminEmailEncodingIT extends AbstractRestIT {
      * @throws Exception
      */
     @Test
-    @Ignore //This fails. I'm not sure if it is by design, but a single quote is valid in an email address
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1117")
+    // This fails. I'm not sure if it is by design, but a single quote is valid in an email address
     public void getTokenQuote() throws Exception {
         doTest("'");
     }


[17/21] usergrid git commit: Adding more JIRA links; all but 10 ignores are now linked to JIRA issues.

Posted by gr...@apache.org.
Adding more JIRA links; all but 10 ignores are now linked to JIRA issues.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/68be7d70
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/68be7d70
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/68be7d70

Branch: refs/heads/master
Commit: 68be7d7027c51940445a43bed5fcb3270ff75846
Parents: 22c29fb
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 10:13:38 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 10:13:38 2015 -0500

----------------------------------------------------------------------
 .../usergrid/batch/job/SchedulerRuntime1IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime2IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime3IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime4IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime5IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime6IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime7IT.java   |  3 ++-
 .../usergrid/batch/job/SchedulerRuntime8IT.java   |  3 ++-
 .../batch/job/SchedulerRuntimeIntervalIT.java     |  3 ++-
 .../java/org/apache/usergrid/mq/MessagesIT.java   |  8 ++++----
 .../org/apache/usergrid/rest/IndexResourceIT.java |  4 +++-
 .../usergrid/rest/management/AdminUsersIT.java    | 18 ++++++++++--------
 .../importer/ImportConnectionsTest.java           |  2 +-
 13 files changed, 36 insertions(+), 23 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
index 380a683..a7f2741 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime1IT.java
@@ -34,7 +34,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes.
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime1IT extends AbstractSchedulerRuntimeIT {
 	private static final Logger logger = LoggerFactory.getLogger(SchedulerRuntime1IT.class.getName());
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
index 2c2aa40..9f18dd0 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime2IT.java
@@ -34,7 +34,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime2IT extends AbstractSchedulerRuntimeIT {
 	private static final Logger logger = LoggerFactory.getLogger(SchedulerRuntime2IT.class.getName());
     /** Test the scheduler ramps up correctly when there are more jobs to be read after a pause */

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
index 5fc6398..60b25ab 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime3IT.java
@@ -33,7 +33,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime3IT extends AbstractSchedulerRuntimeIT {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
index fbd49f2..97acaee 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime4IT.java
@@ -31,7 +31,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime4IT extends AbstractSchedulerRuntimeIT {
     /**
      * Test the scheduler ramps up correctly when there are more jobs to be read after a pause when the job specifies

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
index 1b5cbab..5973fe9 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime5IT.java
@@ -31,7 +31,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime5IT extends AbstractSchedulerRuntimeIT {
     /**
      * Test the scheduler ramps up correctly when there are more jobs to be read after a pause when the job specifies

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
index 5d2e00d..e8e4e54 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
@@ -35,7 +35,8 @@ import static org.junit.Assert.assertTrue;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime6IT extends AbstractSchedulerRuntimeIT {
 
     /**

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
index 000996d..6d4b4a4 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
@@ -32,7 +32,8 @@ import static org.junit.Assert.assertTrue;
  */
 
 //@org.junit.Ignore( "Todd you need to take a look at this since it's not clear to me what was intended in this test." )
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime7IT extends AbstractSchedulerRuntimeIT {
 
     /** Test that we're only running once, even when a job exceeds the heartbeat time */

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
index 65b5e67..bd49781 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime8IT.java
@@ -35,7 +35,8 @@ import static org.junit.Assert.assertFalse;
  * Class to test job runtimes
  */
 
-@Ignore("These tests no longer work with shared Spring context, need to be fixed https://issues.apache.org/jira/browse/USERGRID-1105" )
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntime8IT extends AbstractSchedulerRuntimeIT {
 
     /**

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntimeIntervalIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntimeIntervalIT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntimeIntervalIT.java
index 9504c9f..45d58a9 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntimeIntervalIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntimeIntervalIT.java
@@ -36,7 +36,8 @@ import static org.junit.Assert.assertTrue;
  * Tests saturation at each point of the runtime as well
  */
 
-@Ignore("Ignored awaiting fix for USERGRID-267")
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1105. Spring problems." )
+// These tests no longer work with shared Spring context, need to be fixed
 public class SchedulerRuntimeIntervalIT extends AbstractSchedulerRuntimeIT {
 
 	private static final Logger logger =

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/core/src/test/java/org/apache/usergrid/mq/MessagesIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/mq/MessagesIT.java b/stack/core/src/test/java/org/apache/usergrid/mq/MessagesIT.java
index 70032f7..52ca4d8 100644
--- a/stack/core/src/test/java/org/apache/usergrid/mq/MessagesIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/mq/MessagesIT.java
@@ -48,7 +48,7 @@ public class MessagesIT extends AbstractCoreIT {
     }
 
 
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1116. ")
     @Test
     public void testMessages() throws Exception {
         LOG.info( "MessagesIT.testMessages" );
@@ -118,7 +118,7 @@ public class MessagesIT extends AbstractCoreIT {
     }
 
 
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1116. ")
     @Test
     public void testSubscriberSearch() throws Exception {
         QueueManager qm = app.getQm();
@@ -171,7 +171,7 @@ public class MessagesIT extends AbstractCoreIT {
     }
 
 
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1116. ")
     @Test
     public void testConsumer() throws Exception {
         LOG.info( "Creating messages" );
@@ -214,7 +214,7 @@ public class MessagesIT extends AbstractCoreIT {
     }
 
 
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1116. ")
     @Test
     public void testTransactions() throws Exception {
         QueueManager qm = app.getQm();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/rest/src/test/java/org/apache/usergrid/rest/IndexResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/IndexResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/IndexResourceIT.java
index af91ad7..6a69190 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/IndexResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/IndexResourceIT.java
@@ -20,6 +20,7 @@
 
 package org.apache.usergrid.rest;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.apache.usergrid.rest.test.resource.AbstractRestIT;
 import org.apache.usergrid.rest.test.resource.model.ApiResponse;
 import org.apache.usergrid.rest.test.resource.model.QueryParameters;
@@ -27,6 +28,7 @@ import org.apache.usergrid.rest.test.resource.model.Token;
 import org.glassfish.jersey.client.authentication.HttpAuthenticationFeature;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -99,8 +101,8 @@ public class IndexResourceIT extends AbstractRestIT {
 
     }
 
-    @Ignore
     @Test
+    @Category(ExperimentalTest.class)
     public void TestAddIndex() throws Exception{
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
index fba0c43..09a76aa 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
@@ -187,7 +187,7 @@ public class AdminUsersIT extends AbstractRestIT {
      * TODO:test for parallel test that changing the properties here won't affect other tests
      * @throws Exception
      */
-    @Ignore("breaks other tests")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115. breaks other tests")
     @Test
     public void testUnconfirmedAdminLogin()  throws Exception{
 
@@ -273,7 +273,7 @@ public class AdminUsersIT extends AbstractRestIT {
      * Test that the system admin doesn't need a confirmation email
      * @throws Exception
      */
-    @Ignore("breaks other tests")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115. breaks other tests")
     @Test
     public void testSystemAdminNeedsNoConfirmation() throws Exception{
         //Save original properties to return them to normal at the end of the test
@@ -311,7 +311,8 @@ public class AdminUsersIT extends AbstractRestIT {
      * Test that the test account doesn't need confirmation and is created automatically.
      * @throws Exception
      */
-    @Ignore("Test doesn't pass because the test account isn't getting correct instantiated")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115. Test account problem")
+    // Test doesn't pass because the test account isn't getting correct instantiated
     @Test
     public void testTestUserNeedsNoConfirmation() throws Exception{
         //Save original properties to return them to normal at the end of the test
@@ -347,8 +348,9 @@ public class AdminUsersIT extends AbstractRestIT {
      * Update the current management user and make sure the change persists
      * @throws Exception
      */
-    @Ignore("Fails because we cannot GET a management user with a super user token - only with an Admin level token."
-        + "But, we can PUT with a superuser token. This test will work once that issue has been resolved.")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115.")
+    // Fails because we cannot GET a management user with a super user token - only with an Admin level token.
+    // But, we can PUT with a superuser token. This test will work once that issue has been resolved.
     @Test
     public void updateManagementUser() throws Exception {
 
@@ -398,7 +400,8 @@ public class AdminUsersIT extends AbstractRestIT {
         assertFalse( inbox.isEmpty() );
     }
 
-    @Ignore("Build problems - Test is broken due to viewables not being properly returned in the embedded tomcat")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115. Viewables issue.")
+    // Test is broken due to viewables not being properly returned in the embedded tomcat
     @Test
     public void checkFormPasswordReset() throws Exception {
 
@@ -435,7 +438,6 @@ public class AdminUsersIT extends AbstractRestIT {
         assertTrue( html.contains( "invalid token" ) );
     }
 
-//    // TODO: will work once resetpw viewables work
 //    @Test
 //    public void passwordResetIncorrectUserName() throws Exception {
 //
@@ -636,7 +638,7 @@ public class AdminUsersIT extends AbstractRestIT {
         }
     }
 
-    @Ignore("breaks other tests")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1115. breaks other tests")
     @Test
     public void testProperties(){
         ApiResponse originalTestPropertiesResponse = clientSetup.getRestClient().testPropertiesResource().get();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/68be7d70/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportConnectionsTest.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportConnectionsTest.java b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportConnectionsTest.java
index 8ca3caa..05948ff 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportConnectionsTest.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportConnectionsTest.java
@@ -58,7 +58,7 @@ public class ImportConnectionsTest {
 
 
     @Test
-    @Ignore("Because getTargetEntities() is broken")
+    @Ignore("Pending merge of export-feature branch. Because getTargetEntities() is broken")
     public void testCreateAndCountConnectionsViaGet() throws Exception {
 
         doTestCreateAndCountConnections(new ConnectionCounter() {


[16/21] usergrid git commit: Add pending comment to import/export tests that are awaiting merge of the export-feature branch.

Posted by gr...@apache.org.
Add pending comment to import/export tests that are awaiting merge of the export-feature branch.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/22c29fb7
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/22c29fb7
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/22c29fb7

Branch: refs/heads/master
Commit: 22c29fb7ca455c05739d1c2724e33dd0429bab1a
Parents: a9c1e84
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 09:32:08 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 09:32:08 2015 -0500

----------------------------------------------------------------------
 .../usergrid/rest/management/ExportResourceIT.java      |  2 +-
 .../usergrid/rest/management/ImportResourceIT.java      |  2 +-
 .../usergrid/management/export/ExportServiceIT.java     |  9 +++++----
 .../usergrid/management/importer/ImportServiceIT.java   | 12 ++++++------
 4 files changed, 13 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/22c29fb7/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
index 79590b8..81ff2d3 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ExportResourceIT.java
@@ -32,7 +32,7 @@ import java.util.UUID;
 import static org.junit.Assert.*;
 
 
-@Ignore
+@Ignore("Pending merge of export-feature branch. ")
 public class ExportResourceIT extends AbstractRestIT {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/22c29fb7/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
index ee6ed8e..60b872d 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/ImportResourceIT.java
@@ -60,7 +60,7 @@ import java.util.Properties;
 import static org.junit.Assert.*;
 
 
-@Ignore
+@Ignore("Pending merge of export-feature branch. ")
 public class ImportResourceIT extends AbstractRestIT {
 
     private static final Logger logger = LoggerFactory.getLogger(ImportResourceIT.class);

http://git-wip-us.apache.org/repos/asf/usergrid/blob/22c29fb7/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
index 4f78b92..c31f07c 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/export/ExportServiceIT.java
@@ -150,7 +150,8 @@ public class ExportServiceIT {
 
 
     //Tests to make sure we can call the job with mock data and it runs.
-    @Ignore("Connections won't save when run with maven, but on local builds it will.")
+    // Connections won't save when run with maven, but on local builds it will.
+    @Ignore("Pending merge of export-feature branch")
     public void testConnectionsOnCollectionExport() throws Exception {
 
         File f = null;
@@ -746,7 +747,7 @@ public class ExportServiceIT {
 
 
     @Test
-    @Ignore // TODO: fix this test...
+    @Ignore("Pending merge of export-feature branch")
     public void testIntegration100EntitiesOn() throws Exception {
 
         logger.debug("testIntegration100EntitiesOn(): starting...");
@@ -834,8 +835,8 @@ public class ExportServiceIT {
         }
     }
 
-    @Ignore("Why is this ignored?")
     @Test
+    @Ignore("Pending merge of export-feature branch")
     public void testIntegration100EntitiesForAllApps() throws Exception {
 
         S3Export s3Export = new S3ExportImpl();
@@ -917,8 +918,8 @@ public class ExportServiceIT {
     }
 
 
-    @Ignore("Why is this ignored")
     @Test
+    @Ignore("Pending merge of export-feature branch")
     public void testIntegration100EntitiesOnOneOrg() throws Exception {
 
         S3Export s3Export = new S3ExportImpl();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/22c29fb7/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
index d7d7028..7dc075c 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/importer/ImportServiceIT.java
@@ -138,7 +138,7 @@ public class ImportServiceIT {
 
     // test case to check if application is imported correctly
     @Test
-    @Ignore("Import organization not supported")
+    @Ignore("Pending merge of export-feature branch. Import organization not supported")
     public void testImportApplication() throws Exception {
 
         EntityManager em = setup.getEmf().getEntityManager( applicationId );
@@ -260,7 +260,7 @@ public class ImportServiceIT {
 
     // test case to check if all applications file for an organization are imported correctly
     @Test
-    @Ignore("Import organization not supported")
+    @Ignore("Pending merge of export-feature branch. Import organization not supported")
     public void testImportOrganization() throws Exception {
 
         // creates 5 entities in usertests collection
@@ -455,7 +455,7 @@ public class ImportServiceIT {
      * Test to the doImport method with null organziation ID
      */
     @Test
-    @Ignore("Import organization not supported")
+    @Ignore("Pending merge of export-feature branch. Import organization not supported")
     public void testDoImportWithNullOrganizationID() throws Exception {
         // import
         S3Import s3Import = new S3ImportImpl();
@@ -481,7 +481,7 @@ public class ImportServiceIT {
      * Test to the doImport method with fake organization ID
      */
     @Test
-    @Ignore("Import organization not supported")
+    @Ignore("Pending merge of export-feature branch. Import organization not supported")
     public void testDoImportWithFakeOrganizationID() throws Exception {
 
         UUID fakeOrgId = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -511,7 +511,7 @@ public class ImportServiceIT {
      * Test to the doImport method with fake application ID
      */
     @Test
-    @Ignore("Import application not supported")
+    @Ignore("Pending merge of export-feature branch. Import application not supported")
     public void testDoImportWithFakeApplicationID() throws Exception {
 
         UUID fakeappId = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );
@@ -543,7 +543,7 @@ public class ImportServiceIT {
      * Test to the doImport Collection method with fake application ID
      */
     @Test
-    @Ignore("Import application not supported")
+    @Ignore("Pending merge of export-feature branch. Import application not supported")
     public void testDoImportCollectionWithFakeApplicationID() throws Exception {
 
         UUID fakeappId = UUID.fromString( "AAAAAAAA-FFFF-FFFF-FFFF-AAAAAAAAAAAA" );


[14/21] usergrid git commit: Adding JIRA issue links to ignore annotations

Posted by gr...@apache.org.
Adding JIRA issue links to ignore annotations


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/6af43dc9
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/6af43dc9
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/6af43dc9

Branch: refs/heads/master
Commit: 6af43dc9b5c3963eefb7a93afb029a4e365cc65f
Parents: 4cf69e6
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 09:30:57 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 09:30:57 2015 -0500

----------------------------------------------------------------------
 .../org/apache/usergrid/persistence/EntityManagerIT.java     | 3 ++-
 .../test/java/org/apache/usergrid/rest/NotificationsIT.java  | 2 +-
 .../usergrid/rest/applications/ApplicationResourceIT.java    | 2 +-
 .../usergrid/services/notifications/NotifiersServiceIT.java  | 8 ++++----
 .../services/notifications/apns/NotificationsServiceIT.java  | 8 ++++----
 .../services/notifications/gcm/NotificationsServiceIT.java   | 8 ++++----
 6 files changed, 16 insertions(+), 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/core/src/test/java/org/apache/usergrid/persistence/EntityManagerIT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityManagerIT.java b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityManagerIT.java
index d81cad2..658c808 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/EntityManagerIT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/EntityManagerIT.java
@@ -381,7 +381,8 @@ public class EntityManagerIT extends AbstractCoreIT {
 
 
     @Test
-    @Ignore( "There is a concurrency issue due to counters not being thread safe: see USERGRID-1753" )
+    @Ignore( "Pending https://issues.apache.org/jira/browse/USERGRID-1753. Concurrency issue.")
+    // There is a concurrency issue due to counters not being thread safe
     public void testEntityCounters() throws Exception {
         LOG.info( "EntityManagerIT#testEntityCounters" );
         EntityManager em = app.getEntityManager();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
index 63e7b49..61dc419 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/NotificationsIT.java
@@ -44,7 +44,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Test creating, sending and paging through Notifications via the REST API.
  */
-@Ignore
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. ")
 public class NotificationsIT extends org.apache.usergrid.rest.test.resource.AbstractRestIT {
     private static final Logger logger = LoggerFactory.getLogger( NotificationsIT.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
index 5915eca..8d29290 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
@@ -648,7 +648,7 @@ public class ApplicationResourceIT extends AbstractRestIT {
      * Retrieve an access token using HTTP Basic authentication
      */
     @Test
-    @Ignore("Should fix - https://issues.apache.org/jira/browse/USERGRID-1106")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1106")
     //Are we trying to generate token with token? Couldn't find endpoint that accepts token for generating token
     public void clientCredentialsFlowWithHeaderAuthorization() throws Exception {
         //retrieve the credentials

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/services/src/test/java/org/apache/usergrid/services/notifications/NotifiersServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/NotifiersServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/NotifiersServiceIT.java
index fbf8290..58172dd 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/NotifiersServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/NotifiersServiceIT.java
@@ -83,7 +83,7 @@ public class NotifiersServiceIT extends AbstractServiceIT {
         }
     }
 
-    @Ignore("Mock doesn't work")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. Mock doesn't work")
     @Test
     public void badGcmToken() throws Exception {
         app.clear();
@@ -146,7 +146,7 @@ public class NotifiersServiceIT extends AbstractServiceIT {
         assertArrayEquals(notifier.getP12Certificate(), certBytes);
     }
 
-    @Ignore("Mock doesn't work")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. Mock doesn't work")
     @Test
     public void badAPNsCertificate() throws Exception {
 
@@ -170,7 +170,7 @@ public class NotifiersServiceIT extends AbstractServiceIT {
         }
     }
 
-    @Ignore("Mock doesn't work")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. Mock doesn't work")
     @Test
     public void badAPNsPassword() throws Exception {
 
@@ -195,7 +195,7 @@ public class NotifiersServiceIT extends AbstractServiceIT {
     }
 
     @Test
-    @Ignore("No longer needed to verify")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. No longer needed to verify")
     public void encryption() throws Exception {
 
         app.clear();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
index a3ff449..60b02a6 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
@@ -41,7 +41,7 @@ import static org.apache.usergrid.services.notifications.impl.ApplicationQueueMa
 
 // todo: test reschedule on delivery time change
 // todo: test restart of queuing
-@Ignore
+@Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. ")
 public class NotificationsServiceIT extends AbstractServiceNotificationIT {
 
     private static final Logger LOG = LoggerFactory.getLogger(NotificationsServiceIT.class);
@@ -383,7 +383,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
         }
     }
 
-    @Ignore("todo: how can I mock this?")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. todo: how can I mock this?")
     @Test
     public void badToken() throws Exception {
 
@@ -576,7 +576,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
         checkReceipts(notification, 2);
     }
 
-    @Ignore("todo: how can I mock this?")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. todo: how can I mock this?")
     @Test
     public void badCertificate() throws Exception {
 
@@ -649,7 +649,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
         assertNotNull(entity);
     }
 
-    @Ignore("todo: how can I mock this?")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. todo: how can I mock this?")
     @Test
     public void inactiveDeviceUpdate() throws Exception {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6af43dc9/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
index 3369786..282055c 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
@@ -142,7 +142,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
     }
 
     @Test
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. ")
     public void singlePushNotification() throws Exception {
 
         app.clear();
@@ -168,7 +168,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
     }
 
     @Test
-    @Ignore
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. ")
     public void singlePushNotificationViaUser() throws Exception {
 
         app.clear();
@@ -303,7 +303,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
         }
     }
 
-    @Ignore("todo: how can I mock this?")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. todo: how can I mock this?")
     @Test
     public void badToken() throws Exception {
 
@@ -346,7 +346,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
         assertEquals("InvalidRegistration", receipt.getErrorCode());
     }
 
-    @Ignore("todo: how can I mock this?")
+    @Ignore("Pending https://issues.apache.org/jira/browse/USERGRID-1113. todo: how can I mock this?")
     @Test
     public void badAPIKey() throws Exception {
 


[09/21] usergrid git commit: Fix spelling error in class names.

Posted by gr...@apache.org.
Fix spelling error in class names.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/f62e0aa5
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/f62e0aa5
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/f62e0aa5

Branch: refs/heads/master
Commit: f62e0aa5869785b459571cab306579ca21bf0be1
Parents: 1385366
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 07:51:29 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 07:51:29 2015 -0500

----------------------------------------------------------------------
 .../usergrid/batch/job/OnlyOnceExceution.java   | 123 -----------------
 .../usergrid/batch/job/OnlyOnceExecution.java   | 122 +++++++++++++++++
 .../job/OnlyOnceUnlockOnFailExceution.java      | 133 -------------------
 .../job/OnlyOnceUnlockOnFailExecution.java      | 132 ++++++++++++++++++
 4 files changed, 254 insertions(+), 256 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/f62e0aa5/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExceution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExceution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExceution.java
deleted file mode 100644
index 6109f1f..0000000
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExceution.java
+++ /dev/null
@@ -1,123 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.usergrid.batch.job;
-
-
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.TimeUnit;
-
-import org.junit.Ignore;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.springframework.stereotype.Component;
-
-import org.apache.usergrid.batch.JobExecution;
-
-
-/**
- * A job that will sleep for the amount of time specified. Used to check that our counter is only ever run once.
- *
- * @author tnine
- */
-@Component("onlyOnceExceution")
-@Ignore("Not a test")
-public class OnlyOnceExceution extends OnlyOnceJob {
-
-    private static final Logger logger = LoggerFactory.getLogger( OnlyOnceExceution.class );
-
-    private CountDownLatch latch = null;
-    private CountDownLatch sleptLatch = new CountDownLatch( 1 );
-    private long timeout;
-    private boolean slept = false;
-    private long delay;
-
-
-    /**
-     *
-     */
-    public OnlyOnceExceution() {
-    }
-
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see
-     * org.apache.usergrid.batch.job.OnlyOnceJob#doJob(org.apache.usergrid.batch.JobExecution)
-     */
-    @Override
-    protected void doJob( JobExecution execution ) throws Exception {
-        logger.info( "Running only once execution" );
-
-        latch.countDown();
-
-
-        if ( !slept ) {
-            logger.info( "Sleeping in only once execution" );
-            Thread.sleep( timeout );
-            slept = true;
-            sleptLatch.countDown();
-        }
-    }
-
-
-    /* (non-Javadoc)
-     * @see org.apache.usergrid.batch.job.OnlyOnceJob#getDelay(org.apache.usergrid.batch.JobExecution)
-     */
-    @Override
-    protected long getDelay( JobExecution execution ) throws Exception {
-        return delay;
-    }
-
-
-    public void setDelay( long delay ) {
-        this.delay = delay;
-    }
-
-
-    public void setLatch( int calls ) {
-        latch = new CountDownLatch( calls );
-    }
-
-
-    public boolean waitForCount( long timeout, TimeUnit unit ) throws InterruptedException {
-        return latch.await( timeout, unit );
-    }
-
-
-    public boolean waitForSleep( long timeout, TimeUnit unit ) throws InterruptedException {
-        return sleptLatch.await( timeout, unit );
-    }
-
-
-    /** @return the timeout */
-    public long getTimeout() {
-        return timeout;
-    }
-
-
-    /** @param timeout the timeout to set */
-    public void setTimeout( long timeout ) {
-        this.timeout = timeout;
-    }
-
-
-    @Override
-    public void dead( final JobExecution execution ) throws Exception {
-        //To change body of implemented methods use File | Settings | File Templates.
-    }
-}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f62e0aa5/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExecution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExecution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExecution.java
new file mode 100644
index 0000000..ed9e8f4
--- /dev/null
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceExecution.java
@@ -0,0 +1,122 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.usergrid.batch.job;
+
+
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.TimeUnit;
+
+import org.junit.Ignore;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.springframework.stereotype.Component;
+
+import org.apache.usergrid.batch.JobExecution;
+
+
+/**
+ * A job that will sleep for the amount of time specified. Used to check that our counter is only ever run once.
+ *
+ * @author tnine
+ */
+@Component("onlyOnceExecution")
+public class OnlyOnceExecution extends OnlyOnceJob {
+
+    private static final Logger logger = LoggerFactory.getLogger( OnlyOnceExecution.class );
+
+    private CountDownLatch latch = null;
+    private CountDownLatch sleptLatch = new CountDownLatch( 1 );
+    private long timeout;
+    private boolean slept = false;
+    private long delay;
+
+
+    /**
+     *
+     */
+    public OnlyOnceExecution() {
+    }
+
+
+    /*
+     * (non-Javadoc)
+     *
+     * @see
+     * org.apache.usergrid.batch.job.OnlyOnceJob#doJob(org.apache.usergrid.batch.JobExecution)
+     */
+    @Override
+    protected void doJob( JobExecution execution ) throws Exception {
+        logger.info( "Running only once execution" );
+
+        latch.countDown();
+
+
+        if ( !slept ) {
+            logger.info( "Sleeping in only once execution" );
+            Thread.sleep( timeout );
+            slept = true;
+            sleptLatch.countDown();
+        }
+    }
+
+
+    /* (non-Javadoc)
+     * @see org.apache.usergrid.batch.job.OnlyOnceJob#getDelay(org.apache.usergrid.batch.JobExecution)
+     */
+    @Override
+    protected long getDelay( JobExecution execution ) throws Exception {
+        return delay;
+    }
+
+
+    public void setDelay( long delay ) {
+        this.delay = delay;
+    }
+
+
+    public void setLatch( int calls ) {
+        latch = new CountDownLatch( calls );
+    }
+
+
+    public boolean waitForCount( long timeout, TimeUnit unit ) throws InterruptedException {
+        return latch.await( timeout, unit );
+    }
+
+
+    public boolean waitForSleep( long timeout, TimeUnit unit ) throws InterruptedException {
+        return sleptLatch.await( timeout, unit );
+    }
+
+
+    /** @return the timeout */
+    public long getTimeout() {
+        return timeout;
+    }
+
+
+    /** @param timeout the timeout to set */
+    public void setTimeout( long timeout ) {
+        this.timeout = timeout;
+    }
+
+
+    @Override
+    public void dead( final JobExecution execution ) throws Exception {
+        //To change body of implemented methods use File | Settings | File Templates.
+    }
+}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f62e0aa5/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExceution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExceution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExceution.java
deleted file mode 100644
index 6dd826a..0000000
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExceution.java
+++ /dev/null
@@ -1,133 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.usergrid.batch.job;
-
-
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.TimeUnit;
-
-import org.junit.Ignore;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.springframework.stereotype.Component;
-
-import org.apache.usergrid.batch.JobExecution;
-
-
-/**
- * A job that will sleep for the amount of time specified. Used to check that our counter is only ever run once.  Checks
- * the lock is released on fail
- *
- * @author tnine
- */
-@Component("onlyOnceUnlockOnFailExceution")
-@Ignore("Not a test")
-public class OnlyOnceUnlockOnFailExceution extends OnlyOnceJob {
-
-    private static final Logger logger = LoggerFactory.getLogger( OnlyOnceUnlockOnFailExceution.class );
-
-    private CountDownLatch latch = null;
-    private CountDownLatch exception = new CountDownLatch( 1 );
-    private CountDownLatch completed = new CountDownLatch( 1 );
-    private long timeout;
-    private boolean slept = false;
-    private long delay;
-
-
-    /**
-     *
-     */
-    public OnlyOnceUnlockOnFailExceution() {
-    }
-
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see
-     * org.apache.usergrid.batch.job.OnlyOnceJob#doJob(org.apache.usergrid.batch.JobExecution)
-     */
-    @Override
-    protected void doJob( JobExecution execution ) throws Exception {
-        logger.info( "Running only once execution" );
-
-
-        latch.countDown();
-
-        if ( !slept ) {
-            logger.info( "Sleeping in only once execution" );
-            Thread.sleep( timeout );
-            slept = true;
-            exception.countDown();
-            throw new RuntimeException( "I failed to run correctly, I should be retried" );
-        }
-
-        completed.countDown();
-    }
-
-
-    /* (non-Javadoc)
-     * @see org.apache.usergrid.batch.job.OnlyOnceJob#getDelay(org.apache.usergrid.batch.JobExecution)
-     */
-    @Override
-    protected long getDelay( JobExecution execution ) throws Exception {
-        return delay;
-    }
-
-
-    public void setDelay( long delay ) {
-        this.delay = delay;
-    }
-
-
-    public void setLatch( int calls ) {
-        latch = new CountDownLatch( calls );
-    }
-
-
-    public boolean waitForCount( long timeout, TimeUnit unit ) throws InterruptedException {
-        return latch.await( timeout, unit );
-    }
-
-
-    public boolean waitForException( long timeout, TimeUnit unit ) throws InterruptedException {
-        return exception.await( timeout, unit );
-    }
-
-
-    public boolean waitForCompletion( long timeout, TimeUnit unit ) throws InterruptedException {
-        return completed.await( timeout, unit );
-    }
-
-
-    /** @return the timeout */
-    public long getTimeout() {
-        return timeout;
-    }
-
-
-    /** @param timeout the timeout to set */
-    public void setTimeout( long timeout ) {
-        this.timeout = timeout;
-    }
-
-
-    @Override
-    public void dead( final JobExecution execution ) throws Exception {
-        //To change body of implemented methods use File | Settings | File Templates.
-    }
-}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/f62e0aa5/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExecution.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExecution.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExecution.java
new file mode 100644
index 0000000..53fde15
--- /dev/null
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/OnlyOnceUnlockOnFailExecution.java
@@ -0,0 +1,132 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.usergrid.batch.job;
+
+
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.TimeUnit;
+
+import org.junit.Ignore;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.springframework.stereotype.Component;
+
+import org.apache.usergrid.batch.JobExecution;
+
+
+/**
+ * A job that will sleep for the amount of time specified. Used to check that our counter is only ever run once.  Checks
+ * the lock is released on fail
+ *
+ * @author tnine
+ */
+@Component("onlyOnceUnlockOnFailExecution")
+public class OnlyOnceUnlockOnFailExecution extends OnlyOnceJob {
+
+    private static final Logger logger = LoggerFactory.getLogger( OnlyOnceUnlockOnFailExecution.class );
+
+    private CountDownLatch latch = null;
+    private CountDownLatch exception = new CountDownLatch( 1 );
+    private CountDownLatch completed = new CountDownLatch( 1 );
+    private long timeout;
+    private boolean slept = false;
+    private long delay;
+
+
+    /**
+     *
+     */
+    public OnlyOnceUnlockOnFailExecution() {
+    }
+
+
+    /*
+     * (non-Javadoc)
+     *
+     * @see
+     * org.apache.usergrid.batch.job.OnlyOnceJob#doJob(org.apache.usergrid.batch.JobExecution)
+     */
+    @Override
+    protected void doJob( JobExecution execution ) throws Exception {
+        logger.info( "Running only once execution" );
+
+
+        latch.countDown();
+
+        if ( !slept ) {
+            logger.info( "Sleeping in only once execution" );
+            Thread.sleep( timeout );
+            slept = true;
+            exception.countDown();
+            throw new RuntimeException( "I failed to run correctly, I should be retried" );
+        }
+
+        completed.countDown();
+    }
+
+
+    /* (non-Javadoc)
+     * @see org.apache.usergrid.batch.job.OnlyOnceJob#getDelay(org.apache.usergrid.batch.JobExecution)
+     */
+    @Override
+    protected long getDelay( JobExecution execution ) throws Exception {
+        return delay;
+    }
+
+
+    public void setDelay( long delay ) {
+        this.delay = delay;
+    }
+
+
+    public void setLatch( int calls ) {
+        latch = new CountDownLatch( calls );
+    }
+
+
+    public boolean waitForCount( long timeout, TimeUnit unit ) throws InterruptedException {
+        return latch.await( timeout, unit );
+    }
+
+
+    public boolean waitForException( long timeout, TimeUnit unit ) throws InterruptedException {
+        return exception.await( timeout, unit );
+    }
+
+
+    public boolean waitForCompletion( long timeout, TimeUnit unit ) throws InterruptedException {
+        return completed.await( timeout, unit );
+    }
+
+
+    /** @return the timeout */
+    public long getTimeout() {
+        return timeout;
+    }
+
+
+    /** @param timeout the timeout to set */
+    public void setTimeout( long timeout ) {
+        this.timeout = timeout;
+    }
+
+
+    @Override
+    public void dead( final JobExecution execution ) throws Exception {
+        //To change body of implemented methods use File | Settings | File Templates.
+    }
+}


[20/21] usergrid git commit: Error checking in WriteStart, fixing timing in UV serialization test and ignore test broken by disabled delete.

Posted by gr...@apache.org.
Error checking in WriteStart, fixing timing in UV serialization test and ignore test broken by disabled delete.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/c96e2944
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/c96e2944
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/c96e2944

Branch: refs/heads/master
Commit: c96e2944fa170cb4bc68c18943310a42de9fa352
Parents: 78f3da6
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 14:51:08 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 14:51:08 2015 -0500

----------------------------------------------------------------------
 .../persistence/collection/mvcc/stage/write/WriteStart.java   | 7 +++++--
 .../impl/UniqueValueSerializationStrategyImplTest.java        | 4 ++--
 .../persistence/graph/impl/NodeDeleteListenerTest.java        | 1 +
 3 files changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/c96e2944/stack/corepersistence/collection/src/main/java/org/apache/usergrid/persistence/collection/mvcc/stage/write/WriteStart.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/main/java/org/apache/usergrid/persistence/collection/mvcc/stage/write/WriteStart.java b/stack/corepersistence/collection/src/main/java/org/apache/usergrid/persistence/collection/mvcc/stage/write/WriteStart.java
index 7d73b7b..79b2d85 100644
--- a/stack/corepersistence/collection/src/main/java/org/apache/usergrid/persistence/collection/mvcc/stage/write/WriteStart.java
+++ b/stack/corepersistence/collection/src/main/java/org/apache/usergrid/persistence/collection/mvcc/stage/write/WriteStart.java
@@ -26,6 +26,8 @@ import com.netflix.astyanax.connectionpool.exceptions.ConnectionException;
 
 import rx.functions.Func1;
 
+import static org.apache.usergrid.persistence.core.util.ValidationUtils.verifyEntityWrite;
+
 
 /**
  * This is the first stage and should be invoked immediately when a write is started.  It should
@@ -53,7 +55,9 @@ public class WriteStart implements Func1<CollectionIoEvent<Entity>, CollectionIo
 
     @Override
     public CollectionIoEvent<MvccEntity> call( final CollectionIoEvent<Entity> ioEvent ) {
-        {
+
+            verifyEntityWrite( ioEvent.getEvent() );
+
             final Entity entity = ioEvent.getEvent();
             final ApplicationScope applicationScope = ioEvent.getEntityCollection();
 
@@ -84,6 +88,5 @@ public class WriteStart implements Func1<CollectionIoEvent<Entity>, CollectionIo
             //create the mvcc entity for the next stage
 
             return new CollectionIoEvent<>( applicationScope, nextStage );
-        }
     }
 }

http://git-wip-us.apache.org/repos/asf/usergrid/blob/c96e2944/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/UniqueValueSerializationStrategyImplTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/UniqueValueSerializationStrategyImplTest.java b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/UniqueValueSerializationStrategyImplTest.java
index 2fae482..fcf22cf 100644
--- a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/UniqueValueSerializationStrategyImplTest.java
+++ b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/UniqueValueSerializationStrategyImplTest.java
@@ -127,7 +127,7 @@ public abstract class UniqueValueSerializationStrategyImplTest {
         Id entityId = new SimpleId( UUIDGenerator.newTimeUUID(), "entity" );
         UUID version = UUIDGenerator.newTimeUUID();
         UniqueValue stored = new UniqueValueImpl( field, entityId, version );
-        strategy.write( scope, stored, 2 ).execute();
+        strategy.write( scope, stored, 5 ).execute();
 
         Thread.sleep( 1000 );
 
@@ -139,7 +139,7 @@ public abstract class UniqueValueSerializationStrategyImplTest {
         Assert.assertNotNull( retrieved );
         assertEquals( stored, retrieved );
 
-        Thread.sleep( 1500 );
+        Thread.sleep( 5000 );
 
         // wait another second, should be gone now
         fields = strategy.load( scope, entityId.getType(), Collections.<Field>singleton( field ) );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/c96e2944/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
index f2d4d94..5a5df79 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
@@ -319,6 +319,7 @@ public class NodeDeleteListenerTest {
      * since it has no other targets
      */
     @Test
+    @Ignore("Pending re-enable of delete functionality")
     public void testMultiDelete() throws ConnectionException, InterruptedException {
 
         GraphManager em = emf.createEdgeManager( scope );


[02/21] usergrid git commit: Remove some unnecessary ignore annotations and some unnecessary code.

Posted by gr...@apache.org.
Remove some unnecessary ignore annotations and some unnecessary code.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/1434deaa
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/1434deaa
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/1434deaa

Branch: refs/heads/master
Commit: 1434deaace2bd9bb9a237b1a5da9fbaab70ec1e7
Parents: f1aa450
Author: Dave Johnson <sn...@apache.org>
Authored: Fri Nov 13 10:54:13 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Fri Nov 13 10:54:13 2015 -0500

----------------------------------------------------------------------
 .../corepersistence/StaleIndexCleanupTest.java  |  2 +-
 .../usergrid/corepersistence/index/RxTest.java  |  2 +-
 .../collection/CollectionsResourceIT.java       | 18 ------
 .../usergrid/rest/test/resource/DumbClient.java | 58 --------------------
 .../usergrid/cassandra/FakeSchemaManager.java   |  4 --
 5 files changed, 2 insertions(+), 82 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/1434deaa/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java b/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
index 5e0e062..2f96c18 100644
--- a/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
@@ -196,7 +196,7 @@ public class StaleIndexCleanupTest extends AbstractCoreIT {
      * it is building search results.
      */
     @Test
-    @Ignore("Broken until search connections is fixed")
+    @Ignore("Breaks other tests - or would if we reenabled that EVENTS_DISABLED property")
     public void testStaleIndexCleanup() throws Exception {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/1434deaa/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java b/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
index 020371d..1a94aac 100644
--- a/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
@@ -64,7 +64,7 @@ public class RxTest {
 
 
     @Test
-    @Ignore("This seems like it should work, yet blocks forever")
+    @Ignore("Experimental - seems like it should work, yet blocks forever")
     public void testConnectableObserver() throws InterruptedException {
 
         final int count = 10;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/1434deaa/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/CollectionsResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/CollectionsResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/CollectionsResourceIT.java
index 92eac90..57f7b2c 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/CollectionsResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/collection/CollectionsResourceIT.java
@@ -103,24 +103,6 @@ public class CollectionsResourceIT extends AbstractRestIT {
 
     }
 
-    @Ignore("Not sure that this test makes any sense")
-    @Test
-    public void postToEmptyCollection() throws IOException {
-/*
-        Entity payload = new Entity();
-        Entity entity = this.app().collection("cities").post(payload);
-        assertNull(entity.get("name"));
-
-
-        Map<String, String> payload = new HashMap<String, String>();
-
-        JsonNode node = mapper.readTree( resource().path( "/test-organization/test-app/cities" ).queryParam( "access_token", access_token )
-                        .accept( MediaType.APPLICATION_JSON ).type( MediaType.APPLICATION_JSON_TYPE )
-                        .post( String.class, payload ));
-        assertNull( getEntity( node, 0 ) );
-        assertNull( node.get( "count" ) );
-*/
-    }
 
     @Test
     public void postToReservedField() throws Exception {

http://git-wip-us.apache.org/repos/asf/usergrid/blob/1434deaa/stack/rest/src/test/java/org/apache/usergrid/rest/test/resource/DumbClient.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/test/resource/DumbClient.java b/stack/rest/src/test/java/org/apache/usergrid/rest/test/resource/DumbClient.java
deleted file mode 100644
index ae74d4b..0000000
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/test/resource/DumbClient.java
+++ /dev/null
@@ -1,58 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.usergrid.rest.test.resource;
-
-import org.junit.Ignore;
-import org.junit.Test;
-
-import org.apache.usergrid.persistence.index.utils.UUIDUtils;
-
-
-
-/**
- * Test Class used to model if the client is working or doing what it is supposed to be doing.
- * ask if this is for test purposes and if so if I can mark it with junit
- */
-
-public class DumbClient extends AbstractRestIT {
-
-    @Test
-    public void stuff(){
-
-        String name = "stuff"+ UUIDUtils.newTimeUUID();
-       // User user = new User( "derp","derp", "derp"  );
-
-
-        //Organization org = clientSetup.getRestClient().management().orgs().post(  )
-      //  clientSetup.getRestClient().management().orgs().delete(org.getName);
-       // OrganizationResource response =  clientSetup.getRestClient().management().orgs().organization( "" );
-        //assertNotNull( response );
-        //EntityResponse itr  =  client.org( "test" ).getApp( "test" ).users().getEntityResponse();
-        //for(Entity entity: itr){
-    }
-
-
-    @Ignore
-    public void stateful(){
-
-//        EntityResponse itr  =  client.org( "test" ).getApp( "test" ).users().getEntityResponse();
-//
-//        for(Entity entity: itr){
-//
-//        }
-    }
-}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/1434deaa/stack/test-utils/src/test/java/org/apache/usergrid/cassandra/FakeSchemaManager.java
----------------------------------------------------------------------
diff --git a/stack/test-utils/src/test/java/org/apache/usergrid/cassandra/FakeSchemaManager.java b/stack/test-utils/src/test/java/org/apache/usergrid/cassandra/FakeSchemaManager.java
index 7d80c6e..bb6a933 100644
--- a/stack/test-utils/src/test/java/org/apache/usergrid/cassandra/FakeSchemaManager.java
+++ b/stack/test-utils/src/test/java/org/apache/usergrid/cassandra/FakeSchemaManager.java
@@ -17,11 +17,7 @@
 package org.apache.usergrid.cassandra;
 
 
-import org.junit.Ignore;
-
-
 /** @author zznate */
-@Ignore
 public class FakeSchemaManager implements SchemaManager {
     @Override
     public void create() {


[13/21] usergrid git commit: Add an additional reindex/wait to test that fails intermittently.

Posted by gr...@apache.org.
Add an additional reindex/wait to test that fails intermittently.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/4cf69e6f
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/4cf69e6f
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/4cf69e6f

Branch: refs/heads/master
Commit: 4cf69e6f0498dbdc55e55739d38dd78f32a87e58
Parents: 41dde12
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 08:12:30 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 08:12:30 2015 -0500

----------------------------------------------------------------------
 .../src/test/java/org/apache/usergrid/management/RoleIT.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/4cf69e6f/stack/services/src/test/java/org/apache/usergrid/management/RoleIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/RoleIT.java b/stack/services/src/test/java/org/apache/usergrid/management/RoleIT.java
index e2a1941..0bd4f41 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/RoleIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/RoleIT.java
@@ -64,7 +64,6 @@ public class RoleIT {
 
         UUID applicationId = setup.getMgmtSvc().createApplication( organization.getUuid(), "test-app" ).getId();
         EntityManager em = setup.getEmf().getEntityManager( applicationId );
-
         setup.getEntityIndex().refresh(em.getApplicationId());
 
         Map<String, Object> properties = new LinkedHashMap<String, Object>();
@@ -75,6 +74,7 @@ public class RoleIT {
 
         em.createRole( "logged-in", "Logged In", 2000 );
         setup.getEntityIndex().refresh(em.getApplicationId());
+        setup.getEntityIndex().refresh(em.getApplicationId());
         em.addUserToRole( user.getUuid(), "logged-in" );
 
         String accessToken = setup.getMgmtSvc().getAccessTokenForAppUser( applicationId, user.getUuid(), 0 );


[03/21] usergrid git commit: Fix no-org-delete and test.

Posted by gr...@apache.org.
Fix no-org-delete and test.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/59d2a71c
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/59d2a71c
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/59d2a71c

Branch: refs/heads/master
Commit: 59d2a71c761e1419d143acafd02a08f80a3293eb
Parents: 1434dea
Author: Dave Johnson <sn...@apache.org>
Authored: Fri Nov 13 12:52:39 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Fri Nov 13 12:52:39 2015 -0500

----------------------------------------------------------------------
 .../management/organizations/OrganizationResource.java    |  9 +++++++++
 .../apache/usergrid/rest/management/OrganizationsIT.java  | 10 +++++-----
 2 files changed, 14 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/59d2a71c/stack/rest/src/main/java/org/apache/usergrid/rest/management/organizations/OrganizationResource.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/main/java/org/apache/usergrid/rest/management/organizations/OrganizationResource.java b/stack/rest/src/main/java/org/apache/usergrid/rest/management/organizations/OrganizationResource.java
index e1fde46..12fbc63 100644
--- a/stack/rest/src/main/java/org/apache/usergrid/rest/management/organizations/OrganizationResource.java
+++ b/stack/rest/src/main/java/org/apache/usergrid/rest/management/organizations/OrganizationResource.java
@@ -20,6 +20,7 @@ package org.apache.usergrid.rest.management.organizations;
 import com.fasterxml.jackson.jaxrs.json.annotation.JSONP;
 import org.apache.amber.oauth2.common.exception.OAuthSystemException;
 import org.apache.commons.lang.NullArgumentException;
+import org.apache.usergrid.exception.NotImplementedException;
 import org.apache.usergrid.management.ActivationState;
 import org.apache.usergrid.management.OrganizationConfig;
 import org.apache.usergrid.management.OrganizationInfo;
@@ -420,4 +421,12 @@ public class OrganizationResource extends AbstractContextResource {
         return response;
     }
 
+
+    /** Delete organization is not yet supported */
+    //@RequireSystemAccess
+    @DELETE
+    public ApiResponse deleteOrganization() throws Exception {
+        throw new NotImplementedException();
+    }
+
 }

http://git-wip-us.apache.org/repos/asf/usergrid/blob/59d2a71c/stack/rest/src/test/java/org/apache/usergrid/rest/management/OrganizationsIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/OrganizationsIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/OrganizationsIT.java
index 72a2f8c..3acdfd5 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/OrganizationsIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/OrganizationsIT.java
@@ -31,6 +31,7 @@ import org.apache.usergrid.rest.test.resource.AbstractRestIT;
 import org.apache.usergrid.rest.test.resource.RestClient;
 
 import javax.ws.rs.ClientErrorException;
+import javax.ws.rs.ServerErrorException;
 import javax.ws.rs.core.Form;
 import javax.ws.rs.core.Response;
 
@@ -291,18 +292,17 @@ public class OrganizationsIT extends AbstractRestIT {
 
     /**
      * Returns error from unimplemented delete method by trying to call the delete organization endpoint
-     * @throws IOException
      */
-    @Ignore("It should return a 501, so when this is fixed the test can be run")
     @Test
     public void noOrgDelete() throws IOException {
 
         try {
-            //Delete default organization
+            // attempt to delete default organization
             clientSetup.getRestClient().management().orgs().org( clientSetup.getOrganizationName() ).delete();
             fail( "Delete is not implemented yet" );
-        }catch(ClientErrorException uie){
-            assertEquals( Response.Status.NOT_IMPLEMENTED ,uie.getResponse().getStatus());
+
+        } catch( ServerErrorException see ){
+            assertEquals( Response.Status.NOT_IMPLEMENTED.getStatusCode(), see.getResponse().getStatus());
         }
     }
 


[21/21] usergrid git commit: Use variable instead of hardcoded JUnit version number

Posted by gr...@apache.org.
Use variable instead of hardcoded JUnit version number


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/6f3541be
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/6f3541be
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/6f3541be

Branch: refs/heads/master
Commit: 6f3541be60998f23f95c9794a77d7230396db3e3
Parents: c96e294
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 15:14:51 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 15:14:51 2015 -0500

----------------------------------------------------------------------
 stack/build-tools/pom.xml | 2 +-
 stack/config/pom.xml      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/6f3541be/stack/build-tools/pom.xml
----------------------------------------------------------------------
diff --git a/stack/build-tools/pom.xml b/stack/build-tools/pom.xml
index 2001396..91c655d 100644
--- a/stack/build-tools/pom.xml
+++ b/stack/build-tools/pom.xml
@@ -44,7 +44,7 @@
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>
-            <version>4.12</version>
+            <version>${junit-version}</version>
             <scope>test</scope>
         </dependency>
     </dependencies>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/6f3541be/stack/config/pom.xml
----------------------------------------------------------------------
diff --git a/stack/config/pom.xml b/stack/config/pom.xml
index 4ee1a3b..c445c58 100644
--- a/stack/config/pom.xml
+++ b/stack/config/pom.xml
@@ -47,7 +47,7 @@
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>
-            <version>4.12</version>
+            <version>${junit-version}</version>
             <scope>test</scope>
         </dependency>
     </dependencies>


[10/21] usergrid git commit: Fix spelling error in class names.

Posted by gr...@apache.org.
Fix spelling error in class names.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/d1f17107
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/d1f17107
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/d1f17107

Branch: refs/heads/master
Commit: d1f17107631c3e7c83ae0efccb371e3979c174ed
Parents: f62e0aa
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 07:52:21 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 07:52:21 2015 -0500

----------------------------------------------------------------------
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java  | 2 +-
 .../java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/d1f17107/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
index 239101c..5d2e00d 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime6IT.java
@@ -50,7 +50,7 @@ public class SchedulerRuntime6IT extends AbstractSchedulerRuntimeIT {
         long customRetry = sleepTime + 1000;
         int numberOfRuns = 1;
 
-        OnlyOnceExceution job = springResource.getBean( "onlyOnceExceution", OnlyOnceExceution.class );
+        OnlyOnceExecution job = springResource.getBean( "onlyOnceExceution", OnlyOnceExecution.class );
 
         job.setTimeout( customRetry );
         job.setLatch( numberOfRuns );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/d1f17107/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
index 091c875..000996d 100644
--- a/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
+++ b/stack/core/src/test/java/org/apache/usergrid/batch/job/SchedulerRuntime7IT.java
@@ -46,8 +46,8 @@ public class SchedulerRuntime7IT extends AbstractSchedulerRuntimeIT {
         long customRetry = sleepTime * 2;
         int numberOfRuns = 2;
 
-        OnlyOnceUnlockOnFailExceution job =
-                springResource.getBean( "onlyOnceUnlockOnFailExceution", OnlyOnceUnlockOnFailExceution.class );
+        OnlyOnceUnlockOnFailExecution job =
+                springResource.getBean( "onlyOnceUnlockOnFailExceution", OnlyOnceUnlockOnFailExecution.class );
 
         job.setTimeout( customRetry );
         job.setLatch( numberOfRuns );


[07/21] usergrid git commit: Tests run more reliably with 20 iterations.

Posted by gr...@apache.org.
Tests run more reliably with 20 iterations.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/f2cbf1a9
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/f2cbf1a9
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/f2cbf1a9

Branch: refs/heads/master
Commit: f2cbf1a98c723fe75ec6ff967e74c95b09aa82a3
Parents: bba77f0
Author: Dave Johnson <sn...@apache.org>
Authored: Mon Nov 16 13:25:58 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Mon Nov 16 13:25:58 2015 -0500

----------------------------------------------------------------------
 .../java/org/apache/usergrid/corepersistence/CpEntityManager.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/f2cbf1a9/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManager.java
----------------------------------------------------------------------
diff --git a/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManager.java b/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManager.java
index 009266b..522c523 100644
--- a/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManager.java
+++ b/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManager.java
@@ -2887,7 +2887,7 @@ public class CpEntityManager implements EntityManager {
             EntityIndex.IndexRefreshCommandInfo indexRefreshCommandInfo
                 = managerCache.getEntityIndex(applicationScope).refreshAsync().toBlocking().first();
             try {
-                for (int i = 0; i < 10; i++) {
+                for (int i = 0; i < 20; i++) {
                     if (searchCollection(
                         new SimpleEntityRef(org.apache.usergrid.persistence.entities.Application.ENTITY_TYPE, getApplicationId()),
                         InflectionUtils.pluralize("refresh"),


[04/21] usergrid git commit: Un-ignore some tests that now appear to be working (again).

Posted by gr...@apache.org.
Un-ignore some tests that now appear to be working (again).


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/5c85b759
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/5c85b759
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/5c85b759

Branch: refs/heads/master
Commit: 5c85b7598586abe0b11e29918b09be11addec831
Parents: 59d2a71
Author: Dave Johnson <sn...@apache.org>
Authored: Fri Nov 13 16:32:55 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Fri Nov 13 16:32:55 2015 -0500

----------------------------------------------------------------------
 .../applications/ApplicationResourceIT.java     | 25 ++----------
 .../applications/queries/MatrixQueryTests.java  |  2 -
 .../rest/filters/ContentTypeResourceIT.java     |  4 --
 .../usergrid/rest/management/AdminUsersIT.java  |  8 ++--
 .../cassandra/ManagementServiceIT.java          | 41 +++++++-------------
 5 files changed, 21 insertions(+), 59 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/5c85b759/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
index 317b742..cd42262 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/ApplicationResourceIT.java
@@ -230,7 +230,7 @@ public class ApplicationResourceIT extends AbstractRestIT {
      * (for backwards compatibility)
      */
     @Test
-    @Ignore("this form of backwards compatibility no longer needed")
+    @Ignore("no longer relevant - this type of backwards compatibility no longer needed")
     public void jsonForAcceptsTextHtml() throws Exception {
 
         //Create the organization resource
@@ -371,22 +371,6 @@ public class ApplicationResourceIT extends AbstractRestIT {
         assertNotNull(credentials.getClientSecret());
     }
 
-
-    @Test
-    @Ignore //This is implemented now
-    public void noAppDelete() throws IOException {
-        String orgName = clientSetup.getOrganizationName().toLowerCase();
-        String appName = clientSetup.getAppName().toLowerCase();
-
-        ApiResponse apiResponse = target().path( String.format( "/%s/%s", orgName, appName ) )
-            .queryParam( "access_token", this.getAdminToken().getAccessToken() )
-            .request()
-            .accept( MediaType.APPLICATION_JSON )
-            .delete( ApiResponse.class );
-
-        assertNotNull(apiResponse.getError());
-    }
-
     /**
      * Test for an exception when a token's TTL is set greater than the maximum
      */
@@ -591,7 +575,6 @@ public class ApplicationResourceIT extends AbstractRestIT {
      * Retrieve an oauth authorization using invalid credentials
      */
     @Test
-    @Ignore("viewable return types are not working in embedded tomcat 7.x")
     public void authorizationCodeWithWrongCredentials() throws Exception {
         //Create form input with bogus credentials
         Form payload = new Form();
@@ -699,8 +682,8 @@ public class ApplicationResourceIT extends AbstractRestIT {
      * Retrieve an access token using HTTP Basic authentication
      */
     @Test
-    @Ignore
-    //Are we trying to generate token with token? Couldn't find enpoint that accepts token for generating token
+    @Ignore("Should fix - https://issues.apache.org/jira/browse/USERGRID-1106")
+    //Are we trying to generate token with token? Couldn't find endpoint that accepts token for generating token
     public void clientCredentialsFlowWithHeaderAuthorization() throws Exception {
         //retrieve the credentials
         Credentials orgCredentials = getAppCredentials();
@@ -783,7 +766,7 @@ public class ApplicationResourceIT extends AbstractRestIT {
      * Ensure that the Apigee Mobile Analytics config returns valid JSON
      */
     @Test
-    @Ignore
+    @Ignore("No longer relevant - Apigee internal")
     public void validateApigeeApmConfigAPP() throws IOException {
         String orgName = clientSetup.getOrganizationName().toLowerCase();
         String appName = clientSetup.getAppName().toLowerCase();

http://git-wip-us.apache.org/repos/asf/usergrid/blob/5c85b759/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/MatrixQueryTests.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/MatrixQueryTests.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/MatrixQueryTests.java
index de21e0a..7525d51 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/MatrixQueryTests.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/MatrixQueryTests.java
@@ -41,7 +41,6 @@ public class MatrixQueryTests extends AbstractRestIT {
      *
      * @throws Exception
      */
-    @Ignore
     @Test
     public void connectionsTest() throws Exception {
 
@@ -105,7 +104,6 @@ public class MatrixQueryTests extends AbstractRestIT {
         assertEquals(1, user3likes.getResponse().getEntityCount());
     }
 
-    @Ignore
     @Test
     public void largeRootElements() {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/5c85b759/stack/rest/src/test/java/org/apache/usergrid/rest/filters/ContentTypeResourceIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/filters/ContentTypeResourceIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/filters/ContentTypeResourceIT.java
index 47afb8c..ce57c4b 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/filters/ContentTypeResourceIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/filters/ContentTypeResourceIT.java
@@ -50,12 +50,8 @@ import static org.junit.Assert.assertNotNull;
 /**
  * @author tnine
  */
-
-// @Ignore("Client login is causing tests to fail due to socket closure by grizzly.  Need to re-enable once we're not
-// using grizzly to test")
 public class ContentTypeResourceIT extends AbstractRestIT {
 
-
     /**
      * Creates a simple entity of type game. Does not set the content type. The type should be set to json to match the
      * body

http://git-wip-us.apache.org/repos/asf/usergrid/blob/5c85b759/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
index 9864e65..3aed997 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
@@ -398,7 +398,7 @@ public class AdminUsersIT extends AbstractRestIT {
         assertFalse( inbox.isEmpty() );
     }
 
-    @Ignore("Test is broken due to viewables not being properly returned in the embedded tomcat")
+    @Ignore("Build problems - Test is broken due to viewables not being properly returned in the embedded tomcat")
     @Test
     public void checkFormPasswordReset() throws Exception {
 
@@ -434,8 +434,8 @@ public class AdminUsersIT extends AbstractRestIT {
 
         assertTrue( html.contains( "invalid token" ) );
     }
-//
-//     TODO: will work once resetpw viewables work
+
+//    // TODO: will work once resetpw viewables work
 //    @Test
 //    @Ignore( "causes problems in build" )
 //    public void passwordResetIncorrectUserName() throws Exception {
@@ -462,8 +462,6 @@ public class AdminUsersIT extends AbstractRestIT {
 //
 //        assertTrue( html.contains( "password set" ) );
 //    }
-//
-//
 
 
     /**

http://git-wip-us.apache.org/repos/asf/usergrid/blob/5c85b759/stack/services/src/test/java/org/apache/usergrid/management/cassandra/ManagementServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/management/cassandra/ManagementServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/management/cassandra/ManagementServiceIT.java
index f0c0c10..ad2ce66 100644
--- a/stack/services/src/test/java/org/apache/usergrid/management/cassandra/ManagementServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/management/cassandra/ManagementServiceIT.java
@@ -17,25 +17,11 @@
 package org.apache.usergrid.management.cassandra;
 
 
-import java.util.LinkedHashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.UUID;
-
-import org.junit.Before;
-import org.junit.ClassRule;
-import org.junit.Ignore;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import org.apache.usergrid.NewOrgAppAdminRule;
 import org.apache.usergrid.ServiceITSetup;
 import org.apache.usergrid.ServiceITSetupImpl;
-import org.apache.usergrid.cassandra.SpringResource;
 import org.apache.usergrid.cassandra.ClearShiroSubject;
-
+import org.apache.usergrid.cassandra.SpringResource;
 import org.apache.usergrid.count.SimpleBatcher;
 import org.apache.usergrid.management.OrganizationInfo;
 import org.apache.usergrid.management.UserInfo;
@@ -43,7 +29,6 @@ import org.apache.usergrid.persistence.CredentialsInfo;
 import org.apache.usergrid.persistence.Entity;
 import org.apache.usergrid.persistence.EntityManager;
 import org.apache.usergrid.persistence.entities.User;
-import org.apache.usergrid.persistence.index.impl.ElasticSearchResource;
 import org.apache.usergrid.security.AuthPrincipalType;
 import org.apache.usergrid.security.crypto.command.Md5HashCommand;
 import org.apache.usergrid.security.crypto.command.Sha1HashCommand;
@@ -51,19 +36,22 @@ import org.apache.usergrid.security.tokens.TokenCategory;
 import org.apache.usergrid.security.tokens.exceptions.InvalidTokenException;
 import org.apache.usergrid.utils.JsonUtils;
 import org.apache.usergrid.utils.UUIDUtils;
+import org.junit.Before;
+import org.junit.ClassRule;
+import org.junit.Rule;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.UUID;
 
 import static org.apache.commons.codec.binary.Base64.encodeBase64URLSafeString;
-import static org.apache.usergrid.TestHelper.newUUIDString;
-import static org.apache.usergrid.TestHelper.uniqueApp;
-import static org.apache.usergrid.TestHelper.uniqueEmail;
-import static org.apache.usergrid.TestHelper.uniqueOrg;
-import static org.apache.usergrid.TestHelper.uniqueUsername;
+import static org.apache.usergrid.TestHelper.*;
 import static org.apache.usergrid.persistence.Schema.DICTIONARY_CREDENTIALS;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.*;
 
 /**
  * @author zznate
@@ -462,7 +450,6 @@ public class ManagementServiceIT {
     }
 
 
-    @Ignore("Why is this ignored?")
     public void superUserGetOrganizationsPage() throws Exception {
         int beforeSize = setup.getMgmtSvc().getOrganizations().size() - 1;
         // create 15 orgs


[11/21] usergrid git commit: Use JUnut Categories to exclude too heavy (aka stress) tests and experimental tests.

Posted by gr...@apache.org.
Use JUnut Categories to exclude too heavy (aka stress) tests and experimental tests.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/99b932ad
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/99b932ad
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/99b932ad

Branch: refs/heads/master
Commit: 99b932ada3809a33ad8a824b9177d427edb6feba
Parents: d1f1710
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 07:54:39 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 07:54:39 2015 -0500

----------------------------------------------------------------------
 stack/build-tools/pom.xml                       | 11 +++++
 stack/config/pom.xml                            | 10 +++++
 .../clustering/hazelcast/HazelcastTest.java     |  5 ++-
 .../usergrid/corepersistence/index/RxTest.java  |  6 ++-
 .../persistence/PerformanceEntityReadTest.java  |  4 +-
 .../persistence/PerformanceEntityWriteTest.java |  4 +-
 .../EntityCollectionManagerStressTest.java      |  6 ++-
 .../impl/SerializationComparison.java           |  5 ++-
 stack/corepersistence/common/pom.xml            |  1 +
 stack/corepersistence/graph/pom.xml             |  1 +
 .../persistence/graph/GraphManagerLoadTest.java |  7 ++--
 .../graph/GraphManagerShardConsistencyIT.java   |  4 +-
 .../graph/GraphManagerStressTest.java           | 44 +++++++++-----------
 .../serialization/EdgeSerializationTest.java    |  6 ++-
 stack/corepersistence/map/pom.xml               |  1 +
 .../org/apache/usergrid/ExperimentalTest.java   | 22 ++++++++++
 .../java/org/apache/usergrid/StressTest.java    | 22 ++++++++++
 stack/corepersistence/pom.xml                   |  2 +
 stack/corepersistence/queryindex/pom.xml        |  5 +--
 .../index/impl/IndexLoadTestsIT.java            |  3 ++
 stack/pom.xml                                   | 12 ++----
 stack/rest/pom.xml                              |  2 +
 stack/services/pom.xml                          |  8 +---
 .../providers/PingIdentityProviderIT.java       |  4 +-
 .../AbstractServiceNotificationIT.java          |  1 +
 .../apns/NotificationsServiceIT.java            |  1 +
 .../gcm/NotificationsServiceIT.java             |  2 +
 27 files changed, 139 insertions(+), 60 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/build-tools/pom.xml
----------------------------------------------------------------------
diff --git a/stack/build-tools/pom.xml b/stack/build-tools/pom.xml
index 149c1a7..2001396 100644
--- a/stack/build-tools/pom.xml
+++ b/stack/build-tools/pom.xml
@@ -38,4 +38,15 @@
         o to adjust checkstyle settings or suppress reporting please us the
           files contained under src/main/resources/usergrid
   -->
+
+    <!-- the surefire excludedGroups configuration in parent requires this dependency here -->
+    <dependencies>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.12</version>
+            <scope>test</scope>
+        </dependency>
+    </dependencies>
+
 </project>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/config/pom.xml
----------------------------------------------------------------------
diff --git a/stack/config/pom.xml b/stack/config/pom.xml
index 915d32a..4ee1a3b 100644
--- a/stack/config/pom.xml
+++ b/stack/config/pom.xml
@@ -42,4 +42,14 @@
       </resource>
     </resources>
   </build>
+
+    <dependencies>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.12</version>
+            <scope>test</scope>
+        </dependency>
+    </dependencies>
+
 </project>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/core/src/test/java/org/apache/usergrid/clustering/hazelcast/HazelcastTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/clustering/hazelcast/HazelcastTest.java b/stack/core/src/test/java/org/apache/usergrid/clustering/hazelcast/HazelcastTest.java
index 8f6226b..63a181e 100644
--- a/stack/core/src/test/java/org/apache/usergrid/clustering/hazelcast/HazelcastTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/clustering/hazelcast/HazelcastTest.java
@@ -20,10 +20,13 @@ package org.apache.usergrid.clustering.hazelcast;
 import java.util.Collection;
 import java.util.Set;
 
+import org.apache.usergrid.ExperimentalTest;
+import org.apache.usergrid.StressTest;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.config.AutowireCapableBeanFactory;
@@ -38,7 +41,7 @@ import com.hazelcast.core.Member;
 import com.hazelcast.core.MessageListener;
 
 
-@Ignore("Experimental test")
+@Category(ExperimentalTest.class)
 public class HazelcastTest implements InstanceListener, MessageListener<Object> {
 
     private static final Logger logger = LoggerFactory.getLogger( HazelcastTest.class );

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java b/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
index 1a94aac..6bb8947 100644
--- a/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/corepersistence/index/RxTest.java
@@ -23,9 +23,11 @@ package org.apache.usergrid.corepersistence.index;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.junit.Ignore;
 import org.junit.Test;
 
+import org.junit.experimental.categories.Category;
 import rx.Observable;
 import rx.Subscription;
 import rx.observables.ConnectableObservable;
@@ -41,7 +43,7 @@ import static org.junit.Assert.assertTrue;
 public class RxTest {
 
     @Test
-    @Ignore("This fails intermittently.  Possible race condition with Rx.  Need to investigate more.")
+    @Category(ExperimentalTest.class )
     public void testPublish() throws InterruptedException {
 
         final int count = 10;
@@ -64,7 +66,7 @@ public class RxTest {
 
 
     @Test
-    @Ignore("Experimental - seems like it should work, yet blocks forever")
+    @Category(ExperimentalTest.class )
     public void testConnectableObserver() throws InterruptedException {
 
         final int count = 10;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityReadTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityReadTest.java b/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityReadTest.java
index f7435ef..88144c8 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityReadTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityReadTest.java
@@ -24,11 +24,13 @@ import java.util.Map;
 import java.util.UUID;
 import java.util.concurrent.TimeUnit;
 
+import org.apache.usergrid.StressTest;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -44,7 +46,7 @@ import com.codahale.metrics.Slf4jReporter;
 //@RunWith(JukitoRunner.class)
 //@UseModules({ GuiceModule.class })
 
-@Ignore("Kills embedded cassandra")
+@Category(StressTest.class)
 public class PerformanceEntityReadTest extends AbstractCoreIT {
     private static final Logger logger = LoggerFactory.getLogger(PerformanceEntityReadTest.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityWriteTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityWriteTest.java b/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityWriteTest.java
index 341af90..3997615 100644
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityWriteTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityWriteTest.java
@@ -23,11 +23,13 @@ import java.util.List;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
+import org.apache.usergrid.StressTest;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -43,7 +45,7 @@ import com.codahale.metrics.Slf4jReporter;
 //@RunWith(JukitoRunner.class)
 //@UseModules({ GuiceModule.class })
 
-@Ignore("Kills embedded cassandra")
+@Category(StressTest.class)
 public class PerformanceEntityWriteTest extends AbstractCoreIT {
     private static final Logger LOG = LoggerFactory.getLogger( PerformanceEntityWriteTest.class );
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/EntityCollectionManagerStressTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/EntityCollectionManagerStressTest.java b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/EntityCollectionManagerStressTest.java
index 20cf24e..1aabf75 100644
--- a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/EntityCollectionManagerStressTest.java
+++ b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/EntityCollectionManagerStressTest.java
@@ -21,9 +21,11 @@ package org.apache.usergrid.persistence.collection;
 import java.util.HashSet;
 import java.util.Set;
 
+import org.apache.usergrid.StressTest;
 import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.junit.runner.RunWith;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -50,7 +52,7 @@ import static org.junit.Assert.assertNotNull;
 
 @RunWith(ITRunner.class)
 @UseModules(TestCollectionModule.class)
-@Ignore("Stress test should not be run in embedded mode")
+@Category(StressTest.class)
 public class EntityCollectionManagerStressTest {
     private static final Logger log = LoggerFactory.getLogger(
             EntityCollectionManagerStressTest.class );
@@ -58,7 +60,7 @@ public class EntityCollectionManagerStressTest {
     @Inject
     private EntityCollectionManagerFactory factory;
 
-      @Inject
+    @Inject
     @Rule
     public MigrationManagerRule migrationManagerRule;
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/SerializationComparison.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/SerializationComparison.java b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/SerializationComparison.java
index 660a6a9..a2ed6c7 100644
--- a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/SerializationComparison.java
+++ b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/serialization/impl/SerializationComparison.java
@@ -23,8 +23,9 @@ package org.apache.usergrid.persistence.collection.serialization.impl;
 import java.io.IOException;
 import java.util.UUID;
 
-import org.junit.Ignore;
+import org.apache.usergrid.StressTest;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -48,6 +49,7 @@ import com.fasterxml.jackson.dataformat.smile.SmileFactory;
  *
  * @author tnine
  */
+@Category(StressTest.class)
 public class SerializationComparison {
 
     private static final Logger logger = LoggerFactory.getLogger( SerializationComparison.class );
@@ -56,7 +58,6 @@ public class SerializationComparison {
 
 
     @Test
-    @Ignore("Too heavy for normal build process?")
     public void smileSerialization() throws IOException {
         SmileFactory smile = new SmileFactory();
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/common/pom.xml
----------------------------------------------------------------------
diff --git a/stack/corepersistence/common/pom.xml b/stack/corepersistence/common/pom.xml
index 2082b04..0f54896 100644
--- a/stack/corepersistence/common/pom.xml
+++ b/stack/corepersistence/common/pom.xml
@@ -183,6 +183,7 @@
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/graph/pom.xml
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/pom.xml b/stack/corepersistence/graph/pom.xml
index 23621fb..93fbbd5 100644
--- a/stack/corepersistence/graph/pom.xml
+++ b/stack/corepersistence/graph/pom.xml
@@ -104,6 +104,7 @@
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerLoadTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerLoadTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerLoadTest.java
index 22683f6..a0be6a6 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerLoadTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerLoadTest.java
@@ -29,10 +29,12 @@ import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 
+import org.apache.usergrid.StressTest;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.junit.runner.RunWith;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -63,7 +65,7 @@ import static org.junit.Assert.fail;
 
 @RunWith( ITRunner.class )
 @UseModules( TestGraphModule.class )
-@Ignore("Not for testing during build.  Kills embedded Cassandra")
+@Category(StressTest.class)
 public class GraphManagerLoadTest {
     private static final Logger log = LoggerFactory.getLogger( GraphManagerLoadTest.class );
 
@@ -95,7 +97,6 @@ public class GraphManagerLoadTest {
     }
 
 
-//    @Ignore
     @Test
     public void writeThousandsSingleSource() throws InterruptedException, ExecutionException {
         EdgeGenerator generator = new EdgeGenerator() {
@@ -124,7 +125,7 @@ public class GraphManagerLoadTest {
 
 
     @Test
-    @Ignore("Too heavy for normal build process")
+    @Category(StressTest.class)
     public void writeThousandsSingleTarget() throws InterruptedException, ExecutionException {
         EdgeGenerator generator = new EdgeGenerator() {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
index 9000c75..45c3d80 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerShardConsistencyIT.java
@@ -39,10 +39,12 @@ import java.util.concurrent.atomic.AtomicLong;
 
 import javax.annotation.Nullable;
 
+import org.apache.usergrid.StressTest;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -393,7 +395,7 @@ public class GraphManagerShardConsistencyIT {
 
 
     @Test(timeout=120000)
-    @Ignore("This works, but is occasionally causing cassandra to fall over.  Unignore when merged with new shard strategy")
+    @Category(StressTest.class)
     public void writeThousandsDelete()
         throws InterruptedException, ExecutionException, MigrationException, UnsupportedEncodingException {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerStressTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerStressTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerStressTest.java
index 6a2efc9..98065ce 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerStressTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/GraphManagerStressTest.java
@@ -19,20 +19,10 @@
 package org.apache.usergrid.persistence.graph;
 
 
-import java.util.HashSet;
-import java.util.Set;
-import java.util.concurrent.CountDownLatch;
-
-import org.junit.Before;
-import org.junit.Ignore;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
+import com.google.common.base.Optional;
+import com.google.inject.Inject;
 import org.apache.commons.lang3.time.StopWatch;
-
+import org.apache.usergrid.StressTest;
 import org.apache.usergrid.persistence.core.guice.MigrationManagerRule;
 import org.apache.usergrid.persistence.core.scope.ApplicationScope;
 import org.apache.usergrid.persistence.core.test.ITRunner;
@@ -42,25 +32,29 @@ import org.apache.usergrid.persistence.graph.guice.TestGraphModule;
 import org.apache.usergrid.persistence.graph.impl.SimpleSearchByEdgeType;
 import org.apache.usergrid.persistence.model.entity.Id;
 import org.apache.usergrid.persistence.model.util.UUIDGenerator;
-
-import com.google.common.base.Optional;
-import com.google.inject.Inject;
-
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.experimental.categories.Category;
+import org.junit.runner.RunWith;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import rx.Observable;
 import rx.Subscriber;
 
+import java.util.HashSet;
+import java.util.Set;
+import java.util.concurrent.CountDownLatch;
+
 import static org.apache.usergrid.persistence.graph.test.util.EdgeTestUtils.createEdge;
-import static org.apache.usergrid.persistence.core.util.IdGenerator.createId;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
 
 @RunWith(ITRunner.class)
 @UseModules(TestGraphModule.class)
-@Ignore("Stress test should not be run in embedded mode")
+@Category(StressTest.class)
 public class GraphManagerStressTest {
     private static final Logger log = LoggerFactory.getLogger( GraphManagerStressTest.class );
 
@@ -89,7 +83,7 @@ public class GraphManagerStressTest {
 
 
     @Test
-    @Ignore("Too heavy for normal build process")
+    @Category(StressTest.class)
     public void writeThousands() throws InterruptedException {
         EdgeGenerator generator = new EdgeGenerator() {
 
@@ -177,7 +171,7 @@ public class GraphManagerStressTest {
     }
 
 
-    @Ignore("Too heavy for normal build process")
+    @Category(StressTest.class)
     @Test
     public void writeThousandsSingleSource() throws InterruptedException {
         EdgeGenerator generator = new EdgeGenerator() {
@@ -205,7 +199,7 @@ public class GraphManagerStressTest {
 
 
     @Test
-    @Ignore("Too heavy for normal build process")
+    @Category(StressTest.class)
     public void writeThousandsSingleTarget() throws InterruptedException {
         EdgeGenerator generator = new EdgeGenerator() {
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/EdgeSerializationTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/EdgeSerializationTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/EdgeSerializationTest.java
index d875e83..d81413e 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/EdgeSerializationTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/serialization/EdgeSerializationTest.java
@@ -25,10 +25,12 @@ import java.util.Iterator;
 import java.util.Set;
 import java.util.UUID;
 
+import org.apache.usergrid.StressTest;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -650,7 +652,7 @@ public abstract class EdgeSerializationTest {
      * Test paging by resuming the search from the edge
      */
     @Test
-    @Ignore("Kills embedded cassandra")
+    @Category(StressTest.class)
     public void pageIteration() throws ConnectionException {
 
         int size = graphFig.getScanPageSize() * 2;
@@ -690,7 +692,7 @@ public abstract class EdgeSerializationTest {
      * edge types
      */
     @Test
-    @Ignore("Kills embedded cassandra")
+    @Category(StressTest.class)
     public void testIteratorPaging() throws ConnectionException {
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/map/pom.xml
----------------------------------------------------------------------
diff --git a/stack/corepersistence/map/pom.xml b/stack/corepersistence/map/pom.xml
index e6f4d54..8589716 100644
--- a/stack/corepersistence/map/pom.xml
+++ b/stack/corepersistence/map/pom.xml
@@ -81,6 +81,7 @@ limitations under the License.
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/model/src/main/java/org/apache/usergrid/ExperimentalTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/model/src/main/java/org/apache/usergrid/ExperimentalTest.java b/stack/corepersistence/model/src/main/java/org/apache/usergrid/ExperimentalTest.java
new file mode 100644
index 0000000..5138fa4
--- /dev/null
+++ b/stack/corepersistence/model/src/main/java/org/apache/usergrid/ExperimentalTest.java
@@ -0,0 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.usergrid;
+
+public interface ExperimentalTest {}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/model/src/main/java/org/apache/usergrid/StressTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/model/src/main/java/org/apache/usergrid/StressTest.java b/stack/corepersistence/model/src/main/java/org/apache/usergrid/StressTest.java
new file mode 100644
index 0000000..a51439e
--- /dev/null
+++ b/stack/corepersistence/model/src/main/java/org/apache/usergrid/StressTest.java
@@ -0,0 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.usergrid;
+
+public interface StressTest {}

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/pom.xml
----------------------------------------------------------------------
diff --git a/stack/corepersistence/pom.xml b/stack/corepersistence/pom.xml
index dab863b..66129ed 100644
--- a/stack/corepersistence/pom.xml
+++ b/stack/corepersistence/pom.xml
@@ -54,6 +54,7 @@ limitations under the License.
                       <version>${surefire.plugin.version}</version>
                       <configuration>
                           <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                          <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                       </configuration>
                   </plugin>
               </plugins>
@@ -126,6 +127,7 @@ limitations under the License.
                             <archaius.deployment.environment>UNIT</archaius.deployment.environment>
                         </systemPropertyVariables>
                         <argLine>-Xms2G -Xmx4G</argLine>
+                        <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                     </configuration>
                 </plugin>
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/queryindex/pom.xml
----------------------------------------------------------------------
diff --git a/stack/corepersistence/queryindex/pom.xml b/stack/corepersistence/queryindex/pom.xml
index 59f4f56..003281e 100644
--- a/stack/corepersistence/queryindex/pom.xml
+++ b/stack/corepersistence/queryindex/pom.xml
@@ -55,10 +55,7 @@
                         <include>**/*IT.java</include>
                         <include>**/*Test.java</include>
                     </includes>
-                    <!-- run this one manually to stress test -->
-                    <excludes>
-                        <exclude>**/IndexLoadTestsIT.java</exclude>
-                    </excludes>
+                    <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                 </configuration>
 
             </plugin>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/IndexLoadTestsIT.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/IndexLoadTestsIT.java b/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/IndexLoadTestsIT.java
index 1be1195..2343784 100644
--- a/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/IndexLoadTestsIT.java
+++ b/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/IndexLoadTestsIT.java
@@ -23,12 +23,14 @@ import java.util.UUID;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicLong;
 
+import org.apache.usergrid.StressTest;
 import org.apache.usergrid.persistence.core.astyanax.CassandraFig;
 import org.apache.usergrid.persistence.index.*;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 import org.junit.runner.RunWith;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -77,6 +79,7 @@ import static org.junit.Assert.assertEquals;
  */
 @RunWith( EsRunner.class )
 @UseModules( { TestIndexModule.class } )
+@Category( StressTest.class )
 public class IndexLoadTestsIT extends BaseIT {
     private static final Logger log = LoggerFactory.getLogger( IndexLoadTestsIT.class );
     public static final String FIELD_WORKER_INDEX = "workerIndex";

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/pom.xml
----------------------------------------------------------------------
diff --git a/stack/pom.xml b/stack/pom.xml
index 3f7b68b..1df8463 100644
--- a/stack/pom.xml
+++ b/stack/pom.xml
@@ -127,7 +127,7 @@
       <usergrid.it.threads>8</usergrid.it.threads>
 
       <metrics.version>3.0.0</metrics.version>
-        <surefire.plugin.artifactName>surefire-junit47</surefire.plugin.artifactName>
+      <surefire.plugin.artifactName>surefire-junit47</surefire.plugin.artifactName>
       <surefire.plugin.version>2.18.1</surefire.plugin.version>
       <powermock.version>1.6.1</powermock.version>
 
@@ -1344,6 +1344,7 @@
                           <systemPropertyVariables>
                               <jacoco-agent.destfile>target/jacoco.exec</jacoco-agent.destfile>
                           </systemPropertyVariables>
+                          <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                       </configuration>
 
                       <!-- TODO, we may need an exclusion. Appears to be a classloader bug
@@ -1354,14 +1355,6 @@
                               <groupId>org.apache.maven.surefire</groupId>
                               <artifactId>${surefire.plugin.artifactName}</artifactId>
                               <version>${surefire.plugin.version}</version>
-<!--
-                              <exclusions>
-                                  <exclusion>
-                                      <groupId>org.apache.maven.surfire</groupId>
-                                      <artifactId>common-junit3</artifactId>
-                                  </exclusion>
-                              </exclusions>
--->
                           </dependency>
 
                           <!-- override plex utils, otherwise bug from above SO post happens-->
@@ -1585,6 +1578,7 @@
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-javaagent:${settings.localRepository}/com/github/stephenc/jamm/0.2.5/jamm-0.2.5.jar ${ug.argline} -javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/rest/pom.xml
----------------------------------------------------------------------
diff --git a/stack/rest/pom.xml b/stack/rest/pom.xml
index 601ed61..f0ce7b3 100644
--- a/stack/rest/pom.xml
+++ b/stack/rest/pom.xml
@@ -85,6 +85,7 @@
                         <include>**/*IT.java</include>
                         <include>**/*Test.java</include>
                    </includes>
+                    <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
 
                 </configuration>
 
@@ -422,6 +423,7 @@
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec  -Dwebapp.directory=${basedir}/src/main/webapp -Dtest.barrier.timestamp=${maven.build.timestamp} -Dtest.clean.storage=true -Xmx${ug.heapmax} -Xms${ug.heapmin} -Dfile.encoding=UTF-8 -Dsun.jnu.encoding=UTF-8  -javaagent:${settings.localRepository}/com/github/stephenc/jamm/0.2.5/jamm-0.2.5.jar -Djava.util.logging.config.file=${basedir}/src/test/resources/logging.properties ${ug.argline}</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/services/pom.xml
----------------------------------------------------------------------
diff --git a/stack/services/pom.xml b/stack/services/pom.xml
index 9837e76..bfd2f3b 100644
--- a/stack/services/pom.xml
+++ b/stack/services/pom.xml
@@ -103,12 +103,7 @@
                     <include>**/*IT.java</include>
                     <include>**/*Test.java</include>
                 </includes>
-                <excludes>
-                    <exclude>**/*Scheduler*IT.java</exclude>
-                    <exclude>**/*Notification*IT.java</exclude>
-                    <exclude>**/ExportServiceIT.java</exclude>
-                    <exclude>**/ImportServiceIT.java</exclude>
-                </excludes>
+                <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
             </configuration>
             <dependencies>
                 <dependency>
@@ -474,6 +469,7 @@
                         <version>${surefire.plugin.version}</version>
                         <configuration>
                             <argLine>-Dtest.barrier.timestamp=${maven.build.timestamp} -Dtest.clean.storage=true -Xmx${ug.heapmax} -Xms${ug.heapmin} -Dfile.encoding=UTF-8 -Dsun.jnu.encoding=UTF-8 -javaagent:${settings.localRepository}/com/github/stephenc/jamm/0.2.5/jamm-0.2.5.jar ${ug.argline} -javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/jacoco.exec</argLine>
+                            <excludedGroups>org.apache.usergrid.StressTest,org.apache.usergrid.ExperimentalTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/services/src/test/java/org/apache/usergrid/security/providers/PingIdentityProviderIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/security/providers/PingIdentityProviderIT.java b/stack/services/src/test/java/org/apache/usergrid/security/providers/PingIdentityProviderIT.java
index c47b954..df63d1f 100644
--- a/stack/services/src/test/java/org/apache/usergrid/security/providers/PingIdentityProviderIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/security/providers/PingIdentityProviderIT.java
@@ -20,6 +20,7 @@ package org.apache.usergrid.security.providers;
 import java.util.Map;
 import java.util.UUID;
 
+import org.apache.usergrid.ExperimentalTest;
 import org.junit.BeforeClass;
 import org.junit.ClassRule;
 import org.junit.Ignore;
@@ -37,13 +38,14 @@ import org.apache.usergrid.persistence.entities.Application;
 import org.apache.usergrid.persistence.entities.User;
 import org.apache.usergrid.persistence.index.impl.ElasticSearchResource;
 import org.apache.usergrid.utils.MapUtils;
+import org.junit.experimental.categories.Category;
 
 import static junit.framework.Assert.assertNotNull;
 
 
 /** @author zznate */
-@Ignore("Experimental Ping Indentiyy test")
 
+@Category(ExperimentalTest.class )
 public class PingIdentityProviderIT {
     private static UserInfo adminUser;
     private static OrganizationInfo organization;

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/services/src/test/java/org/apache/usergrid/services/notifications/AbstractServiceNotificationIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/AbstractServiceNotificationIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/AbstractServiceNotificationIT.java
index 798f183..d0d2c29 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/AbstractServiceNotificationIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/AbstractServiceNotificationIT.java
@@ -23,6 +23,7 @@ import org.apache.usergrid.persistence.Query;
 import org.apache.usergrid.services.ServiceManagerFactory;
 import org.junit.Before;
 import org.junit.BeforeClass;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.rules.TestName;
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
index 8b94661..a3ff449 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/apns/NotificationsServiceIT.java
@@ -41,6 +41,7 @@ import static org.apache.usergrid.services.notifications.impl.ApplicationQueueMa
 
 // todo: test reschedule on delivery time change
 // todo: test restart of queuing
+@Ignore
 public class NotificationsServiceIT extends AbstractServiceNotificationIT {
 
     private static final Logger LOG = LoggerFactory.getLogger(NotificationsServiceIT.class);

http://git-wip-us.apache.org/repos/asf/usergrid/blob/99b932ad/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
----------------------------------------------------------------------
diff --git a/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java b/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
index 3218137..3369786 100644
--- a/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
+++ b/stack/services/src/test/java/org/apache/usergrid/services/notifications/gcm/NotificationsServiceIT.java
@@ -142,6 +142,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
     }
 
     @Test
+    @Ignore
     public void singlePushNotification() throws Exception {
 
         app.clear();
@@ -167,6 +168,7 @@ public class NotificationsServiceIT extends AbstractServiceNotificationIT {
     }
 
     @Test
+    @Ignore
     public void singlePushNotificationViaUser() throws Exception {
 
         app.clear();


[18/21] usergrid git commit: Un-ignore some working tests.

Posted by gr...@apache.org.
Un-ignore some working tests.


Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/43f7ee05
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/43f7ee05
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/43f7ee05

Branch: refs/heads/master
Commit: 43f7ee05b20b57f37f28eac5ff3baa41cee1326c
Parents: 68be7d7
Author: Dave Johnson <sn...@apache.org>
Authored: Tue Nov 17 10:52:47 2015 -0500
Committer: Dave Johnson <sn...@apache.org>
Committed: Tue Nov 17 10:52:47 2015 -0500

----------------------------------------------------------------------
 .../corepersistence/StaleIndexCleanupTest.java  | 99 ++++++++++----------
 .../mvcc/stage/AbstractEntityStageTest.java     |  1 -
 .../graph/impl/NodeDeleteListenerTest.java      |  1 -
 3 files changed, 51 insertions(+), 50 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/usergrid/blob/43f7ee05/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
----------------------------------------------------------------------
diff --git a/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java b/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
index 2f96c18..8e6d21f 100644
--- a/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/corepersistence/StaleIndexCleanupTest.java
@@ -194,16 +194,16 @@ public class StaleIndexCleanupTest extends AbstractCoreIT {
     /**
      * Test that the CpRelationManager cleans up and stale indexes that it finds when
      * it is building search results.
+     * TODO: does this test still make sense?
      */
     @Test
-    @Ignore("Breaks other tests - or would if we reenabled that EVENTS_DISABLED property")
     public void testStaleIndexCleanup() throws Exception {
 
 
         logger.info( "Started testStaleIndexCleanup()" );
 
-        // turn off post processing stuff that cleans up stale entities
-        System.setProperty( EVENTS_DISABLED, "true" );
+        // this EVENTS_DISABLED property is no longer supported
+        //System.setProperty( EVENTS_DISABLED, "true" );
 
         final EntityManager em = app.getEntityManager();
 
@@ -255,54 +255,57 @@ public class StaleIndexCleanupTest extends AbstractCoreIT {
         app.refreshIndex();
 
         // query Core Persistence directly for total number of result candidates
-        crs = queryCollectionCp( "things", "thing", "select * order by updateCount asc" );
-        Assert.assertEquals( "Expect stale candidates", numEntities * ( numUpdates + 1 ), crs.size() );
-
-        // query EntityManager for results and page through them
-        // should return numEntities because it filters out the stale entities
-        final int limit = 8;
-
-        // we order by updateCount asc, this forces old versions to appear first, otherwise,
-        // we don't clean them up in our versions
-        Query q = Query.fromQL( "select * order by updateCount asc" );
-        q.setLimit( limit );
-
-        int thingCount = 0;
-        int index = 0;
-        String cursor;
-
-        do {
-            Results results = em.searchCollection( em.getApplicationRef(), "things", q );
-            thingCount += results.size();
-
-            logger.debug( "Retrieved total of {} entities", thingCount );
-
-            cursor = results.getCursor();
-            if ( cursor != null && thingCount < numEntities ) {
-                assertEquals( limit, results.size() );
-            }
 
-            for ( int i = 0; i < results.size(); i++, index++ ) {
+        // Because events are not disabled there will be no stale candidates
 
-                final Entity returned = results.getEntities().get( i );
+//        crs = queryCollectionCp( "things", "thing", "select * order by updateCount asc" );
+//        Assert.assertEquals( "Expect stale candidates", numEntities * ( numUpdates + 1 ), crs.size() );
 
-                // last entities appear first
-                final Entity expected = maxVersions.get( index );
-                assertEquals("correct entity returned", expected, returned);
-
-            }
-        }
-        while ( cursor != null );
-
-        assertEquals( "Expect no stale candidates", numEntities, thingCount );
-
-
-        app.refreshIndex();
-
-
-        // query for total number of result candidates = numEntities
-        crs = queryCollectionCp( "things", "thing", "select *" );
-        Assert.assertEquals( "Expect stale candidates de-indexed", numEntities, crs.size() );//20,21
+        // query EntityManager for results and page through them
+        // should return numEntities because it filters out the stale entities
+//        final int limit = 8;
+//
+//        // we order by updateCount asc, this forces old versions to appear first, otherwise,
+//        // we don't clean them up in our versions
+//        Query q = Query.fromQL( "select * order by updateCount asc" );
+//        q.setLimit( limit );
+//
+//        int thingCount = 0;
+//        int index = 0;
+//        String cursor;
+//
+//        do {
+//            Results results = em.searchCollection( em.getApplicationRef(), "things", q );
+//            thingCount += results.size();
+//
+//            logger.debug( "Retrieved total of {} entities", thingCount );
+//
+//            cursor = results.getCursor();
+//            if ( cursor != null && thingCount < numEntities ) {
+//                assertEquals( limit, results.size() );
+//            }
+//
+//            for ( int i = 0; i < results.size(); i++, index++ ) {
+//
+//                final Entity returned = results.getEntities().get( i );
+//
+//                // last entities appear first
+//                final Entity expected = maxVersions.get( index );
+//                assertEquals("correct entity returned", expected, returned);
+//
+//            }
+//        }
+//        while ( cursor != null );
+//
+//        assertEquals( "Expect no stale candidates", numEntities, thingCount );
+//
+//
+//        app.refreshIndex();
+//
+//
+//        // query for total number of result candidates = numEntities
+//        crs = queryCollectionCp( "things", "thing", "select *" );
+//        Assert.assertEquals( "Expect stale candidates de-indexed", numEntities, crs.size() );//20,21
     }
 
 

http://git-wip-us.apache.org/repos/asf/usergrid/blob/43f7ee05/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/AbstractEntityStageTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/AbstractEntityStageTest.java b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/AbstractEntityStageTest.java
index 201aa8c..62e1403 100644
--- a/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/AbstractEntityStageTest.java
+++ b/stack/corepersistence/collection/src/test/java/org/apache/usergrid/persistence/collection/mvcc/stage/AbstractEntityStageTest.java
@@ -47,7 +47,6 @@ public abstract class AbstractEntityStageTest {
 
 
     /** Test every Entity with */
-    @Ignore("Why is this ignored?")
     @Test( expected = IllegalArgumentException.class )
     @Theory
     public void testWrongEntityType(@InvalidEntityGenerator.IllegalFields final Entity entity, @InvalidIdGenerator.IllegalFields final Id id) throws Exception {

http://git-wip-us.apache.org/repos/asf/usergrid/blob/43f7ee05/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
index dd92261..f2d4d94 100644
--- a/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
+++ b/stack/corepersistence/graph/src/test/java/org/apache/usergrid/persistence/graph/impl/NodeDeleteListenerTest.java
@@ -319,7 +319,6 @@ public class NodeDeleteListenerTest {
      * since it has no other targets
      */
     @Test
-    @Ignore("This needs to be re-enable.  The counters for sharding fall over in cass, needs fixes")
     public void testMultiDelete() throws ConnectionException, InterruptedException {
 
         GraphManager em = emf.createEdgeManager( scope );