From c38b8ac59533a597fb9955140143a8bfa964cea8 Mon Sep 17 00:00:00 2001 From: Darin Kelkhoff Date: Wed, 28 Jun 2023 13:40:25 -0500 Subject: [PATCH] Fix test test and propagate exceptions more --- .../implementations/scripts/TestScriptProcessStep.java | 2 ++ .../implementations/scripts/TestScriptProcessStepTest.java | 3 +++ 2 files changed, 5 insertions(+) diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStep.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStep.java index 68d03f1f..b9a99eab 100644 --- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStep.java +++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStep.java @@ -145,6 +145,7 @@ public class TestScriptProcessStep implements BackendStep if(testScriptOutput.getException() != null) { output.addValue("exception", testScriptOutput.getException()); + output.setException(testScriptOutput.getException()); } } catch(Exception e) @@ -153,6 +154,7 @@ public class TestScriptProcessStep implements BackendStep // is this the kind of exception meant here? or is it more for one thrown by the script execution? or are those the same?? // /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// output.addValue("exception", e); + output.setException(e); } } diff --git a/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStepTest.java b/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStepTest.java index 25e1680a..448e57a9 100644 --- a/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStepTest.java +++ b/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/processes/implementations/scripts/TestScriptProcessStepTest.java @@ -71,11 +71,14 @@ class TestScriptProcessStepTest extends BaseTest insertInput.setRecords(List.of(new Script() .withName("TestScript") .withScriptTypeId(insertOutput.getRecords().get(0).getValueInteger("id")) + .withTableName(TestUtils.TABLE_NAME_SHAPE) .toQRecord())); insertOutput = new InsertAction().execute(insertInput); RunBackendStepInput input = new RunBackendStepInput(); input.addValue("scriptId", insertOutput.getRecords().get(0).getValueInteger("id")); + TestUtils.insertDefaultShapes(qInstance); + input.addValue("recordPrimaryKeyList", "1"); input.addValue("fileNames", new ArrayList<>(List.of("script.js"))); input.addValue("fileContents:script.js", "logger.log('oh my.')");