1
68%
master: 97%

Ran 26 Oct 2015 03:26PM UTC

Files 87

Run time 8s

Badge

Embed ▾

Committed 26 Oct 2015 03:17PM UTC coverage: 68.408%. First build

Job # 8788.1

Build Type

push

travis-ci

Committed by

robertknight

Commit Message

Deliver up-to-date groups list in groups leave/join notifications

When pushing a session state change notification to clients
after joining or leaving a group, the session model could
be out of date because the groups view published the notification
prior to committing the database transaction.

If the websocket server picked up the message from NSQ and queried
the DB for the session state prior to the commit happening,
the client would receive a notification but the payload
would be out of date.

We considered two possible approaches to fix this - one
was to commit the transaction in the groups views before
publishing the notification, the other was to serialize the
session model as part of the notification.

This commit takes the latter approach for consistency with how
annotation notifications work and also as slightly more
explicit to read. We may want to reconsider in future.

Fixes #2671

Run Details

2501 of 3656 relevant lines covered (68.41%)

0.68 hits per line

hypothesis / h / 8788 / 1
68%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8788.1

hypothesis / h / 8788 / 1 68% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8788.1

hypothesis / h / 8788 / 1
68%
master: 97%

README BADGES
x