# frozen_string_literal: true require 'spec_helper' EXT_PREFIXES = %w[ext ex x xt # :].freeze describe PhonyRails do it 'should not pollute the global namespace with a Country class' do should_not be_const_defined 'Country' end describe 'phony_format String extension' do describe 'the phony_formatted method' do it 'does not modify the original options Hash' do options = { normalize: :NL, format: :international } '0101234123'.phony_formatted(options) expect(options).to eql(normalize: :NL, format: :international) end describe 'with the bang!' do it 'changes the String using the bang method' do # Mutable String s = +'0101234123' rescue '0101234123' expect(s.phony_formatted!(normalize: :NL, format: :international)).to eql('+31 10 123 4123') expect(s).to eql('+31 10 123 4123') end end describe 'with strict option' do it 'returns nil with non plausible number' do number = '+319090' # not valid expect(Phony.plausible?(number)).to be false expect(number.phony_formatted(strict: true)).to eql(nil) end it 'should not return nil with plausible number' do number = '+31101234123' # valid expect(Phony.plausible?(number)).to be true expect(number.phony_formatted(strict: true)).to_not eql(nil) end end describe 'with normalize option' do it 'should phony_format' do expect('101234123'.phony_formatted(normalize: :NL)).to eql('010 123 4123') expect('101234123'.phony_formatted(normalize: :NL, format: :international)).to eql('+31 10 123 4123') end it 'should not change original String' do s = '0101234123' expect(s.phony_formatted(normalize: :NL)).to eql('010 123 4123') expect(s).to eql('0101234123') end it 'should phony_format String with country code' do expect('31101234123'.phony_formatted(normalize: :NL)).to eql('010 123 4123') end it 'should phony_format String with country code' do expect('31101234123'.phony_formatted(normalize: :NL)).to eql('010 123 4123') end it 'should accept strings with non-digits in it' do expect('+31-10-1234123'.phony_formatted(normalize: :NL, format: :international, spaces: '-')).to eql('+31-10-123-4123') end it 'should phony_format String with country code different than normalized value' do expect('+4790909090'.phony_formatted(normalize: :SE, format: :international)).to eql('+47 909 09 090') end end describe 'with raise option' do # https://github.com/joost/phony_rails/issues/79 context 'when raise is true' do it 'should raise the error' do expect(lambda do '8887716095'.phony_formatted(format: :international, raise: true) end).to raise_error(Phony::FormattingError) end end context 'when raise is false (default)' do it 'returns original String on exception' do expect('8887716095'.phony_formatted(format: :international)).to eq('8887716095') end end end describe 'with extensions' do EXT_PREFIXES.each do |prefix| it "should format number with #{prefix} extension" do expect("+319090#{prefix}123".phony_formatted(strict: true)).to eql(nil) expect("101234123#{prefix}123".phony_formatted(normalize: :NL)).to eql('010 123 4123 x123') expect("101234123#{prefix}123".phony_formatted(normalize: :NL, format: :international)).to eql('+31 10 123 4123 x123') expect("31101234123#{prefix}123".phony_formatted(normalize: :NL)).to eql('010 123 4123 x123') expect("8887716095#{prefix}123".phony_formatted(format: :international, normalize: 'US', raise: true)).to eq('+1 (888) 771-6095 x123') expect("+12145551212#{prefix}123".phony_formatted).to eq('(214) 555-1212 x123') end end end describe 'specific tests from issues' do # https://github.com/joost/phony_rails/issues/79 it 'should pass Github issue #42' do expect('8887716095'.phony_formatted(format: :international, normalize: 'US', raise: true)).to eq('+1 (888) 771-6095') end # https://github.com/joost/phony_rails/issues/42 it 'should pass Github issue #42' do expect(PhonyRails.normalize_number('0606060606', default_country_code: 'FR')).to eq('+33606060606') end it 'should pass Github issue #85' do expect(PhonyRails.normalize_number('47386160', default_country_code: 'NO')).to eq('+4747386160') expect(PhonyRails.normalize_number('47386160', country_number: '47')).to eq('+4747386160') end it 'should pass Github issue #87' do expect(PhonyRails.normalize_number('2318725305', country_code: 'US')).to eq('+12318725305') expect(PhonyRails.normalize_number('2318725305', default_country_code: 'US')).to eq('+12318725305') expect(PhonyRails.normalize_number('+2318725305', default_country_code: 'US')).to eq('+2318725305') # expect(Phony.plausible?("#{PhonyRails.country_number_for('US')}02318725305")).to be_truthy expect(PhonyRails.normalize_number('02318725305', default_country_code: 'US')).to eq('+12318725305') end it 'should pass Github issue #89' do number = '+33 (0)6 87 36 18 75' expect(Phony.plausible?(number)).to be true expect(PhonyRails.normalize_number(number, country_code: 'FR')).to eq('+33687361875') end it 'should pass Github issue #90' do number = '(0)30 1234 123' expect(number.phony_normalized(country_code: 'NL')).to eq('+31301234123') end it 'should pass Github issue #107' do number = '04575700834' expect(number.phony_normalized(country_code: 'FI')).to eq('+3584575700834') # Seems this number can be interpreted as from multiple countries, following fails: # expect(number.phony_normalized(default_country_code: 'FI')).to eq('+3584575700834') # expect("04575700834".phony_formatted(normalize: 'FI', format: :international)).to eql('+358 45 757 00 834') end it 'should pass Github issue #113' do number = '(951) 703-593' expect(lambda do number.phony_formatted!(normalize: 'US', spaces: '-', strict: true) end).to raise_error(ArgumentError) end it 'should pass Github issue #95' do number = '02031234567' expect(number.phony_normalized(default_country_code: 'GB')).to eq('+442031234567') end it 'should pass Github issue #121' do number = '06-87-73-83-58' expect(number.phony_normalized(default_country_code: 'FR')).to eq('+33687738358') end it 'returns the original input if all goes wrong' do expect(Phony).to receive(:plausible?).and_raise('unexpected error') number = '(0)30 1234 123' expect(number.phony_normalized(country_code: 'NL')).to eq number end it 'should pass Github issue #126 (country_code)' do phone = '0143590213' # A plausible FR number phone = PhonyRails.normalize_number(phone, country_code: 'FR') expect(phone).to eq('+33143590213') expect(Phony.plausible?(phone)).to be_truthy phone = PhonyRails.normalize_number(phone, country_code: 'FR') expect(phone).to eq('+33143590213') end # Adding a country code is expected behavior when a # number is nog plausible. it 'should pass Github issue #126 (country_code) (intended)' do phone = '06123456789' # A non-plausible FR number phone = PhonyRails.normalize_number(phone, country_code: 'FR') expect(phone).to eq('+336123456789') expect(Phony.plausible?(phone)).to be_falsy phone = PhonyRails.normalize_number(phone, country_code: 'FR') expect(phone).to eq('+33336123456789') end it 'should pass Github issue #126 (default_country_code)' do phone = '06123456789' # French phone numbers have to be 10 chars long phone = PhonyRails.normalize_number(phone, default_country_code: 'FR') expect(phone).to eq('+336123456789') phone = PhonyRails.normalize_number(phone, default_country_code: 'FR') expect(phone).to eq('+336123456789') end it 'should pass Github issue #92 (invalid number with normalization)' do ActiveRecord::Schema.define do create_table :normal_homes do |table| table.column :phone_number, :string end end # rubocop:disable Lint/ConstantDefinitionInBlock class NormalHome < ActiveRecord::Base attr_accessor :phone_number phony_normalize :phone_number, default_country_code: 'US' validates :phone_number, phony_plausible: true end # rubocop:enable Lint/ConstantDefinitionInBlock normal = NormalHome.new normal.phone_number = 'HAHA' expect(normal).to_not be_valid expect(normal.phone_number).to eq('HAHA') expect(normal.errors.messages.to_hash).to include(phone_number: ['is an invalid number']) end it 'should pass Github issue #170' do phone = '(+49) 175 123 4567' phone = PhonyRails.normalize_number(phone) expect(phone).to eq('+491751234567') end it 'should pass Github issue #170 (part 2)' do phone = '(+49) 175 123 4567' phone = PhonyRails.normalize_number(phone, default_country_code: 'DE') expect(phone).to eq('+491751234567') end it 'should pass Github issue #175' do phone = '0041 23456789' phone = PhonyRails.normalize_number(phone, default_country_code: 'DE') expect(phone).to eq('+4123456789') end it 'should pass Github issue #175' do phone = '+1 260-437-9123' phone = PhonyRails.normalize_number(phone, default_country_code: 'DE') expect(phone).to eq('+12604379123') end it 'should pass Github issue #187' do phone1 = '0037253400030' phone1 = PhonyRails.normalize_number(phone1, default_country_code: 'EE') expect(phone1).to eq('+37253400030') phone2 = '0037275016183' phone2 = PhonyRails.normalize_number(phone2, default_country_code: 'EE') expect(phone2).to eq('+37275016183') end it 'should pass Github issue #180' do phone = '5555555555' phone = PhonyRails.normalize_number(phone, default_country_code: 'AU') expect(phone).to eq('+615555555555') phone = PhonyRails.normalize_number(phone, default_country_code: 'AU') expect(phone).to eq('+615555555555') end end it 'should not change original String' do s = '0101234123' expect(s.phony_formatted(normalize: :NL)).to eql('010 123 4123') expect(s).to eql('0101234123') end it 'should phony_format a digits string with spaces String' do expect('31 10 1234123'.phony_formatted(format: :international, spaces: '-')).to eql('+31-10-123-4123') end it 'should phony_format a digits String' do expect('31101234123'.phony_formatted(format: :international, spaces: '-')).to eql('+31-10-123-4123') end it 'returns nil if implausible phone' do expect('this is not a phone'.phony_formatted).to be_nil end it 'returns nil on blank string' do expect(''.phony_formatted).to be_nil end end describe 'the phony_normalized method' do it 'returns blank on blank string' do expect(''.phony_normalized).to be_nil end it 'should not modify the original options Hash' do options = { normalize: :NL, format: :international } '0101234123'.phony_normalized(options) expect(options).to eql(normalize: :NL, format: :international) end context 'when String misses a country_code' do it 'should normalize with :country_code option' do expect('010 1231234'.phony_normalized(country_code: :NL)).to eql('+31101231234') end it 'should normalize without :country_code option' do expect('010 1231234'.phony_normalized).to eql('101231234') end it 'should normalize with :add_plus option' do expect('010 1231234'.phony_normalized(country_code: :NL, add_plus: false)).to eql('31101231234') end end it 'should normalize with :add_plus option' do expect('+31 (0)10 1231234'.phony_normalized(add_plus: false)).to eql('31101231234') end it 'should normalize a String' do expect('+31 (0)10 1231234'.phony_normalized).to eql('+31101231234') end end end describe 'PhonyRails#normalize_number' do context 'number with a country code' do it 'should not add default_country_code' do expect(PhonyRails.normalize_number('+4790909090', default_country_code: 'SE')).to eql('+4790909090') # SE = +46 expect(PhonyRails.normalize_number('004790909090', default_country_code: 'SE')).to eql('+4790909090') expect(PhonyRails.normalize_number('4790909090', default_country_code: 'NO')).to eql('+4790909090') # NO = +47 end it 'should force add country_code' do expect(PhonyRails.normalize_number('+4790909090', country_code: 'SE')).to eql('+464790909090') expect(PhonyRails.normalize_number('004790909090', country_code: 'SE')).to eql('+4604790909090') # FIXME: differs due to Phony.normalize in v2.7.1?! expect(PhonyRails.normalize_number('4790909090', country_code: 'SE')).to eql('+464790909090') end it 'should recognize lowercase country codes' do expect(PhonyRails.normalize_number('4790909090', country_code: 'se')).to eql('+464790909090') end end context 'number without a country code' do it 'should normalize with a default_country_code' do expect(PhonyRails.normalize_number('010-1234123', default_country_code: 'NL')).to eql('+31101234123') end it 'should normalize with a country_code' do expect(PhonyRails.normalize_number('010-1234123', country_code: 'NL', default_country_code: 'DE')).to eql('+31101234123') expect(PhonyRails.normalize_number('010-1234123', country_code: 'NL')).to eql('+31101234123') end it 'should handle different countries' do expect(PhonyRails.normalize_number('(030) 8 61 29 06', country_code: 'DE')).to eql('+49308612906') expect(PhonyRails.normalize_number('0203 330 8897', country_code: 'GB')).to eql('+442033308897') end it 'should prefer country_code over default_country_code' do expect(PhonyRails.normalize_number('(030) 8 61 29 06', country_code: 'DE', default_country_code: 'NL')).to eql('+49308612906') end it 'should recognize lowercase country codes' do expect(PhonyRails.normalize_number('010-1234123', country_code: 'nl')).to eql('+31101234123') end end context 'number with an extension' do EXT_PREFIXES.each do |prefix| it "should handle some edge cases (with country_code) and #{prefix} extension" do expect(PhonyRails.normalize_number("some nasty stuff in this +31 number 10-1234123 string #{prefix}123", country_code: 'NL')).to eql('+31101234123 x123') expect(PhonyRails.normalize_number("070-4157134#{prefix}123", country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("0031-70-4157134#{prefix}123", country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("+31-70-4157134#{prefix}123", country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("0322-69497#{prefix}123", country_code: 'BE')).to eql('+3232269497 x123') expect(PhonyRails.normalize_number("+32 3 226 94 97#{prefix}123", country_code: 'BE')).to eql('+3232269497 x123') expect(PhonyRails.normalize_number("0450 764 000#{prefix}123", country_code: 'AU')).to eql('+61450764000 x123') end it "should handle some edge cases (with default_country_code) and #{prefix}" do expect(PhonyRails.normalize_number("some nasty stuff in this +31 number 10-1234123 string #{prefix}123", country_code: 'NL')).to eql('+31101234123 x123') expect(PhonyRails.normalize_number("070-4157134#{prefix}123", default_country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("0031-70-4157134#{prefix}123", default_country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("+31-70-4157134#{prefix}123", default_country_code: 'NL')).to eql('+31704157134 x123') expect(PhonyRails.normalize_number("0322-69497#{prefix}123", default_country_code: 'BE')).to eql('+3232269497 x123') expect(PhonyRails.normalize_number("+32 3 226 94 97#{prefix}123", default_country_code: 'BE')).to eql('+3232269497 x123') expect(PhonyRails.normalize_number("0450 764 000#{prefix}123", default_country_code: 'AU')).to eql('+61450764000 x123') end it "should remove #{prefix} extension (with extension: false)" do expect(PhonyRails.normalize_number("0031-70-4157134#{prefix}123", extension: false, country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number("+31-70-4157134#{prefix}123", extension: false, country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number("0322-69497#{prefix}123", extension: false, country_code: 'BE')).to eql('+3232269497') end end end it 'should handle some edge cases (with country_code)' do expect(PhonyRails.normalize_number('some nasty stuff in this +31 number 10-1234123 string', country_code: 'NL')).to eql('+31101234123') expect(PhonyRails.normalize_number('070-4157134', country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('0031-70-4157134', country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('+31-70-4157134', country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('0322-69497', country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('+32 3 226 94 97', country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('0450 764 000', country_code: 'AU')).to eql('+61450764000') end it 'should handle some edge cases (with default_country_code)' do expect(PhonyRails.normalize_number('some nasty stuff in this +31 number 10-1234123 string', country_code: 'NL')).to eql('+31101234123') expect(PhonyRails.normalize_number('070-4157134', default_country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('0031-70-4157134', default_country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('+31-70-4157134', default_country_code: 'NL')).to eql('+31704157134') expect(PhonyRails.normalize_number('0322-69497', default_country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('+32 3 226 94 97', default_country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('0450 764 000', default_country_code: 'AU')).to eql('+61450764000') end it 'should normalize even an implausible number' do expect(PhonyRails.normalize_number('01')).to eql('1') end context 'with default_country_code set' do before { PhonyRails.default_country_code = 'NL' } after { PhonyRails.default_country_code = nil } it 'normalize using the default' do expect(PhonyRails.normalize_number('010-1234123')).to eql('+31101234123') expect(PhonyRails.normalize_number('010-1234123')).to eql('+31101234123') expect(PhonyRails.normalize_number('070-4157134')).to eql('+31704157134') expect(PhonyRails.normalize_number('0031-70-4157134')).to eql('+31704157134') expect(PhonyRails.normalize_number('+31-70-4157134')).to eql('+31704157134') end it 'allows default_country_code to be overridden' do expect(PhonyRails.normalize_number('0322-69497', country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('+32 3 226 94 97', country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('0450 764 000', country_code: 'AU')).to eql('+61450764000') expect(PhonyRails.normalize_number('0322-69497', default_country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('+32 3 226 94 97', default_country_code: 'BE')).to eql('+3232269497') expect(PhonyRails.normalize_number('0450 764 000', default_country_code: 'AU')).to eql('+61450764000') end end end describe 'PhonyRails.plausible_number?' do subject { described_class } let(:valid_number) { '1 555 555 5555' } let(:invalid_number) { '123456789 123456789 123456789 123456789' } let(:another_invalid_number) { '441212' } let(:normalizable_number) { '555 555 5555' } let(:formatted_french_number_with_country_code) { '+33 627899541' } let(:empty_number) { '' } let(:nil_number) { nil } it 'returns true for a valid number' do is_expected.to be_plausible_number valid_number, country_code: 'US' end it 'returns false for an invalid numbers' do is_expected.not_to be_plausible_number invalid_number is_expected.not_to be_plausible_number invalid_number, country_code: 'US' is_expected.not_to be_plausible_number another_invalid_number is_expected.not_to be_plausible_number another_invalid_number, country_code: 'US' end it 'returns true for a normalizable number' do is_expected.to be_plausible_number normalizable_number, country_code: 'US' end it 'returns false for a valid number with the wrong country code' do is_expected.not_to be_plausible_number normalizable_number, country_code: 'FR' end it 'returns true for a well formatted valid number' do is_expected.to be_plausible_number formatted_french_number_with_country_code, country_code: 'FR' end it 'returns false for an empty number' do is_expected.not_to be_plausible_number empty_number, country_code: 'US' end it 'returns false for a nil number' do is_expected.not_to be_plausible_number nil_number, country_code: 'US' end it 'returns false when no country code is supplied' do is_expected.not_to be_plausible_number normalizable_number end it 'returns false if something goes wrong' do expect(Phony).to receive(:plausible?).twice.and_raise('unexpected error') is_expected.not_to be_plausible_number normalizable_number, country_code: 'US' end it 'should pass Github issue #95' do is_expected.to be_plausible_number '+358414955444', default_country_code: :de end context 'with default_country_code set' do before { PhonyRails.default_country_code = 'FR' } after { PhonyRails.default_country_code = nil } it 'uses the default' do is_expected.not_to be_plausible_number normalizable_number is_expected.to be_plausible_number formatted_french_number_with_country_code end it 'allows default_country_code to be overridden' do is_expected.not_to be_plausible_number empty_number, country_code: 'US' is_expected.not_to be_plausible_number nil_number, country_code: 'US' end end end describe 'PhonyRails.default_country' do before { PhonyRails.default_country_code = 'US' } after { PhonyRails.default_country_code = nil } it 'can set a global default country code' do expect(PhonyRails.default_country_code).to eq 'US' end it 'can set a global default country code' do PhonyRails.default_country_number = '1' expect(PhonyRails.default_country_number).to eq '1' end it 'default country code affects default country number' do expect(PhonyRails.default_country_number).to eq '1' end end describe 'PhonyRails#extract_extension' do it 'returns [nil, nil] on nil input' do expect(PhonyRails.extract_extension(nil)).to eq [nil, nil] end it 'returns [number, nil] when number does not have an extension' do expect(PhonyRails.extract_extension('123456789')).to eq ['123456789', nil] end EXT_PREFIXES.each do |prefix| it "returns [number, ext] when number has a #{prefix} extension" do expect(PhonyRails.extract_extension("123456789#{prefix}123")).to eq %w[123456789 123] end end end describe 'PhonyRails.country_code_from_number' do it 'returns the code of the plausible phone number' do expect(PhonyRails.country_code_from_number('+32475000000')).to eq '32' end end describe 'PhonyRails.country_from_number' do it 'returns the country of the plausible phone number' do expect(PhonyRails.country_from_number('+32475000000')).to eq 'BE' end end describe 'PhonyRails#format_extension' do it 'returns just number if no extension' do expect(PhonyRails.format_extension('+123456789', nil)).to eq '+123456789' end it 'returns number with extension if extension exists' do expect(PhonyRails.format_extension('+123456789', '123')).to eq '+123456789 x123' end end shared_examples_for 'model with PhonyRails' do describe 'defining model#phony_normalized_method' do it 'should add a normalized_phone_attribute method' do expect(model_klass.new).to respond_to(:normalized_phone_attribute) end it 'should add a normalized_phone_method method' do expect(model_klass.new).to respond_to(:normalized_phone_method) end it 'should raise error on existing methods' do expect(lambda do model_klass.phony_normalized_method(:phone_method) end).to raise_error(StandardError) end it 'should raise error on not existing attribute' do model_klass.phony_normalized_method(:phone_non_existing_method) expect(lambda do model_klass.new.normalized_phone_non_existing_method end).to raise_error(ArgumentError) end end describe 'defining model#phony_normalize' do it 'should not accept :as option with multiple attribute names' do expect(lambda do model_klass.phony_normalize(:phone_number, :phone1_method, as: 'non_existing_attribute') end).to raise_error(ArgumentError) end it 'should accept :as option with non existing attribute name' do expect(lambda do dummy_klass.phony_normalize(:non_existing_attribute, as: 'non_existing_attribute') end).to_not raise_error end it 'should accept :as option with single non existing attribute name' do expect(lambda do dummy_klass.phony_normalize(:phone_number, as: 'something_else') end).to_not raise_error end it 'should accept :as option with single existing attribute name' do expect(lambda do model_klass.phony_normalize(:phone_number, as: 'phone_number_as_normalized') end).to_not raise_error end it 'should accept a non existing attribute name' do expect(lambda do dummy_klass.phony_normalize(:non_existing_attribute) end).to_not raise_error end it 'should accept supported options' do options = %i[country_number default_country_number country_code default_country_code add_plus as enforce_record_country] options.each do |option_sym| expect(lambda do dummy_klass.phony_normalize(:phone_number, option_sym => false) end).to_not raise_error end end it 'should not accept unsupported options' do expect(lambda do dummy_klass.phony_normalize(:phone_number, unsupported_option: false) end).to raise_error(ArgumentError) end end describe 'using model#phony_normalized_method' do # Following examples have complete number (with country code!) it 'returns a normalized version of an attribute' do model = model_klass.new(phone_attribute: '+31-(0)10-1234123') expect(model.normalized_phone_attribute).to eql('+31101234123') end it 'returnsa normalized version of a method' do model = model_klass.new(phone_method: '+31-(0)10-1234123') expect(model.normalized_phone_method).to eql('+31101234123') end # Following examples have incomplete number it 'should normalize even a unplausible number (no country code)' do model = model_klass.new(phone_attribute: '(0)10-1234123') expect(model.normalized_phone_attribute).to eql('101234123') end it 'should use country_code option' do model = model_klass.new(phone_attribute: '(0)10-1234123') expect(model.normalized_phone_attribute(country_code: 'NL')).to eql('+31101234123') end it 'should use country_code object method' do model = model_klass.new(phone_attribute: '(0)10-1234123', country_code: 'NL') expect(model.normalized_phone_attribute).to eql('+31101234123') end it 'should fallback to default_country_code option' do model = model_klass.new(phone1_method: '(030) 8 61 29 06') expect(model.normalized_phone1_method).to eql('+49308612906') end it 'should overwrite default_country_code option with object method' do model = model_klass.new(phone1_method: '(030) 8 61 29 06', country_code: 'NL') expect(model.normalized_phone1_method).to eql('+31308612906') end it 'should overwrite default_country_code option with option' do model = model_klass.new(phone1_method: '(030) 8 61 29 06') expect(model.normalized_phone1_method(country_code: 'NL')).to eql('+31308612906') end it 'should use last passed options' do model = model_klass.new(phone1_method: '(030) 8 61 29 06') expect(model.normalized_phone1_method(country_code: 'NL')).to eql('+31308612906') expect(model.normalized_phone1_method(country_code: 'DE')).to eql('+49308612906') expect(model.normalized_phone1_method(country_code: nil)).to eql('+49308612906') end it 'should use last object method' do model = model_klass.new(phone1_method: '(030) 8 61 29 06') model.country_code = 'NL' expect(model.normalized_phone1_method).to eql('+31308612906') model.country_code = 'DE' expect(model.normalized_phone1_method).to eql('+49308612906') model.country_code = nil expect(model.normalized_phone1_method(country_code: nil)).to eql('+49308612906') end it 'should accept a symbol when setting country_code options' do model = model_klass.new(symboled_phone_method: '02031234567', country_code_attribute: 'GB') expect(model.normalized_symboled_phone_method).to eql('+442031234567') end end describe 'using model#phony_normalize' do it 'should not change normalized numbers (see #76)' do model = model_klass.new(phone_number: '+31-(0)10-1234123') expect(model).to be_valid expect(model.phone_number).to eql('+31101234123') end it 'should nilify attribute when it is set to nil' do model = model_klass.new(phone_number: '+31-(0)10-1234123') model.phone_number = nil expect(model).to be_valid expect(model.phone_number).to eql(nil) end it 'should nilify attribute when it is set to nil' do model = ActiveRecordModel.create!(phone_number: '+31-(0)10-1234123') model.phone_number = nil expect(model).to be_valid expect(model.save).to be(true) expect(model.reload.phone_number).to eql(nil) end it 'should empty attribute when it is set to ""' do # Github issue #149 model = ActiveRecordModel.create!(phone_number: '+31-(0)10-1234123') model.phone_number = '' expect(model).to be_valid expect(model.save).to be(true) expect(model.reload.phone_number).to eql('') end it 'should set a normalized version of an attribute using :as option' do model_klass.phony_normalize :phone_number, as: :phone_number_as_normalized model = model_klass.new(phone_number: '+31-(0)10-1234123') expect(model).to be_valid expect(model.phone_number_as_normalized).to eql('+31101234123') end it 'should nilify normalized version of an attribute when it is set to nil using :as option ' do model_klass.phony_normalize :phone_number, as: :phone_number_as_normalized model = model_klass.new(phone_number: '+31-(0)10-1234123', phone_number_as_normalized: '+31101234123') model.phone_number = nil expect(model).to be_valid expect(model.phone_number_as_normalized).to eq(nil) end it 'should not add a + using :add_plus option' do model_klass.phony_normalize :phone_number, add_plus: false model = model_klass.new(phone_number: '+31-(0)10-1234123') expect(model).to be_valid expect(model.phone_number).to eql('31101234123') end it 'should raise a RuntimeError at validation if the attribute doesn\'t exist' do dummy_klass.phony_normalize :non_existing_attribute dummy = dummy_klass.new expect(lambda do dummy.valid? end).to raise_error(RuntimeError) end it 'should raise a RuntimeError at validation if the :as option attribute doesn\'t exist' do dummy_klass.phony_normalize :phone_number, as: :non_existing_attribute dummy = dummy_klass.new expect(lambda do dummy.valid? end).to raise_error(RuntimeError) end it 'should accept a symbol when setting country_code options' do model = model_klass.new(symboled_phone: '0606060606', country_code_attribute: 'FR') expect(model).to be_valid expect(model.symboled_phone).to eql('+33606060606') end context 'relational normalization' do it 'should normalize based on custom attribute of the current model' do model_klass.phony_normalize :phone_number, default_country_code: ->(instance) { instance.custom_country_code } model = model_klass.new phone_number: '012 416 0001', custom_country_code: 'MY' expect(model).to be_valid expect(model.phone_number).to eq('+60124160001') end it 'should normalize based on specific attribute of the associative model' do model_klass.phony_normalize :phone_number, default_country_code: ->(instance) { instance.home_country.country_code } home_country = double('home_country', country_code: 'MY') model = model_klass.new phone_number: '012 416 0001', home_country: home_country expect(model).to be_valid expect(model.phone_number).to eq('+60124160001') end it 'should normalize based on default value if missing associative model' do model_klass.phony_normalize :phone_number, default_country_code: ->(instance) { instance.home_country&.country_code || 'MY' } model = model_klass.new phone_number: '012 416 0001', home_country: nil expect(model).to be_valid expect(model.phone_number).to eq('+60124160001') end end context 'conditional normalization' do context 'standalone methods' do it 'should only normalize if the :if conditional is true' do model_klass.phony_normalize :recipient, default_country_code: 'US', if: :use_phone? sms_alarm = model_klass.new recipient: '222 333 4444', delivery_method: 'sms' email_alarm = model_klass.new recipient: 'foo123@example.com', delivery_method: 'email' expect(sms_alarm).to be_valid expect(email_alarm).to be_valid expect(sms_alarm.recipient).to eq('+12223334444') expect(email_alarm.recipient).to eq('foo123@example.com') end it 'should only normalize if the :unless conditional is false' do model_klass.phony_normalize :recipient, default_country_code: 'US', unless: :use_email? sms_alarm = model_klass.new recipient: '222 333 4444', delivery_method: 'sms' email_alarm = model_klass.new recipient: 'foo123@example.com', delivery_method: 'email' expect(sms_alarm).to be_valid expect(email_alarm).to be_valid expect(sms_alarm.recipient).to eq('+12223334444') expect(email_alarm.recipient).to eq('foo123@example.com') end end context 'using lambdas' do it 'should only normalize if the :if conditional is true' do model_klass.phony_normalize :recipient, default_country_code: 'US', if: -> { delivery_method == 'sms' } sms_alarm = model_klass.new recipient: '222 333 4444', delivery_method: 'sms' email_alarm = model_klass.new recipient: 'foo123@example.com', delivery_method: 'email' expect(sms_alarm).to be_valid expect(email_alarm).to be_valid expect(sms_alarm.recipient).to eq('+12223334444') expect(email_alarm.recipient).to eq('foo123@example.com') end it 'should only normalize if the :unless conditional is false' do model_klass.phony_normalize :recipient, default_country_code: 'US', unless: -> { delivery_method == 'email' } sms_alarm = model_klass.new recipient: '222 333 4444', delivery_method: 'sms' email_alarm = model_klass.new recipient: 'foo123@example.com', delivery_method: 'email' expect(sms_alarm).to be_valid expect(email_alarm).to be_valid expect(sms_alarm.recipient).to eq('+12223334444') expect(email_alarm.recipient).to eq('foo123@example.com') end end end end end describe 'ActiveModel + ActiveModel::Validations::Callbacks' do let(:model_klass) { ActiveModelModel } let(:dummy_klass) { ActiveModelDummy } it_behaves_like 'model with PhonyRails' end describe 'ActiveRecord' do let(:model_klass) { ActiveRecordModel } let(:dummy_klass) { ActiveRecordDummy } it_behaves_like 'model with PhonyRails' it 'should correctly keep a hard set country_code' do model = model_klass.new(fax_number: '+1 978 555 0000') expect(model.valid?).to be true expect(model.fax_number).to eql('+19785550000') expect(model.save).to be true expect(model.save).to be true # revalidate model.reload expect(model.fax_number).to eql('+19785550000') model.fax_number = '(030) 8 61 29 06' expect(model.save).to be true # revalidate model.reload expect(model.fax_number).to eql('+61308612906') end context 'when enforce_record_country is turned off' do let(:model_klass) { RelaxedActiveRecordModel } let(:record) { model_klass.new } before do record.phone_number = phone_number record.country_code = 'DE' record.valid? # run the empty validation chain to execute the before hook (normalized the number) end context 'when the country_code attribute does not match the country number' do context 'when the number is prefixed with a country number and a plus' do let(:phone_number) { '+436601234567' } it 'should not add the records country number' do expect(record.phone_number).to eql('+436601234567') end end # In this case it's not clear if there is a country number, so it should be added context 'when the number is prefixed with a country number' do let(:phone_number) { '436601234567' } it 'should add the records country number' do expect(record.phone_number).to eql('+49436601234567') end end end # This should be the case anyways context 'when the country_code attribute matches the country number' do context 'when the number includes a country number and a plus' do let(:phone_number) { '+491721234567' } it 'should not add the records country number' do expect(record.phone_number).to eql('+491721234567') end end context 'when the number has neither country number nor plus' do let(:phone_number) { '01721234567' } it 'should not add the records country number' do expect(record.phone_number).to eql('+491721234567') end end end end end # describe 'Mongoid' do # let(:model_klass) { MongoidModel } # let(:dummy_klass) { MongoidDummy } # it_behaves_like 'model with PhonyRails' # end end