Auth Google IO 23 (#1255)

* 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 <[email protected]>

* 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 <[email protected]>

---------

Co-authored-by: chkuang-g <[email protected]>
Co-authored-by: Jon Simantov <[email protected]>
Co-authored-by: Jon Simantov <[email protected]>

Author: 3 people

app_check/integration_test/src/integration_test.cc

@@ -406,12 +406,12 @@ void FirebaseAppCheckTest::TerminateFunctions() {
 }
 
 void FirebaseAppCheckTest::SignIn() {
-  if (auth_->current_user() != nullptr) {
+  if (auth_->current_user().is_valid()) {
     // Already signed in.
     return;
   }
   LogDebug("Signing in.");
-  firebase::Future<firebase::auth::User*> sign_in_future =
+  firebase::Future<firebase::auth::AuthResult> sign_in_future =
       auth_->SignInAnonymously();
   WaitForCompletion(sign_in_future, "SignInAnonymously");
   if (sign_in_future.error() != 0) {
@@ -426,15 +426,15 @@ void FirebaseAppCheckTest::SignOut() {
     // Auth is not set up.
     return;
   }
-  if (auth_->current_user() == nullptr) {
+  if (!auth_->current_user().is_valid()) {
     // Already signed out.
     return;
   }
-  if (auth_->current_user()->is_anonymous()) {
+  if (auth_->current_user().is_anonymous()) {
     // If signed in anonymously, delete the anonymous user.
-    WaitForCompletion(auth_->current_user()->Delete(), "DeleteAnonymousUser");
+    WaitForCompletion(auth_->current_user().Delete(), "DeleteAnonymousUser");
     // If there was a problem deleting the user, try to sign out at least.
-    if (auth_->current_user()) {
+    if (auth_->current_user().is_valid()) {
       auth_->SignOut();
     }
   } else {
@@ -443,11 +443,11 @@ void FirebaseAppCheckTest::SignOut() {
     auth_->SignOut();
 
     // Wait for the sign-out to finish.
-    while (auth_->current_user() != nullptr) {
+    while (auth_->current_user().is_valid()) {
       if (ProcessEvents(100)) break;
     }
   }
-  EXPECT_EQ(auth_->current_user(), nullptr);
+  EXPECT_FALSE(auth_->current_user().is_valid());
 }
 
 firebase::database::DatabaseReference FirebaseAppCheckTest::CreateWorkingPath(
@@ -559,7 +559,7 @@ TEST_F(FirebaseAppCheckTest, TestSignIn) {
   InitializeAppCheckWithDebug();
   InitializeApp();
   InitializeAuth();
-  EXPECT_NE(auth_->current_user(), nullptr);
+  EXPECT_TRUE(auth_->current_user().is_valid());
 }
 
 TEST_F(FirebaseAppCheckTest, TestDebugProviderValidToken) {

auth/CMakeLists.txt

@@ -51,6 +51,7 @@ build_flatbuffers("${flatbuffer_schemas}"
 # Common source files used by all platforms
 set(common_SRCS
     src/auth.cc
+    src/credential.cc
     src/common.cc
     src/common.h
     src/data.h