Dredd: Sometimes CLI reporter outputs 'NaNms'

[honzajavorek]

pass: GET /message duration: NaNms

[romansklenar]

+1 for that. I get that every time when test case passes but when it fails it shows actual time in miliseconds (using python hooks).

[akondas]

+1, the same problem, every test case: duration: NaNms but when fails time is correct number.

dredd --version
dredd v1.0.7

Hooks: PHP

[jakubkratina]

Same in dredd v1.0.11

[honzajavorek]

Yes, this is still an issue. I haven't got to fix it due to working heavily on #388 and #389 these days. I think the fix wouldn't be difficult. It's probably somewhere in the CLI reporter. I guess there's some formula to calculate seconds and one of the variables comes as undefined in some cases:

$ coffee
coffee> undefined + 1
NaN
coffee> null + 1
1
coffee> 0 + 1
1

Also, if fixed there, it would be nice to look into the other reporters if there's not a similar bug. Would anyone of you like to work on this?

[jakubkratina]

I'll take a look, but no promises with solving it :)

[honzajavorek]

Released as v3.5.1 🎉