mroderick / codebar-planner / 24497432815

96%

master: 95%

Build Type

push

github

Committed by mroderick

Commit Message

fix: use find_by! to return 404 for invalid tokens

- Changed find_by to find_by! in before_action set_* methods to raise
  ActiveRecord::RecordNotFound (which Rails converts to 404) instead of
  returning nil and causing NoMethodError

High priority (production crash):
- workshop_invitation_concerns.rb - WorkshopInvitation lookup
- admin/meeting_invitations_controller.rb - MeetingInvitation lookup
- admin/invitations_controller.rb - workshop invitation lookup

Medium priority (potential 500 errors):
- admin/meetings_controller.rb - Meeting lookup by slug
- events_controller.rb - Event lookup by slug
- admin/events_controller.rb - Event lookup by slug

Lower priority:
- invitations_controller.rb:80 - MeetingInvitation lookup in cancel_meeting
- contact_preferences_controller.rb:11 - Contact lookup in update
- feedback_controller.rb:22 - FeedbackRequest lookup in submit

Coverage Stats

9 of 9 new or added lines in 9 files covered. (100.0%)

49 existing lines in 18 files now uncovered.

3489 of 3651 relevant lines covered (95.56%)

46.18 hits per line