Skip to content
New issue

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

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

Already on GitHub? # to your account

AsyncChannel: Provide throwing finish method on test stream (#2493) #10

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

1proprogrammerchant
Copy link
Owner

Motivation

We recently provided testing utilities for the NIOAsyncChannelInboundStream. On thing that was missing is a way to finish the stream with an error.

Modification

This PR provides a finish() method that takes an error which is thrown from the inbound stream.

Result

Better way to test code relying on the AsyncChannel work.

[One line description of your change]

Motivation:

[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]

Modifications:

[Describe the modifications you've done.]

Result:

[After your change, what will change.]

# Motivation
We recently provided testing utilities for the `NIOAsyncChannelInboundStream`. On thing that was missing is a way to finish the stream with an error.

# Modification
This PR provides a `finish()` method that takes an error which is thrown from the inbound stream.

# Result
Better way to test code relying on the AsyncChannel work.
@1proprogrammerchant 1proprogrammerchant merged commit 312285c into 1proprogrammerchant:main Aug 8, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants