Remove unnecessary validations
This commit is contained in:
parent
ccd238b1ba
commit
b58c0848c5
|
@ -24,10 +24,6 @@ class Contact < ApplicationRecord
|
||||||
# Validations #
|
# Validations #
|
||||||
###############
|
###############
|
||||||
|
|
||||||
validates :contact_list, presence: true
|
|
||||||
|
|
||||||
validates :contact_network, presence: true
|
|
||||||
|
|
||||||
validates :value,
|
validates :value,
|
||||||
presence: true,
|
presence: true,
|
||||||
uniqueness: { scope: %i[contact_list_id contact_network_id] }
|
uniqueness: { scope: %i[contact_list_id contact_network_id] }
|
||||||
|
|
|
@ -6,15 +6,23 @@ RSpec.describe Contact do
|
||||||
subject { create :some_contact }
|
subject { create :some_contact }
|
||||||
|
|
||||||
describe '#contact_list' do
|
describe '#contact_list' do
|
||||||
it { is_expected.to belong_to(:contact_list) }
|
it { is_expected.to belong_to(:contact_list).required }
|
||||||
|
|
||||||
it { is_expected.to validate_presence_of :contact_list }
|
it do
|
||||||
|
is_expected.to \
|
||||||
|
validate_presence_of(:contact_list)
|
||||||
|
.with_message(:required)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe '#contact_network' do
|
describe '#contact_network' do
|
||||||
it { is_expected.to belong_to(:contact_network) }
|
it { is_expected.to belong_to(:contact_network).required }
|
||||||
|
|
||||||
it { is_expected.to validate_presence_of :contact_network }
|
it do
|
||||||
|
is_expected.to \
|
||||||
|
validate_presence_of(:contact_network)
|
||||||
|
.with_message(:required)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe '#value' do
|
describe '#value' do
|
||||||
|
|
Reference in New Issue