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

Command line paramters not prioritised the way they are in other applications #3845

Closed
j4m3s opened this issue Feb 14, 2021 · 1 comment
Closed
Assignees
Milestone

Comments

@j4m3s
Copy link

j4m3s commented Feb 14, 2021

Not sure if this is a bug or a feature request. The command-line runner is not doing as I would expect :)

Configuration

Operating system: Ubuntu 20.04

PlatformIO Version (platformio --version): PlatformIO Core, version 5.1.0

Description of problem

I would expect that command-line switches override values in the platformio.ini file, because that's the way most (all?) other packages work. E,g. with ansible, I specify config vars in an ini file that override the defaults. If I then specify a command-line argument, the argument overrides both the value in the ini file and the application's defaults. That doesn't appear to be the case with the "pio test --filter" switch at least.

Steps to Reproduce

  1. Add a test_filter = abc, def, xyz value into a [env:native] section in platformio.ini
  2. run pio test -e native -f xyz

Actual Results

The command-line flag is ignored/ overridden by the ini file entry - all test suites specified in the ini file are run

Expected Results

I would expect to only run the xyz test suite in the native environment.

Additional info

This is a problem because a test_filter in an env section is common, as some tests require the microprocessor so will not run natively. Being able to filter to specific tests when running them natively is very useful when adding new features.

@ivankravets ivankravets added this to the 5.2.1 milestone Sep 13, 2021
@ivankravets ivankravets removed this from the 5.2.1 milestone Sep 27, 2021
@ivankravets ivankravets added this to the 5.3.0 milestone Apr 11, 2022
@ivankravets ivankravets self-assigned this Apr 14, 2022
@ivankravets
Copy link
Member

Thanks for the report! Please re-test with pio upgrade --dev.

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

3 participants