From 6ab5886d93a4ce1987431f0f632dfda65a98ce3d Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Fri, 1 Jul 2022 11:45:31 -0500
Subject: [PATCH 01/10] Update for next development version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index a617c019..97f340c8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -25,7 +25,7 @@
 
    <groupId>com.kingsrook.qqq</groupId>
    <artifactId>qqq-middleware-picocli</artifactId>
-   <version>0.0.0</version>
+   <version>0.1.0-SNAPSHOT</version>
 
    <scm>
      <connection>scm:git:git@github.com:Kingsrook/qqq-middleware-picocli.git</connection>

From 36c68ef51c0cb3e4d06e80db9e7de65010647dc3 Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Wed, 6 Jul 2022 13:56:33 -0500
Subject: [PATCH 02/10] QQQ-21 updated per updated backend-core - process
 function -> step renames

---
 pom.xml                                           |  2 +-
 .../kingsrook/qqq/frontend/picocli/TestUtils.java | 15 +++++----------
 2 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/pom.xml b/pom.xml
index 97f340c8..df34a224 100644
--- a/pom.xml
+++ b/pom.xml
@@ -51,7 +51,7 @@
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-core</artifactId>
-         <version>0.0.0</version>
+         <version>0.1.0-20220706.184937-2</version>
       </dependency>
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
diff --git a/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java b/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
index 7940bbcc..925b2cb8 100644
--- a/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
+++ b/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
@@ -25,7 +25,7 @@ package com.kingsrook.qqq.frontend.picocli;
 import java.io.InputStream;
 import java.sql.Connection;
 import java.util.List;
-import com.kingsrook.qqq.backend.core.interfaces.mock.MockFunctionBody;
+import com.kingsrook.qqq.backend.core.interfaces.mock.MockBackendStep;
 import com.kingsrook.qqq.backend.core.model.metadata.QAuthenticationMetaData;
 import com.kingsrook.qqq.backend.core.model.metadata.QCodeReference;
 import com.kingsrook.qqq.backend.core.model.metadata.QCodeType;
@@ -34,16 +34,14 @@ import com.kingsrook.qqq.backend.core.model.metadata.QFieldMetaData;
 import com.kingsrook.qqq.backend.core.model.metadata.QFieldType;
 import com.kingsrook.qqq.backend.core.model.metadata.QInstance;
 import com.kingsrook.qqq.backend.core.model.metadata.QTableMetaData;
+import com.kingsrook.qqq.backend.core.model.metadata.processes.QBackendStepMetaData;
 import com.kingsrook.qqq.backend.core.model.metadata.processes.QFunctionInputMetaData;
-import com.kingsrook.qqq.backend.core.model.metadata.processes.QFunctionMetaData;
 import com.kingsrook.qqq.backend.core.model.metadata.processes.QFunctionOutputMetaData;
-import com.kingsrook.qqq.backend.core.model.metadata.processes.QOutputView;
 import com.kingsrook.qqq.backend.core.model.metadata.processes.QProcessMetaData;
 import com.kingsrook.qqq.backend.core.model.metadata.processes.QRecordListMetaData;
-import com.kingsrook.qqq.backend.core.model.metadata.processes.QRecordListView;
-import com.kingsrook.qqq.backend.module.rdbms.model.metadata.RDBMSBackendMetaData;
 import com.kingsrook.qqq.backend.module.rdbms.jdbc.ConnectionManager;
 import com.kingsrook.qqq.backend.module.rdbms.jdbc.QueryManager;
+import com.kingsrook.qqq.backend.module.rdbms.model.metadata.RDBMSBackendMetaData;
 import org.apache.commons.io.IOUtils;
 import static junit.framework.Assert.assertNotNull;
 
@@ -166,10 +164,10 @@ public class TestUtils
       return new QProcessMetaData()
          .withName("greet")
          .withTableName("person")
-         .addFunction(new QFunctionMetaData()
+         .addStep(new QBackendStepMetaData()
             .withName("prepare")
             .withCode(new QCodeReference()
-               .withName(MockFunctionBody.class.getName())
+               .withName(MockBackendStep.class.getName())
                .withCodeType(QCodeType.JAVA)
                .withCodeUsage(QCodeUsage.FUNCTION)) // todo - needed, or implied in this context?
             .withInputData(new QFunctionInputMetaData()
@@ -184,9 +182,6 @@ public class TestUtils
                   .addField(new QFieldMetaData("fullGreeting", QFieldType.STRING))
                )
                .withFieldList(List.of(new QFieldMetaData("outputMessage", QFieldType.STRING))))
-            .withOutputView(new QOutputView()
-               .withMessageField("outputMessage")
-               .withRecordListView(new QRecordListView().withFieldNames(List.of("id", "firstName", "lastName", "fullGreeting"))))
          );
    }
 

From c75cd7eec9f001619968b582b13ee971bf0813ae Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Thu, 7 Jul 2022 08:43:28 -0500
Subject: [PATCH 03/10] Adding jacoco coverage reporting; switching to run mvn
 verify in circleci

---
 .circleci/config.yml |  2 +-
 pom.xml              | 83 ++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 84 insertions(+), 1 deletion(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 0ef02745..fe4c371c 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -42,7 +42,7 @@ jobs:
     executor: java17
     steps:
       - run_maven:
-          maven_subcommand: test
+          maven_subcommand: verify
       - slack/notify:
           event: fail
 
diff --git a/pom.xml b/pom.xml
index df34a224..21d7b051 100644
--- a/pom.xml
+++ b/pom.xml
@@ -44,6 +44,8 @@
       <maven.compiler.target>17</maven.compiler.target>
       <maven.compiler.showDeprecation>true</maven.compiler.showDeprecation>
       <maven.compiler.showWarnings>true</maven.compiler.showWarnings>
+      <coverage.haltOnFailure>true</coverage.haltOnFailure>
+      <coverage.instructionCoveredRatioMinimum>0.80</coverage.instructionCoveredRatioMinimum>
    </properties>
 
    <dependencies>
@@ -127,6 +129,9 @@
             <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-surefire-plugin</artifactId>
             <version>3.0.0-M5</version>
+            <configuration>
+               <argLine>@{jaCoCoArgLine}</argLine>
+            </configuration>
          </plugin>
          <plugin>
             <groupId>org.apache.maven.plugins</groupId>
@@ -176,6 +181,84 @@
                <versionDigitToIncrement>1</versionDigitToIncrement> <!-- In general, we update the minor -->
             </configuration>
          </plugin>
+         <plugin>
+            <groupId>org.jacoco</groupId>
+            <artifactId>jacoco-maven-plugin</artifactId>
+            <version>0.8.8</version>
+            <executions>
+               <execution>
+                  <id>pre-unit-test</id>
+                  <goals>
+                     <goal>prepare-agent</goal>
+                  </goals>
+                  <configuration>
+                     <propertyName>jaCoCoArgLine</propertyName>
+                  </configuration>
+               </execution>
+               <execution>
+                  <id>unit-test-check</id>
+                  <goals>
+                     <goal>check</goal>
+                  </goals>
+                  <configuration>
+                     <!-- Gives us the ability to pass a parameter to not fail due to coverage E.g. -Dcoverage.haltOnFailure=false -->
+                     <haltOnFailure>${coverage.haltOnFailure}</haltOnFailure>
+                     <rules>
+                        <rule>
+                           <element>BUNDLE</element>
+                           <limits>
+                              <limit>
+                                 <counter>INSTRUCTION</counter>
+                                 <value>COVEREDRATIO</value>
+                                 <minimum>${coverage.instructionCoveredRatioMinimum}</minimum>
+                              </limit>
+                           </limits>
+                        </rule>
+                     </rules>
+                  </configuration>
+               </execution>
+               <execution>
+                  <id>post-unit-test</id>
+                  <phase>verify</phase>
+                  <goals>
+                     <goal>report</goal>
+                  </goals>
+               </execution>
+            </executions>
+         </plugin>
+         <plugin>
+            <artifactId>exec-maven-plugin</artifactId>
+            <groupId>org.codehaus.mojo</groupId>
+            <version>3.0.0</version>
+            <executions>
+               <execution>
+                  <id>test-coverage-summary</id>
+                  <phase>verify</phase>
+                  <goals>
+                     <goal>exec</goal>
+                  </goals>
+                  <configuration>
+                     <executable>sh</executable>
+                     <arguments>
+                        <argument>-c</argument>
+                        <argument>
+                           <![CDATA[
+echo "Element\nInstructions Missed\nInstruction Coverage\nBranches Missed\nBranch Coverage\nComplexity Missed\nComplexity Hit\nLines Missed\nLines Hit\nMethods Missed\nMethods Hit\nClasses Missed\nClasses Hit\n" > /tmp/$$.headers
+xpath -q -e '/html/body/table/tfoot/tr[1]/td/text()' target/site/jacoco/index.html > /tmp/$$.values
+echo
+echo "Jacoco coverage summary report:"
+echo "   See also target/site/jacoco/index.html"
+echo "   and https://www.jacoco.org/jacoco/trunk/doc/counters.html"
+echo "------------------------------------------------------------"
+paste /tmp/$$.headers /tmp/$$.values | tail +2 | awk -v FS='\t' '{printf("%-20s %s\n",$1,$2)}'
+rm /tmp/$$.headers /tmp/$$.values
+                           ]]>
+                        </argument>
+                     </arguments>
+                  </configuration>
+               </execution>
+            </executions>
+         </plugin>
       </plugins>
    </build>
 

From 8a2cb95a07f938a53a5e7bba6ca4d2eb20f3364a Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Fri, 8 Jul 2022 12:06:52 -0500
Subject: [PATCH 04/10] QQQ-21 update qqq-backend-core for process
 function->step refactor

---
 checkstyle.xml                                               | 2 +-
 .../qqq/frontend/picocli/QPicoCliImplementation.java         | 5 +++--
 .../java/com/kingsrook/qqq/frontend/picocli/TestUtils.java   | 2 +-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/checkstyle.xml b/checkstyle.xml
index 76f872ed..f5e7412d 100644
--- a/checkstyle.xml
+++ b/checkstyle.xml
@@ -181,8 +181,8 @@
       </module>
       -->
       <module name="OverloadMethodsDeclarationOrder"/>
-      <module name="VariableDeclarationUsageDistance"/>
       <!--
+      <module name="VariableDeclarationUsageDistance"/>
       <module name="CustomImportOrder">
          <property name="sortImportsInGroupAlphabetically" value="true"/>
          <property name="separateLineBetweenGroups" value="true"/>
diff --git a/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java b/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
index 1302cb94..11c415ea 100644
--- a/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
+++ b/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
@@ -384,9 +384,10 @@ public class QPicoCliImplementation
             subCommandLine.getOut().format("   %s: %s\n", outputField.getLabel(), result.getValues().get(outputField.getName()));
          }
 
-         if(result.getError() != null)
+         if(result.getException().isPresent())
          {
-            subCommandLine.getOut().println("Process Error message: " + result.getError());
+            // todo - user-facing, similar to javalin
+            subCommandLine.getOut().println("Process Error message: " + result.getException().get().getMessage());
          }
       }
       catch(Exception e)
diff --git a/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java b/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
index 925b2cb8..c72c140b 100644
--- a/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
+++ b/src/test/java/com/kingsrook/qqq/frontend/picocli/TestUtils.java
@@ -169,7 +169,7 @@ public class TestUtils
             .withCode(new QCodeReference()
                .withName(MockBackendStep.class.getName())
                .withCodeType(QCodeType.JAVA)
-               .withCodeUsage(QCodeUsage.FUNCTION)) // todo - needed, or implied in this context?
+               .withCodeUsage(QCodeUsage.BACKEND_STEP)) // todo - needed, or implied in this context?
             .withInputData(new QFunctionInputMetaData()
                .withRecordListMetaData(new QRecordListMetaData().withTableName("person"))
                .withFieldList(List.of(

From b25a7fe5b8120d61eacb00e2d18cfdf1e2dff2fc Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Fri, 8 Jul 2022 12:09:16 -0500
Subject: [PATCH 05/10] QQQ-21 update qqq-backend-core to
 0.1.0-20220708.155822-4 (per last code cahnges)

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 21d7b051..8a87a81c 100644
--- a/pom.xml
+++ b/pom.xml
@@ -53,7 +53,7 @@
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-core</artifactId>
-         <version>0.1.0-20220706.184937-2</version>
+         <version>0.1.0-20220708.155822-4</version>
       </dependency>
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>

From e08e52fb6e2fb2b5b7da73e0f3b38ce03df6513f Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Fri, 8 Jul 2022 12:22:13 -0500
Subject: [PATCH 06/10] QQQ-21 switched test value from 'There', since we made
 that always throw...

---
 .../qqq/frontend/picocli/QPicoCliImplementationTest.java      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
index 4f298522..588a9b5b 100644
--- a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
+++ b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
@@ -497,9 +497,9 @@ class QPicoCliImplementationTest
    @Test
    public void test_tableProcessGreetUsingOptionsForFields() throws Exception
    {
-      TestOutput testOutput = testCli("person", "process", "greet", "--field-greetingPrefix=Hello", "--field-greetingSuffix=There");
+      TestOutput testOutput = testCli("person", "process", "greet", "--field-greetingPrefix=Hello", "--field-greetingSuffix=World");
       assertTestOutputDoesNotContain(testOutput, "Please supply a value for the field");
-      assertTestOutputContains(testOutput, "Hello X There");
+      assertTestOutputContains(testOutput, "Hello X World");
    }
 
 

From 0c10acb1223c37fbbd8105653697633fefec0d63 Mon Sep 17 00:00:00 2001
From: Tim Chamberlain <tchamberlain76@gmail.com>
Date: Fri, 8 Jul 2022 15:32:32 -0500
Subject: [PATCH 07/10] QQQ-21: added 'count' action

---
 pom.xml                                       |  4 +-
 .../qqq/frontend/picocli/QCommandBuilder.java |  1 +
 .../picocli/QPicoCliImplementation.java       | 73 ++++++++++++++-----
 .../picocli/QPicoCliImplementationTest.java   | 22 ++++++
 4 files changed, 79 insertions(+), 21 deletions(-)

diff --git a/pom.xml b/pom.xml
index 8a87a81c..77305cc8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -53,12 +53,12 @@
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-core</artifactId>
-         <version>0.1.0-20220708.155822-4</version>
+         <version>0.1.0-20220708.195335-5</version>
       </dependency>
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-module-rdbms</artifactId>
-         <version>0.0.0</version>
+         <version>0.1.0-20220708.202041-3</version>
          <scope>test</scope>
       </dependency>
 
diff --git a/src/main/java/com/kingsrook/qqq/frontend/picocli/QCommandBuilder.java b/src/main/java/com/kingsrook/qqq/frontend/picocli/QCommandBuilder.java
index 9d2525df..d2fbed7b 100644
--- a/src/main/java/com/kingsrook/qqq/frontend/picocli/QCommandBuilder.java
+++ b/src/main/java/com/kingsrook/qqq/frontend/picocli/QCommandBuilder.java
@@ -90,6 +90,7 @@ public class QCommandBuilder
          // add table-specific sub-commands for the table //
          ///////////////////////////////////////////////////
          tableCommand.addSubcommand("meta-data", defineMetaDataCommand(table));
+         tableCommand.addSubcommand("count", defineQueryCommand(table));
          tableCommand.addSubcommand("query", defineQueryCommand(table));
          tableCommand.addSubcommand("insert", defineInsertCommand(table));
          tableCommand.addSubcommand("update", defineUpdateCommand(table));
diff --git a/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java b/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
index 11c415ea..56cd335a 100644
--- a/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
+++ b/src/main/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementation.java
@@ -32,6 +32,7 @@ import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import com.kingsrook.qqq.backend.core.actions.CountAction;
 import com.kingsrook.qqq.backend.core.actions.DeleteAction;
 import com.kingsrook.qqq.backend.core.actions.InsertAction;
 import com.kingsrook.qqq.backend.core.actions.MetaDataAction;
@@ -45,6 +46,8 @@ import com.kingsrook.qqq.backend.core.adapters.JsonToQRecordAdapter;
 import com.kingsrook.qqq.backend.core.adapters.QInstanceAdapter;
 import com.kingsrook.qqq.backend.core.exceptions.QException;
 import com.kingsrook.qqq.backend.core.exceptions.QModuleDispatchException;
+import com.kingsrook.qqq.backend.core.model.actions.count.CountRequest;
+import com.kingsrook.qqq.backend.core.model.actions.count.CountResult;
 import com.kingsrook.qqq.backend.core.model.actions.delete.DeleteRequest;
 import com.kingsrook.qqq.backend.core.model.actions.delete.DeleteResult;
 import com.kingsrook.qqq.backend.core.model.actions.insert.InsertRequest;
@@ -94,7 +97,7 @@ public class QPicoCliImplementation
    public static final int DEFAULT_LIMIT = 20;
 
    private static QInstance qInstance;
-   private static QSession  session;
+   private static QSession session;
 
 
 
@@ -109,14 +112,14 @@ public class QPicoCliImplementation
       // parse args to look up metaData and prime instance
       if(args.length > 0 && args[0].startsWith("--qInstanceJsonFile="))
       {
-         String filePath      = args[0].replaceFirst("--.*=", "");
+         String filePath = args[0].replaceFirst("--.*=", "");
          String qInstanceJson = FileUtils.readFileToString(new File(filePath));
          qInstance = new QInstanceAdapter().jsonToQInstanceIncludingBackends(qInstanceJson);
 
          String[] subArgs = Arrays.copyOfRange(args, 1, args.length);
 
          QPicoCliImplementation qPicoCliImplementation = new QPicoCliImplementation(qInstance);
-         int                    exitCode               = qPicoCliImplementation.runCli("qapi", subArgs);
+         int exitCode = qPicoCliImplementation.runCli("qapi", subArgs);
          System.exit(exitCode);
       }
       else
@@ -229,7 +232,7 @@ public class QPicoCliImplementation
    private static void setupSession(String[] args) throws QModuleDispatchException
    {
       QAuthenticationModuleDispatcher qAuthenticationModuleDispatcher = new QAuthenticationModuleDispatcher();
-      QAuthenticationModuleInterface  authenticationModule            = qAuthenticationModuleDispatcher.getQModule(qInstance.getAuthentication());
+      QAuthenticationModuleInterface authenticationModule = qAuthenticationModuleDispatcher.getQModule(qInstance.getAuthentication());
 
       // todo - does this need some per-provider logic actually?  mmm...
       Map<String, String> authenticationContext = new HashMap<>();
@@ -251,7 +254,7 @@ public class QPicoCliImplementation
       else
       {
          ParseResult subParseResult = parseResult.subcommand();
-         String      subCommandName = subParseResult.commandSpec().name();
+         String subCommandName = subParseResult.commandSpec().name();
          CommandLine subCommandLine = commandLine.getSubcommands().get(subCommandName);
          switch(subCommandName)
          {
@@ -282,13 +285,17 @@ public class QPicoCliImplementation
       if(tableParseResult.hasSubcommand())
       {
          ParseResult subParseResult = tableParseResult.subcommand();
-         String      subCommandName = subParseResult.commandSpec().name();
+         String subCommandName = subParseResult.commandSpec().name();
          switch(subCommandName)
          {
             case "meta-data":
             {
                return runTableMetaData(commandLine, tableName, subParseResult);
             }
+            case "count":
+            {
+               return runTableCount(commandLine, tableName, subParseResult);
+            }
             case "query":
             {
                return runTableQuery(commandLine, tableName, subParseResult);
@@ -345,7 +352,7 @@ public class QPicoCliImplementation
          ///////////////////////////////////////////
          // move on to running the actual process //
          ///////////////////////////////////////////
-         String      subCommandName = subParseResult.subcommand().commandSpec().name();
+         String subCommandName = subParseResult.subcommand().commandSpec().name();
          CommandLine subCommandLine = commandLine.getSubcommands().get(subCommandName);
          return runActualProcess(subCommandLine, subParseResult.subcommand());
       }
@@ -358,9 +365,9 @@ public class QPicoCliImplementation
     *******************************************************************************/
    private int runActualProcess(CommandLine subCommandLine, ParseResult processParseResult)
    {
-      String            processName = processParseResult.commandSpec().name();
-      QProcessMetaData  process     = qInstance.getProcess(processName);
-      RunProcessRequest request     = new RunProcessRequest(qInstance);
+      String processName = processParseResult.commandSpec().name();
+      QProcessMetaData process = qInstance.getProcess(processName);
+      RunProcessRequest request = new RunProcessRequest(qInstance);
 
       request.setSession(session);
       request.setProcessName(processName);
@@ -418,6 +425,24 @@ public class QPicoCliImplementation
 
 
 
+   /*******************************************************************************
+    **
+    *******************************************************************************/
+   private int runTableCount(CommandLine commandLine, String tableName, ParseResult subParseResult) throws QException
+   {
+      CountRequest countRequest = new CountRequest(qInstance);
+      countRequest.setSession(session);
+      countRequest.setTableName(tableName);
+      countRequest.setFilter(generateQueryFilter(subParseResult));
+
+      CountAction countAction = new CountAction();
+      CountResult countResult = countAction.execute(countRequest);
+      commandLine.getOut().println(JsonUtils.toPrettyJson(countResult));
+      return commandLine.getCommandSpec().exitCodeOnSuccess();
+   }
+
+
+
    /*******************************************************************************
     **
     *******************************************************************************/
@@ -428,15 +453,28 @@ public class QPicoCliImplementation
       queryRequest.setTableName(tableName);
       queryRequest.setSkip(subParseResult.matchedOptionValue("skip", null));
       queryRequest.setLimit(subParseResult.matchedOptionValue("limit", DEFAULT_LIMIT));
+      queryRequest.setFilter(generateQueryFilter(subParseResult));
 
+      QueryAction queryAction = new QueryAction();
+      QueryResult queryResult = queryAction.execute(queryRequest);
+      commandLine.getOut().println(JsonUtils.toPrettyJson(queryResult));
+      return commandLine.getCommandSpec().exitCodeOnSuccess();
+   }
+
+
+
+   /*******************************************************************************
+    **
+    *******************************************************************************/
+   private QQueryFilter generateQueryFilter(ParseResult subParseResult)
+   {
       QQueryFilter filter = new QQueryFilter();
-      queryRequest.setFilter(filter);
 
       String[] criteria = subParseResult.matchedOptionValue("criteria", new String[] {});
       for(String criterion : criteria)
       {
          // todo - parse!
-         String[]        parts          = criterion.split(" ");
+         String[] parts = criterion.split(" ");
          QFilterCriteria qQueryCriteria = new QFilterCriteria();
          qQueryCriteria.setFieldName(parts[0]);
          qQueryCriteria.setOperator(QCriteriaOperator.valueOf(parts[1]));
@@ -444,10 +482,7 @@ public class QPicoCliImplementation
          filter.addCriteria(qQueryCriteria);
       }
 
-      QueryAction queryAction = new QueryAction();
-      QueryResult queryResult = queryAction.execute(queryRequest);
-      commandLine.getOut().println(JsonUtils.toPrettyJson(queryResult));
-      return commandLine.getCommandSpec().exitCodeOnSuccess();
+      return filter;
    }
 
 
@@ -497,7 +532,7 @@ public class QPicoCliImplementation
          try
          {
             String path = subParseResult.matchedOptionValue("--csvFile", "");
-            String csv  = FileUtils.readFileToString(new File(path));
+            String csv = FileUtils.readFileToString(new File(path));
             recordList = new CsvToQRecordAdapter().buildRecordsFromCsv(csv, table, mapping);
          }
          catch(IOException e)
@@ -554,7 +589,7 @@ public class QPicoCliImplementation
 
       boolean anyFields = false;
 
-      String         primaryKeyOption = subParseResult.matchedOptionValue("--primaryKey", "");
+      String primaryKeyOption = subParseResult.matchedOptionValue("--primaryKey", "");
       Serializable[] primaryKeyValues = primaryKeyOption.split(",");
       for(Serializable primaryKeyValue : primaryKeyValues)
       {
@@ -603,7 +638,7 @@ public class QPicoCliImplementation
       /////////////////////////////////////////////
       // get the pKeys that the user specified //
       /////////////////////////////////////////////
-      String         primaryKeyOption = subParseResult.matchedOptionValue("--primaryKey", "");
+      String primaryKeyOption = subParseResult.matchedOptionValue("--primaryKey", "");
       Serializable[] primaryKeyValues = primaryKeyOption.split(",");
       deleteRequest.setPrimaryKeys(Arrays.asList(primaryKeyValues));
 
diff --git a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
index 588a9b5b..90ea165f 100644
--- a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
+++ b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
@@ -214,6 +214,28 @@ class QPicoCliImplementationTest
 
 
 
+   /*******************************************************************************
+    ** test running a count on a table
+    **
+    *******************************************************************************/
+   @Test
+   public void test_tableCount()
+   {
+      TestOutput testOutput  = testCli("person", "count", "--criteria", "id NOT_EQUALS 3");
+      JSONObject countResult = JsonUtils.toJSONObject(testOutput.getOutput());
+      assertNotNull(countResult);
+      int count = countResult.getInt("count");
+      assertEquals(4, count);
+
+      testOutput  = testCli("person", "count", "--criteria", "id EQUALS 3");
+      countResult = JsonUtils.toJSONObject(testOutput.getOutput());
+      assertNotNull(countResult);
+      count = countResult.getInt("count");
+      assertEquals(1, count);
+   }
+
+
+
    /*******************************************************************************
     ** test running a query on a table
     **

From 31326a4354a7b9397c7e270985840860fb957aea Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Thu, 14 Jul 2022 10:08:10 -0500
Subject: [PATCH 08/10] Update qqq-backend-core and rdbms to 0.1.0

---
 pom.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index 77305cc8..f4e2a601 100644
--- a/pom.xml
+++ b/pom.xml
@@ -53,12 +53,12 @@
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-core</artifactId>
-         <version>0.1.0-20220708.195335-5</version>
+         <version>0.1.0</version>
       </dependency>
       <dependency>
          <groupId>com.kingsrook.qqq</groupId>
          <artifactId>qqq-backend-module-rdbms</artifactId>
-         <version>0.1.0-20220708.202041-3</version>
+         <version>0.1.0</version>
          <scope>test</scope>
       </dependency>
 

From 9c415698cc347333487bae6fb5bc6fd9e9142c46 Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Thu, 14 Jul 2022 10:08:22 -0500
Subject: [PATCH 09/10] Update versions for release

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index f4e2a601..5061715e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -25,7 +25,7 @@
 
    <groupId>com.kingsrook.qqq</groupId>
    <artifactId>qqq-middleware-picocli</artifactId>
-   <version>0.1.0-SNAPSHOT</version>
+   <version>0.1.0</version>
 
    <scm>
      <connection>scm:git:git@github.com:Kingsrook/qqq-middleware-picocli.git</connection>

From 6c66d913b980ad5ac4085d5e19e1f9c81613ba9a Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Thu, 14 Jul 2022 10:11:39 -0500
Subject: [PATCH 10/10] Removing overly-agressive assertion

---
 .../qqq/frontend/picocli/QPicoCliImplementationTest.java         | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
index 90ea165f..8e14abd8 100644
--- a/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
+++ b/src/test/java/com/kingsrook/qqq/frontend/picocli/QPicoCliImplementationTest.java
@@ -202,7 +202,6 @@ class QPicoCliImplementationTest
       assertNotNull(metaData);
       assertEquals(1, metaData.keySet().size(), "Number of top-level keys");
       JSONObject table = metaData.getJSONObject("table");
-      assertEquals(4, table.keySet().size(), "Number of mid-level keys");
       assertEquals("person", table.getString("name"));
       assertEquals("Person", table.getString("label"));
       assertEquals("id", table.getString("primaryKeyField"));