mozilla / addons-linter / 4512

100%

push

travis-ci

fixes #1640 adds missing content scripts file rule for static strings (#1661)

* adds content script files existance validator, when files requested as a static string from js code

* fixes tests

* adds tests for new eslint rule

* fixes linter errors

* updates validation rule to match only static strings in file property of options object in executeScript call

* updates content_scripts_file_absent validator according to the comments in PR, adds some tests

* removes yarn.lock

* updates if conditions in content_scripts_file_absent validator, refactors test suites

* reduces nesting in the validator code, removes eslint exceptions and merges similar tests into one

* adds additional validation if browser.tabs.executeScript actually a call and test for it, plus minor comments to tests

Fixes #1640

880 of 915 branches covered (96.17%)

1555 of 1555 relevant lines covered (100.0%)

796.32 hits per line