diff options
author | Harald Musum <musum@yahooinc.com> | 2022-06-05 16:16:31 +0200 |
---|---|---|
committer | gjoranv <gv@verizonmedia.com> | 2022-06-08 11:45:29 +0200 |
commit | 0329d09e0b7cf99ab0009308bfef2c09ad7d7123 (patch) | |
tree | 4b898915a5b1e14023f7cd807915ba1978f81f6f /config-model/src/test/java | |
parent | 4ef9e51656a422d9a196ce5e7e6a00b1ba373952 (diff) |
Don't use operation= in array user config
Remove unnecessary config in one test, update other test to have
2 items in list
Diffstat (limited to 'config-model/src/test/java')
3 files changed, 21 insertions, 31 deletions
diff --git a/config-model/src/test/java/com/yahoo/vespa/model/builder/UserConfigBuilderTest.java b/config-model/src/test/java/com/yahoo/vespa/model/builder/UserConfigBuilderTest.java index 3b2b022bbf9..55c10a28a01 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/builder/UserConfigBuilderTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/builder/UserConfigBuilderTest.java @@ -6,7 +6,6 @@ import com.yahoo.config.model.application.provider.BaseDeployLogger; import com.yahoo.config.model.builder.xml.XmlHelper; import com.yahoo.config.model.deploy.ConfigDefinitionStore; import com.yahoo.config.model.producer.UserConfigRepo; -import com.yahoo.test.ArraytypesConfig; import com.yahoo.test.SimpletypesConfig; import com.yahoo.vespa.config.ConfigDefinitionKey; import com.yahoo.vespa.config.ConfigPayload; @@ -15,7 +14,6 @@ import com.yahoo.vespa.configdefinition.SpecialtokensConfig; import org.junit.Test; import org.w3c.dom.Document; import org.w3c.dom.Element; - import java.io.Reader; import java.io.StringReader; import java.util.Optional; @@ -56,11 +54,18 @@ public class UserConfigBuilderTest { public void require_that_arrays_of_structs_are_resolved() { Element configRoot = getDocument( " <config name='vespa.configdefinition.specialtokens'>" + - " <tokenlist operation='append'>" + - " <name>default</name>" + - " <tokens operation='append'>" + - " <token>dvd+-r</token>" + - " </tokens>" + + " <tokenlist>" + + " <item>" + + " <name>default</name>" + + " <tokens>" + + " <item>" + + " <token>dvd+-r</token>" + + " </item>" + + " <item>" + + " <token>c++</token>" + + " </item>" + + " </tokens>" + + " </item>" + " </tokenlist>" + " </config>" ); @@ -74,9 +79,10 @@ public class UserConfigBuilderTest { assertNotNull(map.get(key)); SpecialtokensConfig config = createConfig(SpecialtokensConfig.class, map.get(key)); assertEquals(1, config.tokenlist().size()); - assertEquals("default", config.tokenlist().get(0).name()); - assertEquals(1, config.tokenlist().get(0).tokens().size()); - assertEquals("dvd+-r", config.tokenlist().get(0).tokens().get(0).token()); + SpecialtokensConfig.Tokenlist tokenlist = config.tokenlist().get(0); + assertEquals("default", tokenlist.name()); + assertEquals(2, tokenlist.tokens().size()); + assertEquals("dvd+-r", tokenlist.tokens().get(0).token()); } @Test diff --git a/config-model/src/test/java/com/yahoo/vespa/model/builder/xml/dom/DomConfigPayloadBuilderTest.java b/config-model/src/test/java/com/yahoo/vespa/model/builder/xml/dom/DomConfigPayloadBuilderTest.java index 90b859e09a3..c0db0ac9e09 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/builder/xml/dom/DomConfigPayloadBuilderTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/builder/xml/dom/DomConfigPayloadBuilderTest.java @@ -2,7 +2,6 @@ package com.yahoo.vespa.model.builder.xml.dom; import com.yahoo.config.ConfigurationRuntimeException; -import com.yahoo.config.application.api.DeployLogger; import com.yahoo.config.codegen.DefParser; import com.yahoo.config.model.builder.xml.XmlHelper; import com.yahoo.slime.JsonFormat; @@ -14,13 +13,11 @@ import com.yahoo.vespa.config.ConfigPayloadBuilder; import org.junit.Test; import org.w3c.dom.Document; import org.w3c.dom.Element; - import java.io.ByteArrayOutputStream; import java.io.FileNotFoundException; import java.io.FileReader; import java.io.Reader; import java.io.StringReader; -import java.util.logging.Level; import static org.junit.Assert.assertEquals; import static org.junit.Assert.fail; @@ -33,10 +30,6 @@ import static org.junit.Assert.fail; */ public class DomConfigPayloadBuilderTest { - private static final DeployLogger logger = (level, message) -> { - if (level.intValue() > Level.INFO.intValue()) System.err.println(message); - }; - @Test public void testFunctionTest_DefaultValues() throws FileNotFoundException { Element configRoot = getDocument(new FileReader("src/test/cfg/admin/userconfigs/functiontest-defaultvalues.xml")); @@ -135,7 +128,7 @@ public class DomConfigPayloadBuilderTest { public void testFailWrongTagName() { Element configRoot = getDocument(new StringReader("<configs name=\"foo\"/>")); try { - new DomConfigPayloadBuilder(null, logger).build(configRoot); + new DomConfigPayloadBuilder(null).build(configRoot); fail("Expected exception for wrong tag name."); } catch (ConfigurationRuntimeException e) { assertEquals("The root element must be 'config', but was 'configs'.", e.getMessage()); @@ -147,7 +140,7 @@ public class DomConfigPayloadBuilderTest { public void testFailNoNameAttribute() { Element configRoot = getDocument(new StringReader("<config/>")); try { - new DomConfigPayloadBuilder(null, logger).build(configRoot); + new DomConfigPayloadBuilder(null).build(configRoot); fail("Expected exception for mismatch between def-name and xml name attribute."); } catch (ConfigurationRuntimeException e) { assertEquals("The 'config' element must have a 'name' attribute that matches the name of the config definition.", e.getMessage()); @@ -229,7 +222,7 @@ public class DomConfigPayloadBuilderTest { "<config name=\"test.arraytypes\" version=\"1\">" + " <item>13</item>" + "</config>"); - new DomConfigPayloadBuilder(null, logger).build(configRoot); + new DomConfigPayloadBuilder(null).build(configRoot); } @Test(expected=ConfigurationRuntimeException.class) @@ -241,13 +234,12 @@ public class DomConfigPayloadBuilderTest { DefParser defParser = new DefParser("simpletypes", new FileReader("src/test/resources/configdefinitions/test.simpletypes.def")); ConfigDefinition def = ConfigDefinitionBuilder.createConfigDefinition(defParser.getTree()); - ConfigPayloadBuilder unused = new DomConfigPayloadBuilder(def, logger).build(configRoot); + ConfigPayloadBuilder unused = new DomConfigPayloadBuilder(def).build(configRoot); } - private void assertPayload(String expected, Element configRoot) { ConfigPayload payload = ConfigPayload.fromBuilder( - new DomConfigPayloadBuilder(null, logger).build(configRoot)); + new DomConfigPayloadBuilder(null).build(configRoot)); try { ByteArrayOutputStream a = new ByteArrayOutputStream(); new JsonFormat(true).encode(a, payload.getSlime()); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/content/IndexedTest.java b/config-model/src/test/java/com/yahoo/vespa/model/content/IndexedTest.java index d9dbd6eeca7..a39cbc60a89 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/content/IndexedTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/content/IndexedTest.java @@ -45,14 +45,6 @@ public class IndexedTest extends ContentBaseTest { " <admin version='2.0'>" + " <adminserver hostalias='node0'/>" + " </admin>" + - " <config name='vespa.configdefinition.specialtokens'>" + - " <tokenlist operation='append'>" + - " <name>default</name>" + - " <tokens operation='append'>" + - " <token>dvd+-r</token>" + - " </tokens>" + - " </tokenlist>" + - " </config>" + " <container version='1.0'>" + " <search/>" + " <nodes>" + |