332
99%

Ran 31 Mar 2014 12:11AM UTC

Jobs 1

Files 32

Run time 8s

Badge

Embed ▾

pending completion

Build # 332

Build Type

push

travis-ci

Committed by

trotzig

Commit Message

Always show the snapshot image on the snapshot#show page

A recent commit (8b94b558094) removed the before and after image from
the snapshot#show page. This was a win in case the snapshot was in a
state where it had a non-zero diff because we saved a lot of space by
not rendering the before and after images. However, when the snapshot
had a zero diff and when it didn't have a baseline to compare against,
you wouldn't see the snapshot image at all.

I added the image back in two places. I also added an alert to explain
the state when there is no baseline to compare against.

The spec coverage for all of this is not great. But I did add some smoke
tests just to make sure that the page would render with snapshots in
different states. And while doing that I found a bug in the snapshot
factory that I fixed.

Run Details

591 of 599 relevant lines covered (98.66%)

38.61 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	332.1 (2.0.0)	31 Mar 2014 12:11AM UTC	0	98.66	Travis Job 332.1

Source Files on build 332

Detailed source file information is not available for this build.

trotzig / diffux / 332
99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 332

trotzig / diffux / 332 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 332

trotzig / diffux / 332
99%

README BADGES
x