Skip to content

Commit 011b495

Browse files
committed
add more tests
1 parent a74e261 commit 011b495

File tree

2 files changed

+104
-0
lines changed

2 files changed

+104
-0
lines changed

test/expected/issue_fragment_spread_cycles.out

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,62 @@ begin;
111111
{"errors": [{"message": "Found a cycle between fragments"}]}
112112
(1 row)
113113

114+
select graphql.resolve($$
115+
{
116+
blogCollection {
117+
edges {
118+
node {
119+
... blogFragment
120+
}
121+
}
122+
}
123+
}
124+
125+
fragment blogFragment on Blog {
126+
owner {
127+
blogCollection {
128+
edges {
129+
node {
130+
... blogFragment
131+
}
132+
}
133+
}
134+
}
135+
}
136+
$$);
137+
resolve
138+
--------------------------------------------------------------
139+
{"errors": [{"message": "Found a cycle between fragments"}]}
140+
(1 row)
141+
142+
select graphql.resolve($$
143+
{
144+
blogCollection {
145+
edges {
146+
node {
147+
id
148+
}
149+
}
150+
}
151+
}
152+
153+
fragment blogFragment on Blog {
154+
owner {
155+
blogCollection {
156+
edges {
157+
node {
158+
... blogFragment
159+
}
160+
}
161+
}
162+
}
163+
}
164+
$$);
165+
resolve
166+
--------------------------------------------------------------
167+
{"errors": [{"message": "Found a cycle between fragments"}]}
168+
(1 row)
169+
114170
-- test that a recursion limit of 50 is good enough for most queries
115171
select graphql.resolve($$
116172
{

test/sql/issue_fragment_spread_cycles.sql

Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,54 @@ begin;
107107
}
108108
$$);
109109

110+
select graphql.resolve($$
111+
{
112+
blogCollection {
113+
edges {
114+
node {
115+
... blogFragment
116+
}
117+
}
118+
}
119+
}
120+
121+
fragment blogFragment on Blog {
122+
owner {
123+
blogCollection {
124+
edges {
125+
node {
126+
... blogFragment
127+
}
128+
}
129+
}
130+
}
131+
}
132+
$$);
133+
134+
select graphql.resolve($$
135+
{
136+
blogCollection {
137+
edges {
138+
node {
139+
id
140+
}
141+
}
142+
}
143+
}
144+
145+
fragment blogFragment on Blog {
146+
owner {
147+
blogCollection {
148+
edges {
149+
node {
150+
... blogFragment
151+
}
152+
}
153+
}
154+
}
155+
}
156+
$$);
157+
110158
-- test that a recursion limit of 50 is good enough for most queries
111159
select graphql.resolve($$
112160
{

0 commit comments

Comments
 (0)