Skip to content

test: fix and add mock ProtoBufferEq matchers#8459

Merged
mattklein123 merged 2 commits intoenvoyproxy:masterfrom
fredlas:PRT_proto_debugmsg
Oct 3, 2019
Merged

test: fix and add mock ProtoBufferEq matchers#8459
mattklein123 merged 2 commits intoenvoyproxy:masterfrom
fredlas:PRT_proto_debugmsg

Conversation

@fredlas
Copy link
Copy Markdown
Contributor

@fredlas fredlas commented Oct 2, 2019

Fix ProtoBufferEqIgnoringField's formatting, add ProtoBufferEqIgnoreRepeatedFieldOrdering.
Both are similar to the test/test_common/utility versions.

Risk Level: none
Testing: only changes format of test failure error messages

Signed-off-by: Fred Douglas <fredlas@google.com>
Signed-off-by: Fred Douglas <fredlas@google.com>
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit 7e1ad9c into envoyproxy:master Oct 3, 2019
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Oct 4, 2019
Signed-off-by: Fred Douglas <fredlas@google.com>
nandu-vinodan pushed a commit to nandu-vinodan/envoy that referenced this pull request Oct 17, 2019
Signed-off-by: Fred Douglas <fredlas@google.com>
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.

2 participants