Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Vertex AI] Add test app for integration tests #13960

Merged
merged 14 commits into from
Oct 25, 2024
Merged

Conversation

andrewheard
Copy link
Contributor

@andrewheard andrewheard commented Oct 23, 2024

  • Added a test app to host Vertex AI in Firebase integration tests.
    • This approach provides keychain access, which allows us to use App Check with the Debug Provider.
  • Added tests to check the Firebase Auth / Storage integration.

#no-changelog

@andrewheard andrewheard marked this pull request as ready for review October 25, 2024 21:36
Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

This is great!

My only concern is that I don't see any tests in the log at https://github.com/firebase/firebase-ios-sdk/actions/runs/11525405469/job/32088217991

Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

Maybe the minimum iOS version needs to be set on the xcodebuild command?

@andrewheard
Copy link
Contributor Author

My only concern is that I don't see any tests in the log at firebase/firebase-ios-sdk/actions/runs/11525405469/job/32088217991

Interestingly it only seems to produce output when I intentionally make the tests fail (e.g., this previous run). I was also able to see the calls to App Check in the Firebase Console for this CI run (they now show verified).

Maybe the minimum iOS version needs to be set on the xcodebuild command?

I'll dig into why the tests aren't listed one by one when they succeed in a follow-up PR. I'll also drop the deployment target to iOS 15 (it's unnecessarily high at 17.2 -- I forgot to change the default in the new Xcode project).

@andrewheard andrewheard merged commit 1077879 into main Oct 25, 2024
64 checks passed
@andrewheard andrewheard deleted the ah/vertex-test-app branch October 25, 2024 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants