Collection feature specs

Page: Public Collection Show PageĀ 

link to code:Ā https://github.com/samvera/hyrax/blob/master/spec/features/collection_spec.rb

Test case codeComments & questionsProposed changes to section
Basic segments of public collection show pageLooks for basic contentAdd expectation for parent collections included
Behavior variation when using search feature

Basic segments of public collection show page for a non-nestable collectionIt appears that this is intended to test that the subcollection list in't included when a collection is not nestable. However, it only tests for basic content, and doesn't test that subcollection content is NOT there, so the test is basically a light version of the first test.Add expectation that parent collections and subcollections NOT included when collection isn't nestable.
Pagination of works section of page

Should feature tests really do this sort of test? Would it not be better done as part of a view spec?

There is no wrapper of each section, so how can we differentiate works pagination from subcollection or parent collection pagination?

Recommend that this be removed.
Pagination of subcollections section of pageSame comments as re: pagination of worksRecommend that this be removed.

Page: Dashboard Collection Show PageĀ 

link to code:

Test case codeComments & questionsProposed changes to section