Skip to content

Commit 9fe1113

Browse files
committed
fixing tests due to low cohesion
1 parent 0230775 commit 9fe1113

File tree

3 files changed

+16
-2
lines changed

3 files changed

+16
-2
lines changed

procedures/extension/src/main/java/org/neo4j/gds/extension/OpenGraphDataScienceExtension.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,8 @@ public OpenGraphDataScienceExtension() {
5757
@Override
5858
public Lifecycle newInstance(ExtensionContext extensionContext, Dependencies dependencies) {
5959
var log = new LogAccessor().getLog(dependencies.logService(), getClass());
60+
61+
var dependencySatisfier = extensionContext.dependencySatisfier();
6062
var globalProcedures = dependencies.globalProcedures();
6163
var neo4jConfiguration = dependencies.config();
6264

@@ -77,6 +79,7 @@ public Lifecycle newInstance(ExtensionContext extensionContext, Dependencies dep
7779

7880
var graphDataScienceExtensionBuilderAndAssociatedProducts = OpenGraphDataScienceExtensionBuilder.create(
7981
log,
82+
dependencySatisfier,
8083
globalProcedures,
8184
neo4jConfiguration,
8285
concurrencyValidator,

procedures/integration/src/main/java/org/neo4j/gds/procedures/integration/ComponentRegistration.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
*/
2020
package org.neo4j.gds.procedures.integration;
2121

22+
import org.neo4j.common.DependencySatisfier;
2223
import org.neo4j.function.ThrowingFunction;
2324
import org.neo4j.gds.logging.Log;
2425
import org.neo4j.internal.kernel.api.exceptions.ProcedureException;
@@ -27,10 +28,12 @@
2728

2829
public class ComponentRegistration {
2930
private final Log log;
31+
private final DependencySatisfier dependencySatisfier;
3032
private final GlobalProcedures globalProcedures;
3133

32-
public ComponentRegistration(Log log, GlobalProcedures globalProcedures) {
34+
public ComponentRegistration(Log log, DependencySatisfier dependencySatisfier, GlobalProcedures globalProcedures) {
3335
this.log = log;
36+
this.dependencySatisfier = dependencySatisfier;
3437
this.globalProcedures = globalProcedures;
3538
}
3639

@@ -47,4 +50,8 @@ public <T> void registerComponent(
4750
);
4851
log.info(name + " registered.");
4952
}
53+
54+
void setUpDependency(Object component) {
55+
dependencySatisfier.satisfyDependency(component);
56+
}
5057
}

procedures/integration/src/main/java/org/neo4j/gds/procedures/integration/OpenGraphDataScienceExtensionBuilder.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
package org.neo4j.gds.procedures.integration;
2121

2222
import org.apache.commons.lang3.tuple.Triple;
23+
import org.neo4j.common.DependencySatisfier;
2324
import org.neo4j.gds.applications.algorithms.machinery.AlgorithmProcessingTemplate;
2425
import org.neo4j.gds.applications.graphstorecatalog.ExportLocation;
2526
import org.neo4j.gds.applications.graphstorecatalog.GraphCatalogApplications;
@@ -115,6 +116,7 @@ private OpenGraphDataScienceExtensionBuilder(
115116
*/
116117
public static Triple<OpenGraphDataScienceExtensionBuilder, TaskRegistryFactoryService, TaskStoreService> create(
117118
Log log,
119+
DependencySatisfier dependencySatisfier,
118120
GlobalProcedures globalProcedures,
119121
Configuration neo4jConfiguration,
120122
ConcurrencyValidator concurrencyValidator,
@@ -169,7 +171,7 @@ public static Triple<OpenGraphDataScienceExtensionBuilder, TaskRegistryFactorySe
169171
freeMemoryAfterLastGc
170172
);
171173

172-
var componentRegistration = new ComponentRegistration(log, globalProcedures);
174+
var componentRegistration = new ComponentRegistration(log, dependencySatisfier, globalProcedures);
173175

174176
componentRegistration.registerComponent("GDS Memory Facade", MemoryFacade.class, context -> {
175177
var userAccessor = new UserAccessor();
@@ -265,6 +267,8 @@ private void registerMetricsComponent(Metrics metrics) {
265267
@Deprecated
266268
private void registerModelCatalogComponent(ModelCatalog modelCatalog) {
267269
componentRegistration.registerComponent("Model Catalog", ModelCatalog.class, __ -> modelCatalog);
270+
271+
componentRegistration.setUpDependency(modelCatalog);
268272
}
269273

270274
/**

0 commit comments

Comments
 (0)