2
58%
master: 92%

Ran 14 Aug 2018 11:03PM UTC

Files 35

Run time 30min

Badge

Embed ▾

Committed 14 Aug 2018 09:56PM UTC coverage: 58.142%. Remained the same

Job # 2.4.4

Build Type

push

travis-ci

Committed by

atz

Commit Message

Start cleaning up bundle_spec

Start getting rid of `@instance_variables` in spec.

Treat `bundle_setup` more like a factory, less like a side-effecty
pre-condition.  End goal: git rid of `@b`, which is effectively the
subject.

Use more `let` statements, shrink `before` blocks.  Get rid of global
`before(:all)` block completely.  This appears to have been an attempt
to optimize reading of YAML files off disk (i.e. read them all once
beforehand, cache the results). This spec file takes `0.23589 seconds`
to run. Lesson: don't bother trying to outsmart disk caching.

Use more normal description style of `#method_name`, not
`method_name()`.

Don't use one symbol/string name to refer to the yaml file in tests and
another to actually name the file.  Use the same name. That gets rid of
the `@yaml_filenames`.

Less loops inside `it` blocks. Use `all` matcher for the same
expectation on every element of an array, for example.

Run Details

1264 of 2174 relevant lines covered (58.14%)

14.35 hits per line

sul-dlss / pre-assembly / 217 / 2
58%
master: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 217.2 (2.4.4)

sul-dlss / pre-assembly / 217 / 2 58% master: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 217.2 (2.4.4)

sul-dlss / pre-assembly / 217 / 2
58%
master: 92%

README BADGES
x