improve profile handling, killed warnings

Author: zambrovski

.github/workflows/development.yml

@@ -45,11 +45,6 @@ jobs:
       - name: Build with Maven
         run: ./mvnw clean verify -U -B -ntp -T4
 
-      # itest
-      - name: Run itest
-        run: ./mvnw -Pitest -ntp -U -B -T4
-
-
 #      - name: Upload coverage to Codecov
 #        if: github.event_name == 'push' && github.actor != 'dependabot[bot]'
 #        uses: codecov/codecov-action@v1.0.2

.github/workflows/master.yml

@@ -45,7 +45,7 @@ jobs:
 
       # Publish release
       - name: Deploy a new release version to Maven Central
-        run: ./mvnw clean deploy -B -ntp -DskipTests -DskipExamples -Prelease -Dgpg.keyname="${{ secrets.GPG_KEYNAME }}"
+        run: ./mvnw clean deploy -B -ntp -DskipTests -DskipITests -DskipExamples -Prelease -Dgpg.keyname="${{ secrets.GPG_KEYNAME }}"
         env:
           OSS_CENTRAL_USERNAME: ${{ secrets.SONATYPE_USERNAME }}
           OSS_CENTRAL_PASSWORD: ${{ secrets.SONATYPE_PASSWORD }}

pom.xml

@@ -105,6 +105,11 @@
     </profile>
 
     <profile>
+      <activation>
+        <property>
+          <name>!skipITests</name>
+        </property>
+      </activation>
       <id>itest</id>
       <modules>
         <module>itest</module>

spring-boot-idempotency-registry-jpa/src/main/kotlin/dev/bpmcrafters/processengine/worker/idempotency/MapConverter.kt

@@ -24,6 +24,7 @@ class MapConverter : AttributeConverter<Map<String, Any?>, ByteArray> {
     if (dbData == null) {
       return null
     }
+    @Suppress("UNCHECKED_CAST")
     return ObjectInputStream(ByteArrayInputStream(dbData)).readObject() as Map<String, Any?>
   }
 

spring-boot-starter/src/main/kotlin/dev/bpmcrafters/processengine/worker/registrar/ProcessEngineStarterRegistrar.kt

@@ -168,7 +168,7 @@ class ProcessEngineStarterRegistrar(
           }
           if (autoCompleteTask && !completeBeforeCommit) {
             logger.trace { "PROCESS-ENGINE-WORKER-016: auto completing task ${taskInformation.taskId} after commit" }
-            completeTask(taskInformation, txResult)
+            completeTask(taskInformation, requireNotNull(txResult))
             processEngineWorkerMetrics.taskCompleted(topic)
           }
         } else {

spring-boot-starter/src/test/kotlin/dev/bpmcrafters/processengine/worker/registrar/ProcessEngineIdempotencyTest.kt

@@ -47,6 +47,7 @@ class ProcessEngineIdempotencyTest {
     idempotencyRegistry
   )
 
+  @Suppress("UNCHECKED_CAST")
   @Test
   fun `should not invoke annotated method again for same task`() {
     // Given a worker method and a counting action