5
69%
master: 69%

Ran 08 Jun 2021 02:48AM UTC

Files 165

Run time 10s

Badge

Embed ▾

Committed 08 Jun 2021 02:47AM UTC coverage: 68.894% (+0.2%) from 68.686%

Job # 526.5

Build Type

push

travis-ci-com

Committed by Rodrigo Tobar

Commit Message

Address problems found during testing

Stewart tried out the latest changes to the subscription logic and found
three different problems, all of which are addressed in this commit.

Firstly, a typo in the email message has been fixed. Secondly, a logging
statement was missing a value for a format specifier. These two changes
were minor.

Finally, and most importantly, when a subscription fails to be added
because a similar-looking one already exists, the code tried to remove
the subscription object twice from the list of yet-to-be-created
subscriptions, which raised an unexpected error. However, at the same
time Stewart pointed out that at server shutdown the subscription that
wasn't added *was* removed from the remote server, which is what he
expected to see. This is not exactly great (if we are not adding it we
shouldn't be removing it), but on the other hand I see the appeal on
doing so, and since they are the main users of this feature anyway I'll
follow their preference. To avoid the double removal of the object,
while keeping the code path separate from the fully successful cases, I
created a small function that centralises the marking of a subscription
as active on the local server so it gets removed when the server goes
down.

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

Run Details

13610 of 19755 relevant lines covered (68.89%)

0.69 hits per line

ICRAR / ngas / 526 / 5
69%
master: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 526.5

ICRAR / ngas / 526 / 5 69% master: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 526.5

ICRAR / ngas / 526 / 5
69%
master: 69%

README BADGES
x