Skip to content

coverage: deflake another#18252

Closed
alyssawilk wants to merge 1 commit intoenvoyproxy:mainfrom
alyssawilk:deflake2
Closed

coverage: deflake another#18252
alyssawilk wants to merge 1 commit intoenvoyproxy:mainfrom
alyssawilk:deflake2

Conversation

@alyssawilk
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
@alyssawilk
Copy link
Copy Markdown
Contributor Author

cc @KBaichoo

Copy link
Copy Markdown
Member

@rojkov rojkov left a comment

Choose a reason for hiding this comment

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

LGTM!

@jmarantz
Copy link
Copy Markdown
Contributor

format ...

EXPECT_DEATH(die_function(), "ASSERT_GUARDDOG_ACTION");
}


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To fix the format, can you remove this line?

Suggested change

jmarantz pushed a commit that referenced this pull request Sep 24, 2021
This is a clone of #18252 but with the formatting fixed.

Risk Level: None - test only
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Ryan Hamilton <rch@google.com>
@alyssawilk alyssawilk closed this Sep 27, 2021
soulxu pushed a commit to soulxu/envoy that referenced this pull request Oct 16, 2021
This is a clone of envoyproxy#18252 but with the formatting fixed.

Risk Level: None - test only
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Ryan Hamilton <rch@google.com>
@alyssawilk alyssawilk deleted the deflake2 branch August 4, 2022 01:08
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.

4 participants