Skip to content

Add sanity checks for passed in options and initializers to GIDVerifyAccountDetail. - #401

Merged
brnnmrls merged 20 commits into
briannamorales/vwg-flowfrom
briannamorales/optionsValidation
May 14, 2024
Merged

brnnmrls merged 20 commits into
briannamorales/vwg-flowfrom
briannamorales/optionsValidation

Conversation

@brnnmrls

Copy link
Copy Markdown
Member

Begin implementing -[GIDVerifyAccountDetail verifyAccountDetailsInteractivelyWithOptions:] starting with input validation to ensure that the client has passed in valid parameters, provided a valid presenting view controller, and defined the proper callback schemes. Additionally, add tests for GIDVerifyAccountDetail like ensuring that a configuration was set which will be covered by the created initializers.

@brnnmrls
brnnmrls marked this pull request as ready for review April 25, 2024 22:50
@brnnmrls
brnnmrls requested a review from mdmathias April 25, 2024 22:50
@brnnmrls brnnmrls self-assigned this Apr 26, 2024
@brnnmrls brnnmrls linked an issue Apr 26, 2024 that may be closed by this pull request
@brnnmrls
brnnmrls requested a review from mdmathias May 2, 2024 17:41
Comment thread GoogleSignIn/Tests/Unit/GIDSignInTest.m Resolved
@brnnmrls brnnmrls linked an issue May 9, 2024 that may be closed by this pull request
@brnnmrls
brnnmrls requested a review from mdmathias May 9, 2024 21:51
@brnnmrls
brnnmrls requested a review from mdmathias May 10, 2024 23:45

@mdmathias mdmathias left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just a small comment to improve a doc comment.

@brnnmrls
brnnmrls merged commit 4b7802d into briannamorales/vwg-flow May 14, 2024
@brnnmrls
brnnmrls deleted the briannamorales/optionsValidation branch May 14, 2024 00:30
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.

Fix unstopped strict mocks in GIDSignInTest. Start the interactive add scopes flow.

2 participants