Replace should_fail for should_raise in tests

As agreed on the mid-cycle meetup, the tests that contained
"should_fail" in the name were modified so that it says "should_raise"
instead. This was done so that the tests reflect what is actually going
on, and to avoid ambiguity, while enforcing uniformity in test naming.

Change-Id: Ifae5f6b40748b93ce21202b6b40c391cd12fb7be