From 0cb5d253f8943aeae2e28edd43610ee4780ef7d3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Wed, 22 Oct 2025 12:18:36 +0200 Subject: [PATCH] Use match_array in notifications scopes tests The scopes don't define the order in which the records are returned, so, when using `eq`, the tests failed sometimes. --- spec/models/user_spec.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index a986f556e..fbd61eac2 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -474,7 +474,7 @@ describe User do create(:user, newsletter: true, username: "Subscriber2") create(:user, newsletter: false, username: "NonSubscriber") - expect(User.newsletter.pluck(:username)).to eq ["Subscriber1", "Subscriber2"] + expect(User.newsletter.pluck(:username)).to match_array ["Subscriber1", "Subscriber2"] end end @@ -484,7 +484,7 @@ describe User do create(:user, email_digest: true, username: "Digester2") create(:user, email_digest: false, username: "NonDigester") - expect(User.email_digest.pluck(:username)).to eq ["Digester1", "Digester2"] + expect(User.email_digest.pluck(:username)).to match_array ["Digester1", "Digester2"] end end