diff options
author | Nick Schonning <nschonni@gmail.com> | 2023-02-08 04:39:57 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-08 10:39:57 +0100 |
commit | 0592937264d7a8cb4f1ea56be403f48e0801b2b7 (patch) | |
tree | d2f6bff15de59e700b303bc196044d0e336d4096 | |
parent | 11557d1c5a4ecb44cf602eabf0c9bd2882514b3a (diff) |
Apply Rubocop Rails/WhereNot (#23448)
* Apply Rubocop Rails/WhereNot * Update spec for where.not
-rw-r--r-- | app/lib/vacuum/access_tokens_vacuum.rb | 4 | ||||
-rw-r--r-- | app/models/session_activation.rb | 2 | ||||
-rw-r--r-- | spec/models/session_activation_spec.rb | 4 |
3 files changed, 5 insertions, 5 deletions
diff --git a/app/lib/vacuum/access_tokens_vacuum.rb b/app/lib/vacuum/access_tokens_vacuum.rb index 4f3878027..7b91f74a5 100644 --- a/app/lib/vacuum/access_tokens_vacuum.rb +++ b/app/lib/vacuum/access_tokens_vacuum.rb @@ -9,10 +9,10 @@ class Vacuum::AccessTokensVacuum private def vacuum_revoked_access_tokens! - Doorkeeper::AccessToken.where('revoked_at IS NOT NULL').where('revoked_at < NOW()').delete_all + Doorkeeper::AccessToken.where.not(revoked_at: nil).where('revoked_at < NOW()').delete_all end def vacuum_revoked_access_grants! - Doorkeeper::AccessGrant.where('revoked_at IS NOT NULL').where('revoked_at < NOW()').delete_all + Doorkeeper::AccessGrant.where.not(revoked_at: nil).where('revoked_at < NOW()').delete_all end end diff --git a/app/models/session_activation.rb b/app/models/session_activation.rb index 3a59bad93..0b7fa6fe4 100644 --- a/app/models/session_activation.rb +++ b/app/models/session_activation.rb @@ -59,7 +59,7 @@ class SessionActivation < ApplicationRecord end def exclusive(id) - where('session_id != ?', id).destroy_all + where.not(session_id: id).destroy_all end end diff --git a/spec/models/session_activation_spec.rb b/spec/models/session_activation_spec.rb index 450dc1399..8db06c622 100644 --- a/spec/models/session_activation_spec.rb +++ b/spec/models/session_activation_spec.rb @@ -118,8 +118,8 @@ RSpec.describe SessionActivation, type: :model do let(:id) { '1' } it 'calls where.destroy_all' do - expect(described_class).to receive_message_chain(:where, :destroy_all) - .with('session_id != ?', id).with(no_args) + expect(described_class).to receive_message_chain(:where, :not, :destroy_all) + .with(session_id: id).with(no_args) described_class.exclusive(id) end |