3
69%
master: 69%

Ran 28 Aug 2019 01:43AM UTC

Files 160

Run time 10s

Badge

Embed ▾

Committed 28 Aug 2019 01:29AM UTC coverage: 67.103% (-0.002%) from 67.105%

Job # DB=postgresql

Build Type

push

travis-ci

Committed by

rtobar

Commit Message

Preventing TypeError during date comparison

The type of lastSchedule up to this point is either NoneType or str,
but on the other hand lastDelivery is a float (the type we use
internally to hold date/times). These two values are compared for order,
and while python2 happily compares strings v/s floats, python3 refuses
to do so and bombs out with a TypeError.

Because the subscription code is so convolved, it's a bit difficult to
fix this at the very source of the problem, but ensuring the correct
data representation just before the comparison is sufficient.

This problem was obviously not caught by our unit test suite because the
conditions to trigger it were not coded into any of them. I'm thus
adding a new unit test that reproduces the original problem, and that
will ensure this doesn't happen again.

This patch solves #4.

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

Run Details

12869 of 19178 relevant lines covered (67.1%)

0.67 hits per line

ICRAR / ngas / 342 / 3
69%
master: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 342.3 (DB=postgresql)

ICRAR / ngas / 342 / 3 69% master: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 342.3 (DB=postgresql)

ICRAR / ngas / 342 / 3
69%
master: 69%

README BADGES
x