Port 'Active tense test coverage' changes from EE
Ports changes from https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/642 back into CE
Each change basically rephrases out the should in Rspec tests
Discovered while working on https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8463, and thought it might help prevent future conflicts