We need a changes entry too right?

uschind...@apache.org wrote:
> Author: uschindler
> Date: Mon Oct 26 22:06:40 2009
> New Revision: 829995
>
> URL: http://svn.apache.org/viewvc?rev=829995&view=rev
> Log:
> LUCENE-1929: Merge NumericRangeQuery tests for highlighter from 2.9 branch. 
> The bug was already fixed by a different impl in trunk, but the test was 
> missing.
>
> Modified:
>     lucene/java/trunk/contrib/highlighter/src/test/   (props changed)
>     
> lucene/java/trunk/contrib/highlighter/src/test/org/apache/lucene/search/highlight/HighlighterTest.java
>
> Propchange: lucene/java/trunk/contrib/highlighter/src/test/
> ------------------------------------------------------------------------------
> --- svn:mergeinfo (added)
> +++ svn:mergeinfo Mon Oct 26 22:06:40 2009
> @@ -0,0 +1,3 @@
> +/lucene/java/branches/lucene_2_4/contrib/highlighter/src/test:748824
> +/lucene/java/branches/lucene_2_9/contrib/highlighter/src/test:817269-818600,825998,826775,829134,829816,829881
> +/lucene/java/branches/lucene_2_9_back_compat_tests/contrib/highlighter/src/test:818601-821336
>
> Modified: 
> lucene/java/trunk/contrib/highlighter/src/test/org/apache/lucene/search/highlight/HighlighterTest.java
> URL: 
> http://svn.apache.org/viewvc/lucene/java/trunk/contrib/highlighter/src/test/org/apache/lucene/search/highlight/HighlighterTest.java?rev=829995&r1=829994&r2=829995&view=diff
> ==============================================================================
> --- 
> lucene/java/trunk/contrib/highlighter/src/test/org/apache/lucene/search/highlight/HighlighterTest.java
>  (original)
> +++ 
> lucene/java/trunk/contrib/highlighter/src/test/org/apache/lucene/search/highlight/HighlighterTest.java
>  Mon Oct 26 22:06:40 2009
> @@ -46,6 +46,7 @@
>  import org.apache.lucene.analysis.tokenattributes.TermAttribute;
>  import org.apache.lucene.document.Document;
>  import org.apache.lucene.document.Field;
> +import org.apache.lucene.document.NumericField;
>  import org.apache.lucene.document.Field.Index;
>  import org.apache.lucene.document.Field.Store;
>  import org.apache.lucene.index.IndexReader;
> @@ -60,6 +61,7 @@
>  import org.apache.lucene.search.MultiPhraseQuery;
>  import org.apache.lucene.search.MultiSearcher;
>  import org.apache.lucene.search.MultiTermQuery;
> +import org.apache.lucene.search.NumericRangeQuery;
>  import org.apache.lucene.search.PhraseQuery;
>  import org.apache.lucene.search.Query;
>  import org.apache.lucene.search.TermQuery;
> @@ -88,6 +90,7 @@
>  
>    private IndexReader reader;
>    static final String FIELD_NAME = "contents";
> +  private static final String NUMERIC_FIELD_NAME = "nfield";
>    private Query query;
>    RAMDirectory ramDir;
>    public IndexSearcher searcher = null;
> @@ -302,6 +305,30 @@
>          numHighlights == 4);
>      
>    }
> +  
> +  public void testNumericRangeQuery() throws Exception {
> +    // doesn't currently highlight, but make sure it doesn't cause exception 
> either
> +    query = NumericRangeQuery.newIntRange(NUMERIC_FIELD_NAME, 2, 6, true, 
> true);
> +    searcher = new IndexSearcher(ramDir, true);
> +    hits = searcher.search(query, 100);
> +    int maxNumFragmentsRequired = 2;
> +
> +    QueryScorer scorer = new QueryScorer(query, FIELD_NAME);
> +    Highlighter highlighter = new Highlighter(this, scorer);
> +    
> +    for (int i = 0; i < hits.totalHits; i++) {
> +      String text = 
> searcher.doc(hits.scoreDocs[i].doc).get(NUMERIC_FIELD_NAME);
> +      TokenStream tokenStream = analyzer.tokenStream(FIELD_NAME, new 
> StringReader(text));
> +
> +      highlighter.setTextFragmenter(new SimpleFragmenter(40));
> +
> +      String result = highlighter.getBestFragments(tokenStream, text, 
> maxNumFragmentsRequired,
> +          "...");
> +      //System.out.println("\t" + result);
> +    }
> +
> +
> +  }
>  
>    public void testSimpleQueryScorerPhraseHighlighting2() throws Exception {
>      doSearching("\"text piece long\"~5");
> @@ -1617,7 +1644,26 @@
>      for (int i = 0; i < texts.length; i++) {
>        addDoc(writer, texts[i]);
>      }
> -
> +    Document doc = new Document();
> +    NumericField nfield = new NumericField(NUMERIC_FIELD_NAME, Store.YES, 
> true);
> +    nfield.setIntValue(1);
> +    doc.add(nfield);
> +    writer.addDocument(doc, analyzer);
> +    nfield = new NumericField(NUMERIC_FIELD_NAME, Store.YES, true);
> +    nfield.setIntValue(3);
> +    doc = new Document();
> +    doc.add(nfield);
> +    writer.addDocument(doc, analyzer);
> +    nfield = new NumericField(NUMERIC_FIELD_NAME, Store.YES, true);
> +    nfield.setIntValue(5);
> +    doc = new Document();
> +    doc.add(nfield);
> +    writer.addDocument(doc, analyzer);
> +    nfield = new NumericField(NUMERIC_FIELD_NAME, Store.YES, true);
> +    nfield.setIntValue(7);
> +    doc = new Document();
> +    doc.add(nfield);
> +    writer.addDocument(doc, analyzer);
>      writer.optimize();
>      writer.close();
>      reader = IndexReader.open(ramDir, true);
>
>
>   


-- 
- Mark

http://www.lucidimagination.com




---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org

Reply via email to