query: introduce new ExpectLengthForMultiple query check#607
query: introduce new ExpectLengthForMultiple query check#607
ExpectLengthForMultiple query check#607Conversation
There was a problem hiding this comment.
I'm not attached to this name and open to other suggestions, this just seems the most descriptive at first glance to me.
There was a problem hiding this comment.
Yeah, tough naming 🤔
Two things I think would be nice to communicate with the name are that it's checking total length and that it's checking via a form of matching resource addresses via regex. What do you think of ExpectTotalLengthForMatching? ExpectTotalLengthForMultiple? 🤔
There was a problem hiding this comment.
I went with ExpectTotalLengthForMatching
austinvalle
left a comment
austinvalle
left a comment
There was a problem hiding this comment.
Looking good! I left some thoughts
There was a problem hiding this comment.
Yeah, tough naming 🤔
Two things I think would be nice to communicate with the name are that it's checking total length and that it's checking via a form of matching resource addresses via regex. What do you think of ExpectTotalLengthForMatching? ExpectTotalLengthForMultiple? 🤔
stephybun
force-pushed
the
f/regex-expect-length
branch
from
88908bf to
273cddb
Compare
|
Failing CI is unrelated to this change |
2 participants
Related Issue
Related to #597
Description
ExpectLengthForMultiplefor validating the number of found resources of multiple list blocksExpectLengthAtLeastcheckRollback Plan
Changes to Security Controls
Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.
None