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.

Sign up for GitHub

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

Jump to bottom

Fix test case validity check #273

Merged
merged 1 commit into from
Oct 20, 2024
Merged

Fix test case validity check #273

merged 1 commit into from
Oct 20, 2024

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Oct 20, 2024

Update TestPluginSetOutputTargetIsValidWithInvalidInput to not just assert that plugin.outputSink is set when an invalid output target is specified, but also that it is specifically set to the default plugin output target as intended.

@atc0005
Fix test case validity check
7ddb2d8 
Update `TestPluginSetOutputTargetIsValidWithInvalidInput` to
not just assert that plugin.outputSink is set when an invalid
output target is specified, but also that it is specifically
set to the default plugin output target as intended.

- refs GH-268
- refs GH-267
@atc0005 atc0005 added bug Something isn't working output/plugin plugin output intended for monitoring system ingest tests labels Oct 20, 2024
@atc0005 atc0005 added this to the v0.17.0 milestone Oct 20, 2024
@atc0005 atc0005 self-assigned this Oct 20, 2024
@atc0005 atc0005 merged commit d9dbfd9 into master Oct 20, 2024
30 of 31 checks passed
@atc0005 atc0005 deleted the i267-fix-test-case-validity-check-stable-branch branch October 20, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@atc0005 atc0005

Labels
bug Something isn't working output/plugin plugin output intended for monitoring system ingest tests
Projects
None yet
Milestone
v0.17.0
Development

Successfully merging this pull request may close these issues.
1 participant
@atc0005