Skip to content

Commit 25a0686

Browse files
committed
Fix unit tests
1 parent ceed1f3 commit 25a0686

File tree

2 files changed

+17
-4
lines changed

2 files changed

+17
-4
lines changed

src/test/groovy/graphql/servlet/AbstractGraphQLHttpServletSpec.groovy

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,14 @@ class AbstractGraphQLHttpServletSpec extends Specification {
116116

117117
def "async query over HTTP GET starts async request"() {
118118
setup:
119-
servlet = TestUtils.createServlet({ env -> env.arguments.arg },{ env -> env.arguments.arg }, true)
119+
servlet = TestUtils.createServlet({ env -> env.arguments.arg },{ env -> env.arguments.arg }, { env ->
120+
AtomicReference<SingleSubscriberPublisher<String>> publisherRef = new AtomicReference<>();
121+
publisherRef.set(new SingleSubscriberPublisher<>({ subscription ->
122+
publisherRef.get().offer(env.arguments.arg)
123+
publisherRef.get().noMoreData()
124+
}))
125+
return publisherRef.get()
126+
}, true)
120127
request.addParameter('query', 'query { echo(arg:"test") }')
121128

122129
when:
@@ -350,7 +357,14 @@ class AbstractGraphQLHttpServletSpec extends Specification {
350357

351358
def "async query over HTTP POST starts async request"() {
352359
setup:
353-
servlet = TestUtils.createServlet({ env -> env.arguments.arg },{ env -> env.arguments.arg }, true)
360+
servlet = TestUtils.createServlet({ env -> env.arguments.arg },{ env -> env.arguments.arg }, { env ->
361+
AtomicReference<SingleSubscriberPublisher<String>> publisherRef = new AtomicReference<>();
362+
publisherRef.set(new SingleSubscriberPublisher<>({ subscription ->
363+
publisherRef.get().offer(env.arguments.arg)
364+
publisherRef.get().noMoreData()
365+
}))
366+
return publisherRef.get()
367+
}, true)
354368
request.setContent(mapper.writeValueAsBytes([
355369
query: 'query { echo(arg:"test") }'
356370
]))

src/test/groovy/graphql/servlet/TestUtils.groovy

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,14 @@ class TestUtils {
1212

1313
static def createServlet(DataFetcher queryDataFetcher = { env -> env.arguments.arg },
1414
DataFetcher mutationDataFetcher = { env -> env.arguments.arg },
15-
boolean asyncServletModeEnabled = false,
1615
DataFetcher subscriptionDataFetcher = { env ->
1716
AtomicReference<SingleSubscriberPublisher<String>> publisherRef = new AtomicReference<>();
1817
publisherRef.set(new SingleSubscriberPublisher<>({ subscription ->
1918
publisherRef.get().offer(env.arguments.arg)
2019
publisherRef.get().noMoreData()
2120
}))
2221
return publisherRef.get()
23-
}) {
22+
}, boolean asyncServletModeEnabled = false) {
2423
GraphQLHttpServlet servlet = GraphQLHttpServlet.with(GraphQLConfiguration
2524
.with(createGraphQlSchema(queryDataFetcher, mutationDataFetcher, subscriptionDataFetcher))
2625
.with(createInstrumentedQueryInvoker())

0 commit comments

Comments
 (0)