Skip to content

Feature/tvos support integration tests functions #471

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 functions 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_functions.xcframework

All tests pass on simulator and device.

@vimanyu vimanyu requested a review from DellaBitta June 16, 2021 02:48
@vimanyu vimanyu self-assigned this Jun 16, 2021
@google-cla google-cla bot added the cla: yes label 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 7798615
Last updated: Tue Jun 15 20:19:11 PDT 2021
View integration test results

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

I think you should remove files from xcuserdata/vimanyujain*, right?

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

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

@sunmou99
Copy link
Contributor

I think you should remove files from xcuserdata/vimanyujain*, right?

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

I remember that "integration_test.xcodeproj/project.pbxproj" is the only file we want under "integration_test.xcodeproj", we can safely remove the rest.
This may apply to "integration_test.xcworkspace" as well.
@jonsimantov Could you provide more information?

@vimanyu
Copy link
Contributor Author

vimanyu commented Jun 17, 2021

Removed user specific files.

@vimanyu vimanyu merged commit 2e307d7 into main Jun 17, 2021
@vimanyu vimanyu deleted the feature/tvos-support-integration-tests-functions branch June 17, 2021 19:32
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. and removed tests: succeeded This PR's integration tests succeeded. labels Jun 17, 2021
@github-actions
Copy link

⏳  Integration test in progress...

Requested by @vimanyu on commit 2e307d7
Last updated: Thu Jun 17 12:33 PDT 2021
View integration test log & download artifacts

@github-actions github-actions bot added the tests: failed This PR's integration tests failed. 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. tests: in-progress This PR's integration tests are in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants