From 6545f07f3052af7de8d54772cc63fc943062dbc6 Mon Sep 17 00:00:00 2001 From: robfrank Date: Wed, 22 Apr 2026 10:51:06 +0200 Subject: [PATCH] test: fix GitHubIssue3878Test by committing setup data before materialized view refresh The test used autoStartTx=true which left INSERT data in an uncommitted transaction. MaterializedViewRefresher.fullRefresh() creates a dedicated transaction (joinCurrentTx=false) that cannot see uncommitted data under snapshot isolation, resulting in empty materialized views. Fix: remove autoStartTx, wrap INSERTs in database.transaction() so the data is committed before the view's initial fullRefresh runs. Co-Authored-By: Claude Sonnet 4.6 --- .../query/sql/executor/GitHubIssue3878Test.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/engine/src/test/java/com/arcadedb/query/sql/executor/GitHubIssue3878Test.java b/engine/src/test/java/com/arcadedb/query/sql/executor/GitHubIssue3878Test.java index a14144f227..330e8b151e 100644 --- a/engine/src/test/java/com/arcadedb/query/sql/executor/GitHubIssue3878Test.java +++ b/engine/src/test/java/com/arcadedb/query/sql/executor/GitHubIssue3878Test.java @@ -32,15 +32,13 @@ */ public class GitHubIssue3878Test extends TestHelper { - public GitHubIssue3878Test() { - autoStartTx = true; - } - @Override public void beginTest() { database.command("sql", "CREATE DOCUMENT TYPE TestDoc"); - database.command("sql", "INSERT INTO TestDoc SET name = 'Alice', active = true"); - database.command("sql", "INSERT INTO TestDoc SET name = 'Bob', active = false"); + database.transaction(() -> { + database.command("sql", "INSERT INTO TestDoc SET name = 'Alice', active = true"); + database.command("sql", "INSERT INTO TestDoc SET name = 'Bob', active = false"); + }); } /**