[SLT] Run one single slt test on every merge

mihaibudiu · mihaibudiu · commit 4384b20378c5 · 2025-09-05T18:15:15.000Z
Signed-off-by: Mihai Budiu &lt;mbudiu@feldera.com&gt;
diff --git a/.github/workflows/test-java.yml b/.github/workflows/test-java.yml
@@ -58,6 +58,11 @@ jobs:
         working-directory: ./sql-to-dbsp-compiler
         if: ${{ vars.CI_DRY_RUN != 'true' }}
 
+      - name: Run one quick SLT test
+        if: ${{ vars.CI_DRY_RUN != 'true' }}
+        run: mvn test --no-transfer-progress -q -B -Dsurefire.failIfNoSpecifiedTests=false -Dtest=RotateTests#quick
+        working-directory: ./sql-to-dbsp-compiler
+
       - name: Print sccache stats
         run: |
           sccache --show-stats
diff --git a/sql-to-dbsp-compiler/slt/src/main/java/org/dbsp/sqllogictest/Main.java b/sql-to-dbsp-compiler/slt/src/main/java/org/dbsp/sqllogictest/Main.java
@@ -84,6 +84,19 @@ public static void rotateTests() throws IOException, ClassNotFoundException {
         main(argv);
     }
 
+    /** Executes one SLT program, to make sure everything is fine */
+    public static void quick() throws IOException, ClassNotFoundException {
+        String[] args = new String[] { "-v", "-x", "-inc", "-e", "hybrid", "-skip", "1",
+                "test/random/expr/slt_good_102.test" };
+
+        String wd = System.getProperty("user.dir");
+        File directory = new File(wd + "/..").getAbsoluteFile();
+        System.setProperty("user.dir", directory.getAbsolutePath());
+        wd = System.getProperty("user.dir");
+        System.out.println("working directory is " + wd);
+        main(args);
+    }
+
     @SuppressWarnings("SpellCheckingInspection")
     public static void main(String[] argv) throws IOException, ClassNotFoundException {
         Class.forName("org.hsqldb.jdbcDriver");
diff --git a/sql-to-dbsp-compiler/slt/src/main/java/org/dbsp/sqllogictest/executors/DBSPExecutor.java b/sql-to-dbsp-compiler/slt/src/main/java/org/dbsp/sqllogictest/executors/DBSPExecutor.java
@@ -472,7 +472,7 @@ static ProgramAndTester createTesterCode(
         }
         DBSPLetStatement step =
                 new DBSPLetStatement("_",
-                        new DBSPApplyMethodExpression("step", DBSPTypeAny.getDefault(),
+                        new DBSPApplyMethodExpression("transaction", DBSPTypeAny.getDefault(),
                         cas.getVarReference().field(0)).resultUnwrap());
         list.add(step);
         DBSPLetStatement outputStatement =
@@ -550,6 +550,7 @@ static ProgramAndTester createTesterCode(
         return new ProgramAndTester(circuit, function);
     }
 
+    @SuppressWarnings("UnusedReturnValue")
     public boolean statement(SltSqlStatement statement) throws SQLException {
         this.options.message("Executing " + statement + "\n", 2);
         String command = statement.statement.toLowerCase();
diff --git a/sql-to-dbsp-compiler/slt/src/test/java/org/dbsp/sqllogictest/RotateTests.java b/sql-to-dbsp-compiler/slt/src/test/java/org/dbsp/sqllogictest/RotateTests.java
@@ -10,4 +10,9 @@ public class RotateTests {
     public void rotate() throws IOException, ClassNotFoundException {
         Main.rotateTests();
     }
+
+    @Test
+    public void quick() throws IOException, ClassNotFoundException {
+        Main.quick();
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -10,4 +10,9 @@ public class RotateTests {`
`10`	`10`	`public void rotate() throws IOException, ClassNotFoundException {`
`11`	`11`	`Main.rotateTests();`
`12`	`12`	`}`
	`13`	`+`
	`14`	`+ @Test`
	`15`	`+ public void quick() throws IOException, ClassNotFoundException {`
	`16`	`+ Main.quick();`
	`17`	`+ }`
`13`	`18`	`}`