From 3a8bfe5f486e02ffd8706748e201cdeac9028d75 Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Mon, 3 Mar 2025 07:49:57 -0600
Subject: [PATCH] Minor cleanup from code review (comments, fixed a few
 exceptions);

---
 .../qqq/backend/core/actions/tables/InsertAction.java          | 2 +-
 .../core/actions/values/SearchPossibleValueSourceAction.java   | 2 +-
 .../qqq/backend/core/instances/QInstanceEnricher.java          | 3 ++-
 .../metadata/tables/TablesCustomPossibleValueProvider.java     | 3 ++-
 qqq-backend-module-filesystem/pom.xml                          | 1 +
 5 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/tables/InsertAction.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/tables/InsertAction.java
index 846ed476..5ce2ce38 100644
--- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/tables/InsertAction.java
+++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/tables/InsertAction.java
@@ -114,7 +114,7 @@ public class InsertAction extends AbstractQActionFunction<InsertInput, InsertOut
 
       if(!StringUtils.hasContent(insertInput.getTableName()))
       {
-         throw (new QException("Table name was not specified in update input"));
+         throw (new QException("Table name was not specified in insert input"));
       }
 
       QTableMetaData table = insertInput.getTable();
diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/values/SearchPossibleValueSourceAction.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/values/SearchPossibleValueSourceAction.java
index 2063f39f..6416ff50 100644
--- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/values/SearchPossibleValueSourceAction.java
+++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/values/SearchPossibleValueSourceAction.java
@@ -435,7 +435,7 @@ public class SearchPossibleValueSourceAction
       {
          String message = "Error searching custom possible value source [" + input.getPossibleValueSourceName() + "]";
          LOG.warn(message, e);
-         throw (new QException(message));
+         throw (new QException(message, e));
       }
    }
 
diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/instances/QInstanceEnricher.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/instances/QInstanceEnricher.java
index a234bfe1..14dda867 100644
--- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/instances/QInstanceEnricher.java
+++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/instances/QInstanceEnricher.java
@@ -1484,7 +1484,8 @@ public class QInstanceEnricher
 
 
    /***************************************************************************
-    **
+    ** scan the classpath for classes in the specified package name which
+    ** implement the QInstanceEnricherPluginInterface - any found get added
     ***************************************************************************/
    public static void discoverAndAddPluginsInPackage(String packageName) throws QException
    {
diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/model/metadata/tables/TablesCustomPossibleValueProvider.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/model/metadata/tables/TablesCustomPossibleValueProvider.java
index df2b55a3..e72d188f 100644
--- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/model/metadata/tables/TablesCustomPossibleValueProvider.java
+++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/model/metadata/tables/TablesCustomPossibleValueProvider.java
@@ -37,7 +37,8 @@ import com.kingsrook.qqq.backend.core.utils.ValueUtils;
 
 
 /*******************************************************************************
- **
+ ** possible-value source provider for the `Tables` PVS - a list of all tables
+ ** in an application/qInstance.
  *******************************************************************************/
 public class TablesCustomPossibleValueProvider implements QCustomPossibleValueProvider<String>
 {
diff --git a/qqq-backend-module-filesystem/pom.xml b/qqq-backend-module-filesystem/pom.xml
index 9344db4a..c389a018 100644
--- a/qqq-backend-module-filesystem/pom.xml
+++ b/qqq-backend-module-filesystem/pom.xml
@@ -75,6 +75,7 @@
          <scope>test</scope>
       </dependency>
       <dependency>
+         <!-- this was added to help make testcontainers work -->
          <groupId>net.java.dev.jna</groupId>
          <artifactId>jna</artifactId>
          <version>5.7.0</version>