Skip to content

Add unit tests for alertspb ToProto and ParseTemplates#7353

Open
archy-rock3t-cloud wants to merge 2 commits intocortexproject:masterfrom
sophotechlabs:test/alertspb-compat-unit-tests
Open

Add unit tests for alertspb ToProto and ParseTemplates#7353
archy-rock3t-cloud wants to merge 2 commits intocortexproject:masterfrom
sophotechlabs:test/alertspb-compat-unit-tests

Conversation

@archy-rock3t-cloud
Copy link
Copy Markdown
Contributor

What this PR does:

Adds unit tests for ToProto and ParseTemplates in pkg/alertmanager/alertspb

Which issue(s) this PR fixes:
N/A — test coverage improvement

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Artem Muterko <artem@sopho.tech>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 17, 2026
@SungJin1212
Copy link
Copy Markdown
Member

@archy-rock3t-cloud
Could you fix the lint CI?

@archy-rock3t-cloud
Copy link
Copy Markdown
Contributor Author

@SungJin1212 sure, updated

@archy-rock3t-cloud archy-rock3t-cloud force-pushed the test/alertspb-compat-unit-tests branch 2 times, most recently from a4fdfcf to 889ca8f Compare March 28, 2026 17:47
@archy-rock3t-cloud
Copy link
Copy Markdown
Contributor Author

@SungJin1212 please take a look at the last commit, it should fix flaky test if I understood the issue correctly

@SungJin1212
Copy link
Copy Markdown
Member

@archy-rock3t-cloud
Can you make another PR for the block cleaner flaky test?

Signed-off-by: Artem Muterko <artem@sopho.tech>
@archy-rock3t-cloud archy-rock3t-cloud force-pushed the test/alertspb-compat-unit-tests branch from c31a621 to 4c4a00f Compare March 30, 2026 13:07
@archy-rock3t-cloud
Copy link
Copy Markdown
Contributor Author

@SungJin1212 sure, here it is #7386

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/alertmanager lgtm This PR has been approved by a maintainer size/L type/tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants