Skip to content

Conversation

injectives
Copy link
Contributor

Cherry-picks: #992, #993, #995, #994, #998.

injectives and others added 5 commits September 6, 2021 09:45
This commit remoces the command processor from the TestKit state. The processor is supposed to use that state, not to be part of it.
However, some requests, like `NewDriver` need the processor to trigger further state.
Therefor the processor is now provided as injectable value via Jackson, so that any request can indicate that it needs a processor via a constructor argument.
* Migrate Testkit backend to Netty implementation

* Remove unnessary Optional in processAsync.

This brings both sync and asynchronous paths in TestKitState in symmetry.
In addition, I noticed that the transaction related tests behaved differently in sync and async (when a tx is not found). I fixed that.

* Remove redundant lines

Co-authored-by: Michael Simons <[email protected]>
This update brings reactive backend support.

Its scope is limited to providing partial transparent support for existing test cases. More updates are expected in future PRs.
@injectives injectives merged commit 979b13e into neo4j:4.3 Sep 6, 2021
@injectives injectives deleted the feature/cherry branch September 6, 2021 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants