Auth I/O '23 - update other SDK iTests to use new API #1268
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chkuang-g
approved these changes
Apr 13, 2023
github-actions
bot
added
the
tests: in-progress
❌ Integration test FAILEDRequested by @DellaBitta on commit 4351bcd
Add flaky tests to go/fpl-cpp-flake-tracker |
firebase-workflow-trigger
bot
removed
the
tests: in-progress
jonsimantov
added a commit
that referenced
this pull request
May 3, 2023
* initial Auth deprecations * all three platforms build * FirebaseAuth.java and auth_test.cc fixes * format * bundle identifer * removed explicit from CurrentUserBlockListener constructor * added tests, desktop impl, desktop passes * iOS builds and tests pass * android builds and passes tests * [Auth] PhoneAuthCredential support (#1253) Add a new `PhoneAuthCredential` and return it from a new `PhoneAuthProvider::GetCredential` method. Update the `OnVerificationCompleted` method for Phone Auth to push `PhoneAuthCredential` objects to the application. This new PhoneAuthCredential will be used in other new API methods that will introduced later. For now it returns a `sms_code` string on Android platforms. * [Auth] PhoneAuthOptions struct for VerifyPhoneNumber (#1257) Add a new `PhoneAuthProvider::VerifyPhoneNumber` method that takes a `PhoneAuthOptions` structure. For the C++ implementation on Android, update the current `VerifyPhoneNumber` method to call into the new offering. The older Android SDK `VerifyPhoneNumber` method is deprecated and will probably be removed from the Android SDK at Google I/O, so let's fix this before it becomes a problem. * Add new Auth and User methods to replace the deprecated ones (#1264) Adds the methods to replace those that were deprecated in the Auth and User classes. Duplicated every Integration Test so that there's one test for the deprecated API and one test for the new API. **Note:** `User::LinkAndRetreiveveDataWithCredential` has been removed as it has the same functionality as the new `User::LinkWithCredential`. The major changes of this PR are in four different areas: 1. The addition of tests to `integration_test.cc`. 2. The addition of new methods in the User and Auth implementation. 3. The plaform specific functions that close Futures on callbacks. Functions that faciliate `Future<SignInResult>` and `Future<User*>` completions, marshalling native data into those two return types have been two new compatriot versions for `Future<AuthResult>` and `Future<User>` return types. 4. The internal experimental implementation of FederatedAuth for desktop calls into the user's application to facilitate a auth. The class `FederatedAuthProvider` has been expanded with callbacks that support `AuthResult` objects in addition to the deprecated `SignInResult` objects. Fixes b/138788092. * Auth I/O '23 - update other SDK iTests to use new API (#1268) Numerous product SDKs have iTests which interact with Auth. In a previous feature branch change these tests were changed to use `Future<User*> Auth::current_user_DEPRECATED()`, for instance, as the new Auth API wasn't yet written. This PR updates the tests to use the new APIs, such as `Future<AuthResult> Auth::current_user()`. * Default no-op for deprecated virtual function (#1275) `Listener::OnVerificationCompleted(Credential)` is going to be deprecated and should not be required to implement in the derived class. * Fix Doxygen warnings. * Additional docs fixes. * Fix lint warning. * Added readme note about deprecated methods. * Remove "_DEPRECATED" from LinkAndRetrieveDataWithCredential (#1290) * Remove "_DEPRECATED" from LinkAndRetrieveDataWithCredential No need a DEPRECATED suffix since there's no conflict in a new overloaded version of the method. * Fix kUserFn_LinkAndRetrieveDataWithCredential * Format files. --------- Co-authored-by: Jon Simantov <jsimantov@gmail.com> * Auth Google I/0 23 - Update doc examples to use new API (#1271) Update auth user doc examples to use the new Auth API. * Add comparison operator for Auth::User (#1301) * Add comparison operator for Auth::User * Change to use is_valid() * Add tests * format_code * Add != operator * add comments * Add documentation * update TODO * format code --------- Co-authored-by: Jon Simantov <jsimantov@google.com> --------- Co-authored-by: chkuang-g <31869252+chkuang-g@users.noreply.github.com> Co-authored-by: Jon Simantov <jsimantov@google.com> Co-authored-by: Jon Simantov <jsimantov@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Numerous product SDKs have iTests which interact with Auth. In a previous feature branch change these tests were changed to use
Future<User*> Auth::current_user_DEPRECATED(), for instance, as the new Auth API wasn't yet written.This PR updates the tests to use the new APIs, such as
Future<AuthResult> Auth::current_user().Testing
Integration Test Run
Type of Change
Place an
xthe applicable box:Notes
Release Notessection ofrelease_build_files/readme.md.