You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2015/02/23 15:39:09 UTC
svn commit: r1661685 - in /lucene/dev/trunk/solr: CHANGES.txt
core/src/test/org/apache/solr/request/TestFaceting.java
Author: shalin
Date: Mon Feb 23 14:39:09 2015
New Revision: 1661685
URL: http://svn.apache.org/r1661685
Log:
SOLR-7142: Fix TestFaceting.testFacets
Modified:
lucene/dev/trunk/solr/CHANGES.txt
lucene/dev/trunk/solr/core/src/test/org/apache/solr/request/TestFaceting.java
Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1661685&r1=1661684&r2=1661685&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Mon Feb 23 14:39:09 2015
@@ -182,6 +182,8 @@ Other Changes
* SOLR-7032: Clean up test remnants of old-style solr.xml (Erick Erickson)
+* SOLR-7142: Fix TestFaceting.testFacets. (Michal Kroliczek via shalin)
+
================== 5.0.0 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/request/TestFaceting.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/request/TestFaceting.java?rev=1661685&r1=1661684&r2=1661685&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/request/TestFaceting.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/request/TestFaceting.java Mon Feb 23 14:39:09 2015
@@ -160,7 +160,7 @@ public class TestFaceting extends SolrTe
assertU(commit());
assertQ("check many tokens",
- req("q", "id:1","indent","true"
+ req("q", "*:*","indent","true"
,"facet", "true", "facet.method","fc"
,"facet.field", "many_ws"
,"facet.limit", "-1"
@@ -183,34 +183,25 @@ public class TestFaceting extends SolrTe
,"//lst[@name='many_ws']/int[@name='" + t(4999) + "'][.='1']"
);
- // test gaps that take more than one byte
+ // add second document, check facets for items with count =2
sb = new StringBuilder();
sb.append(t(0)).append(' ');
sb.append(t(150)).append(' ');
- sb.append(t(301)).append(' ');
- sb.append(t(453)).append(' ');
- sb.append(t(606)).append(' ');
- sb.append(t(1000)).append(' ');
- sb.append(t(2010)).append(' ');
- sb.append(t(3050)).append(' ');
sb.append(t(4999)).append(' ');
assertU(adoc("id", "2", "many_ws", sb.toString()));
+ assertU(commit());
assertQ("check many tokens",
- req("q", "id:1","indent","true"
+ req("q", "*:*","indent","true"
,"facet", "true", "facet.method","fc"
,"facet.field", "many_ws"
,"facet.limit", "-1"
)
,"*[count(//lst[@name='many_ws']/int)=5000]"
- ,"//lst[@name='many_ws']/int[@name='" + t(0) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(150) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(301) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(453) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(606) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(1000) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(2010) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(3050) + "'][.='1']"
- ,"//lst[@name='many_ws']/int[@name='" + t(4999) + "'][.='1']"
+ ,"//lst[@name='many_ws']/int[@name='" + t(0) + "'][.='2']"
+ ,"//lst[@name='many_ws']/int[@name='" + t(1) + "'][.='1']"
+ ,"//lst[@name='many_ws']/int[@name='" + t(150) + "'][.='2']"
+ ,"//lst[@name='many_ws']/int[@name='" + t(4998) + "'][.='1']"
+ ,"//lst[@name='many_ws']/int[@name='" + t(4999) + "'][.='2']"
);
}