diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2018-09-14 19:38:45 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-14 19:38:45 +0200 |
commit | 0f4aa6b514bcca1ed48969680c71e740448fce85 (patch) | |
tree | ee21c3c53a9fb50a1c065e3b1472e1f4762ababa /docproc/src/test | |
parent | 1d96a18bb0079b626af384748164c15dbee92c8c (diff) |
Revert "Revert "Do not expose fieldupdates as a list. Hide implementation details ins…""
Diffstat (limited to 'docproc/src/test')
-rw-r--r-- | docproc/src/test/java/com/yahoo/docproc/ProcessingUpdateTestCase.java | 10 | ||||
-rw-r--r-- | docproc/src/test/java/com/yahoo/docproc/proxy/SchemaMappingAndAccessesTest.java | 9 |
2 files changed, 11 insertions, 8 deletions
diff --git a/docproc/src/test/java/com/yahoo/docproc/ProcessingUpdateTestCase.java b/docproc/src/test/java/com/yahoo/docproc/ProcessingUpdateTestCase.java index 9a3a29e55b1..a89dbfcc782 100644 --- a/docproc/src/test/java/com/yahoo/docproc/ProcessingUpdateTestCase.java +++ b/docproc/src/test/java/com/yahoo/docproc/ProcessingUpdateTestCase.java @@ -37,8 +37,10 @@ public class ProcessingUpdateTestCase { @Test public void testProcessingUpdates() { DocumentType articleType = new DocumentType("article"); - articleType.addField(new Field("body", DataType.STRING, true)); - articleType.addField(new Field("title", DataType.STRING, true)); + Field bodyField = new Field("body", DataType.STRING, true); + Field titleField = new Field("title", DataType.STRING, true); + articleType.addField(bodyField); + articleType.addField(titleField); dtm = new DocumentTypeManager(); dtm.registerDocumentType(articleType); @@ -69,12 +71,12 @@ public class ProcessingUpdateTestCase { assertEquals(new StringFieldValue("body blah blah blah "), first.getFieldValue("title")); DocumentUpdate second = (DocumentUpdate) operations.get(1); - FieldUpdate firstUpd = second.getFieldUpdate(0); + FieldUpdate firstUpd = second.getFieldUpdate(bodyField); assertEquals(ValueUpdate.ValueUpdateClassID.ASSIGN, firstUpd.getValueUpdate(0).getValueUpdateClassID()); assertEquals(new StringFieldValue("this is the updated body of the article, blahdi blahdi blahdi"), firstUpd.getValueUpdate(0) .getValue()); - FieldUpdate secondUpd = second.getFieldUpdate(1); + FieldUpdate secondUpd = second.getFieldUpdate(titleField); assertEquals(ValueUpdate.ValueUpdateClassID.ASSIGN, secondUpd.getValueUpdate(0).getValueUpdateClassID()); assertEquals(new StringFieldValue("body blahdi blahdi blahdi "), secondUpd.getValueUpdate(0).getValue()); } diff --git a/docproc/src/test/java/com/yahoo/docproc/proxy/SchemaMappingAndAccessesTest.java b/docproc/src/test/java/com/yahoo/docproc/proxy/SchemaMappingAndAccessesTest.java index 05a03480173..e6de3190156 100644 --- a/docproc/src/test/java/com/yahoo/docproc/proxy/SchemaMappingAndAccessesTest.java +++ b/docproc/src/test/java/com/yahoo/docproc/proxy/SchemaMappingAndAccessesTest.java @@ -328,16 +328,17 @@ public class SchemaMappingAndAccessesTest { Document doc = getDoc(); DocumentType type = doc.getDataType(); DocumentUpdate dud = new DocumentUpdate(type, new DocumentId("doc:map:test:1")); - FieldUpdate assignSingle = FieldUpdate.createAssign(type.getField("title"), new StringFieldValue("something")); + com.yahoo.document.Field title = type.getField("title"); + FieldUpdate assignSingle = FieldUpdate.createAssign(title, new StringFieldValue("something")); Map<String, String> fieldMap = new HashMap<>(); fieldMap.put("t", "title"); fieldMap.put("a", "artist"); ProxyDocumentUpdate pup = new ProxyDocumentUpdate(dud, fieldMap); pup.addFieldUpdate(assignSingle); - assertEquals(pup.getFieldUpdates(), dud.getFieldUpdates()); + assertEquals(pup.fieldUpdates().toString(), dud.fieldUpdates().toString()); assertEquals(pup.getDocumentType(), dud.getDocumentType()); - assertEquals(pup.getFieldUpdate(new com.yahoo.document.Field("title")).size(), 1); - assertEquals(pup.getFieldUpdate(0), dud.getFieldUpdate(0)); + assertEquals(pup.getFieldUpdate(title).size(), 1); + assertEquals(pup.getFieldUpdate(title), dud.fieldUpdates().iterator().next()); assertEquals(pup.getFieldUpdate("title"), dud.getFieldUpdate("title")); assertEquals(pup.getId(), dud.getId()); assertEquals(pup.getType(), dud.getType()); |