Merge pull request #325 from eclipse-jnosql/neo4j-adjustments

dearrudam · web-flow · commit 1e864060abb4 · 2025-04-11T12:36:59.000-03:00
Changed Neo4JDatabaseManagerFactory's constructor visibility to protected
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -11,6 +11,7 @@ and this project adheres to https://semver.org/spec/v2.0.0.html[Semantic Version
 === Changed
 
 - Update Elasticsearch driver to 8.17.4
+- Changed Neo4JDatabaseManagerFactory's constructor visibility to protected
 
 === Fixed
 
diff --git a/jnosql-neo4j/src/main/java/org/eclipse/jnosql/databases/neo4j/communication/Neo4JDatabaseManagerFactory.java b/jnosql-neo4j/src/main/java/org/eclipse/jnosql/databases/neo4j/communication/Neo4JDatabaseManagerFactory.java
@@ -50,8 +50,8 @@ public class Neo4JDatabaseManagerFactory implements DatabaseManagerFactory {
 
     private final Driver driver;
 
-    private Neo4JDatabaseManagerFactory(Driver driver) {
-        this.driver = driver;
+    Neo4JDatabaseManagerFactory(Driver driver) {
+        this.driver = Objects.requireNonNull(driver, "driver is required");
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -50,8 +50,8 @@ public class Neo4JDatabaseManagerFactory implements DatabaseManagerFactory {`
`50`	`50`
`51`	`51`	`private final Driver driver;`
`52`	`52`
`53`		`- private Neo4JDatabaseManagerFactory(Driver driver) {`
`54`		`- this.driver = driver;`
	`53`	`+ Neo4JDatabaseManagerFactory(Driver driver) {`
	`54`	`+ this.driver = Objects.requireNonNull(driver, "driver is required");`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`/**`