Skip to content

Prefer `none?` over `.none` to match null relations

Peter Leitzen requested to merge pl-rails7-fix-null-relation-failures into master

What does this MR do and why?

Use list.none? instead of Model.none. Comparing null relations got stricter in Rails 7 and now also takes the resulting SQL statement into account which might differ in specs.

See !90907 (comment 1253691580)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Peter Leitzen

Merge request reports