Munter / hyperlink / 230

79%

Pull #143

travis-ci

Expect one more failing external-redirect test in an unrelated test

Instead of

  push(
    null,
    {
      ok: true,
      operator: 'external-redirect',
      name: 'external-redirect https://elsewhere.com/',
      at: 'https://example.com/ (1:39) <script src="https://elsewhere.com/">...</script>',
      expected: '302 https://elsewhere.com/ --> 200 http://elsewhere.com/redirectTarget'
    }
  ); at reportTest (lib/index.js:104:9)
  push(
    null,
    {
      ok: true,
      operator: 'external-redirect',
      name: 'external-redirect http://elsewhere.com/redirectTarget',
      at: 'https://elsewhere.com/',
      expected: '302 http://elsewhere.com/redirectTarget --> 200 https://elsewhere.com/redirectTarget'
    }
  ); at reportTest (lib/index.js:104:9)

we now get

  push(
    null,
    {
      ok: false,
      operator: 'external-redirect',
      name: 'external-redirect https://elsewhere.com/',
      at: 'https://example.com/ (1:39) <script src="https://elsewhere.com/">...</script>',
      expected: '302 https://elsewhere.com/ --> 200 https://elsewhere.com/redirectTarget',
      actual: '302 https://elsewhere.com/ --> 302 http://elsewhere.com/redirectTarget --> 200 https://elsewhere.com/redirectTarget'
    }
  ); at reportTest (lib/index.js:114:7)

which is correct

186 of 239 branches covered (77.82%)

46 of 46 new or added lines in 1 file covered. (100.0%)

246 of 307 relevant lines covered (80.13%)

145.6 hits per line