Add test cases.

Relates to #60

Author: zantvoort

…core/BuilderExtendedIntegrationTest.java …/st/orm/core/BuilderIntegrationTest.javastorm-core/src/test/java/st/orm/core/BuilderExtendedIntegrationTest.java renamed to storm-core/src/test/java/st/orm/core/BuilderIntegrationTest.java storm-core/src/test/java/st/orm/core/BuilderExtendedIntegrationTest.java renamed to storm-core/src/test/java/st/orm/core/BuilderIntegrationTest.java

@@ -37,7 +37,7 @@ public class DatabaseSchemaIntegrationTest {
     @Autowired
     private DataSource dataSource;
 
-    // ---- Table existence ----
+    // Table existence
 
     @Test
     public void testCoreTablesExist() throws SQLException {
@@ -73,7 +73,7 @@ public void testNonExistentTableNotFound() throws SQLException {
         }
     }
 
-    // ---- Views ----
+    // Views
 
     @Test
     public void testViewsExist() throws SQLException {
@@ -95,7 +95,7 @@ public void testViewColumns() throws SQLException {
         }
     }
 
-    // ---- Column metadata ----
+    // Column metadata
 
     @Test
     public void testCityColumns() throws SQLException {
@@ -145,7 +145,7 @@ public void testVetSpecialtySpecialtyIdNotNullable() throws SQLException {
         }
     }
 
-    // ---- Primary keys ----
+    // Primary keys
 
     @Test
     public void testCityPrimaryKey() throws SQLException {
@@ -176,7 +176,7 @@ public void testPrimaryKeysForViewEmpty() throws SQLException {
         }
     }
 
-    // ---- Foreign keys ----
+    // Foreign keys
 
     @Test
     public void testOwnerForeignKeyToCity() throws SQLException {
@@ -237,7 +237,7 @@ public void testForeignKeysForTableWithNone() throws SQLException {
         }
     }
 
-    // ---- Unique keys ----
+    // Unique keys
 
     @Test
     public void testUniqueKeysForNonExistentTable() throws SQLException {
@@ -248,7 +248,7 @@ public void testUniqueKeysForNonExistentTable() throws SQLException {
         }
     }
 
-    // ---- JDBC_METADATA strategy ----
+    // JDBC_METADATA strategy
 
     @Test
     public void testReadWithJdbcMetadataStrategy() throws SQLException {
@@ -275,7 +275,7 @@ public void testReadWithJdbcMetadataStrategy() throws SQLException {
         }
     }
 
-    // ---- Polymorphic tables ----
+    // Polymorphic tables
 
     @Test
     public void testPolymorphicTablesExist() throws SQLException {
@@ -304,7 +304,7 @@ public void testJoinedInheritanceForeignKeys() throws SQLException {
         }
     }
 
-    // ---- DbColumn record properties ----
+    // DbColumn record properties
 
     @Test
     public void testDbColumnRecordProperties() throws SQLException {
@@ -318,7 +318,7 @@ public void testDbColumnRecordProperties() throws SQLException {
         }
     }
 
-    // ---- DbPrimaryKey record properties ----
+    // DbPrimaryKey record properties
 
     @Test
     public void testDbPrimaryKeyRecordProperties() throws SQLException {
@@ -332,7 +332,7 @@ public void testDbPrimaryKeyRecordProperties() throws SQLException {
         }
     }
 
-    // ---- Sequence discovery (H2 supports INFORMATION_SCHEMA.SEQUENCES) ----
+    // Sequence discovery (H2 supports INFORMATION_SCHEMA.SEQUENCES)
 
     @Test
     public void testSequenceNotExistsByDefault() throws SQLException {
@@ -343,7 +343,7 @@ public void testSequenceNotExistsByDefault() throws SQLException {
         }
     }
 
-    // ---- Null schema/catalog handling ----
+    // Null schema/catalog handling
 
     @Test
     public void testReadWithNullCatalogAndSchema() throws SQLException {

storm-core/src/test/java/st/orm/core/DatabaseSchemaIntegrationTest.java

@@ -107,9 +107,7 @@ public record VisitFieldValue(
             @Version Instant timestamp
     ) implements Entity<Integer> {}
 
-    // ------------------------------------------------------------
     // Expected SQL
-    // ------------------------------------------------------------
 
     private static final String FULL_UPDATE_SQL = """
             UPDATE visit
@@ -121,9 +119,7 @@ public record VisitFieldValue(
             SET pet_id = ?, "timestamp" = CURRENT_TIMESTAMP
             WHERE id = ? AND "timestamp" = ?""";
 
-    // ------------------------------------------------------------
     // OFF: always updates all columns, dirtyCheck irrelevant
-    // ------------------------------------------------------------
 
     @Test
     void off_default_alwaysUpdates_evenIfNoChanges() {
@@ -150,11 +146,9 @@ void off_value_alwaysUpdates_evenIfNoChanges() {
         assertEquals(FULL_UPDATE_SQL, sql.statement());
     }
 
-    // ------------------------------------------------------------
     // ENTITY: full update SQL, dirty detection is per-column
     // DEFAULT = instance-based dirty check
     // VALUE   = semantic value-based dirty check
-    // ------------------------------------------------------------
 
     @Test
     void entity_default_skipsUpdate_whenNoMappedColumnChanges() {
@@ -231,11 +225,9 @@ void entity_value_skipsUpdate_whenFkIdSameEvenIfInstanceChanges() {
         assertNull(sql);
     }
 
-    // ------------------------------------------------------------
     // FIELD: updates only changed columns (plus version)
     // DEFAULT = instance-based dirty check
     // VALUE   = semantic value-based dirty check
-    // ------------------------------------------------------------
 
     @Test
     void field_default_skipsUpdate_whenNoMappedColumnChanges() {
@@ -312,9 +304,7 @@ void field_value_skipsUpdate_whenFkIdSameEvenIfInstanceChanges() {
         assertNull(sql);
     }
 
-    // ------------------------------------------------------------
     // Helpers
-    // ------------------------------------------------------------
 
     private Sql captureFirstUpdateSql(ThrowingSupplier<?> action) {
         AtomicReference<Sql> ref = new AtomicReference<>();