From 246e2593bb80b2421a3e659235826d987fd0ddb4 Mon Sep 17 00:00:00 2001 From: Harald Musum Date: Sat, 11 Feb 2017 12:09:09 +0100 Subject: 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() --- .../src/test/java/com/yahoo/vespa/config/server/ModelStub.java | 5 ++--- .../java/com/yahoo/vespa/config/server/application/MockModel.java | 5 ++--- 2 files changed, 4 insertions(+), 6 deletions(-) (limited to 'configserver/src/test/java/com/yahoo') 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(); } -- cgit v1.2.3