Improve connection API and test infrastructure (#181)

* Add ConnectionSource interface and unify pooled/non-pooled data sources

- Add ConnectionSource interface unifying pooled and non-pooled connection sources
- Add SimpleDataSource for non-pooled connections (wraps DriverManager)
- Add ConnectionSettings for connection behavior (isolation, autoCommit, readOnly, etc.)
- Move connection settings from PoolConfig to ConnectionSettings
- Add transactor() shortcuts to DatabaseConfig for common use cases
- Update PooledDataSource to implement ConnectionSource
- Remove config::connect from Transactor (use ConnectionSource instead)

This provides a clean, unified API:
- config.transactor() for quick non-pooled access
- config.transactor(settings, strategy) for customized non-pooled
- HikariDataSourceFactory.create(config, settings, pool).transactor() for pooled

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Bump PostgreSQL version from 14 to 16 in deploy-docs workflow

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add test-utils with file locking for parallel snapshot tests

When tests run in parallel across multiple JVMs, git operations can conflict.
This adds proper file-based locking to ensure exclusive access to git add.

- Add test-utils project with SnapshotTestUtils (Java)
- withGitLock() uses FileChannel.lock() for cross-JVM file locking
- Update all SnapshotTest files to use shared utility
- Update WithConnection files to use new Transactor API
- Add .snapshot-test.lock to .gitignore

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Update tester withConnection files to use new Transactor API

Migrate from `new Transactor(config, strategy)` to
`config.transactor(strategy)` for all database testers.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Parallelize type tests for faster execution

Use parallel streams and unique table names per test case to enable
concurrent execution across PgTypeTest, MariaTypeTest, DuckDbTypeTest,
OracleTypeTest, SqlServerTypeTest, and Db2TypeTest.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix remaining Kotlin and Java test helpers to use new Transactor API

- Update DuckDB Kotlin to use DriverManager.getConnection
- Update DB2 Java to use Transactor pattern
- Update all Kotlin test helpers to use CONFIG.transactor() method

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: oyvindberg

.github/workflows/deploy-docs.yml

@@ -23,7 +23,7 @@ jobs:
     runs-on: ubuntu-latest
     if: "!contains(github.event.head_commit.message, 'ci skip')"
     env:
-      PG_MAJOR: 14
+      PG_MAJOR: 16
     steps:
       - uses: actions/checkout@v4
       - uses: bleep-build/bleep-setup-action@0.0.1

.gitignore

@@ -23,4 +23,5 @@ __pycache__/
 /testdb.wal
 db/duckdb/test.db
 db/duckdb/test.db.wal
-duckdb-test/
+duckdb-test/
+.snapshot-test.lock

bleep.yaml

@@ -87,6 +87,11 @@ projects:
       options: -proc:none
     platform:
       name: jvm
+  test-utils:
+    java:
+      options: -proc:none
+    platform:
+      name: jvm
   testers/db2/java:
     dependencies:
     - com.fasterxml.jackson.core:jackson-annotations:2.17.2
@@ -346,7 +351,9 @@ projects:
     - com.typesafe.play::play-json:2.10.6
     - org.postgresql:postgresql:42.7.3
     - org.scalatest::scalatest:3.2.18
-    dependsOn: typr-dsl-anorm
+    dependsOn:
+    - typr-dsl-anorm
+    - test-utils
     extends: template-cross
     isTestProject: true
   testers/pg/scala/doobie:
@@ -359,7 +366,9 @@ projects:
     - io.circe::circe-core:0.14.7
     - org.postgresql:postgresql:42.7.3
     - org.scalatest::scalatest:3.2.18
-    dependsOn: typr-dsl-doobie
+    dependsOn:
+    - typr-dsl-doobie
+    - test-utils
     extends: template-cross
     isTestProject: true
     sources: ./generated-and-checked-in
@@ -373,7 +382,9 @@ projects:
     - junit:junit:4.13.2
     - org.postgresql:postgresql:42.7.3
     - org.scalatest::scalatest:3.2.18
-    dependsOn: foundations-jdbc-dsl
+    dependsOn:
+    - foundations-jdbc-dsl
+    - test-utils
     extends: template-scala-3
     isTestProject: true
     sources: ./generated-and-checked-in
@@ -387,7 +398,9 @@ projects:
     - junit:junit:4.13.2
     - org.postgresql:postgresql:42.7.3
     - org.scalatest::scalatest:3.2.18
-    dependsOn: foundations-jdbc-dsl-scala
+    dependsOn:
+    - foundations-jdbc-dsl-scala
+    - test-utils
     extends: template-scala-3
     isTestProject: true
     sources: ./generated-and-checked-in
@@ -401,7 +414,9 @@ projects:
     - dev.zio::zio-json:0.7.0
     - org.postgresql:postgresql:42.7.3
     - org.scalatest::scalatest:3.2.18
-    dependsOn: typr-dsl-zio-jdbc
+    dependsOn:
+    - typr-dsl-zio-jdbc
+    - test-utils
     extends: template-cross
     isTestProject: true
   testers/sqlserver/java:

foundations-jdbc-hikari/src/java/dev/typr/foundations/hikari/HikariDataSourceFactory.java

@@ -2,55 +2,43 @@
 
 import com.zaxxer.hikari.HikariConfig;
 import com.zaxxer.hikari.HikariDataSource;
+import dev.typr.foundations.connect.ConnectionSettings;
 import dev.typr.foundations.connect.DatabaseConfig;
 
 /**
- * Factory for creating PooledDataSource instances from DatabaseConfig and PoolConfig.
+ * Factory for creating PooledDataSource instances.
  *
  * <p>Example usage:
  *
  * <pre>{@code
- * // Minimal - all defaults
+ * // With connection settings
  * var ds = HikariDataSourceFactory.create(
- *     PostgresConfig.builder("localhost", 5432, "mydb", "user", "pass").build());
- *
- * // Use the transactor
- * var tx = ds.transactor();
- * tx.execute(conn -> repo.selectAll(conn));
- *
- * // With pool configuration
- * var ds = HikariDataSourceFactory.create(
- *     PostgresConfig.builder("localhost", 5432, "mydb", "user", "pass")
- *         .sslmode(PgSslMode.REQUIRE)
+ *     PostgresConfig.builder("localhost", 5432, "mydb", "user", "pass").build(),
+ *     ConnectionSettings.builder()
+ *         .transactionIsolation(TransactionIsolation.READ_UNCOMMITTED)
  *         .build(),
  *     PoolConfig.builder()
  *         .maximumPoolSize(20)
- *         .connectionTimeout(Duration.ofSeconds(10))
  *         .build());
+ *
+ * var tx = ds.transactor();
+ * tx.execute(conn -> repo.selectAll(conn));
  * }</pre>
  */
 public final class HikariDataSourceFactory {
 
   private HikariDataSourceFactory() {}
 
   /**
-   * Create a PooledDataSource from DatabaseConfig with default pool settings.
-   *
-   * @param config database configuration
-   * @return configured PooledDataSource with transactor() method
-   */
-  public static PooledDataSource create(DatabaseConfig config) {
-    return create(config, PoolConfig.defaults());
-  }
-
-  /**
-   * Create a PooledDataSource from DatabaseConfig and PoolConfig.
+   * Create a PooledDataSource with connection settings and pool configuration.
    *
-   * @param config database configuration
-   * @param pool pool configuration
-   * @return configured PooledDataSource with transactor() method
+   * @param config database configuration (URL, credentials, driver properties)
+   * @param settings connection settings (isolation, autoCommit, readOnly, etc.)
+   * @param pool pool configuration (sizing, timeouts, etc.)
+   * @return configured PooledDataSource
    */
-  public static PooledDataSource create(DatabaseConfig config, PoolConfig pool) {
+  public static PooledDataSource create(
+      DatabaseConfig config, ConnectionSettings settings, PoolConfig pool) {
     HikariConfig hikari = new HikariConfig();
 
     // Connection settings from DatabaseConfig
@@ -73,25 +61,27 @@ public static PooledDataSource create(DatabaseConfig config, PoolConfig pool) {
     hikari.setKeepaliveTime(pool.keepaliveTime().toMillis());
     hikari.setLeakDetectionThreshold(pool.leakDetectionThreshold().toMillis());
 
-    // Connection defaults
-    if (pool.transactionIsolation() != null) {
-      hikari.setTransactionIsolation(pool.transactionIsolation().jdbcName());
+    // Connection settings
+    if (settings.transactionIsolation() != null) {
+      hikari.setTransactionIsolation(settings.transactionIsolation().jdbcName());
     }
-    if (pool.autoCommit() != null) {
-      hikari.setAutoCommit(pool.autoCommit());
+    if (settings.autoCommit() != null) {
+      hikari.setAutoCommit(settings.autoCommit());
     }
-    if (pool.readOnly() != null) {
-      hikari.setReadOnly(pool.readOnly());
+    if (settings.readOnly() != null) {
+      hikari.setReadOnly(settings.readOnly());
     }
-    if (pool.catalog() != null) {
-      hikari.setCatalog(pool.catalog());
+    if (settings.catalog() != null) {
+      hikari.setCatalog(settings.catalog());
     }
-    if (pool.schema() != null) {
-      hikari.setSchema(pool.schema());
+    if (settings.schema() != null) {
+      hikari.setSchema(settings.schema());
     }
-    if (pool.connectionInitSql() != null) {
-      hikari.setConnectionInitSql(pool.connectionInitSql());
+    if (settings.connectionInitSql() != null) {
+      hikari.setConnectionInitSql(settings.connectionInitSql());
     }
+
+    // Pool-specific: connection test query
     if (pool.connectionTestQuery() != null) {
       hikari.setConnectionTestQuery(pool.connectionTestQuery());
     }
@@ -117,4 +107,36 @@ public static PooledDataSource create(DatabaseConfig config, PoolConfig pool) {
 
     return new PooledDataSource(new HikariDataSource(hikari));
   }
+
+  /**
+   * Create a PooledDataSource with connection settings and default pool configuration.
+   *
+   * @param config database configuration
+   * @param settings connection settings
+   * @return configured PooledDataSource
+   */
+  public static PooledDataSource create(DatabaseConfig config, ConnectionSettings settings) {
+    return create(config, settings, PoolConfig.defaults());
+  }
+
+  /**
+   * Create a PooledDataSource with default settings.
+   *
+   * @param config database configuration
+   * @return configured PooledDataSource with driver defaults
+   */
+  public static PooledDataSource create(DatabaseConfig config) {
+    return create(config, ConnectionSettings.EMPTY, PoolConfig.defaults());
+  }
+
+  /**
+   * Create a PooledDataSource with pool configuration but default connection settings.
+   *
+   * @param config database configuration
+   * @param pool pool configuration
+   * @return configured PooledDataSource
+   */
+  public static PooledDataSource create(DatabaseConfig config, PoolConfig pool) {
+    return create(config, ConnectionSettings.EMPTY, pool);
+  }
 }