Fix loop for ctas table with multiple indexes (#97)

* Fix multiple indexes syntax

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

* Add index tests

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

* Add description to tests

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

* Fix test

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

* Add row to changelog file

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

---------

Signed-off-by: VasylRatushniuk <vasylrat@gmail.com>

Author: VasylRatushniuk

CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - `get_catalog` macro (#89)
+-  Multiple indexes issue (#97)
 
 ## [1.11.0] - 2025-10-16
 

dbt/include/starrocks/macros/materializations/models/table.sql

@@ -23,13 +23,12 @@
   {{ sql_header if sql_header is not none }}
 
   create table {{ relation.include(database=False) }}
-  {%- if indexs is not none -%}
-    {%- for index in indexs -%}
-      {%- set columns = index.get('columns') -%}
-      (
-        INDEX idx_{{ columns | replace(" ", "") | replace(",", "_") }} ({{ columns }}) USING BITMAP
-      )
-    {%- endfor -%}
+  {%- if indexs is not none %} (
+    {%- for index in indexs %}
+      {%- set columns = index.get('columns') %}
+    INDEX idx_{{ columns | replace(" ", "") | replace(",", "_") }} ({{ columns }}) USING BITMAP{{ "," if not loop.last }}
+    {%- endfor %}
+  )
   {%- endif -%}
 
   {%- if is_external -%}

tests/functional/adapter/test_indexes.py

@@ -0,0 +1,84 @@
+import pytest
+from dbt.tests.util import run_dbt
+import logging as log
+
+single_index_model = """
+{{ config(
+    materialized='table',
+    engine='OLAP',
+    distributed_by=['id'],
+    indexs=[{'columns': 'name'}]
+) }}
+select 1 as id, 'test' as name
+"""
+
+multiple_indexes_model = """
+{{ config(
+    materialized='table',
+    engine='OLAP',
+    distributed_by=['id'],
+    indexs=[
+        {'columns': 'name'},
+        {'columns': 'email'},
+        {'columns': 'status'}
+    ]
+) }}
+select 1 as id, 'test' as name, 'test@example.com' as email, 'active' as status
+"""
+
+multiple_columns_index_model = """
+{{ config(
+    materialized='table',
+    engine='OLAP',
+    distributed_by=['id'],
+    indexs=[{'columns': 'name, email'}]
+) }}
+select 1 as id, 'test' as name, 'test@example.com' as email
+"""
+
+class TestSingleIndex:
+    """Test that a single index is created correctly with proper naming and BITMAP type."""
+    
+    @pytest.fixture(scope="class")
+    def models(self):
+        return {"single_index.sql": single_index_model}
+
+    def test_single_index(self, project):
+        results = run_dbt(["run"])
+        assert len(results) == 1
+        
+        relation = results[0].node.relation_name
+        result = project.run_sql(f"SHOW CREATE TABLE {relation}", fetch="one")[1]
+        assert "INDEX idx_name" in result
+        assert "USING BITMAP" in result
+
+
+class TestMultipleIndexes:
+    """Test that multiple indexes are created without trailing commas and all indexes are present."""
+    
+    @pytest.fixture(scope="class")
+    def models(self):
+        return {"multiple_indexes.sql": multiple_indexes_model}
+
+    def test_multiple_indexes(self, project):
+        results = run_dbt(["run"])
+        assert len(results) == 1
+        
+        relation = results[0].node.relation_name
+        result = project.run_sql(f"SHOW CREATE TABLE {relation}", fetch="one")[1]
+        assert "INDEX idx_name" in result
+        assert "INDEX idx_email" in result
+        assert "INDEX idx_status" in result
+        assert result.count("USING BITMAP") == 3
+
+
+class TestMultipleColumnsIndex:
+    """Test that creating an index with multiple columns fails as expected (not supported)."""
+    
+    @pytest.fixture(scope="class")
+    def models(self):
+        return {"multiple_columns_index.sql": multiple_columns_index_model}
+
+    def test_multiple_columns_index(self, project):
+        results = run_dbt(["run"], expect_pass=False)
+        assert "index can only apply to a single column" in results.results[0].message