Adding Integration Test for Phone Number Authentication in Firebase

[srushtisv]

Implement integration test for verifying phone number sign-in with Firebase Auth. Please review and approve.

[google-oss-bot]

	1 Warning
⚠️	Did you forget to add a changelog entry? (Add #no-changelog to the PR description to silence this warning.)

Generated by 🚫 Danger

[paulb777]

LLC is missing. See other files in the repo or https://github.com/firebase/firebase-ios-sdk/blob/main/scripts/check_copyright.sh#L21

[paulb777]

Thanks for working on tests. The new file should be added to the SwiftApiTests target in the Xcode project so it builds and runs.

When I do that locally, it has some failures.

Also, it would be great to test the async variations of the APIs as well.

[ncooke3]

The check CI can be fixed by running:

scripts/style.sh FirebaseAuth/Tests/SampleSwift/SwiftApiTests/PhoneAuthTest.swift

You can also run it without args though, but I find it sometimes slows when it checks all the source. I like to run git diff --name-only main | xargs scripts/style.sh to style everything on my branch different from main.

[ncooke3]

Since the plist wasn't changed in this PR, I think it makes sense to revert all xcode proj changes that mention the plists

[ncooke3]

Thanks for working on tests. The new file should be added to the SwiftApiTests target in the Xcode project so it builds and runs.

When I do that locally, it has some failures.

Also, it would be great to test the async variations of the APIs as well.

confirmed they are building and running:

PhoneAuthTests
    ✓ testSignInWithPhoneNumber (0.997 seconds)
    ✓ testSignInWithPhoneNumberAsync_Success (1.066 seconds)

from https://github.com/firebase/firebase-ios-sdk/actions/runs/10625088625/job/29454486166?pr=13538#step:6:340

[srushtisv]

Thanks for working on tests. The new file should be added to the SwiftApiTests target in the Xcode project so it builds and runs.

When I do that locally, it has some failures.

Also, it would be great to test the async variations of the APIs as well.

done

[paulb777]

@srushtisv Any reason this code hasn't yet merged to main? Ideally, we should merge larger PRs to main as soon as possible to avoid unnecessary branching.