64
100%

Ran 26 Nov 2019 03:23AM UTC

Jobs 8

Files 66

Run time 1min

Badge

Embed ▾

pending completion

Build # 64

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

2892 of 3180 relevant lines covered (90.94%)

5.84 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	64.1 (YAJL_VERSION=1.0.12)	26 Nov 2019 03:23AM UTC	62.64	Travis Job 64.1
2	64.2	26 Nov 2019 03:23AM UTC	80.94	Travis Job 64.2
3	64.3 (YAJL_VERSION=1.0.9)	26 Nov 2019 03:24AM UTC	63.77	Travis Job 64.3
4	64.4	26 Nov 2019 03:23AM UTC	81.51	Travis Job 64.4
5	64.5	26 Nov 2019 03:23AM UTC	81.51	Travis Job 64.5
6	64.6	26 Nov 2019 03:24AM UTC	81.51	Travis Job 64.6
7	64.7	26 Nov 2019 03:24AM UTC	81.51	Travis Job 64.7
8	64.8 (YAJL_VERSION=none)	26 Nov 2019 03:25AM UTC	50.94	Travis Job 64.8

Source Files on build 64

Detailed source file information is not available for this build.

ICRAR / ijson / 64
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 64

ICRAR / ijson / 64 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 64

ICRAR / ijson / 64
100%

README BADGES
x