8
100%
master: 100%

Ran 26 Nov 2019 03:25AM UTC

Files 11

Run time 0s

Badge

Embed ▾

Committed 26 Nov 2019 03:22AM UTC coverage: 50.943% (+0.09%) from 50.851%

Job # YAJL_VERSION=none

Build Type

push

travis-ci-com

Committed by

rtobar

Commit Message

Catching only numeric conversion errors

The new set of tests introduced in the previous commit highlighted a
corner case that wasn't properly covered, and that was mentioned in #16:
the yield expression within the try/expect block can also actually raise
an exception if the generator is being destroyed before it is consumed
fully (e.g., if manually iterated over via next() and then discarded).
In these cases an UnexpectedSymbol exception would be raised during the
destruction of the generator object, which would be ignored by the
interpreter.

The solution is simple: include *only* the number parsing logic within
the try/catch, leaving the yield expression outside to get a normal
logic flow.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

Run Details

270 of 530 relevant lines covered (50.94%)

0.51 hits per line

ICRAR / ijson / 64 / 8
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 64.8 (YAJL_VERSION=none)

ICRAR / ijson / 64 / 8 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 64.8 (YAJL_VERSION=none)

ICRAR / ijson / 64 / 8
100%
master: 100%

README BADGES
x