• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

ICRAR / ngas / 525
69%
master: 69%

Build:
Build:
LAST BUILD BRANCH: config_use_prepared_stmts_for_clients
DEFAULT BRANCH: master
Ran 01 Jun 2021 03:51AM UTC
Jobs 6
Files 165
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
525

push

travis-ci-com

Rodrigo Tobar
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>

7 of 7 new or added lines in 1 file covered. (100.0%)

13656 of 19755 relevant lines covered (69.13%)

4.13 hits per line

Jobs
ID Job ID Ran Files Coverage
1 525.1 (DB=sqlite3) 01 Jun 2021 03:51AM UTC 0
68.77
Travis Job 525.1
2 525.2 (DB=mysql) 01 Jun 2021 03:52AM UTC 0
68.59
Travis Job 525.2
3 525.3 (DB=postgresql) 01 Jun 2021 03:52AM UTC 0
68.5
Travis Job 525.3
4 525.4 01 Jun 2021 03:51AM UTC 0
68.82
Travis Job 525.4
5 525.5 01 Jun 2021 03:51AM UTC 0
68.91
Travis Job 525.5
6 525.6 01 Jun 2021 03:51AM UTC 0
68.92
Travis Job 525.6
Source Files on build 525
Detailed source file information is not available for this build.
  • Back to Repo
  • Build #525
  • 52427387 on github
  • Prev Build on issue-50 (#524)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc