6660
38%

Ran 30 Jun 2017 09:05AM UTC

Jobs 4

Files 243

Run time 10min

Badge

Embed ▾

pending completion

Build # 6660

Build Type

push

travis-ci

Committed by

stevenday

Commit Message

Use request that's passed in when updating summaries, not local one

RequestSummary#update_from_request is called by code that has access to
a guaranteed-most-up-to-date copy of the request, but it discards it
and accessess it's own copy through .summarisable instead. In the case
of status updates through InfoRequest#set_described_state (and perhaps
in other cases) this meant that the summary wasn't seeing changes to
its request's status, because it's local copy was cached. I could have
forced a reload in the method but it seemed better to utilise the copy
of the request that was already present by passing it in, making the
update and create methods more consistent in the process.

For mysociety/alaveteli-professional#310

Coverage Stats

9739 of 10488 relevant lines covered (92.86%)

3082.39 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	6660.1 (1.9.3)	30 Jun 2017 09:15AM UTC	92.94	Travis Job 6660.1
2	6660.2 (2.0.0)	30 Jun 2017 09:07AM UTC	92.93	Travis Job 6660.2
3	6660.3 (2.1.5)	30 Jun 2017 09:05AM UTC	92.93	Travis Job 6660.3
4	6660.4 (2.3.0)	30 Jun 2017 09:06AM UTC	92.93	Travis Job 6660.4

Source Files on build 6660

Detailed source file information is not available for this build.

mysociety / alaveteli / 6660
38%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6660

mysociety / alaveteli / 6660 38%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6660

mysociety / alaveteli / 6660
38%

README BADGES
x