sul-dlss / pre-assembly / 890

58%

master: 92%

Pull #289

travis-ci

Tweak BundleContextController and rework tests

No need to separately `find_by` when rspec gives us the `assigns`
method.  It gives the tests access to the **same** object the template would
receive (even better than a different object -- or `nil` -- returned by
`find_by`).

Uses devise helper `sign_in` method and removes hardcoded `User.create`
in controller (meaning only one object could ever be created, since the
subsequent `User.create` would certainly fail).

Controller is not concerned with Job directly, only with creating a
`JobRun` object.

`current_user` is one of the `bundle_context_params`, logically.  Pass
it to `.create`.

`bundle_input_b` doesn't exist anymore, and the model still requires the
`bundle_dir` to exist to be valid (and saved). Changed it to `smpl_multimedia`.

This fixes the build.

4 of 4 new or added lines in 1 file covered. (100.0%)

528 of 912 relevant lines covered (57.89%)

15.01 hits per line