[native] add a test case for text reader

Akanksha-kedia · Nov 6, 2023 · f0bd6a5 · f0bd6a5
1 parent 8a0da75
commit f0bd6a5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/presto-native-execution/pom.xml b/presto-native-execution/pom.xml
@@ -192,7 +192,7 @@
                 <configuration>
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
-                    <excludedGroups>remote-function</excludedGroups>
+                    <excludedGroups>remote-function,textfile_reader</excludedGroups>
                     <systemPropertyVariables>
                         <PRESTO_SERVER>/root/project/build/debug/presto_cpp/main/presto_server</PRESTO_SERVER>
                         <DATA_DIR>/tmp/velox</DATA_DIR>

diff --git a/...tion/src/test/java/com/facebook/presto/nativeworker/AbstractTestNativeGeneralQueries.java b/...tion/src/test/java/com/facebook/presto/nativeworker/AbstractTestNativeGeneralQueries.java
@@ -992,13 +992,23 @@ private String generateRandomTableName()
         return tableName;
     }
 
-    @Test
-    public void testReadTableWithUnsupportedFormats()
+    @Test(groups = {"no_json_reader"})
+    public void testReadTableWithUnsupportedJsonFormat()
     {
         assertQueryFails("SELECT * FROM nation_json", ".*ReaderFactory is not registered for format json.*");
+    }
+
+    @Test(groups = {"no_textfile_reader"})
+    public void testReadTableWithUnsupportedTextfileFormat()
+    {
         assertQueryFails("SELECT * FROM nation_text", ".*ReaderFactory is not registered for format text.*");
     }
 
+    @Test(groups = {"textfile_reader"})
+    public void testReadTableWithTextfileFormat()
+    {
+        assertQuery("SELECT * FROM nation_text");
+    }
     private void dropTableIfExists(String tableName)
     {
         // An ugly workaround for the lack of getExpectedQueryRunner()