Add CLI workflow tests

Author: denyshorman

.github/workflows/publish-release.yml

@@ -7,14 +7,8 @@ on:
       - release
 
 jobs:
-  test:
-    name: Test
-    permissions:
-      contents: read
-    uses: ./.github/workflows/func-test.yml
   build-jar:
     name: Build Jar
-    needs: test
     permissions:
       contents: read
     uses: ./.github/workflows/func-build-jar.yml

README.md

@@ -68,14 +68,20 @@ You can also run directly via Gradle during development:
 
 ## Quick Start
 
-All state is stored under `~/.jdbcli/`:
+All state is stored under `~/.jdbcli/` by default.
 
 | Path | Contents |
 |---|---|
 | `~/.jdbcli/dbs/` | Database descriptors (JSON) |
 | `~/.jdbcli/drivers/` | Downloaded JDBC driver JARs |
 | `~/.jdbcli/profiles/` | Connection profiles (JSON) |
 
+Override with `-Djdbcli.home=<path>`:
+
+```bash
+java -Djdbcli.home=/opt/jdbcli -jar jdbcli.jar db list
+```
+
 ### 1. Browse built-in database descriptors
 
 ```bash

src/main/java/io/github/denyshorman/jdbcli/cli/DbCommand.java

@@ -2,6 +2,7 @@
 
 import io.github.denyshorman.jdbcli.config.DbDescriptor;
 import io.github.denyshorman.jdbcli.descriptor.DbDescriptorLoader;
+import io.github.denyshorman.jdbcli.exception.JdbcliException;
 import io.github.denyshorman.jdbcli.util.FileSystemUtil;
 import io.github.denyshorman.jdbcli.util.JsonUtil;
 import org.slf4j.Logger;
@@ -56,11 +57,11 @@ public void list() {
     public void show(@Parameters(index = "0", paramLabel = "<id>", description = "Descriptor ID (e.g. postgres, sqlserver, oracle, mongodb)") String dbName) {
         var db = DbDescriptorLoader.load(dbName);
 
-        if (db != null) {
-            System.out.println(JsonUtil.MAPPER.writeValueAsString(db));
-        } else {
-            LOGGER.error("Database not found: {}", dbName);
+        if (db == null) {
+            throw new JdbcliException("Database not found: " + dbName);
         }
+
+        System.out.println(JsonUtil.MAPPER.writeValueAsString(db));
     }
 
     @Command(

src/main/java/io/github/denyshorman/jdbcli/cli/MainCommand.java

@@ -18,6 +18,7 @@
                 "  3. jdbcli profile init --db mydb --name dev   # create a connection profile",
                 "  4. jdbcli query --profile dev --sql \"SELECT 1\"  # run a query",
                 "",
+                "Set -Djdbcli.home=<path> to use a custom home directory instead of ~/.jdbcli.",
                 "Set JDBCLI_DEBUG=1 to print full stack traces on error.",
         },
         subcommands = {

src/main/java/io/github/denyshorman/jdbcli/cli/ProfileCommand.java

@@ -4,21 +4,19 @@
 import io.github.denyshorman.jdbcli.config.Profile;
 import io.github.denyshorman.jdbcli.config.SafetyConfig;
 import io.github.denyshorman.jdbcli.descriptor.DbDescriptorLoader;
+import io.github.denyshorman.jdbcli.driver.DriverInstaller;
 import io.github.denyshorman.jdbcli.exception.JdbcliException;
 import io.github.denyshorman.jdbcli.profile.ProfileLoader;
 import io.github.denyshorman.jdbcli.profile.ProfileWriter;
 import io.github.denyshorman.jdbcli.util.FileSystemUtil;
 import io.github.denyshorman.jdbcli.util.JsonUtil;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import picocli.CommandLine.Command;
 import picocli.CommandLine.Option;
 
 import java.util.Map;
 
-import io.github.denyshorman.jdbcli.driver.DriverInstaller;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 @Command(
         name = "profile",
         description = {
@@ -103,10 +101,10 @@ public void list() {
     public void show(@Option(names = "--profile", required = true, paramLabel = "<name>", description = "Profile name to display") String name) {
         var p = ProfileLoader.load(name);
 
-        if (p != null) {
-            System.out.println(JsonUtil.MAPPER.writeValueAsString(p));
-        } else {
-            LOGGER.error("Profile not found: {}", name);
+        if (p == null) {
+            throw new JdbcliException("Profile not found: " + name);
         }
+
+        System.out.println(JsonUtil.MAPPER.writeValueAsString(p));
     }
 }

src/main/java/io/github/denyshorman/jdbcli/util/FileSystemUtil.java

@@ -4,6 +4,12 @@
 
 public class FileSystemUtil {
     public static Path getJdbcliHome() {
+        var override = System.getProperty("jdbcli.home");
+
+        if (override != null) {
+            return Path.of(override);
+        }
+
         return Path.of(System.getProperty("user.home"), ".jdbcli");
     }
 }