1
96%
master: 92%

Ran 23 Mar 2017 11:04AM UTC

Files 30

Run time 1s

Badge

Embed ▾

Committed 23 Mar 2017 10:51AM UTC coverage: 95.874% (+2.0%) from 93.907%

Job # 3168.1

Build Type

push

travis-ci

Committed by

honzajavorek

Commit Message

fix: Test and fix the Content-Length calculation behavior

Dredd should prevent user errors by always calculating the Content-Length
header value. If it differs from what user provided, it should warn
and use the calculated value anyway. There is no reason for user to
override the calculated value and having the value out of sync with
body size just brings all sorts of strange behavior.

Also, Dredd needs to calculate the value before the 'request' library
does it, because

1) it needs to ensure the behavior described above,
2) setting Content-Length to zero if there's no body seems to be like
   a good thing to do (and 'request' doesn't do it),
3) the added header needs to be propagated back to the transaction
   object so Dredd is transparent about what was actually sent to the
   server under test

Close https://github.com/apiaryio/dredd/issues/751
Close https://github.com/apiaryio/dredd/issues/408

Run Details

1952 of 2036 relevant lines covered (95.87%)

243.65 hits per line

apiaryio / dredd / 3168 / 1
96%
master: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3168.1

apiaryio / dredd / 3168 / 1 96% master: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3168.1

apiaryio / dredd / 3168 / 1
96%
master: 92%

README BADGES
x