Refactoring: Add example descriptions to existing Cti::CallerId spec

2019-01-14 17:30:51 +08:00 · 2019-01-14 17:30:51 +08:00 · 44822d5483
commit 44822d5483
parent 6839f040e0
1 changed files with 116 additions and 51 deletions
--- a/spec/models/cti/caller_id_spec.rb
+++ b/spec/models/cti/caller_id_spec.rb
@ -1,66 +1,131 @@
 require 'rails_helper'

 RSpec.describe Cti::CallerId do
-  subject { create(:cti_caller_id, caller_id: phone) }
-  let(:phone) { '1234567890' }
+  describe '.extract_numbers' do
+    context 'for strings containing arbitrary numbers (<6 digits long)' do
+      let(:input) { <<~INPUT }
+        some text
+        test 123
+      INPUT

-  describe 'extract_numbers' do
-    it { expect(described_class.extract_numbers("some text\ntest 123")).to eq [] }
-    it { expect(described_class.extract_numbers('Lorem ipsum dolor sit amet, consectetuer +49 (0) 30 60 00 00 00-0 adipiscing elit. Aenean commodo ligula eget dolor. Aenean massa. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Donec quam felis, ultricies nec, pellentesque eu, pretium quis, sem. Nulla consequat massa quis enim. Donec pede justo, fringilla vel.')).to eq ['4930600000000'] }
-    it { expect(described_class.extract_numbers("GS Oberalteich\nTelefon  09422 1000 Telefax 09422 805000\nE-Mail:  ")).to eq %w[4994221000 499422805000] }
-    it { expect(described_class.extract_numbers('Tel +41 81 288 63 93 / +41 76 346 72 14 ...')).to eq %w[41812886393 41763467214] }
-    it { expect(described_class.extract_numbers("P: +1 (949) 431 0000\nF: +1 (949) 431 0001\nW: http://znuny")).to eq %w[19494310000 19494310001] }
-  end
-
-  describe 'normalize_number' do
-    # can be anything
-    it { expect(described_class.normalize_number('5754321')).to eq '5754321' }
-    it { expect(described_class.normalize_number('622 32281')).to eq '62232281' }
-    it { expect(described_class.normalize_number('0049 1234 123456789')).to eq '491234123456789' }
-    it { expect(described_class.normalize_number('022 1234567')).to eq '49221234567' }
-    it { expect(described_class.normalize_number('0271233211')).to eq '49271233211' }
-    it { expect(described_class.normalize_number('021-233-9123')).to eq '49212339123' }
-    it { expect(described_class.normalize_number('09 123 32112')).to eq '49912332112' }
-    it { expect(described_class.normalize_number('021 2331231')).to eq '49212331231' }
-    it { expect(described_class.normalize_number('021 321123123')).to eq '4921321123123' }
-    it { expect(described_class.normalize_number('0150 12345678')).to eq '4915012345678' }
-    it { expect(described_class.normalize_number('092213212')).to eq '4992213212' }
-    it { expect(described_class.normalize_number('(09)1234321')).to eq '4991234321' }
-    it { expect(described_class.normalize_number('+49 30 53 00 00 000')).to eq '4930530000000' }
-    it { expect(described_class.normalize_number('+49 160 0000000')).to eq '491600000000' }
-    it { expect(described_class.normalize_number('+49 (0) 30 60 00 00 00-0')).to eq '4930600000000' }
-    it { expect(described_class.normalize_number('0043 (0) 30 60 00 00 00-0')).to eq '4330600000000' }
-    it { expect(described_class.normalize_number('0043 30 60 00 00 00-0')).to eq '4330600000000' }
-    it { expect(described_class.normalize_number('1-888-407-4747')).to eq '18884074747' }
-  end
-
-  context 'on creation' do
-    it 'adopts CTI Logs from same number (via UpdateCtiLogsByCallerJob)' do
-      allow(UpdateCtiLogsByCallerJob).to receive(:perform_later).with(any_args)
-
-      subject # create CallerId record
-
-      expect(UpdateCtiLogsByCallerJob).to have_received(:perform_later)
+      it 'returns an empty array' do
+        expect(described_class.extract_numbers(input)).to be_empty
+      end
    end

-    it 'splits job into fg and bg (for more responsive UI – see #2057)' do
-      allow(UpdateCtiLogsByCallerJob).to receive(:perform_now).with(any_args)
-      allow(UpdateCtiLogsByCallerJob).to receive(:perform_later).with(any_args)
+    context 'for strings containing a phone number with "(0)" after country code' do
+      let(:input) { <<~INPUT }
+        Lorem ipsum dolor sit amet, consectetuer +49 (0) 30 60 00 00 00-0 adipiscing elit. Aenean commodo ligula eget dolor. Aenean massa. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Donec quam felis, ultricies nec, pellentesque eu, pretium quis, sem. Nulla consequat massa quis enim. Donec pede justo, fringilla vel.
+      INPUT

-      subject # create CallerId record
+      it 'returns the number in an array, without the leading "(0)"' do
+        expect(described_class.extract_numbers(input)).to eq(['4930600000000'])
+      end
+    end

-      expect(UpdateCtiLogsByCallerJob).to have_received(:perform_now).with(phone, limit: 20)
-      expect(UpdateCtiLogsByCallerJob).to have_received(:perform_later).with(phone, limit: 40, offset: 20)
+    context 'for strings containing a phone number with leading 0 (no country code)' do
+      let(:input) { <<~INPUT }
+        GS Oberalteich
+        Telefon  09422 1000
+        E-Mail:
+      INPUT
+
+      it 'returns the number in an array, using default country code (49)' do
+        expect(described_class.extract_numbers(input)).to eq(['4994221000'])
+      end
+    end
+
+    context 'for strings containing multiple phone numbers' do
+      let(:input) { <<~INPUT }
+        Tel +41 81 288 63 93 / +41 76 346 72 14 ...
+      INPUT
+
+      it 'returns all numbers in an array' do
+        expect(described_class.extract_numbers(input)).to eq(%w[41812886393 41763467214])
+      end
+    end
+
+    context 'for strings containing US-formatted numbers' do
+      let(:input) { <<~INPUT }
+        P: +1 (949) 431 0000
+        F: +1 (949) 431 0001
+        W: http://znuny
+      INPUT
+
+      it 'returns the numbers in an array correctly' do
+        expect(described_class.extract_numbers(input)).to eq(%w[19494310000 19494310001])
+      end
    end
  end

-  context 'on destruction' do
-    before { subject }
+  describe '.normalize_number' do
+    it 'does not modify digit-only strings (starting with 1-9)' do
+      expect(described_class.normalize_number('5754321')).to eq('5754321')
+    end

-    it 'orphans CTI Logs from same number (via UpdateCtiLogsByCallerJob)' do
-      allow(UpdateCtiLogsByCallerJob).to receive(:perform_later).with(phone)
-      subject.destroy
-      expect(UpdateCtiLogsByCallerJob).to have_received(:perform_later)
+    it 'strips whitespace' do
+      expect(described_class.normalize_number('622 32281')).to eq('62232281')
+    end
+
+    it 'strips hyphens' do
+      expect(described_class.normalize_number('1-888-407-4747')).to eq('18884074747')
+    end
+
+    it 'strips leading pluses' do
+      expect(described_class.normalize_number('+49 30 53 00 00 000')).to eq('4930530000000')
+      expect(described_class.normalize_number('+49 160 0000000')).to eq('491600000000')
+    end
+
+    it 'replaces a single leading zero with the default country code (49)' do
+      expect(described_class.normalize_number('092213212')).to eq('4992213212')
+      expect(described_class.normalize_number('0271233211')).to eq('49271233211')
+      expect(described_class.normalize_number('022 1234567')).to eq('49221234567')
+      expect(described_class.normalize_number('09 123 32112')).to eq('49912332112')
+      expect(described_class.normalize_number('021 2331231')).to eq('49212331231')
+      expect(described_class.normalize_number('021 321123123')).to eq('4921321123123')
+      expect(described_class.normalize_number('0150 12345678')).to eq('4915012345678')
+      expect(described_class.normalize_number('021-233-9123')).to eq('49212339123')
+    end
+
+    it 'strips two leading zeroes' do
+      expect(described_class.normalize_number('0049 1234 123456789')).to eq('491234123456789')
+      expect(described_class.normalize_number('0043 30 60 00 00 00-0')).to eq('4330600000000')
+    end
+
+    it 'strips leading zero from "(0x)" at start of number or after country code' do
+      expect(described_class.normalize_number('(09)1234321')).to eq('4991234321')
+      expect(described_class.normalize_number('+49 (0) 30 60 00 00 00-0')).to eq('4930600000000')
+      expect(described_class.normalize_number('0043 (0) 30 60 00 00 00-0')).to eq('4330600000000')
+    end
+  end
+
+  describe 'callbacks' do
+    subject!(:caller_id) { build(:cti_caller_id, caller_id: phone) }
+    let(:phone) { '1234567890' }
+
+    describe 'on creation' do
+      it 'adopts CTI Logs from same number (via UpdateCtiLogsByCallerJob)' do
+        expect(UpdateCtiLogsByCallerJob).to receive(:perform_later)
+
+        caller_id.save
+      end
+
+      it 'splits job into fg and bg (for more responsive UI – see #2057)' do
+        expect(UpdateCtiLogsByCallerJob).to receive(:perform_now).with(phone, limit: 20)
+        expect(UpdateCtiLogsByCallerJob).to receive(:perform_later).with(phone, limit: 40, offset: 20)
+
+        caller_id.save
+      end
+    end
+
+    describe 'on destruction' do
+      before { caller_id.save }
+
+      it 'orphans CTI Logs from same number (via UpdateCtiLogsByCallerJob)' do
+        expect(UpdateCtiLogsByCallerJob).to receive(:perform_later).with(phone)
+
+        caller_id.destroy
+      end
    end
  end
 end