Polymer / tools

Polymer Tools Monorepo
BSD 3-Clause "New" or "Revised" License
430 stars 200 forks source link

`polymer test --config-file` does not work. #2582

Open hastebrot opened 6 years ago

hastebrot commented 6 years ago

Description

There is a discrepancy between polymer test --config-file and wct --configFile. As a result polymer test completely ignores the file given to it using --config-file.

Source code of the respective CLI option definitions:

This seems to be a bug in polymer-cli, not in web-component-tester.

Versions & Environment

Steps to Reproduce

Create wct-headless.conf.json:

{
  "suites": [
    "test/index.html"
  ],
  "environmentImports": [
    "test-fixture/test-fixture.html"
  ],
  "plugins": {
    "local": {
      "browserOptions": {
        "chrome": [
          "headless",
          "disable-gpu",
          "no-sandbox"
        ],
        "firefox": [
          "-headless"
        ]
      }
    }
  }
}

Run ./node_modules/.bin/polymer test --local chrome --local firefox --config-file wct-headless.config.json.

Expected Results

Should run browsers in headless mode.

Actual Results

Does run browsers with window, i.e. ignores the config file. It works, when I rename wct-headless.config.json to wct.config.json. It complains when I use --configFile instead of --config-file. It does not complain when I give an unexistent file, e.g. --config-file wct-foobar.config.json.

hastebrot commented 6 years ago

This is a duplicate of #919.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.