Skip to content

Feature/tvos support integration tests storage #470

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 4 commits into from
Jun 17, 2021

Conversation

vimanyu
Copy link
Contributor

@vimanyu vimanyu commented Jun 16, 2021

Changes done to XCode project for storage 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_storage.xcframework

All tests pass on simulator but one (TestPutFileAndGetFile) is failing on device ( internal ticket: b/91209621).

@google-cla google-cla bot added the cla: yes label Jun 16, 2021
@vimanyu vimanyu requested a review from DellaBitta June 16, 2021 02:12
@vimanyu vimanyu self-assigned this Jun 16, 2021
@vimanyu vimanyu added the tests-requested: quick Trigger a quick set of integration tests. label Jun 16, 2021
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. and removed tests-requested: quick Trigger a quick set of integration tests. labels Jun 16, 2021
@github-actions
Copy link

github-actions bot commented Jun 16, 2021

✅  Integration test succeeded!

Requested by @vimanyu on commit c6e6816
Last updated: Tue Jun 15 19:46:51 PDT 2021
View integration test results

@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Jun 16, 2021
@github-actions github-actions bot added the tests: succeeded This PR's integration tests succeeded. 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:

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

@vimanyu
Copy link
Contributor Author

vimanyu commented Jun 17, 2021

Thanks for pointing it out! Done.

@vimanyu vimanyu merged commit fccb88c into main Jun 17, 2021
@vimanyu vimanyu deleted the feature/tvos-support-integration-tests-storage 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 fccb88c
Last updated: Thu Jun 17 15:07 PDT 2021
View integration test log & download artifacts

Failures Configs
firestore [TEST] [FAILURE] [MacOS] [openssl]
(1 failed tests)  AppendObjectToArrayViaUpdate
installations [TEST] [FAILURE] [Android] [macos] [android_latest]
(1 failed tests)  TestGettingTokenTwiceMatches
remote_config [TEST] [FAILURE] [iOS] [macos] [ios_target]
(1 failed tests)  TestFetchInterval

@github-actions github-actions bot added tests: failed This PR's integration tests failed. and removed tests: succeeded This PR's integration tests succeeded. labels 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