2684
100%

Ran 28 Aug 2017 08:48PM UTC

Jobs 4

Files 13

Run time 4min

Badge

Embed ▾

pending completion

Build # 2684

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

4025 of 4117 relevant lines covered (97.77%)

13706.35 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	2684.1	28 Aug 2017 08:51PM UTC	97.74	Travis Job 2684.1
2	2684.2	28 Aug 2017 08:50PM UTC	97.69	Travis Job 2684.2
3	2684.3	28 Aug 2017 08:53PM UTC	97.69	Travis Job 2684.3
4	2684.4	28 Aug 2017 08:48PM UTC	97.69	Travis Job 2684.4

jshint / jshint / 2684
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2684

jshint / jshint / 2684 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2684

jshint / jshint / 2684
100%

README BADGES
x