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/08/10 20:40:31 UTC

incubator-usergrid git commit: Added comment in case a test fails. Also removed a query from the EntityIndexTest due to giving additional meaning to spaces in usergrid. ( now they can be queried )

Repository: incubator-usergrid
Updated Branches:
  refs/heads/USERGRID-903 4d75b6b10 -> 2d27fd0d7


Added comment in case a test fails.
Also removed a query from the EntityIndexTest due to giving additional meaning to spaces in usergrid. ( now they can be queried )


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

Branch: refs/heads/USERGRID-903
Commit: 2d27fd0d7b75efa00508da395c830bc807cf8d21
Parents: 4d75b6b
Author: GERey <gr...@apigee.com>
Authored: Mon Aug 10 11:40:28 2015 -0700
Committer: GERey <gr...@apigee.com>
Committed: Mon Aug 10 11:40:28 2015 -0700

----------------------------------------------------------------------
 .../apache/usergrid/persistence/index/impl/EntityIndexTest.java  | 4 +++-
 .../apache/usergrid/rest/applications/queries/OrderByTest.java   | 3 ++-
 2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/2d27fd0d/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/EntityIndexTest.java
----------------------------------------------------------------------
diff --git a/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/EntityIndexTest.java b/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/EntityIndexTest.java
index 9154382..1da1751 100644
--- a/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/EntityIndexTest.java
+++ b/stack/corepersistence/queryindex/src/test/java/org/apache/usergrid/persistence/index/impl/EntityIndexTest.java
@@ -450,7 +450,9 @@ public class EntityIndexTest extends BaseIT {
 
         testQuery( scope, searchTypes, "name = 'Minerva Harrell' and age <= 40", 1 );
 
-        testQuery( scope, searchTypes, "name = 'Morgan* '", 1 );
+        //Commented out because we no longer trim the spaces at the end ergo this looking for a string with a
+        //space at the end ( of which isn't created
+//        testQuery( scope, searchTypes, "name = 'Morgan* '", 1 );
 
         testQuery( scope, searchTypes, "name = 'Morgan*'", 1 );
 

http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/2d27fd0d/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/OrderByTest.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/OrderByTest.java b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/OrderByTest.java
index 869d5b1..7b5b13b 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/OrderByTest.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/applications/queries/OrderByTest.java
@@ -76,9 +76,10 @@ public class OrderByTest extends QueryTestBase {
         refreshIndex();
 
         QueryParameters params = new QueryParameters()
-            .setQuery("select * where random = 'fury ' ");
+            .setQuery("select * where random = 'fury '");
         Collection activities = this.app().collection(collectionName).get(params);
         assertEquals(1, activities.getResponse().getEntityCount());
+        //note the space below
         assertEquals( "fury ",activities.getResponse().getEntities().get( 0 ).get( "random" ) );
     }