Merge pull request #2844 from dolthub/zachmu/skip-transaction-test

Correctly skip skipped assertions in transaction tests
dolthub · Feb 6, 2025 · c216e59 · c216e59
2 parents ace3b04 + cad92ac
commit c216e59
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/enginetest/evaluation.go b/enginetest/evaluation.go
@@ -301,6 +301,10 @@ func TestTransactionScriptWithEngine(t *testing.T, e QueryEngine, harness Harnes
 				t.Skip()
 			}
 
+			if assertion.Skip {
+				t.Skip()
+			}
+
 			if assertion.ExpectedErr != nil {
 				AssertErrWithCtx(t, e, harness, clientSession, assertion.Query, assertion.Bindings, assertion.ExpectedErr)
 			} else if assertion.ExpectedErrStr != "" {