add test case about add index

JSQLParser · manticore-projects · Apr 4, 2025 · Apr 4, 2025 · Apr 4, 2025 · Apr 4, 2025
commit 0bd163af32e6cbfb29e37de415cae8fc33932e92
diff --git a/src/main/java/net/sf/jsqlparser/statement/create/table/Index.java b/src/main/java/net/sf/jsqlparser/statement/create/table/Index.java
@@ -81,7 +81,9 @@ public String getName() {
 
     public void setName(String name) {
         this.name.clear();
-        this.name.add(name);
+        if (name != null) {
+            this.name.add(name);
+        }
     }
 
     public void setName(List<String> name) {

diff --git a/src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt b/src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt
@@ -7818,8 +7818,13 @@ AlterExpression AlterExpression():
                 |
                 LOOKAHEAD(2) (
                   (tk=<K_KEY> { alterExp.setUk(true); } | tk=<K_INDEX>)
-                  sk3 = RelObjectName()
-                  columnNames = ColumnsNamesList()
+                  (
+                      LOOKAHEAD(2)
+                      sk3 = RelObjectName()
+                      columnNames = ColumnsNamesList()
+                      |
+                      columnNames = ColumnsNamesList()
+                  )
                   {
                     index = new Index().withType(tk.image).withName(sk3).withColumnsNames(columnNames);
                     alterExp.setIndex(index);

diff --git a/src/test/java/net/sf/jsqlparser/statement/alter/AlterTest.java b/src/test/java/net/sf/jsqlparser/statement/alter/AlterTest.java
@@ -1916,4 +1916,66 @@ public void testAlterTableAddFullTextIndexWithOptions() throws JSQLParserExcepti
 
         assertSqlCanBeParsedAndDeparsed(sql);
     }
+
+    @Test
+    public void testAlterTableAddUnnamedIndex() throws JSQLParserException {
+        String sql = "ALTER TABLE employees ADD INDEX (name1, name2)";
+
+        Statement stmt = CCJSqlParserUtil.parse(sql);
+        Alter alter = (Alter) stmt;
+
+        assertEquals("employees", alter.getTable().getFullyQualifiedName());
+
+        List<AlterExpression> alterExpressions = alter.getAlterExpressions();
+        assertNotNull(alterExpressions);
+        assertEquals(1, alterExpressions.size());
+
+        AlterExpression indexExp = alterExpressions.get(0);
+        assertEquals(AlterOperation.ADD, indexExp.getOperation());
+
+        Index index = indexExp.getIndex();
+        assertNotNull(index);
+        assertNull(index.getName());
+
+        List<String> columnNames = index.getColumnsNames();
+        assertEquals(2, columnNames.size());
+        assertEquals("name1", columnNames.get(0));
+        assertEquals("name2", columnNames.get(1));
+
+        assertSqlCanBeParsedAndDeparsed(sql);
+    }
+
+    @Test
+    public void testAlterTableAddIndexWithOptions() throws JSQLParserException {
+        String sql = "ALTER TABLE employees ADD INDEX idx_lastname (last_name) " +
+                "USING BTREE KEY_BLOCK_SIZE = 16 COMMENT 'Performance tuning' VISIBLE";
+
+        Statement stmt = CCJSqlParserUtil.parse(sql);
+        Alter alter = (Alter) stmt;
+
+        assertEquals("employees", alter.getTable().getFullyQualifiedName());
+
+        List<AlterExpression> alterExpressions = alter.getAlterExpressions();
+        assertNotNull(alterExpressions);
+        assertEquals(1, alterExpressions.size());
+
+        AlterExpression indexExp = alterExpressions.get(0);
+        assertEquals(AlterOperation.ADD, indexExp.getOperation());
+
+        Index index = indexExp.getIndex();
+        assertNotNull(index);
+        assertEquals("INDEX", index.getIndexKeyword()); // 명시적 "INDEX" 키워드
+        assertEquals("idx_lastname", index.getName());
+        assertEquals("last_name", index.getColumnsNames().get(0));
+
+        List<String> indexSpec = index.getIndexSpec();
+        assertNotNull(indexSpec);
+        assertEquals(3, indexSpec.size());
+        assertEquals("USING BTREE", indexSpec.get(0));
+        assertEquals("KEY_BLOCK_SIZE=16", indexSpec.get(1));
+        assertEquals("COMMENT 'Performance tuning'", indexSpec.get(2));
+        assertEquals("VISIBLE", indexSpec.get(3));
+
+        assertSqlCanBeParsedAndDeparsed(sql);
+    }
 }