You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@manifoldcf.apache.org by kw...@apache.org on 2016/04/30 20:00:33 UTC

svn commit: r1741784 - /manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java

Author: kwright
Date: Sat Apr 30 18:00:33 2016
New Revision: 1741784

URL: http://svn.apache.org/viewvc?rev=1741784&view=rev
Log:
Fix the tests

Modified:
    manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java

Modified: manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java
URL: http://svn.apache.org/viewvc/manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java?rev=1741784&r1=1741783&r2=1741784&view=diff
==============================================================================
--- manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java (original)
+++ manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java Sat Apr 30 18:00:33 2016
@@ -201,30 +201,30 @@ public class SearchBloxDocumentTest {
         assertTrue(meta.has("share_allow"));
         assertTrue(meta.get("share_allow") instanceof JSONArray);
         assertEquals(3, ((JSONArray) meta.get("share_allow")).length());
-        assertEquals("user1", ((JSONArray) meta.get("share_allow")).getString(0));
+        assertEquals("user3", ((JSONArray) meta.get("share_allow")).getString(0));
         assertEquals("user2", ((JSONArray) meta.get("share_allow")).getString(1));
-        assertEquals("user3", ((JSONArray) meta.get("share_allow")).getString(2));
+        assertEquals("user1", ((JSONArray) meta.get("share_allow")).getString(2));
 
         assertTrue(meta.has("document_deny"));
         assertTrue(meta.get("document_deny") instanceof JSONArray);
         assertEquals(2, ((JSONArray) meta.get("document_deny")).length());
-        assertEquals("user42", ((JSONArray) meta.get("document_deny")).getString(0));
-        assertEquals("user52", ((JSONArray) meta.get("document_deny")).getString(1));
+        assertEquals("user52", ((JSONArray) meta.get("document_deny")).getString(0));
+        assertEquals("user42", ((JSONArray) meta.get("document_deny")).getString(1));
 
 
         assertTrue(meta.has("share_deny"));
         assertTrue(meta.get("share_deny") instanceof JSONArray);
         assertEquals(2, ((JSONArray) meta.get("share_deny")).length());
-        assertEquals("user5", ((JSONArray) meta.get("share_deny")).getString(0));
-        assertEquals("user4", ((JSONArray) meta.get("share_deny")).getString(1));
+        assertEquals("user4", ((JSONArray) meta.get("share_deny")).getString(0));
+        assertEquals("user5", ((JSONArray) meta.get("share_deny")).getString(1));
 
 
         assertTrue(meta.has("document_allow"));
         assertTrue(meta.get("document_allow") instanceof JSONArray);
         assertEquals(3, ((JSONArray) meta.get("document_allow")).length());
         assertEquals("user22", ((JSONArray) meta.get("document_allow")).getString(0));
-        assertEquals("user33", ((JSONArray) meta.get("document_allow")).getString(1));
-        assertEquals("user12", ((JSONArray) meta.get("document_allow")).getString(2));
+        assertEquals("user12", ((JSONArray) meta.get("document_allow")).getString(1));
+        assertEquals("user33", ((JSONArray) meta.get("document_allow")).getString(2));
 
 
         assertTrue(document.has("description"));



Re: svn commit: r1741784 - /manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java

Posted by Ahmet Arslan <io...@yahoo.com.INVALID>.
Hi,

It looks like I overlooked a json test issue.
I checked in a fix. This test should be truly order free now.
Please let me know if this test ever fails again.

Sorry for the noise.

Thanks,
Ahmet



On Saturday, April 30, 2016 9:00 PM, "kwright@apache.org" <kw...@apache.org> wrote:
Author: kwright
Date: Sat Apr 30 18:00:33 2016
New Revision: 1741784

URL: http://svn.apache.org/viewvc?rev=1741784&view=rev
Log:
Fix the tests

Modified:
    manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java

Modified: manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java
URL: http://svn.apache.org/viewvc/manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java?rev=1741784&r1=1741783&r2=1741784&view=diff
==============================================================================
--- manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java (original)
+++ manifoldcf/branches/CONNECTORS-1308/connectors/searchblox/connector/src/test/java/org/apache/manifoldcf/agents/output/searchblox/SearchBloxDocumentTest.java Sat Apr 30 18:00:33 2016
@@ -201,30 +201,30 @@ public class SearchBloxDocumentTest {
         assertTrue(meta.has("share_allow"));
         assertTrue(meta.get("share_allow") instanceof JSONArray);
         assertEquals(3, ((JSONArray) meta.get("share_allow")).length());
-        assertEquals("user1", ((JSONArray) meta.get("share_allow")).getString(0));
+        assertEquals("user3", ((JSONArray) meta.get("share_allow")).getString(0));
         assertEquals("user2", ((JSONArray) meta.get("share_allow")).getString(1));
-        assertEquals("user3", ((JSONArray) meta.get("share_allow")).getString(2));
+        assertEquals("user1", ((JSONArray) meta.get("share_allow")).getString(2));

         assertTrue(meta.has("document_deny"));
         assertTrue(meta.get("document_deny") instanceof JSONArray);
         assertEquals(2, ((JSONArray) meta.get("document_deny")).length());
-        assertEquals("user42", ((JSONArray) meta.get("document_deny")).getString(0));
-        assertEquals("user52", ((JSONArray) meta.get("document_deny")).getString(1));
+        assertEquals("user52", ((JSONArray) meta.get("document_deny")).getString(0));
+        assertEquals("user42", ((JSONArray) meta.get("document_deny")).getString(1));


         assertTrue(meta.has("share_deny"));
         assertTrue(meta.get("share_deny") instanceof JSONArray);
         assertEquals(2, ((JSONArray) meta.get("share_deny")).length());
-        assertEquals("user5", ((JSONArray) meta.get("share_deny")).getString(0));
-        assertEquals("user4", ((JSONArray) meta.get("share_deny")).getString(1));
+        assertEquals("user4", ((JSONArray) meta.get("share_deny")).getString(0));
+        assertEquals("user5", ((JSONArray) meta.get("share_deny")).getString(1));


         assertTrue(meta.has("document_allow"));
         assertTrue(meta.get("document_allow") instanceof JSONArray);
         assertEquals(3, ((JSONArray) meta.get("document_allow")).length());
         assertEquals("user22", ((JSONArray) meta.get("document_allow")).getString(0));
-        assertEquals("user33", ((JSONArray) meta.get("document_allow")).getString(1));
-        assertEquals("user12", ((JSONArray) meta.get("document_allow")).getString(2));
+        assertEquals("user12", ((JSONArray) meta.get("document_allow")).getString(1));
+        assertEquals("user33", ((JSONArray) meta.get("document_allow")).getString(2));


         assertTrue(document.has("description"));