Skip to content

Conversation

fscgolden
Copy link
Contributor

Co-authored-by: Chris LeBrett [email protected]
Co-authored-by: Robby Tipton [email protected]
Co-authored-by: Joseph Park [email protected]
Co-authored-by: David Kim [email protected]
Co-authored-by: Kevin HoEun Lee [email protected]

Co-authored-by: Chris LeBrett <[email protected]>
Co-authored-by: Robby Tipton <[email protected]>
Co-authored-by: Joseph Park <[email protected]>
Co-authored-by: David Kim <[email protected]>
Co-authored-by: Kevin HoEun Lee <[email protected]>
Copy link
Contributor

@khobread khobread left a comment

Choose a reason for hiding this comment

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

Good unit tests. Thank you Chris!!

Copy link
Contributor

@davidkim7773 davidkim7773 left a comment

Choose a reason for hiding this comment

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

Looks like a good test file Chris.

Copy link
Contributor

@joeepark joeepark left a comment

Choose a reason for hiding this comment

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

Nice tests Chris!

Copy link
Contributor

@RobbyTipton RobbyTipton left a comment

Choose a reason for hiding this comment

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

Looks good. Glad to have test coverage for our new component.

@fscgolden fscgolden merged commit cd93166 into master Jun 15, 2022
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.

5 participants