Skip to content

Firestore integration tests on tvos #472

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

Merged
merged 3 commits into from
Jun 17, 2021

Conversation

vimanyu
Copy link
Contributor

@vimanyu vimanyu commented Jun 16, 2021

Changes done to XCode project for firestore that allows us to run integration tests on device and simulator.

added a tvos target to XCode project for storage.
updated Podfile to build pods for tvos platform for this newly added target
To test this,

Make sure to have a valid GoogleService-Info.plist.
pod install
open integration_test.xcworkspace
Drag and drop firebase.xcframework, firebase_auth.xcframework and firebase_firestore.xcframework

All tests pass on simulator and device. Exposed a problem in packaging with missing some required firestore headers.

@vimanyu vimanyu requested a review from DellaBitta June 16, 2021 03:40
@google-cla google-cla bot added the cla: yes label Jun 16, 2021
Copy link
Contributor

@DellaBitta DellaBitta left a comment

Choose a reason for hiding this comment

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

Remove user-specific files:

  • firestore/integration_test/integration_test.xcodeproj/project.xcworkspace/xcuserdata/vimanyujain.xcuserdatad/UserInterfaceState.xcuserstate
  • firestore/integration_test/integration_test.xcodeproj/xcuserdata/vimanyujain.xcuserdatad/xcschemes/xcschememanagement.plist
  • firestore/integration_test/integration_test.xcworkspace/xcuserdata/vimanyujain.xcuserdatad/UserInterfaceState.xcuserstate

@vimanyu
Copy link
Contributor Author

vimanyu commented Jun 17, 2021

Removed user specific files.

@vimanyu vimanyu merged commit 84f3fc6 into main Jun 17, 2021
@vimanyu vimanyu deleted the feature/tvos-support-integration-tests-firestore branch June 17, 2021 19:32
@github-actions github-actions bot added the tests: in-progress This PR's integration tests are in progress. label Jun 17, 2021
@github-actions
Copy link

github-actions bot commented Jun 17, 2021

❌  Integration test FAILED

Requested by @vimanyu on commit 84f3fc6
Last updated: Thu Jun 17 15:25 PDT 2021
View integration test log & download artifacts

Failures Configs
remote_config [TEST] [FAILURE] [iOS] [macos] [ios_target]
(1 failed tests)  TestFetchInterval

@github-actions github-actions bot added the tests: failed This PR's integration tests failed. label Jun 17, 2021
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Jun 17, 2021
@firebase firebase locked and limited conversation to collaborators Jul 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes tests: failed This PR's integration tests failed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants