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

#835 added printing all test environment information #901

Merged
merged 2 commits into from
Jul 20, 2020

Conversation

piotradamczyk5
Copy link
Contributor

Fixes #835

Test Plan

How do we know the code works?

Use on of a new CLI option to test this PR:

  • flank firebase test android|ios test-environment or flank android|ios test-environment - print Android|iOS available versions and devices to test against

More listing options of the test environment will be added in the next tasks

Checklist

  • printing Android software versions and devices list together
  • printing iOS software versions and devices list together
  • Documented
  • Unit tested
  • release_notes.md updated

@bootstraponline
Copy link
Contributor

$ gcloud firebase test

Available groups for gcloud firebase test:

      android                 Command group for Android application testing.
      ios                     Command group for iOS application testing.
      network-profiles        Explore network profiles available for testing.
$ gcloud firebase test android

Available groups for gcloud firebase test android:

      locales                 Explore Android locales available for testing.
      models                  Explore Android models available in the Test
                              Environment catalog.
      versions                Explore Android versions available for testing.

Available commands for gcloud firebase test android:

      run                     Invoke a test in Firebase Test Lab for Android and
                              view test results.

Flank CLI commands should mirror the gcloud commands. The reason Flank supports the more verbose format: "flank firebase test android test-environment" is so that you can replace "flank" with "gcloud" and get the same result. This also helps simplify testing as we can compare Flank against the reference implementation.

It doesn't look like test-environment has a parallel command in the gcloud CLI tool?

@piotradamczyk5
Copy link
Contributor Author

piotradamczyk5 commented Jul 20, 2020

$ gcloud firebase test

Available groups for gcloud firebase test:

      android                 Command group for Android application testing.
      ios                     Command group for iOS application testing.
      network-profiles        Explore network profiles available for testing.
$ gcloud firebase test android

Available groups for gcloud firebase test android:

      locales                 Explore Android locales available for testing.
      models                  Explore Android models available in the Test
                              Environment catalog.
      versions                Explore Android versions available for testing.

Available commands for gcloud firebase test android:

      run                     Invoke a test in Firebase Test Lab for Android and
                              view test results.

Flank CLI commands should mirror the gcloud commands. The reason Flank supports the more verbose format: "flank firebase test android test-environment" is so that you can replace "flank" with "gcloud" and get the same result. This also helps simplify testing as we can compare Flank against the reference implementation.

It doesn't look like test-environment has a parallel command in the gcloud CLI tool?

It does not have similar command for gcloud. This is special for flank :) Cause flank is more than gcloud.
Basically this command allow you to print all environment information just in one command

@piotradamczyk5 piotradamczyk5 merged commit 611e0f0 into master Jul 20, 2020
@piotradamczyk5 piotradamczyk5 deleted the #835-all-config branch July 20, 2020 10:55
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.

Show available devices
3 participants