1
100%
master: 100%

Ran 28 Aug 2017 08:51PM UTC

Files 13

Run time 1s

Badge

Embed ▾

Committed 28 Aug 2017 08:41PM UTC coverage: 97.741% (+0.05%) from 97.692%

Job # 2684.1

Build Type

push

travis-ci

Committed by

rwaldron

Commit Message

[[TEST]] Require all tests to specify char no.

The character of reported warnings/errors is of critical importance to
tooling built on top of JSHint. These values should be accurate, and
they should be stable between releases of the project.

Update the project's "test" API to require explicit specification of
expected character number. Update each test to provide this value.

Note: due to a deficiency in the test harness, lines that emitted
multiple instances of the same error/warning (with distinct line
numbers) would be satisfied by a single assertion for the relevant
error/warning. This change exposed that deficiency and necessitated the
introduction of additional assertions for such cases, e.g. the following
modification to `tests/unit/options.js`:

    -    .addError(11, "'j' used out of scope.") // 3x
    +    .addError(11, 14, "'j' used out of scope.")
    +    .addError(11, 21, "'j' used out of scope.")
    +    .addError(11, 28, "'j' used out of scope.")

Run Details

4024 of 4117 relevant lines covered (97.74%)

3426.58 hits per line

jshint / jshint / 2684 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2684.1

jshint / jshint / 2684 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2684.1

jshint / jshint / 2684 / 1
100%
master: 100%

README BADGES
x