fix(webUI): fixed the ability to specify function's return value before performing a command

Closes: #65

Author: nergal-perm

src/main/java/ru/ewc/checklogic/FileStateFactory.java

@@ -71,6 +71,12 @@ public State initialState() {
         return new State(this.locators);
     }
 
+    @Override
+    public boolean functionSpecified(final String arg) {
+        return ((FunctionsLocator) new State(this.locators)
+            .locatorFor(this.config.functionsLocatorName())).functionSpecified(arg);
+    }
+
     private void loadFunctionsLocator() {
         this.locators.add(
             new FunctionsLocator(

src/main/java/ru/ewc/checklogic/MockStateFactory.java

@@ -44,4 +44,9 @@ public State initialState() {
         );
     }
 
+    @Override
+    public boolean functionSpecified(final String arg) {
+        return false;
+    }
+
 }

src/main/java/ru/ewc/checklogic/ServerInstance.java

@@ -28,6 +28,7 @@
 import java.nio.file.Paths;
 import java.util.List;
 import java.util.Map;
+import lombok.Getter;
 import ru.ewc.decisions.api.ComputationContext;
 import ru.ewc.decisions.api.DecisionTables;
 import ru.ewc.decisions.api.DecitaException;
@@ -45,6 +46,7 @@ public final class ServerInstance {
     /**
      * The root path for the external business logic resources.
      */
+    @Getter
     private final String root;
 
     /**
@@ -130,28 +132,20 @@ public boolean isAvailable(final String command, final String field) {
     }
 
     public void update(final List<String> values) {
-        final InMemoryLocator request = InMemoryLocator.empty(this.requestLocatorName());
+        final InMemoryLocator request = InMemoryLocator.empty(this.server.requestLocatorName());
         values.forEach(
             value -> {
                 final String[] split = value.split(":");
                 request.setFragmentValue(split[0].trim(), split[1].trim());
             });
-        this.state.locators().put(this.requestLocatorName(), request);
+        this.state.locators().put(this.server.requestLocatorName(), request);
         this.context = new ComputationContext(this.state, this.getAllTables());
     }
 
-    public String requestLocatorName() {
-        return this.server.requestLocatorName();
-    }
-
     public boolean isEmpty() {
         return this.state.locators().isEmpty();
     }
 
-    public String getRoot() {
-        return this.root;
-    }
-
     public void initialize() {
         this.state = this.states.initialState();
         this.context = new ComputationContext(this.state, this.getAllTables());
@@ -166,6 +160,13 @@ public void createTestFolder() {
         this.context = new ComputationContext(this.state, this.getAllTables());
     }
 
+    public boolean isNotSpecified(final String arg) {
+        final String[] args = arg.split("::");
+        final boolean function = this.server.functionsLocatorName().equals(args[0]);
+        final boolean request = this.server.requestLocatorName().equals(args[0]);
+        return function && !this.states.functionSpecified(args[1]) || request;
+    }
+
     private DecisionTables getAllTables() {
         return DecisionTables.using(new CombinedCsvFileReader(this.tables, ".csv", ";"));
     }

src/main/java/ru/ewc/checklogic/StateFactory.java

@@ -34,4 +34,6 @@
  */
 interface StateFactory {
     State initialState();
+
+    boolean functionSpecified(String arg);
 }

src/main/java/ru/ewc/checklogic/server/CommandMetadata.java

@@ -82,7 +82,7 @@ public String namesAsHtmlList(final ServerInstance computation, final String out
     public String commandArgsAsHtmlForm(final String command, final ServerInstance context) {
         return this.metadata.get(command).stream()
             .distinct()
-            .filter(arg -> context.requestLocatorName().equals(arg.split("::")[0]))
+            .filter(context::isNotSpecified)
             .map(
                 arg -> new StringBuilder()
                     .append("<div class='mb-3'>")

src/main/java/ru/ewc/checklogic/testing/FunctionsLocator.java

@@ -46,6 +46,11 @@ public final class FunctionsLocator implements Locator {
      */
     private final String name;
 
+    /**
+     * The path to the Groovy scripts.
+     */
+    private final Path path;
+
     /**
      * The in-memory locator used to store overridden values.
      */
@@ -59,6 +64,7 @@ public final class FunctionsLocator implements Locator {
     @SneakyThrows
     public FunctionsLocator(final String name, final Path path) {
         this.name = name;
+        this.path = path;
         this.locator = new InMemoryLocator("locator", new HashMap<>());
         this.engine = new GroovyScriptEngine(new URL[]{path.toUri().toURL()});
     }
@@ -72,10 +78,12 @@ public String fragmentBy(
         final String result;
         if (this.locator.state().containsKey(fragment)) {
             result = this.locator.fragmentBy(fragment, context);
-        } else {
+        } else if (this.hasScript(fragment)) {
             final Binding binding = new Binding();
             binding.setVariable("context", context);
             result = this.engine.run("%s.groovy".formatted(fragment), binding).toString();
+        } else {
+            result = "undefined";
         }
         return result;
     }
@@ -89,4 +97,16 @@ public void setFragmentValue(final String fragment, final String value) {
     public String locatorName() {
         return this.name;
     }
+
+    public boolean functionSpecified(final String arg) {
+        return this.hasValue(arg) || this.hasScript(arg);
+    }
+
+    private boolean hasScript(final String arg) {
+        return this.path.resolve("%s.groovy".formatted(arg)).toFile().exists();
+    }
+
+    private boolean hasValue(final String arg) {
+        return !"undefined".equals(this.locator.state().getOrDefault(arg, "undefined"));
+    }
 }