This is *really* not a merge blocker, but just thought I'd point out an opportunity to use a "subject {canvas_provide_course_site.user_currently_teaching?}" & one-liner tests like you do in auth_policy_spec.