Converted order_by and copy tests to the new format (#1526)

Added test.group for copy and order_by and removed the cpp code.
kuzudb · May 10, 2023 · 1d45b99 · 1d45b99
1 parent a669eeb
commit 1d45b99
Show file tree

Hide file tree

Showing 9 changed files with 63 additions and 45 deletions.
diff --git a/test/copy/copy_csv_parquet_test.cpp b/test/copy/copy_csv_parquet_test.cpp
@@ -4,26 +4,6 @@ using namespace kuzu::common;
 using namespace kuzu::storage;
 using namespace kuzu::testing;
 
-class CopyNodeFromCSVTest : public DBTest {
-    std::string getInputDir() override {
-        return TestHelper::appendKuzuRootPath("dataset/copy-test/node/csv/");
-    }
-};
-
-TEST_F(CopyNodeFromCSVTest, CopyNodeTest) {
-    runTest(TestHelper::appendKuzuRootPath("test/test_files/copy/copy_node.test"));
-}
-
-class CopyNodeFromParquetTest : public DBTest {
-    std::string getInputDir() override {
-        return TestHelper::appendKuzuRootPath("dataset/copy-test/node/parquet/");
-    }
-};
-
-TEST_F(CopyNodeFromParquetTest, CopyNodeTest) {
-    runTest(TestHelper::appendKuzuRootPath("test/test_files/copy/copy_node.test"));
-}
-
 class CopyNodeWrongPathTest : public BaseGraphTest {
 public:
     void SetUp() override {

diff --git a/test/include/test_helper/test_helper.h b/test/include/test_helper/test_helper.h
@@ -67,9 +67,9 @@ class TestHelper {
     static void initializeConnection(TestQueryConfig* config, Connection& conn);
     static bool testQuery(TestQueryConfig* config, Connection& conn);
     static void setConfigValue(
-        const std::string& line, std::string& configValue, const std::string& configKey);
+        const std::string& line, std::string& configItem, const std::string& configKey);
     static void setConfigValue(
-        const std::string& line, bool& configValue, const std::string& configKey);
+        const std::string& line, bool& configItem, const std::string& configKey);
     static std::string extractConfigValue(const std::string& line, const std::string& configKey);
 };
 

diff --git a/test/runner/CMakeLists.txt b/test/runner/CMakeLists.txt
@@ -2,7 +2,6 @@ add_kuzu_test(e2e_copy_csv_transaction_test e2e_copy_transaction_test.cpp)
 add_kuzu_test(e2e_ddl_test e2e_ddl_test.cpp)
 add_kuzu_test(e2e_delete_create_transaction_test e2e_delete_create_transaction_test.cpp)
 add_kuzu_test(e2e_exception_test e2e_exception_test.cpp)
-add_kuzu_test(e2e_order_by_test e2e_order_by_test.cpp)
 add_kuzu_test(e2e_read_test e2e_read_test.cpp)
 add_kuzu_test(e2e_set_transaction_test e2e_set_transaction_test.cpp)
 add_kuzu_test(e2e_update_node_test e2e_update_node_test.cpp)

diff --git a/test/runner/e2e_order_by_test.cpp b/test/runner/e2e_order_by_test.cpp
diff --git a/test/test_files/copy/copy_node.test → test/test_files/copy/csv/copy_node.test b/test/test_files/copy/copy_node.test → test/test_files/copy/csv/copy_node.test
@@ -1,4 +1,3 @@
-
 -NAME SubsetTest
 -QUERY MATCH (row:tableOfTypes) WHERE row.id >= 20 AND row.id <= 24 RETURN *;
 ---- 5

diff --git a/test/test_files/copy/csv/test.group b/test/test_files/copy/csv/test.group
@@ -0,0 +1,4 @@
+-GROUP CopyNodeFromCSVTest
+-TEST CopyNodeTest
+-DATASET copy-test/node/csv
+-READ_ONLY TRUE
diff --git a/test/test_files/copy/parquet/copy_node.test b/test/test_files/copy/parquet/copy_node.test
@@ -0,0 +1,48 @@
+-NAME SubsetTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.id >= 20 AND row.id <= 24 RETURN *;
+---- 5
+(label:tableOfTypes, 0:20, {id:20, int64Column:0, doubleColumn:57.579280, booleanColumn:True, dateColumn:1731-09-26, timestampColumn:1731-09-26 03:30:08, stringColumn:OdM})
+(label:tableOfTypes, 0:21, {id:21, int64Column:7, doubleColumn:64.630960, booleanColumn:False, dateColumn:1307-01-26, timestampColumn:1307-01-26 03:31:08, stringColumn:AjbxHQThEtDDlOjbzMjCQSXlvGQEjcFLykESrnFHwPKX})
+(label:tableOfTypes, 0:22, {id:22, int64Column:71, doubleColumn:37.963386, booleanColumn:True, dateColumn:1455-07-26, timestampColumn:1455-07-26 03:07:03, stringColumn:dRvHHdyNXYfSUcicaxBoQEKQUfgex})
+(label:tableOfTypes, 0:23, {id:23, int64Column:58, doubleColumn:42.774957, booleanColumn:False, dateColumn:1181-10-16, timestampColumn:1181-10-16 18:19:43, stringColumn:ISImRVpUjynGMFRQyYmeIUVjM})
+(label:tableOfTypes, 0:24, {id:24, int64Column:75, doubleColumn:53.813224, booleanColumn:False, dateColumn:1942-10-24, timestampColumn:1942-10-24 09:30:16, stringColumn:naDlQ})
+
+-NAME CheckNumLinesTest
+-QUERY MATCH (row:tableOfTypes) RETURN count(*)
+---- 1
+49999
+
+-NAME CountTrueTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.booleanColumn = True RETURN count(*)
+---- 1
+24997
+
+-NAME CountFalseTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.booleanColumn = False RETURN count(*)
+---- 1
+25002
+
+-NAME CheckSumTest
+-QUERY MATCH (row:tableOfTypes) RETURN sum(row.id), sum(row.int64Column), sum(row.doubleColumn)
+---- 1
+1249925001|2500180|2504542.349696
+
+-NAME EmptyStringTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.id = 49992 RETURN *;
+---- 1
+(label:tableOfTypes, 0:49992, {id:49992, int64Column:50, doubleColumn:31.582059, booleanColumn:False, dateColumn:1551-07-19, timestampColumn:1551-07-19 16:28:31, stringColumn:})
+
+-NAME FloatTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.doubleColumn = 68.73718401556897 RETURN row.dateColumn;
+---- 1
+1042-06-05
+
+-NAME DateTest
+-QUERY MATCH (row:tableOfTypes) WHERE row.id = 25531 RETURN *;
+---- 1
+(label:tableOfTypes, 0:25531, {id:25531, int64Column:77, doubleColumn:28.417543, booleanColumn:False, dateColumn:1895-03-13, timestampColumn:1895-03-13 04:31:22, stringColumn:XB})
+
+-NAME IntervalTest
+-QUERY MATCH (row:tableOfTypes) WHERE 0 <= row.doubleColumn AND row.doubleColumn <= 10 AND 0 <= row.int64Column AND row.int64Column <= 10 RETURN count(*);
+---- 1
+546
diff --git a/test/test_files/copy/parquet/test.group b/test/test_files/copy/parquet/test.group
@@ -0,0 +1,4 @@
+-GROUP CopyNodeFromParquetTest
+-TEST CopyNodeTest
+-DATASET copy-test/node/parquet
+-READ_ONLY TRUE
diff --git a/test/test_files/order_by/test.group b/test/test_files/order_by/test.group
@@ -0,0 +1,5 @@
+-GROUP OrderByTests
+-TEST OrderByLargeDatasetTest
+-DATASET order-by-tests
+-CHECK_ORDER TRUE
+-READ_ONLY TRUE