Skip to content

chore: write shell script for running swiftui tests from terminal#1346

Merged
russellwheatley merged 2 commits into
mainfrom
run-tests-from-script
May 19, 2026
Merged

chore: write shell script for running swiftui tests from terminal#1346
russellwheatley merged 2 commits into
mainfrom
run-tests-from-script

Conversation

@russellwheatley
Copy link
Copy Markdown
Member

Hey there! So you want to contribute to FirebaseUI? Before you file this pull request, follow these steps:

  • Read the contribution guidelines.
  • If this has been discussed in an issue, make sure to mention the issue number here. If not, go file an issue about this to make sure this is a desirable change.
  • If this is a new feature please co-ordinate with someone on FirebaseUI-Android to make sure that we can implement this on both platforms and maintain feature parity.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new bash script, swiftui-tests.sh, designed to run SwiftUI Auth unit, integration, and UI tests locally, mirroring the GitHub Actions workflow. It also updates .gitignore to exclude test logs and results, and adds documentation to CONTRIBUTING.md. Review feedback suggests improving the script's flexibility and performance by using a more widely available default simulator device, reducing the wait time for the Firebase emulator to start, and allowing the Firebase project ID to be configured via an environment variable.

Comment thread swiftui-tests.sh
Comment thread swiftui-tests.sh Outdated
Comment thread swiftui-tests.sh Outdated
@russellwheatley russellwheatley merged commit 2be3c12 into main May 19, 2026
10 checks passed
@russellwheatley russellwheatley deleted the run-tests-from-script branch May 19, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants