diff options
author | Harald Musum <musum@yahooinc.com> | 2023-01-02 09:15:53 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2023-01-02 09:15:53 +0100 |
commit | 76700dda98d7d0d7c28d4755dba314289e6c4ae4 (patch) | |
tree | adb29cd22d200a9c59b94d27402edb0e32d14ca7 /configserver | |
parent | 37bd63b9f4590d295cb82758f6b3f13325b5acfe (diff) |
Simplify
Diffstat (limited to 'configserver')
6 files changed, 7 insertions, 14 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java index 61f447597a2..8f104a27000 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java @@ -129,8 +129,7 @@ public class ApplicationRepositoryTest { .withClock(clock) .withConfigserverConfig(configserverConfig) .withCurator(curator) - .withFileDistributionFactory( - new MockFileDistributionFactory(configserverConfig, fileDirectory)) + .withFileDistributionFactory(new MockFileDistributionFactory(configserverConfig)) .withFlagSource(flagSource) .build(); tenantRepository.addTenant(TenantRepository.HOSTED_VESPA_TENANT); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java index e5b8a743b35..62bb4a77916 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java @@ -26,7 +26,6 @@ import com.yahoo.vespa.config.server.MockProvisioner; import com.yahoo.vespa.config.server.TimeoutBudget; import com.yahoo.vespa.config.server.application.ConfigConvergenceChecker; import com.yahoo.vespa.config.server.application.OrchestratorMock; -import com.yahoo.vespa.config.server.filedistribution.FileDirectory; import com.yahoo.vespa.config.server.filedistribution.MockFileDistributionFactory; import com.yahoo.vespa.config.server.http.v2.PrepareResult; import com.yahoo.vespa.config.server.modelfactory.ModelFactoryRegistry; @@ -291,8 +290,7 @@ public class DeployTester { .withClock(clock) .withConfigserverConfig(configserverConfig) .withCurator(curator) - .withFileDistributionFactory( - new MockFileDistributionFactory(configserverConfig, new FileDirectory(configserverConfig))) + .withFileDistributionFactory(new MockFileDistributionFactory(configserverConfig)) .withMetrics(Optional.ofNullable(metrics).orElse(Metrics.createTestMetrics())) .withModelFactoryRegistry((new ModelFactoryRegistry(modelFactories))) .withZone(zone); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/MockFileDistributionFactory.java b/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/MockFileDistributionFactory.java index 706a550a387..ed8ee7ac927 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/MockFileDistributionFactory.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/MockFileDistributionFactory.java @@ -11,8 +11,8 @@ import java.io.File; */ public class MockFileDistributionFactory extends FileDistributionFactory { - public MockFileDistributionFactory(ConfigserverConfig configserverConfig, FileDirectory fileDirectory) { - super(configserverConfig, fileDirectory); + public MockFileDistributionFactory(ConfigserverConfig configserverConfig) { + super(configserverConfig, new FileDirectory(configserverConfig)); } @Override diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandlerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandlerTest.java index 9251bbfcf39..4cb69290e2f 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandlerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandlerTest.java @@ -29,7 +29,6 @@ import com.yahoo.vespa.config.server.application.ClusterReindexing.Status; import com.yahoo.vespa.config.server.application.HttpProxy; import com.yahoo.vespa.config.server.application.OrchestratorMock; import com.yahoo.vespa.config.server.deploy.DeployTester; -import com.yahoo.vespa.config.server.filedistribution.FileDirectory; import com.yahoo.vespa.config.server.filedistribution.MockFileDistributionFactory; import com.yahoo.vespa.config.server.http.HandlerTest; import com.yahoo.vespa.config.server.http.HttpErrorResponse; @@ -125,7 +124,7 @@ public class ApplicationHandlerTest { tenantRepository = new TestTenantRepository.Builder() .withClock(clock) .withConfigserverConfig(configserverConfig) - .withFileDistributionFactory(new MockFileDistributionFactory(configserverConfig, new FileDirectory(configserverConfig))) + .withFileDistributionFactory(new MockFileDistributionFactory(configserverConfig)) .withHostProvisionerProvider(HostProvisionerProvider.withProvisioner(provisioner, false)) .withModelFactoryRegistry(new ModelFactoryRegistry(modelFactories)) .build(); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionPreparerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionPreparerTest.java index 871338931fe..c7113bbf803 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionPreparerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionPreparerTest.java @@ -32,7 +32,6 @@ import com.yahoo.vespa.config.server.MockSecretStore; import com.yahoo.vespa.config.server.TestConfigDefinitionRepo; import com.yahoo.vespa.config.server.TimeoutBudgetTest; import com.yahoo.vespa.config.server.deploy.DeployHandlerLogger; -import com.yahoo.vespa.config.server.filedistribution.FileDirectory; import com.yahoo.vespa.config.server.filedistribution.MockFileDistributionFactory; import com.yahoo.vespa.config.server.host.HostRegistry; import com.yahoo.vespa.config.server.http.InvalidApplicationException; @@ -125,7 +124,7 @@ public class SessionPreparerTest { HostProvisionerProvider hostProvisionerProvider) { return new SessionPreparer( modelFactoryRegistry, - new MockFileDistributionFactory(configserverConfig, new FileDirectory(configserverConfig)), + new MockFileDistributionFactory(configserverConfig), new InThreadExecutorService(), hostProvisionerProvider, configserverConfig, diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionRepositoryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionRepositoryTest.java index dc3441237ab..a5360fbc01c 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionRepositoryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionRepositoryTest.java @@ -22,7 +22,6 @@ import com.yahoo.vespa.config.server.ApplicationRepository; import com.yahoo.vespa.config.server.MockProvisioner; import com.yahoo.vespa.config.server.application.ApplicationSet; import com.yahoo.vespa.config.server.application.OrchestratorMock; -import com.yahoo.vespa.config.server.filedistribution.FileDirectory; import com.yahoo.vespa.config.server.filedistribution.MockFileDistributionFactory; import com.yahoo.vespa.config.server.http.InvalidApplicationException; import com.yahoo.vespa.config.server.modelfactory.ModelFactoryRegistry; @@ -98,8 +97,7 @@ public class SessionRepositoryTest { .withConfigserverConfig(configserverConfig) .withCurator(curator) .withFlagSource(flagSource) - .withFileDistributionFactory( - new MockFileDistributionFactory(configserverConfig, new FileDirectory(configserverConfig))) + .withFileDistributionFactory(new MockFileDistributionFactory(configserverConfig)) .withModelFactoryRegistry(modelFactoryRegistry) .build(); tenantRepository.addTenant(SessionRepositoryTest.tenantName); |