improve coverage

AnEmortalKid · AnEmortalKid · commit 5b80ec4c9640 · 2020-05-21T19:34:50.000-05:00
diff --git a/src/main/java/net/sf/jsqlparser/util/TablesNamesFinder.java b/src/main/java/net/sf/jsqlparser/util/TablesNamesFinder.java
@@ -878,9 +878,11 @@ public void visit(ArrayExpression array) {
 
     @Override
     public void visit(CreateSequence createSequence) {
+        throw new UnsupportedOperationException("Finding tables from CreateSequence is not supported");
     }
 
     @Override
     public void visit(AlterSequence alterSequence) {
+        throw new UnsupportedOperationException("Finding tables from AlterSequence is not supported");
     }
 }
diff --git a/src/test/java/net/sf/jsqlparser/util/TablesNamesFinderTest.java b/src/test/java/net/sf/jsqlparser/util/TablesNamesFinderTest.java
@@ -9,11 +9,6 @@
  */
 package net.sf.jsqlparser.util;
 
-import java.io.BufferedReader;
-import java.io.InputStreamReader;
-import java.io.StringReader;
-import java.util.Iterator;
-import java.util.List;
 import net.sf.jsqlparser.JSQLParserException;
 import net.sf.jsqlparser.expression.Expression;
 import net.sf.jsqlparser.expression.OracleHint;
@@ -34,9 +29,17 @@
 import net.sf.jsqlparser.statement.update.Update;
 import net.sf.jsqlparser.statement.upsert.Upsert;
 import net.sf.jsqlparser.test.TestException;
-import static org.junit.Assert.*;
 import org.junit.Test;
 
+import java.io.BufferedReader;
+import java.io.InputStreamReader;
+import java.io.StringReader;
+import java.util.Iterator;
+import java.util.List;
+
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.junit.Assert.*;
+
 public class TablesNamesFinderTest {
 
     private static CCJSqlParserManager pm = new CCJSqlParserManager();
@@ -594,4 +597,22 @@ public void testRemoteLink() throws JSQLParserException {
         assertEquals(1, tableList.size());
         assertTrue(tableList.contains("table1@remote"));
     }
+
+    @Test
+    public void testCreateSequence_throwsException() throws JSQLParserException {
+        String sql = "CREATE SEQUENCE my_seq";
+        Statement stmt = CCJSqlParserUtil.parse(sql);
+        TablesNamesFinder tablesNamesFinder = new TablesNamesFinder();
+        assertThatThrownBy(() -> tablesNamesFinder.getTableList(stmt)).isInstanceOf(UnsupportedOperationException.class)
+                .hasMessage("Finding tables from CreateSequence is not supported");
+    }
+
+    @Test
+    public void testAlterSequence_throwsException() throws JSQLParserException {
+        String sql = "ALTER SEQUENCE my_seq";
+        Statement stmt = CCJSqlParserUtil.parse(sql);
+        TablesNamesFinder tablesNamesFinder = new TablesNamesFinder();
+        assertThatThrownBy(() -> tablesNamesFinder.getTableList(stmt)).isInstanceOf(UnsupportedOperationException.class)
+                .hasMessage("Finding tables from AlterSequence is not supported");
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -878,9 +878,11 @@ public void visit(ArrayExpression array) {`
`878`	`878`
`879`	`879`	`@Override`
`880`	`880`	`public void visit(CreateSequence createSequence) {`
	`881`	`+ throw new UnsupportedOperationException("Finding tables from CreateSequence is not supported");`
`881`	`882`	`}`
`882`	`883`
`883`	`884`	`@Override`
`884`	`885`	`public void visit(AlterSequence alterSequence) {`
	`886`	`+ throw new UnsupportedOperationException("Finding tables from AlterSequence is not supported");`
`885`	`887`	`}`
`886`	`888`	`}`