-
Notifications
You must be signed in to change notification settings - Fork 13
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
Test Firebase Test Lab Integration #28
Test Firebase Test Lab Integration #28
Conversation
sjcqs
commented
Apr 21, 2020
- Add a daily runner running both tests and firebase test lab integration
- Run firebase test lab integration test when releasing
Use the same version as the other tests
95a5d5e
to
e75f7c0
Compare
e75f7c0
to
c14a7c2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall great PR! Thank you for this major addition!
ci_cd.sh
Outdated
@@ -22,19 +22,22 @@ function usage { | |||
echo " --ndk_version <version> Install a specific Android NDK version from \`sdkmanager --list\`" | |||
echo " --build Build image" | |||
echo " --test Test image" | |||
echo " --test-lab Test instrumented tests on Firebase Test Lab" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not we pick a name a bit more different from test
parameter as it might be confusing? test
will run all unit
or local
test. Maybe we can use the same semantic as https://developer.android.com/reference/android/test/suitebuilder/annotation/LargeTest. So we would have test
and large-test
options. We could evolve these parameters documentation this way:
test
: Test image with local tests local testslarge-test
: Test image with remote services (Firebase Test Lab)
What do you think about that?
ci_cd.sh
Outdated
@@ -22,22 +22,22 @@ function usage { | |||
echo " --ndk_version <version> Install a specific Android NDK version from \`sdkmanager --list\`" | |||
echo " --build Build image" | |||
echo " --test Test image" | |||
echo " --test-lab Test instrumented tests on Firebase Test Lab" | |||
echo " --large-test Test instrumented tests on Firebase Test Lab" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You updated value name, but not the description, is this intended? See #28 (comment). Also the doc should be align l25