Skip to content

Commit a57c470

Browse files
replacing clusterservice with inference context in tests
1 parent c1345c4 commit a57c470

File tree

40 files changed

+227
-348
lines changed

40 files changed

+227
-348
lines changed

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/alibabacloudsearch/AlibabaCloudSearchService.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.ValidationException;
1615
import org.elasticsearch.common.util.LazyInitializable;
1716
import org.elasticsearch.core.Nullable;
@@ -87,11 +86,6 @@ public class AlibabaCloudSearchService extends SenderService {
8786
InputType.INTERNAL_SEARCH
8887
);
8988

90-
// for testing
91-
public AlibabaCloudSearchService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
92-
super(factory, serviceComponents, clusterService);
93-
}
94-
9589
public AlibabaCloudSearchService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
9690
super(factory, serviceComponents, context);
9791
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/amazonbedrock/AmazonBedrockService.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.Strings;
1615
import org.elasticsearch.common.ValidationException;
1716
import org.elasticsearch.common.util.LazyInitializable;
@@ -92,17 +91,6 @@ public class AmazonBedrockService extends SenderService {
9291
InputType.UNSPECIFIED
9392
);
9493

95-
// for testing
96-
public AmazonBedrockService(
97-
HttpRequestSender.Factory httpSenderFactory,
98-
AmazonBedrockRequestSender.Factory amazonBedrockFactory,
99-
ServiceComponents serviceComponents,
100-
ClusterService clusterService
101-
) {
102-
super(httpSenderFactory, serviceComponents, clusterService);
103-
this.amazonBedrockSender = amazonBedrockFactory.createSender();
104-
}
105-
10694
public AmazonBedrockService(
10795
HttpRequestSender.Factory httpSenderFactory,
10896
AmazonBedrockRequestSender.Factory amazonBedrockFactory,

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/anthropic/AnthropicService.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.ValidationException;
1615
import org.elasticsearch.common.util.LazyInitializable;
1716
import org.elasticsearch.core.Nullable;
@@ -60,11 +59,6 @@ public class AnthropicService extends SenderService {
6059

6160
private static final EnumSet<TaskType> supportedTaskTypes = EnumSet.of(TaskType.COMPLETION);
6261

63-
// for testing
64-
public AnthropicService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
65-
super(factory, serviceComponents, clusterService);
66-
}
67-
6862
public AnthropicService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
6963
super(factory, serviceComponents, context);
7064
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/azureaistudio/AzureAiStudioService.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.Strings;
1615
import org.elasticsearch.common.ValidationException;
1716
import org.elasticsearch.common.util.LazyInitializable;
@@ -85,11 +84,6 @@ public class AzureAiStudioService extends SenderService {
8584
InputType.INTERNAL_SEARCH
8685
);
8786

88-
// for testing
89-
public AzureAiStudioService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
90-
super(factory, serviceComponents, clusterService);
91-
}
92-
9387
public AzureAiStudioService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
9488
super(factory, serviceComponents, context);
9589
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/azureopenai/AzureOpenAiService.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.ValidationException;
1615
import org.elasticsearch.common.util.LazyInitializable;
1716
import org.elasticsearch.core.Nullable;
@@ -71,11 +70,6 @@ public class AzureOpenAiService extends SenderService {
7170
private static final String SERVICE_NAME = "Azure OpenAI";
7271
private static final EnumSet<TaskType> supportedTaskTypes = EnumSet.of(TaskType.TEXT_EMBEDDING, TaskType.COMPLETION);
7372

74-
// for testing
75-
public AzureOpenAiService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
76-
super(factory, serviceComponents, clusterService);
77-
}
78-
7973
public AzureOpenAiService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
8074
super(factory, serviceComponents, context);
8175
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/CohereService.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -86,11 +86,6 @@ public class CohereService extends SenderService {
8686
// The reason it needs to be done here is that the batching logic needs to hold state but the *RequestManagers are instantiated
8787
// on every request
8888

89-
// for testing
90-
public CohereService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
91-
super(factory, serviceComponents, clusterService);
92-
}
93-
9489
public CohereService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
9590
super(factory, serviceComponents, context);
9691
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/custom/CustomService.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.ValidationException;
1615
import org.elasticsearch.common.util.LazyInitializable;
1716
import org.elasticsearch.core.Nullable;
@@ -76,10 +75,6 @@ public class CustomService extends SenderService {
7675
TaskType.COMPLETION
7776
);
7877

79-
public CustomService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
80-
super(factory, serviceComponents, clusterService);
81-
}
82-
8378
public CustomService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
8479
super(factory, serviceComponents, context);
8580
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/deepseek/DeepSeekService.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import org.elasticsearch.TransportVersion;
1111
import org.elasticsearch.TransportVersions;
1212
import org.elasticsearch.action.ActionListener;
13-
import org.elasticsearch.cluster.service.ClusterService;
1413
import org.elasticsearch.common.ValidationException;
1514
import org.elasticsearch.common.util.LazyInitializable;
1615
import org.elasticsearch.core.Strings;
@@ -60,11 +59,6 @@ public class DeepSeekService extends SenderService {
6059
);
6160
private static final EnumSet<TaskType> SUPPORTED_TASK_TYPES_FOR_STREAMING = EnumSet.of(TaskType.COMPLETION, TaskType.CHAT_COMPLETION);
6261

63-
// for testing
64-
public DeepSeekService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
65-
super(factory, serviceComponents, clusterService);
66-
}
67-
6862
public DeepSeekService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
6963
super(factory, serviceComponents, context);
7064
}

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/ElasticInferenceService.java

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import org.elasticsearch.TransportVersion;
1212
import org.elasticsearch.TransportVersions;
1313
import org.elasticsearch.action.ActionListener;
14-
import org.elasticsearch.cluster.service.ClusterService;
1514
import org.elasticsearch.common.Strings;
1615
import org.elasticsearch.common.ValidationException;
1716
import org.elasticsearch.common.util.LazyInitializable;
@@ -138,31 +137,6 @@ public ElasticInferenceService(
138137
);
139138
}
140139

141-
// for testing
142-
public ElasticInferenceService(
143-
HttpRequestSender.Factory factory,
144-
ServiceComponents serviceComponents,
145-
ElasticInferenceServiceSettings elasticInferenceServiceSettings,
146-
ModelRegistry modelRegistry,
147-
ElasticInferenceServiceAuthorizationRequestHandler authorizationRequestHandler,
148-
ClusterService service
149-
) {
150-
super(factory, serviceComponents, service);
151-
this.elasticInferenceServiceComponents = new ElasticInferenceServiceComponents(
152-
elasticInferenceServiceSettings.getElasticInferenceServiceUrl()
153-
);
154-
authorizationHandler = new ElasticInferenceServiceAuthorizationHandler(
155-
serviceComponents,
156-
modelRegistry,
157-
authorizationRequestHandler,
158-
initDefaultEndpoints(elasticInferenceServiceComponents),
159-
IMPLEMENTED_TASK_TYPES,
160-
this,
161-
getSender(),
162-
elasticInferenceServiceSettings
163-
);
164-
}
165-
166140
private static Map<String, DefaultModelConfig> initDefaultEndpoints(
167141
ElasticInferenceServiceComponents elasticInferenceServiceComponents
168142
) {

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/googleaistudio/GoogleAiStudioService.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,6 @@ public class GoogleAiStudioService extends SenderService {
8484
InputType.INTERNAL_SEARCH
8585
);
8686

87-
// for testing
88-
public GoogleAiStudioService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, ClusterService clusterService) {
89-
super(factory, serviceComponents, clusterService);
90-
}
91-
9287
public GoogleAiStudioService(HttpRequestSender.Factory factory, ServiceComponents serviceComponents, InferenceServiceExtension.InferenceServiceFactoryContext context) {
9388
super(factory, serviceComponents, context);
9489
}

0 commit comments

Comments
 (0)