Skip to content

Commit 151d67f

Browse files
akoclaude
andcommitted
fix: use correct entity syntax in mapping integration tests
Entity creation uses parentheses `( )` with comma-separated attributes, not braces `{ }` with semicolons. This fixes CI integration test failures. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent de46d17 commit 151d67f

File tree

2 files changed

+45
-45
lines changed

2 files changed

+45
-45
lines changed

mdl/executor/roundtrip_export_mapping_test.go

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ func TestRoundtripExportMapping_NoSchema(t *testing.T) {
1616
defer env.teardown()
1717

1818
// First create the entity that will be exported
19-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMPet {
20-
Id: Integer;
21-
Name: String(200);
22-
};`); err != nil {
19+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMPet (
20+
Id: Integer,
21+
Name: String(200)
22+
);`); err != nil {
2323
t.Fatalf("CREATE ENTITY failed: %v", err)
2424
}
2525

@@ -42,10 +42,10 @@ func TestRoundtripExportMapping_WithJsonStructureRef(t *testing.T) {
4242
env := setupTestEnv(t)
4343
defer env.teardown()
4444

45-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMOrder {
46-
OrderId: Integer;
47-
Total: Decimal(10,2);
48-
};`); err != nil {
45+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMOrder (
46+
OrderId: Integer,
47+
Total: Decimal(10,2)
48+
);`); err != nil {
4949
t.Fatalf("CREATE ENTITY failed: %v", err)
5050
}
5151

@@ -77,9 +77,9 @@ func TestRoundtripExportMapping_NullValueOption(t *testing.T) {
7777
env := setupTestEnv(t)
7878
defer env.teardown()
7979

80-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMNullPet {
81-
Id: Integer;
82-
};`); err != nil {
80+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMNullPet (
81+
Id: Integer
82+
);`); err != nil {
8383
t.Fatalf("CREATE ENTITY failed: %v", err)
8484
}
8585

@@ -107,9 +107,9 @@ func TestRoundtripExportMapping_Drop(t *testing.T) {
107107
env := setupTestEnv(t)
108108
defer env.teardown()
109109

110-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMDropPet {
111-
Id: Integer;
112-
};`); err != nil {
110+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMDropPet (
111+
Id: Integer
112+
);`); err != nil {
113113
t.Fatalf("CREATE ENTITY failed: %v", err)
114114
}
115115

@@ -138,9 +138,9 @@ func TestRoundtripExportMapping_ShowAppearsInList(t *testing.T) {
138138
env := setupTestEnv(t)
139139
defer env.teardown()
140140

141-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMListPet {
142-
Id: Integer;
143-
};`); err != nil {
141+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.EMListPet (
142+
Id: Integer
143+
);`); err != nil {
144144
t.Fatalf("CREATE ENTITY failed: %v", err)
145145
}
146146

@@ -172,10 +172,10 @@ func TestMxCheck_ExportMapping_Basic(t *testing.T) {
172172
env := setupTestEnv(t)
173173
defer env.teardown()
174174

175-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.MxCheckEMPet {
176-
Id: Integer;
177-
Name: String(200);
178-
};`); err != nil {
175+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.MxCheckEMPet (
176+
Id: Integer,
177+
Name: String(200)
178+
);`); err != nil {
179179
t.Fatalf("CREATE ENTITY failed: %v", err)
180180
}
181181

mdl/executor/roundtrip_import_mapping_test.go

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ func TestRoundtripImportMapping_NoSchema(t *testing.T) {
1616
defer env.teardown()
1717

1818
// First create the entity that will be mapped to
19-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMPet {
20-
Id: Integer;
21-
Name: String(200);
22-
};`); err != nil {
19+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMPet (
20+
Id: Integer,
21+
Name: String(200)
22+
);`); err != nil {
2323
t.Fatalf("CREATE ENTITY failed: %v", err)
2424
}
2525

@@ -43,10 +43,10 @@ func TestRoundtripImportMapping_WithJsonStructureRef(t *testing.T) {
4343
defer env.teardown()
4444

4545
// Create required entity and JSON structure first
46-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMOrder {
47-
OrderId: Integer;
48-
Total: Decimal(10,2);
49-
};`); err != nil {
46+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMOrder (
47+
OrderId: Integer,
48+
Total: Decimal(10,2)
49+
);`); err != nil {
5050
t.Fatalf("CREATE ENTITY failed: %v", err)
5151
}
5252

@@ -78,12 +78,12 @@ func TestRoundtripImportMapping_ValueTypes(t *testing.T) {
7878
env := setupTestEnv(t)
7979
defer env.teardown()
8080

81-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMAllTypes {
82-
IntVal: Integer;
83-
DecVal: Decimal(10,2);
84-
BoolVal: Boolean DEFAULT false;
85-
DateVal: DateTime;
86-
};`); err != nil {
81+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMAllTypes (
82+
IntVal: Integer,
83+
DecVal: Decimal(10,2),
84+
BoolVal: Boolean DEFAULT false,
85+
DateVal: DateTime
86+
);`); err != nil {
8787
t.Fatalf("CREATE ENTITY failed: %v", err)
8888
}
8989

@@ -108,9 +108,9 @@ func TestRoundtripImportMapping_Drop(t *testing.T) {
108108
env := setupTestEnv(t)
109109
defer env.teardown()
110110

111-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMDropPet {
112-
Id: Integer;
113-
};`); err != nil {
111+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMDropPet (
112+
Id: Integer
113+
);`); err != nil {
114114
t.Fatalf("CREATE ENTITY failed: %v", err)
115115
}
116116

@@ -139,9 +139,9 @@ func TestRoundtripImportMapping_ShowAppearsInList(t *testing.T) {
139139
env := setupTestEnv(t)
140140
defer env.teardown()
141141

142-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMListPet {
143-
Id: Integer;
144-
};`); err != nil {
142+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.IMListPet (
143+
Id: Integer
144+
);`); err != nil {
145145
t.Fatalf("CREATE ENTITY failed: %v", err)
146146
}
147147

@@ -173,10 +173,10 @@ func TestMxCheck_ImportMapping_Basic(t *testing.T) {
173173
env := setupTestEnv(t)
174174
defer env.teardown()
175175

176-
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.MxCheckIMPet {
177-
Id: Integer;
178-
Name: String(200);
179-
};`); err != nil {
176+
if err := env.executeMDL(`CREATE ENTITY ` + testModule + `.MxCheckIMPet (
177+
Id: Integer,
178+
Name: String(200)
179+
);`); err != nil {
180180
t.Fatalf("CREATE ENTITY failed: %v", err)
181181
}
182182

0 commit comments

Comments
 (0)