Skip to content
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.

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

Release 2.14.1 #4175

Merged
merged 6 commits into from
May 28, 2020
Merged

Release 2.14.1 #4175

merged 6 commits into from
May 28, 2020

Conversation

abernix
Copy link
Member

@abernix abernix commented May 28, 2020

As with release PRs in the past, this is a PR tracking a release-2.14.1 branch for an upcoming patch release of Apollo Server. 🙌 The version in the title of this PR should correspond to the appropriate branch.

This, in particular, is a quick release to fix:

And to also land:

The `apollo-server-testing` package uses an internal Apollo Server method
called `executeOperation` (introduced in [#1909]) in order to power its
`createTestClient` functionality.  This is the testing practice which is
documented within [Integration testing] in the Apollo Server documentation.

However, it failed to introduce the same context-cloning which [takes place
in `runHttpQuery`][Ref 1], prior to arriving at the main request pipeline.

Since the context was not cloned, and we had made the expectation in [#3988]
that it was a unique context on every single request (which it was, in a
non-testing context), the Symbol we use to implement `willResolveField` was
already present [on the request pipeline][Ref 2] when running a subsequent
test via `createTestClient`!

This commit introduces the same cloning that takes place in
`buildRequestContext` within `runHttpQuery`, and adds tests to ensure the
behavior is preserved.

[Fixes #4170]: #4170
[#1909]: #1909
[Integration testing]: https://www.apollographql.com/docs/apollo-server/testing/testing/
[Ref 1]: https://git.io/Jfou6
[#3988]: #3988
[Ref 2]: https://git.io/Jfouy
@abernix abernix added the 📦 release Applied to PRs which track upcoming releases. label May 28, 2020
@abernix abernix added this to the Release 2.14.1 milestone May 28, 2020
@abernix abernix self-assigned this May 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
📦 release Applied to PRs which track upcoming releases.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant