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

gcloud test-targets flag can match multiple classes even if only one is configured #1227

Closed
hasan-selman-kara-leshop opened this issue Oct 9, 2020 · 1 comment · Fixed by #1233
Assignees

Comments

@hasan-selman-kara-leshop

Describe the bug

A clear and concise description of what the bug is.

If you have a package with the following files:

  • com.xyz.MyTest1.kt
  • com.xyz.MyTest2.kt
  • com.xyz.MyTest3.kt
  • com.xyz.MyTest10.kt
  • com.xyz.MyTest11.kt
  • com.xyz.MyTest12.kt
  • com.xyz.MyTest13.kt

And configure the flank yaml as follows:

gcloud:
  test-targets:
    - class com.xyz.MyTest1

Flank will match the following test classes: MyTest1; MyTest10; MyTest11; MyTest12; MyTest13

And — let's say each test class has one test method — flank will print the following output

Uploading app.apk ....
Uploading app-androidTest.apk .
5 tests / 1 shard

To Reproduce

  1. Create the test files as described above, each with one test method

My flank.yml

gcloud:
  test-targets:
    - class com.xyz.MyTest1
  record-video: true
  timeout: 60m
  async: false
  app: ./app.apk
  test: ./app-androidTest.apk
  auto-google-login: true
  use-orchestrator: true
  environment-variables:
    clearPackageData: true
  directories-to-pull:
    - /sdcard/screenshots
  performance-metrics: true
  device:
    - model: NexusLowRes
      version: 28
flank:
  testShards: 1
  ignore-failed-tests: true
  repeatTests: 1

Expected behavior
It should only run the tests of the specified class.

Details (please complete the following information):

Flank version 20.09.3

@bootstraponline
Copy link
Contributor

Thanks for reporting! We’ll take a look.

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

Successfully merging a pull request may close this issue.

3 participants