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

feat(pubsub/pstest): Add reactor options to pstest server #2916

Merged
merged 1 commit into from Sep 25, 2020

Conversation

zhongduo
Copy link
Contributor

Add reactor options to pstest server.

This allows users to define customized reactor to a sub command handler. An example is to inject error in topic deletion.

Note: only the DeleteTopic is shown as an example here, after we settle down the naming and formatting, similar reactor will be added to all the functions.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 24, 2020
@zhongduo
Copy link
Contributor Author

/assign @hongalex

@hongalex hongalex self-requested a review September 24, 2020 18:44
@hongalex hongalex added the kokoro:run Add this label to force Kokoro to re-run the tests. label Sep 24, 2020
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Sep 24, 2020
@codyoss codyoss added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@zhongduo zhongduo changed the title WIP: Add reactor options to pstest server Add reactor options to pstest server Sep 25, 2020
@hongalex hongalex changed the title Add reactor options to pstest server feat(pubsub/pstest): Add reactor options to pstest server Sep 25, 2020
Copy link
Member

@hongalex hongalex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor formatting/comment nits, otherwise LGTM

pubsub/pstest/fake.go Outdated Show resolved Hide resolved
pubsub/pstest/fake.go Outdated Show resolved Hide resolved
pubsub/pstest/fake.go Outdated Show resolved Hide resolved
pubsub/pstest/fake_test.go Outdated Show resolved Hide resolved
pubsub/pstest/fake.go Outdated Show resolved Hide resolved
pubsub/pstest/fake.go Outdated Show resolved Hide resolved
@hongalex hongalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@hongalex hongalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@zhongduo zhongduo force-pushed the pstest-reactor-option branch 4 times, most recently from 7c5fd6c to 57044b2 Compare September 25, 2020 18:29
This allows users to define customized reactor to a sub command handler.
An example is to inject error in topic deletion. A generic ErrorInjection
is also provided to inject error to handlers.
@hongalex hongalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2020
@hongalex hongalex merged commit fc5b336 into googleapis:master Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants