lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Neal Granroth (JIRA)" <j...@apache.org>
Subject [jira] Commented: (LUCENENET-97) HitIterator does not correctly implement System.Collections.IEnumerator
Date Wed, 29 Aug 2007 21:54:31 GMT

    [ https://issues.apache.org/jira/browse/LUCENENET-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12523709
] 

Neal Granroth commented on LUCENENET-97:
----------------------------------------

Correction:

The test section for verifying that Current does not alter the position requires a MoveNext(),
as documented in .NET System.Collections.IEnumerator "the initial position is before the first
element of the collection".  So replace the list 13 lines with:

// ..................

HitIterator iterator2 = (HitIterator) hits.Iterator(); 

iterator2.MoveNext();

Hit hit = (Hit) iterator2.Current;
Assert.AreEqual("iterator test doc 1", hit.Get("field"));

// Verify that "Current" does not move, when MoveNext is not called.

Hit hitAgain = (Hit) iterator2.Current;
Assert.AreEqual("iterator test doc 1", hitAgain.Get("field")); 



> HitIterator does not correctly implement System.Collections.IEnumerator
> -----------------------------------------------------------------------
>
>                 Key: LUCENENET-97
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-97
>             Project: Lucene.Net
>          Issue Type: Bug
>         Environment: .NET
>            Reporter: Neal Granroth
>
> method MoveNext() is supposed to increment the position within the list; it does not.
> method Current() is supposed to return the object at the current position, but not change
the position within the list.
> It incorrectly moves to the next position.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message