Skip to content

Commit d18ce4c

Browse files
committed
[CockroachDB] Update tests
1 parent 2fe7b6b commit d18ce4c

5 files changed

Lines changed: 29 additions & 7 deletions

File tree

.github/workflows/main.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,9 @@ jobs:
144144
- name: Create SQLancer user
145145
run: cd cockroach-v24.1.0-rc.2.linux-amd64/ && ./cockroach sql --insecure -e "CREATE USER sqlancer; GRANT admin to sqlancer" && cd ..
146146
- name: Run Tests
147-
run: COCKROACHDB_AVAILABLE=true mvn -Dtest=TestCockroachDB test
147+
run: |
148+
COCKROACHDB_AVAILABLE=true mvn -Dtest=TestCockroachDBNoREC test
149+
COCKROACHDB_AVAILABLE=true mvn -Dtest=TestCockroachDBTLP test
148150
149151
cockroachdb-qpg:
150152
name: QPG Tests (CockroachDB)

src/sqlancer/cockroachdb/CockroachDBErrors.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,8 @@ public static List<String> getExpressionErrors() {
206206
errors.add("argument of OFFSET must be type int, not type decimal");
207207
errors.add("ERROR: for SELECT DISTINCT, ORDER BY expressions must appear in select list");
208208

209+
errors.add("incompatible IF expressions");
210+
209211
errors.addAll(getArrayErrors());
210212
errors.addAll(getComputedColumnErrors());
211213

test/sqlancer/dbms/TestCockroachDB.java renamed to test/sqlancer/dbms/TestCockroachDBNoREC.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,13 @@
77

88
import sqlancer.Main;
99

10-
public class TestCockroachDB {
10+
public class TestCockroachDBNoREC {
1111

1212
@Test
13-
public void testMySQL() {
14-
String cockroachDB = System.getenv("COCKROACHDB_AVAILABLE");
15-
boolean cockroachDBIsAvailable = cockroachDB != null && cockroachDB.equalsIgnoreCase("true");
16-
assumeTrue(cockroachDBIsAvailable);
13+
public void testCockroachDBNoREC() {
14+
assumeTrue(TestConfig.isEnvironmentTrue(TestConfig.COCKROACHDB_ENV));
1715
assertEquals(0, Main.executeMain(new String[] { "--random-seed", "0", "--timeout-seconds", TestConfig.SECONDS,
18-
"--num-queries", TestConfig.NUM_QUERIES, "cockroachdb" }));
16+
"--num-queries", TestConfig.NUM_QUERIES, "cockroachdb", "--oracle", "NOREC" }));
1917
}
2018

2119
}
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
package sqlancer.dbms;
2+
3+
import static org.junit.jupiter.api.Assertions.assertEquals;
4+
import static org.junit.jupiter.api.Assumptions.assumeTrue;
5+
6+
import org.junit.jupiter.api.Test;
7+
8+
import sqlancer.Main;
9+
10+
public class TestCockroachDBTLP {
11+
12+
@Test
13+
public void testCockroachDBTLP() {
14+
assumeTrue(TestConfig.isEnvironmentTrue(TestConfig.COCKROACHDB_ENV));
15+
assertEquals(0, Main.executeMain(new String[] { "--random-seed", "0", "--timeout-seconds", TestConfig.SECONDS,
16+
"--num-queries", TestConfig.NUM_QUERIES, "cockroachdb", "--oracle", "QUERY_PARTITIONING" }));
17+
}
18+
19+
}

test/sqlancer/dbms/TestConfig.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ public class TestConfig {
44
public static final String NUM_QUERIES = "1000";
55
public static final String SECONDS = "300";
66

7+
public static final String COCKROACHDB_ENV = "COCKROACHDB_AVAILABLE";
78
public static final String DATABEND_ENV = "DATABEND_AVAILABLE";
89
public static final String DORIS_ENV = "DORIS_AVAILABLE";
910
public static final String MARIADB_ENV = "MARIADB_AVAILABLE";

0 commit comments

Comments
 (0)