Skip to content

Added ability to cache the whole graphql response #247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Nov 19, 2020
Merged

Added ability to cache the whole graphql response #247

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4d4020d
Added ability to cache the whole graphql response.
donbeave Apr 23, 2020
b66190e
Fixed SingleQueryResponseWriterTest.
donbeave Apr 23, 2020
8836fab
Pass HttpServletRequest to GraphQLResponseCache methods.
donbeave Apr 23, 2020
c52e45b
Added comments.
donbeave Apr 23, 2020
2322c25
Pass request to cacheResponse method.
donbeave Apr 23, 2020
9153aa5
Added method which check is this request must be cached.
donbeave May 1, 2020
86cc378
Fixed HttpRequestHandlerImpl.
donbeave May 1, 2020
1cf9e73
Merge branch 'master' of https://github.com/graphql-java-kickstart/gr…
donbeave May 24, 2020
6c95ef3
Merge branch 'graphql-java-kickstart-master'
donbeave May 24, 2020
4b4edfe
Code review changes.
donbeave May 24, 2020
aa936f9
Fixed tests.
donbeave May 25, 2020
4d25d64
Fixed tests.
donbeave May 25, 2020
2316944
Code review fixes.
donbeave Jun 1, 2020
ac6fc42
Moved createCacheWriter from QueryResponseWriter to CachingQueryRespo…
donbeave Jun 1, 2020
315b77f
Added Override annotation.
donbeave Jun 1, 2020
bc93099
Merge branch 'master' of https://github.com/graphql-java-kickstart/gr…
donbeave Nov 18, 2020
2902ee7
Merge branch 'graphql-java-kickstart-master'
donbeave Nov 18, 2020
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed SingleQueryResponseWriterTest.
  • Loading branch information
@donbeave
donbeave committed Apr 23, 2020
commit b66190ea4d892d2465499ea4215e887faa1d0f8a
4 changes: 2 additions & 2 deletions ...va-servlet/src/test/groovy/graphql/kickstart/servlet/SingleQueryResponseWriterTest.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ class SingleQueryResponseWriterTest extends Specification {
1 * responseMock.getOutputStream().write(expectedResponseContent.getBytes(StandardCharsets.UTF_8))

expect:
def writer = new SingleQueryResponseWriter(new ExecutionResultImpl(result, []), graphQLObjectMapperMock)
writer.write(requestMock, responseMock)
def writer = new SingleQueryResponseWriter(new ExecutionResultImpl(result, []), graphQLObjectMapperMock, null)
writer.write(requestMock, responseMock, null)

where:
result || expectedContentLenght | expectedResponseContent
Expand Down