diff options
author | Harald Musum <musum@yahoo-inc.com> | 2017-02-11 12:09:09 +0100 |
---|---|---|
committer | Harald Musum <musum@yahoo-inc.com> | 2017-02-11 12:09:09 +0100 |
commit | 246e2593bb80b2421a3e659235826d987fd0ddb4 (patch) | |
tree | bafd4fb0793cc1b5b9d046938e3cc0c194c4a6b2 /configserver/src/test/java/com/yahoo | |
parent | 77026d390cb95bd203c5a594ecb869a856947751 (diff) |
Clean up interface
* Remove 'throws IOException now that no implementors use it and handle
getConfig now returning null instead of exception when no config found
* Remove getConfig method with InnerCNode as 2nd argument
* Prepare for removing 3-argument getConfig()
Diffstat (limited to 'configserver/src/test/java/com/yahoo')
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/ModelStub.java | 5 | ||||
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/application/MockModel.java | 5 |
2 files changed, 4 insertions, 6 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/ModelStub.java b/configserver/src/test/java/com/yahoo/vespa/config/server/ModelStub.java index ed9ced7b49b..3347bf28af3 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/ModelStub.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/ModelStub.java @@ -1,7 +1,6 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config.server; -import com.yahoo.config.codegen.InnerCNode; import com.yahoo.config.model.api.FileDistribution; import com.yahoo.config.model.api.HostInfo; import com.yahoo.config.model.api.Model; @@ -20,12 +19,12 @@ import java.util.Set; public class ModelStub implements Model { @Override - public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef, ConfigPayload override) { + public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef) { return null; } @Override - public ConfigPayload getConfig(ConfigKey<?> configKey, InnerCNode targetDef, ConfigPayload override) { + public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef, ConfigPayload override) { return null; } diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/application/MockModel.java b/configserver/src/test/java/com/yahoo/vespa/config/server/application/MockModel.java index b22e8648eae..f50bc95442e 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/application/MockModel.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/application/MockModel.java @@ -1,7 +1,6 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config.server.application; -import com.yahoo.config.codegen.InnerCNode; import com.yahoo.config.model.api.FileDistribution; import com.yahoo.config.model.api.HostInfo; import com.yahoo.config.model.api.Model; @@ -69,12 +68,12 @@ class MockModel implements Model { } @Override - public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef, ConfigPayload override) { + public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef) { throw new UnsupportedOperationException(); } @Override - public ConfigPayload getConfig(ConfigKey<?> configKey, InnerCNode targetDef, ConfigPayload override) { + public ConfigPayload getConfig(ConfigKey<?> configKey, ConfigDefinition targetDef, ConfigPayload override) { throw new UnsupportedOperationException(); } |