feat: Support for Environment to flyway service

doc/src/asciidoc/module_dbflyway.adoc

@@ -38,7 +38,7 @@ The migrate command accepts an optional --out-of-order parameter.
 
 In addition to CLI support, there's also a `FlywayService` that can be
 configured as a QBean. We recommend to use a low filename (such as
-`01_flyway.xml` so that the service starts before other services that may
+`01_flyway.xml`) so that the service starts before other services that may
 require the schema to be impacted in the database.
 
 The QBean descriptor looks like this:

modules/db-flyway/src/main/java/org/jpos/flyway/FlywayService.java

@@ -24,7 +24,7 @@
 import org.flywaydb.core.api.MigrationVersion;
 import org.flywaydb.core.internal.info.MigrationInfoDumper;
 import org.jdom2.Element;
-import org.jpos.core.ConfigurationException;
+import org.jpos.core.Environment;
 import org.jpos.core.XmlConfigurable;
 import org.jpos.q2.QBeanSupport;
 
@@ -77,9 +77,7 @@ protected void initService() {
     }
 
     @Override
-    public void setConfiguration(Element e) throws ConfigurationException {
-        commands = e.getChildText("commands");
-        if (commands == null)
-            commands = "info";
+    public void setConfiguration(Element e) {
+        commands = Environment.get(e.getChildText("commands"), "info");
     }
 }

modules/db-flyway/src/test/java/org/jpos/flyway/FlywayServiceTest.java

@@ -0,0 +1,45 @@
+package org.jpos.flyway;
+
+import org.jdom2.Element;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+/**
+ * @author Arturo Volpe
+ * @since 2022-05-12
+ */
+class FlywayServiceTest {
+
+    @Test
+    void test_empty_config() throws Exception {
+
+        FlywayService flywayService = new FlywayService();
+        flywayService.setConfiguration(new Element("flyway"));
+
+        assertEquals("info", flywayService.commands);
+    }
+
+    @Test
+    void test_static_config() throws Exception {
+
+        FlywayService flywayService = new FlywayService();
+        flywayService.setConfiguration(new Element("flyway")
+                .addContent(new Element("commands").addContent("info"))
+        );
+
+        assertEquals("info", flywayService.commands);
+    }
+
+    @Test
+    void test_env_config() throws Exception {
+
+        System.setProperty("flyway_commands", "info migrate");
+        FlywayService flywayService = new FlywayService();
+        flywayService.setConfiguration(new Element("flyway")
+                .addContent(new Element("commands").addContent("${flyway_commands}"))
+        );
+
+        assertEquals("info migrate", flywayService.commands);
+    }
+}