Oh - now I see that the `get_lms_link_for_item` method used to take an optional `course_id` parameter, which it doesn't now.  But I think we should still add a test where it passes a location with `category=course`.