comment

TouK · Oct 30, 2024 · 6a93a16 · 6a93a16
1 parent 6b79612
commit 6a93a16
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/designer/server/src/test/scala/pl/touk/nussknacker/test/base/it/NuResourcesTest.scala b/designer/server/src/test/scala/pl/touk/nussknacker/test/base/it/NuResourcesTest.scala
@@ -470,6 +470,9 @@ trait NuResourcesTest
         forwardedUserName = None
       )
     for {
+      // FIXME: Using method `runInSerializableTransactionWithRetry` is a workaround for problem with flaky tests
+      // (some tests failed with [java.sql.SQLTransactionRollbackException: transaction rollback: serialization failure])
+      // the underlying cause of that errors needs investigating
       _  <- dbioRunner.runInSerializableTransactionWithRetry(writeProcessRepository.saveNewProcess(action))
       id <- futureFetchingScenarioRepository.fetchProcessId(processName).map(_.get)
     } yield id

diff --git a/designer/server/src/test/scala/pl/touk/nussknacker/test/utils/domain/ScenarioHelper.scala b/designer/server/src/test/scala/pl/touk/nussknacker/test/utils/domain/ScenarioHelper.scala
@@ -190,6 +190,9 @@ private[test] class ScenarioHelper(dbRef: DbRef, clock: Clock, designerConfig: C
       forwardedUserName = None
     )
     for {
+      // FIXME: Using method `runInSerializableTransactionWithRetry` is a workaround for problem with flaky tests
+      // (some tests failed with [java.sql.SQLTransactionRollbackException: transaction rollback: serialization failure])
+      // the underlying cause of that errors needs investigating
       _  <- dbioRunner.runInSerializableTransactionWithRetry(writeScenarioRepository.saveNewProcess(action))
       id <- futureFetchingScenarioRepository.fetchProcessId(scenarioName).map(_.get)
     } yield id