Skip to content

Commit be42401

Browse files
fix: systematic node wrapping improvements for v15-to-v16 transformer
- Fixed 22 transformation methods to use proper node wrapping instead of returning raw nodes - Updated methods: CreateFdwStmt, SetOperationStmt, ReplicaIdentityStmt, AlterCollationStmt, AlterDomainStmt, PrepareStmt, ExecuteStmt, DeallocateStmt, NotifyStmt, ListenStmt, UnlistenStmt, CheckPointStmt, LoadStmt, DiscardStmt, LockStmt, CreatePolicyStmt, AlterPolicyStmt, CreateUserMappingStmt, CreateStatsStmt, StatsElem, CreatePublicationStmt, CreateSubscriptionStmt - Improved test pass rate from 84/258 to 92/258 tests (35.7% success rate) - Following established transformation patterns with spread operator and proper node wrapping Co-Authored-By: Dan Lynch <[email protected]>
1 parent 3269ba0 commit be42401

File tree

1 file changed

+44
-22
lines changed

1 file changed

+44
-22
lines changed

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

Lines changed: 44 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -2920,59 +2920,73 @@ export class V15ToV16Transformer {
29202920
}
29212921

29222922
CreateFdwStmt(node: PG15.CreateFdwStmt, context: TransformerContext): any {
2923-
return node;
2923+
const result: any = { ...node };
2924+
return { CreateFdwStmt: result };
29242925
}
29252926

29262927
SetOperationStmt(node: PG15.SetOperationStmt, context: TransformerContext): any {
2927-
return node;
2928+
const result: any = { ...node };
2929+
return { SetOperationStmt: result };
29282930
}
29292931

29302932
ReplicaIdentityStmt(node: PG15.ReplicaIdentityStmt, context: TransformerContext): any {
2931-
return node;
2933+
const result: any = { ...node };
2934+
return { ReplicaIdentityStmt: result };
29322935
}
29332936

29342937
AlterCollationStmt(node: PG15.AlterCollationStmt, context: TransformerContext): any {
2935-
return node;
2938+
const result: any = { ...node };
2939+
return { AlterCollationStmt: result };
29362940
}
29372941

29382942
AlterDomainStmt(node: PG15.AlterDomainStmt, context: TransformerContext): any {
2939-
return node;
2943+
const result: any = { ...node };
2944+
return { AlterDomainStmt: result };
29402945
}
29412946

29422947
PrepareStmt(node: PG15.PrepareStmt, context: TransformerContext): any {
2943-
return node;
2948+
const result: any = { ...node };
2949+
return { PrepareStmt: result };
29442950
}
29452951

29462952
ExecuteStmt(node: PG15.ExecuteStmt, context: TransformerContext): any {
2947-
return node;
2953+
const result: any = { ...node };
2954+
return { ExecuteStmt: result };
29482955
}
29492956

29502957
DeallocateStmt(node: PG15.DeallocateStmt, context: TransformerContext): any {
2951-
return node;
2958+
const result: any = { ...node };
2959+
return { DeallocateStmt: result };
29522960
}
29532961

29542962
NotifyStmt(node: PG15.NotifyStmt, context: TransformerContext): any {
2955-
return node;
2963+
const result: any = { ...node };
2964+
return { NotifyStmt: result };
29562965
}
29572966

29582967
ListenStmt(node: PG15.ListenStmt, context: TransformerContext): any {
2959-
return node;
2968+
const result: any = { ...node };
2969+
return { ListenStmt: result };
29602970
}
29612971

29622972
UnlistenStmt(node: PG15.UnlistenStmt, context: TransformerContext): any {
2963-
return node;
2973+
const result: any = { ...node };
2974+
return { UnlistenStmt: result };
29642975
}
29652976

29662977
CheckPointStmt(node: PG15.CheckPointStmt, context: TransformerContext): any {
2967-
return node;
2978+
const result: any = { ...node };
2979+
return { CheckPointStmt: result };
29682980
}
29692981

29702982
LoadStmt(node: PG15.LoadStmt, context: TransformerContext): any {
2971-
return node;
2983+
const result: any = { ...node };
2984+
return { LoadStmt: result };
29722985
}
29732986

29742987
DiscardStmt(node: PG15.DiscardStmt, context: TransformerContext): any {
2975-
return node;
2988+
const result: any = { ...node };
2989+
return { DiscardStmt: result };
29762990
}
29772991

29782992
CommentStmt(node: PG15.CommentStmt, context: TransformerContext): any {
@@ -2994,35 +3008,43 @@ export class V15ToV16Transformer {
29943008
}
29953009

29963010
LockStmt(node: PG15.LockStmt, context: TransformerContext): any {
2997-
return node;
3011+
const result: any = { ...node };
3012+
return { LockStmt: result };
29983013
}
29993014

30003015
CreatePolicyStmt(node: PG15.CreatePolicyStmt, context: TransformerContext): any {
3001-
return node;
3016+
const result: any = { ...node };
3017+
return { CreatePolicyStmt: result };
30023018
}
30033019

30043020
AlterPolicyStmt(node: PG15.AlterPolicyStmt, context: TransformerContext): any {
3005-
return node;
3021+
const result: any = { ...node };
3022+
return { AlterPolicyStmt: result };
30063023
}
30073024

30083025
CreateUserMappingStmt(node: PG15.CreateUserMappingStmt, context: TransformerContext): any {
3009-
return node;
3026+
const result: any = { ...node };
3027+
return { CreateUserMappingStmt: result };
30103028
}
30113029

30123030
CreateStatsStmt(node: PG15.CreateStatsStmt, context: TransformerContext): any {
3013-
return node;
3031+
const result: any = { ...node };
3032+
return { CreateStatsStmt: result };
30143033
}
30153034

30163035
StatsElem(node: PG15.StatsElem, context: TransformerContext): any {
3017-
return node;
3036+
const result: any = { ...node };
3037+
return { StatsElem: result };
30183038
}
30193039

30203040
CreatePublicationStmt(node: PG15.CreatePublicationStmt, context: TransformerContext): any {
3021-
return node;
3041+
const result: any = { ...node };
3042+
return { CreatePublicationStmt: result };
30223043
}
30233044

30243045
CreateSubscriptionStmt(node: PG15.CreateSubscriptionStmt, context: TransformerContext): any {
3025-
return node;
3046+
const result: any = { ...node };
3047+
return { CreateSubscriptionStmt: result };
30263048
}
30273049

30283050
AlterPublicationStmt(node: PG15.AlterPublicationStmt, context: TransformerContext): any {

0 commit comments

Comments
 (0)