Pour le rubocop, une boucle + une rupture est recommandée.
class Nonce
def set_nonce
return true if nonce.present? #Garde pour éviter l'écrasement
self.nonace = loop do
random_str = SecureRandom.urlsafe_base64
break random_str unless self.class.exists?(nonce: random_str)
end
end
end
class Nonce
def set_nonce
return true if nonce.present? #Garde pour éviter l'écrasement
begin
self.nonce = SecureRandom.urlsafe_base64
end while self.class.exists?(nonce: nonce)
end
end
before_create or before_validation
Si vous écrivez validates comme ceci, vous devez utiliser before_validation car vous serez bloqué dans la validation si vous utilisez before_create.
class SomeClass
validates :nonce, presence: true, uniqueness: true
before_validation :set_nonce
private
def set_nonce
return true if nonce.present?
self.nonace = loop do
random_str = SecureRandom.urlsafe_base64
break random_str unless self.class.exists?(nonce: random_str)
end
end
end
# frozen_string_literal: true
require 'rails_helper'
describe Nonce, type: :model do
describe '#set_nonce' do
it do
nonce = Nonce.create!
expect(nonce.nonce.size).to eq(8)
end
it do
nonce = Nonce.new
expect(nonce.nonce).to be_nil
nonce.save!
expect(nonce.nonce.size).to eq(8)
end
it do
nonce = Nonce.create!
nonce_str = nonce.nonce
nonce.update!(updated_at: Time.current)
expect(nonce.nonce).to eq(nonce_str)
end
end
end
Recommended Posts