You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by kr...@apache.org on 2016/11/14 16:06:53 UTC

[12/16] lucene-solr:jira/solr-8593: SOLR-9718: replace assert and assertTrue() to assertEquals()

SOLR-9718: replace assert and assertTrue() to assertEquals()


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/907bed88
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/907bed88
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/907bed88

Branch: refs/heads/jira/solr-8593
Commit: 907bed887d6be1aaec832c4f6d395d051bb49b17
Parents: 7936f74
Author: Noble Paul <no...@apache.org>
Authored: Mon Nov 14 12:41:59 2016 +0530
Committer: Noble Paul <no...@apache.org>
Committed: Mon Nov 14 12:41:59 2016 +0530

----------------------------------------------------------------------
 .../client/solrj/io/stream/StreamingTest.java   | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/907bed88/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamingTest.java
----------------------------------------------------------------------
diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamingTest.java b/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamingTest.java
index 3888a41..7d6e1d3 100644
--- a/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamingTest.java
+++ b/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamingTest.java
@@ -1718,7 +1718,7 @@ public class StreamingTest extends SolrCloudTestCase {
     CloudSolrStream stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
     List<Tuple> tuples = getTuples(stream);
 
-    assert(tuples.size() == 5);
+    assertEquals(5,tuples.size());
     assertOrder(tuples, 4, 3, 2, 1, 0);
 
     //With Ascending Sort
@@ -1726,7 +1726,7 @@ public class StreamingTest extends SolrCloudTestCase {
     stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
     tuples = getTuples(stream);
 
-    assert(tuples.size() == 5);
+    assertEquals(5, tuples.size());
     assertOrder(tuples, 0,1,2,3,4);
 
 
@@ -1735,7 +1735,7 @@ public class StreamingTest extends SolrCloudTestCase {
     stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
     tuples = getTuples(stream);
 
-    assert(tuples.size() == 5);
+    assertEquals(5, tuples.size());
     assertOrder(tuples, 2,0,1,3,4);
 
 
@@ -1743,7 +1743,7 @@ public class StreamingTest extends SolrCloudTestCase {
     stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
     tuples = getTuples(stream);
 
-    assert (tuples.size() == 5);
+    assertEquals(5, tuples.size());
     assertOrder(tuples, 0, 2, 1, 3, 4);
 
   }
@@ -1771,7 +1771,7 @@ public class StreamingTest extends SolrCloudTestCase {
     try  {
       List<Tuple> tuples = getTuples(stream);
 
-      assert (tuples.size() == 5);
+      assertEquals(5, tuples.size());
       assertOrder(tuples, 0, 2, 1, 3, 4);
 
       //Basic CloudSolrStream Test bools desc
@@ -1779,7 +1779,7 @@ public class StreamingTest extends SolrCloudTestCase {
       stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
       tuples = getTuples(stream);
 
-      assert (tuples.size() == 5);
+      assertEquals (5,tuples.size());
       assertOrder(tuples, 4, 3, 1, 2, 0);
 
       //Basic CloudSolrStream Test dates desc
@@ -1787,7 +1787,7 @@ public class StreamingTest extends SolrCloudTestCase {
       stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
       tuples = getTuples(stream);
 
-      assert (tuples.size() == 5);
+      assertEquals (5,tuples.size());
       assertOrder(tuples, 2, 0, 1, 4, 3);
 
       //Basic CloudSolrStream Test ates desc
@@ -1795,7 +1795,7 @@ public class StreamingTest extends SolrCloudTestCase {
       stream = new CloudSolrStream(zkHost, COLLECTION, sParams);
       tuples = getTuples(stream);
 
-      assert (tuples.size() == 5);
+      assertEquals (5,tuples.size());
       assertOrder(tuples, 3, 4, 1, 0, 2);
     } finally {
       if (stream != null) {
@@ -1840,8 +1840,8 @@ public class StreamingTest extends SolrCloudTestCase {
       assertEquals("MV should be returned for i_multi", 12, tuple.getLongs("i_multi").get(0).longValue());
       assertEquals("MV should be returned for i_multi", 13, tuple.getLongs("i_multi").get(1).longValue());
 
-      assertEquals("longs should be returned", 14,tuple.getLong("l_sing").longValue());
-      assertEquals("MV should be returned for l_multi",15, tuple.getLongs("l_multi").get(0).longValue());
+      assertEquals("longs should be returned", 14, tuple.getLong("l_sing").longValue());
+      assertEquals("MV should be returned for l_multi", 15, tuple.getLongs("l_multi").get(0).longValue());
       assertEquals("MV should be returned for l_multi", 16, tuple.getLongs("l_multi").get(1).longValue());
 
       assertEquals("floats should be returned", 1.7, tuple.getDouble("f_sing").doubleValue(), 0.001);