summaryrefslogtreecommitdiffstats
path: root/configserver
diff options
context:
space:
mode:
authorHarald Musum <musum@yahooinc.com>2023-01-02 09:15:53 +0100
committerHarald Musum <musum@yahooinc.com>2023-01-02 09:15:53 +0100
commit76700dda98d7d0d7c28d4755dba314289e6c4ae4 (patch)
treeadb29cd22d200a9c59b94d27402edb0e32d14ca7 /configserver
parent37bd63b9f4590d295cb82758f6b3f13325b5acfe (diff)
Simplify
Diffstat (limited to 'configserver')
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java3
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java4
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/MockFileDistributionFactory.java4
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandlerTest.java3
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionPreparerTest.java3
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/session/SessionRepositoryTest.java4
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);