Fix before_each method to handle nil parameter correctly

[ericproulx]

Fix before_each method to handle nil parameter correctly

Problem

When calling before_each(nil) to clear the before_each callbacks, a NoMethodError was raised because the code attempted to assign nil to @before_each instead of clearing the array.

Solution

• Modified the before_each method in lib/grape/endpoint.rb to properly handle the nil parameter by clearing the @before_each array instead of assigning nil
• Updated test expectations in spec/grape/endpoint_spec.rb to be more specific about error types (NameError vs NoMethodError) and include error message patterns for better test reliability

Changes

• lib/grape/endpoint.rb: Added else clause to clear @before_each when new_setup is nil
• spec/grape/endpoint_spec.rb: Updated error expectations to match specific error types and messages

Testing

All existing tests pass, and the fix ensures that before_each(nil) correctly clears the callbacks without raising an error.

[github-actions]

Danger Report

No issues found.

View run