Merge pull request #200 from launchql/pg/create-index-345

fix: preserve num field in AlterTableCmd 14->15 transformation

Author: pyramation

packages/transform/src/transformers/v14-to-v15.ts

@@ -786,6 +786,10 @@ export class V14ToV15Transformer {
       result.name = node.name;
     }
 
+    if (node.num !== undefined) {
+      result.num = node.num;
+    }
+
     if (node.newowner !== undefined) {
       result.newowner = this.transform(node.newowner as any, context);
     }

packages/transform/test-utils/skip-tests/transformer-errors.ts

@@ -16,7 +16,7 @@ export const transformerErrors: SkipTest[] = [
     [15, 16, "latest/postgres/create_index-326.sql", "15-16 transformer fails with syntax error at end of input"],
     // [15, 16, "latest/postgres/create_index-184.sql", "15-16 transformer fails with missing nulls_not_distinct property"],
 
-    [14, 15, "latest/postgres/create_index-345.sql", "AST transformation mismatch (extra \"num\": 1 field)"],
+    // [14, 15, "latest/postgres/create_index-345.sql", "AST transformation mismatch (extra \"num\": 1 field)"],
 
     [13, 14, "original/upstream/rangetypes-300.sql", "AST transformer bug - converts FUNC_PARAM_DEFAULT to FUNC_PARAM_IN for function parameters"],
     [13, 14, "original/upstream/rangetypes-294.sql", "AST transformer bug - converts FUNC_PARAM_DEFAULT to FUNC_PARAM_IN for function parameters"],