6465: Noticed a slight dip in code coverage in two areas and added tests to cover...
authorradhika <radhika@curoverse.com>
Wed, 22 Jul 2015 12:34:04 +0000 (08:34 -0400)
committerradhika <radhika@curoverse.com>
Wed, 22 Jul 2015 12:34:04 +0000 (08:34 -0400)
commit4b8ecdc4d88c9dccbbcf07b5037ba4dcf4ea2d20
tree6d6b531743c06f9fcdafa6c98dd19201c4edd7de
parentfe0f4c45a3e3bf2c28f389dce064aafd92a30b0b
6465: Noticed a slight dip in code coverage in two areas and added tests to cover those situations. Details:
1. Removal of integration "Test dashboard button all {jobs}" resulted in no test at all to visit /jobs page.
Added a controller test for this.
2. Not visiting the hash match page when only one collection matches (the controller test just verified the redirection),
resulted in the unforeseen side effect of not testing preview allowed status for a file name extention that is not listed
in application_mimetypes_with_view_icon (tar file in this case). Added this to test.
apps/workbench/test/controllers/jobs_controller_test.rb
apps/workbench/test/helpers/collections_helper_test.rb