Skip to content
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

firebase appdistribution:distribute attempts to run tests even though --test-devices flag was not used #6749

Closed
sofi-kry opened this issue Feb 7, 2024 · 5 comments

Comments

@sofi-kry
Copy link

sofi-kry commented Feb 7, 2024

[REQUIRED] Environment info

firebase-tools: 13.2.0

Platform: Ubuntu

[REQUIRED] Test case

firebase appdistribution:distribute x.apk --app id  --groups "android-internal-testers" --release-notes "notes"

[REQUIRED] Steps to reproduce

firebase appdistribution:distribute x.apk --app id  --groups "android-internal-testers" --release-notes "notes"

[REQUIRED] Expected behavior

Automated tests are not run, given that the flag --test-devices is not included. The job ends successfully.

[REQUIRED] Actual behavior

Screenshot 2024-02-07 at 14 27 52
@google-oss-bot
Copy link
Contributor

This issue does not have all the information required by the template. Looks like you forgot to fill out some sections. Please update the issue with more information.

@MaesterChestnut
Copy link
Contributor

Apologies for this issue. Releases are currently being uploaded despite the error message. We're currently working on a fix, but in the meantime please use 13.1.0 to remove the error message. We'll post up to date information on our status page: https://status.firebase.google.com/

@MaesterChestnut
Copy link
Contributor

We just pushed a fix so that firebase-tools@13.2.0 should no longer throw unexpected errors while uploading releases. Please try again to confirm @sofi-kry

@joehan
Copy link
Contributor

joehan commented Feb 7, 2024

Closing this as fixed by #6750 - we are release 13.2.1 including this fix shortly. Apologies for the misleading errors all!

@sofi-kry
Copy link
Author

sofi-kry commented Feb 8, 2024

It works now, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants