Skip to content

Update TestkitRequestProcessorHandler to adhere to req/res pattern #1500

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

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

injectives
Copy link
Contributor

@injectives injectives commented Oct 31, 2023

This update aims to make sure the Teskit backend does not break the request / response pattern. For instance, by sending 2 responses together in case of internal callbacks.

@injectives injectives force-pushed the feature/update-backend-handler branch from 20a0c75 to e812208 Compare October 31, 2023 10:18
@injectives injectives changed the title Feature/update backend handler [WIP] testing Oct 31, 2023
@injectives injectives marked this pull request as draft October 31, 2023 10:18
@injectives injectives force-pushed the feature/update-backend-handler branch from e812208 to 2d160d3 Compare October 31, 2023 15:10
@injectives injectives changed the title [WIP] testing Update TestkitRequestProcessorHandler to adhere to req/res pattern Oct 31, 2023
@injectives injectives force-pushed the feature/update-backend-handler branch 6 times, most recently from 53c382e to 906252c Compare October 31, 2023 23:10
This update aims to make sure the Teskit backend does not break the request / response pattern. For instance, by sending 2 responses together in case of internal callbacks.
@injectives injectives force-pushed the feature/update-backend-handler branch from 906252c to 3bbc806 Compare November 1, 2023 11:15
@injectives injectives marked this pull request as ready for review November 1, 2023 14:18
@injectives injectives merged commit 99d32cb into neo4j:5.0 Nov 1, 2023
@injectives injectives deleted the feature/update-backend-handler branch November 1, 2023 14:26
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.

2 participants