1006
71%
master: 71%

Ran 19 Jul 2016 05:24PM UTC

Jobs 1

Files 8

Run time 1s

Badge

Embed ▾

pending completion

Build # 1006

Build Type

push

travis-ci

Committed by

jeremiak

Commit Message

Fix user store so tests pass

We had previously been setting .fetching at the top of each
of the handler functions in the stores. The problem with this is
that changing the fetching value will trigger a change event,
which might fire before the data that was retrieved from the API
has been merged into the stores. The result is a flash of content
like bug where the user sees a flash of "No apps", etc.

It looks pretty weird to set the value in two places like this,
but I set it after the merge data operation has been completed
and in the case that the request retrieved no data.

Run Details

237 of 334 relevant lines covered (70.96%)

3.17 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	1006.1	19 Jul 2016 05:24PM UTC	0	70.96	Travis Job 1006.1

Source Files on build 1006

Detailed source file information is not available for this build.

18F / cg-deck / 1006
71%
master: 71%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1006

18F / cg-deck / 1006 71% master: 71%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1006

18F / cg-deck / 1006
71%
master: 71%

README BADGES
x