Author: aroush
Date: Mon Oct 9 20:50:53 2006
New Revision: 454598
URL: http://svn.apache.org/viewvc?view=rev&rev=454598
Log:
Lucene.Net.Documents.Document.Fields() -- changed from returning an IEnumerator
to IEnumerable
Modified:
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestBinaryDocument.cs
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestDocument.cs
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/DocHelper.cs
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/TestSegmentReader.cs
Modified:
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestBinaryDocument.cs
URL:
http://svn.apache.org/viewvc/incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestBinaryDocument.cs?view=diff&rev=454598&r1=454597&r2=454598
==============================================================================
---
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestBinaryDocument.cs
(original)
+++
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestBinaryDocument.cs
Mon Oct 9 20:50:53 2006
@@ -68,10 +68,7 @@
doc.Add(stringFldCompressed);
/** test for field count */
- System.Collections.IEnumerator iter = doc.Fields();
- int count = 0;
- while (iter.MoveNext()) count++;
- Assert.AreEqual(4, count);
+ Assert.AreEqual(4, doc.GetFieldsCount());
/** add the doc to a ram index */
RAMDirectory dir = new RAMDirectory();
Modified:
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestDocument.cs
URL:
http://svn.apache.org/viewvc/incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestDocument.cs?view=diff&rev=454598&r1=454597&r2=454598
==============================================================================
---
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestDocument.cs
(original)
+++
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Document/TestDocument.cs
Mon Oct 9 20:50:53 2006
@@ -56,10 +56,7 @@
doc.Add(stringFld);
doc.Add(binaryFld);
- System.Collections.IEnumerator iter = doc.Fields();
- int count = 0;
- while (iter.MoveNext()) count++;
- Assert.AreEqual(2, count);
+ Assert.AreEqual(2, doc.GetFieldsCount());
Assert.IsTrue(binaryFld.IsBinary());
Assert.IsTrue(binaryFld.IsStored());
@@ -74,10 +71,7 @@
doc.Add(binaryFld2);
- iter = doc.Fields();
- count = 0;
- while (iter.MoveNext()) count++;
- Assert.AreEqual(3, count);
+ Assert.AreEqual(3, doc.GetFieldsCount());
byte[][] binaryTests = doc.GetBinaryValues("binary");
@@ -92,16 +86,10 @@
Assert.IsTrue(binaryTest2.Equals(binaryVal2));
doc.RemoveField("string");
- iter = doc.Fields();
- count = 0;
- while (iter.MoveNext()) count++;
- Assert.AreEqual(2, count);
+ Assert.AreEqual(2, doc.GetFieldsCount());
doc.RemoveFields("binary");
- iter = doc.Fields();
- count = 0;
- while (iter.MoveNext()) count++;
- Assert.AreEqual(0, count);
+ Assert.AreEqual(0, doc.GetFieldsCount());
}
/// <summary> Tests [EMAIL PROTECTED]
Document#RemoveField(String)} method for a brand new Document
@@ -112,40 +100,27 @@
[Test]
public virtual void TestRemoveForNewDocument()
{
- int count;
- System.Collections.IEnumerator iter;
-
Lucene.Net.Documents.Document doc =
MakeDocumentWithFields();
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(8, count);
+ Assert.AreEqual(8, doc.GetFieldsCount());
doc.RemoveFields("keyword");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(6, count);
+ Assert.AreEqual(6, doc.GetFieldsCount());
doc.RemoveFields("doesnotexists"); // removing
non-existing fields is siltenlty ignored
doc.RemoveFields("keyword"); // removing a field more
than once
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(6, count);
+ Assert.AreEqual(6, doc.GetFieldsCount());
doc.RemoveField("text");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(5, count);
+ Assert.AreEqual(5, doc.GetFieldsCount());
doc.RemoveField("text");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(4, count);
+ Assert.AreEqual(4, doc.GetFieldsCount());
doc.RemoveField("text");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(4, count);
+ Assert.AreEqual(4, doc.GetFieldsCount());
doc.RemoveField("doesnotexists"); // removing
non-existing fields is siltenlty ignored
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(4, count);
+ Assert.AreEqual(4, doc.GetFieldsCount());
doc.RemoveFields("unindexed");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(2, count);
+ Assert.AreEqual(2, doc.GetFieldsCount());
doc.RemoveFields("unstored");
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(0, count);
+ Assert.AreEqual(0, doc.GetFieldsCount());
doc.RemoveFields("doesnotexists"); // removing
non-existing fields is siltenlty ignored
- iter = doc.Fields(); count = 0; while (iter.MoveNext())
count++;
- Assert.AreEqual(0, count);
+ Assert.AreEqual(0, doc.GetFieldsCount());
}
[Test]
Modified:
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/DocHelper.cs
URL:
http://svn.apache.org/viewvc/incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/DocHelper.cs?view=diff&rev=454598&r1=454597&r2=454598
==============================================================================
---
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/DocHelper.cs
(original)
+++
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/DocHelper.cs
Mon Oct 9 20:50:53 2006
@@ -155,11 +155,10 @@
public static int NumFields(Lucene.Net.Documents.Document doc)
{
- System.Collections.IEnumerator fields = doc.Fields();
int result = 0;
- while (fields.MoveNext())
+ foreach (Field field in doc.Fields())
{
- System.String name = fields.Current.ToString();
+ System.String name = field.Name();
name += ""; // avoid compiler warning
result++;
}
Modified:
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/TestSegmentReader.cs
URL:
http://svn.apache.org/viewvc/incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/TestSegmentReader.cs?view=diff&rev=454598&r1=454597&r2=454598
==============================================================================
---
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/TestSegmentReader.cs
(original)
+++
incubator/lucene.net/tags/Lucene.Net_1_9_RC1_final/src/Test/Index/TestSegmentReader.cs
Mon Oct 9 20:50:53 2006
@@ -77,10 +77,8 @@
//There are 2 unstored fields on the document that are
not preserved across writing
Assert.IsTrue(DocHelper.NumFields(result) ==
DocHelper.NumFields(testDoc) - DocHelper.unstored.Count);
- System.Collections.IEnumerator fields = result.Fields();
- while (fields.MoveNext())
+ foreach (Field field in result.Fields())
{
- Field field = (Field) fields.Current;
Assert.IsTrue(field != null);
Assert.IsTrue(DocHelper.nameValues.Contains(field.Name()));
}