Skip to content

Commit b9e3435

Browse files
bilby91claude
andcommitted
Fix CI failures for PostgreSQL array column integration tests
Add ArrayType entity to postgresql-commands.txt so CI's config generator includes it, and skip list-type fields in InputTypeBuilder filter/orderby generation to prevent array columns from being incorrectly processed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent fac4382 commit b9e3435

2 files changed

Lines changed: 14 additions & 0 deletions

File tree

config-generators/postgresql-commands.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,8 @@ update series --config "dab-config.PostgreSql.json" --permissions "TestNestedFil
171171
update series --config "dab-config.PostgreSql.json" --permissions "TestNestedFilterOneMany_ColumnForbidden:read"
172172
update series --config "dab-config.PostgreSql.json" --permissions "TestNestedFilterOneMany_EntityReadForbidden:read"
173173
update DefaultBuiltInFunction --config "dab-config.PostgreSql.json" --permissions "anonymous:create" --fields.exclude "current_date,next_date"
174+
add ArrayType --config "dab-config.PostgreSql.json" --source "array_type_table" --permissions "anonymous:read" --rest true --graphql "arrayType:arrayTypes"
175+
update ArrayType --config "dab-config.PostgreSql.json" --permissions "authenticated:read"
174176
add dbo_DimAccount --config "dab-config.PostgreSql.json" --source "dimaccount" --permissions "anonymous:*" --rest true --graphql true
175177
update dbo_DimAccount --config "dab-config.PostgreSql.json" --map "parentaccountkey:ParentAccountKey,accountkey:AccountKey"
176178
update dbo_DimAccount --config "dab-config.PostgreSql.json" --relationship parent_account --target.entity dbo_DimAccount --cardinality one --relationship.fields "parentaccountkey:accountkey"

src/Service.GraphQLBuilder/Queries/InputTypeBuilder.cs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,12 @@ private static List<InputValueDefinitionNode> GenerateOrderByInputFieldsForBuilt
5050
List<InputValueDefinitionNode> inputFields = new();
5151
foreach (FieldDefinitionNode field in node.Fields)
5252
{
53+
// Skip array/list type fields (e.g., PostgreSQL array columns) - they cannot be ordered.
54+
if (field.Type.IsListType())
55+
{
56+
continue;
57+
}
58+
5359
if (IsBuiltInType(field.Type))
5460
{
5561
inputFields.Add(
@@ -110,6 +116,12 @@ private static List<InputValueDefinitionNode> GenerateFilterInputFieldsForBuiltI
110116
List<InputValueDefinitionNode> inputFields = new();
111117
foreach (FieldDefinitionNode field in objectTypeDefinitionNode.Fields)
112118
{
119+
// Skip array/list type fields (e.g., PostgreSQL array columns) - they cannot be filtered.
120+
if (field.Type.IsListType())
121+
{
122+
continue;
123+
}
124+
113125
string fieldTypeName = field.Type.NamedType().Name.Value;
114126
if (IsBuiltInType(field.Type))
115127
{

0 commit comments

Comments
 (0)