Skip to content

Commit 54d3702

Browse files
committed
pass test
1 parent 71b488d commit 54d3702

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

src/utils/dependent-type-utils.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,10 @@ export function getDependentInterfaceNames(
4949
return "interfaces" in node
5050
? (node.interfaces
5151
?.map((interfaceNode) => interfaceNode.name.value)
52-
.filter((interfaceName) =>
53-
shouldIncludeTypeDefinition(interfaceName, config),
52+
.filter(
53+
(interfaceName) =>
54+
shouldIncludeTypeDefinition(interfaceName, config) ||
55+
config.includeDependentTypes,
5456
) ?? [])
5557
: [];
5658
}
@@ -75,5 +77,9 @@ export function getDependentUnionsForType(
7577
union.getTypes().some((type) => type.name === node.name.value),
7678
)
7779
.map((union) => union.name)
78-
.filter((unionName) => shouldIncludeTypeDefinition(unionName, config));
80+
.filter(
81+
(unionName) =>
82+
shouldIncludeTypeDefinition(unionName, config) ||
83+
config.includeDependentTypes,
84+
);
7985
}

test/unit/should_honor_onlyTypes_when_implementing_external_interfaces_and_unions/codegen.config.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,5 @@ import { GraphQLKotlinCodegenConfig } from "../../../src/plugin";
22

33
export default {
44
onlyTypes: ["MyIncludedType", "MyIncludedInterfaceInOnlyTypes"],
5+
includeDependentTypes: false,
56
} satisfies GraphQLKotlinCodegenConfig;

0 commit comments

Comments
 (0)