@wazo/sdk 0.41.1 → 0.42.0-rc.0

package/src/utils/__tests__/PhoneNumberUtil.test.ts

@@ -1,4 +1,4 @@
-import { getDisplayableNumber, getCallableNumber } from '../PhoneNumberUtil';
+import { getDisplayableNumber, getCallableNumber, guessParsePhoneNumber } from '../PhoneNumberUtil';
 
 describe('Formatting phone numbers', () => {
   it('should not format all phone numbers', () => {
@@ -8,30 +8,34 @@ describe('Formatting phone numbers', () => {
     expect(getDisplayableNumber('#23445433', 'DE')).toBe('#23445433');
     expect(getDisplayableNumber('0080510', 'FR')).toBe('0080510');
   });
+
   it('should format real phone numbers', () => {
     expect(getDisplayableNumber('+33675456545', 'FR')).toBe('06 75 45 65 45');
     expect(getDisplayableNumber('0675456545', 'FR')).toBe('06 75 45 65 45');
     expect(getDisplayableNumber('+33675456545', 'US')).toBe('+33 6 75 45 65 45');
-    expect(getDisplayableNumber('0675456545', 'US')).toBe('+1 0675456545');
+    expect(getDisplayableNumber('0675456545', 'US')).toBe('+33 6 75 45 65 45');
     expect(getDisplayableNumber('4188884356', 'US')).toBe('(418) 888-4356');
   });
+
   it('should format real phone numbers when typing', () => {
     expect(getDisplayableNumber('+3367545', 'FR', true)).toBe('+33 6 75 45');
     expect(getDisplayableNumber('067545', 'FR', true)).toBe('06 75 45');
     expect(getDisplayableNumber('+3367545', 'US', true)).toBe('+33 6 75 45');
     expect(getDisplayableNumber('067545', 'US', true)).toBe('067545');
-    expect(getDisplayableNumber('418808', 'US', true)).toBe('418-808');
+    expect(getDisplayableNumber('418808', 'US', true)).toBe('(418) 808');
   });
+
   it('should use the international number if the country number is different from the given country', () => {
-    expect(getDisplayableNumber('00201005803648', 'FR')).toBe('+20 100 580 3648');
-    expect(getDisplayableNumber('+14188004554', 'FR')).toBe('+1 418-800-4554');
-    expect(getDisplayableNumber('00201005803648', 'EG')).toBe('0100 580 3648');
+    expect(getDisplayableNumber('00201005803648', 'FR')).toBe('+20 10 05803648');
+    expect(getDisplayableNumber('+14188004554', 'FR')).toBe('+1 418 800 4554');
+    expect(getDisplayableNumber('00201005803648', 'EG')).toBe('010 05803648');
     expect(getDisplayableNumber('0634543450', 'FR')).toBe('06 34 54 34 50');
     expect(getDisplayableNumber('4188034302', 'US')).toBe('(418) 803-4302');
     expect(getDisplayableNumber('8775863230', 'CA')).toBe('(877) 586-3230');
     expect(getDisplayableNumber('8775863230', 'US')).toBe('(877) 586-3230');
   });
 });
+
 describe('getCallableNumber', () => {
   it('works with a country', () => {
     expect(getCallableNumber('+33 6 75 45 12 34', 'FR')).toBe('0675451234');
@@ -43,6 +47,7 @@ describe('getCallableNumber', () => {
     expect(getCallableNumber('*10', 'US')).toBe('*10');
     expect(getCallableNumber('9000', 'US')).toBe('9000');
   });
+
   it('works without a country', () => {
     expect(getCallableNumber('06 75 45')).toBe('067545');
     expect(getCallableNumber('067-545')).toBe('067545');
@@ -52,3 +57,23 @@ describe('getCallableNumber', () => {
     expect(getCallableNumber('9000')).toBe('9000');
   });
 });
+
+describe('guessParsePhoneNumber', () => {
+  it('should guess with default countries', () => {
+    const parsedNumber = guessParsePhoneNumber('0675456545');
+    expect(parsedNumber?.country).toBe('FR');
+    expect(parsedNumber?.isValid()).toBe(true);
+  });
+
+  it('should guess with specified countries', () => {
+    const parsedNumber = guessParsePhoneNumber('752097514', undefined, ['JP']);
+    expect(parsedNumber?.country).toBe('JP');
+    expect(parsedNumber?.isValid()).toBe(true);
+  });
+
+  it('should used use defaultCountry has guesser', () => {
+    const parsedNumber = guessParsePhoneNumber('752097514', 'JP', []);
+    expect(parsedNumber?.country).toBe('JP');
+    expect(parsedNumber?.isValid()).toBe(true);
+  });
+});

package/src/utils/__tests__/api-requester.test.ts

@@ -79,12 +79,14 @@ describe('Retrieving headers', () => {
 });
 describe('Calling fetch', () => {
   it('should call fetch without body but query string in get method', async () => {
-    global.fetch = jest.fn(() => Promise.resolve({
-      json: () => Promise.resolve({}),
-      headers: {
-        get: () => '',
-      },
-    })) as any;
+    Object.defineProperty(global, 'fetch', {
+      value: jest.fn(() => Promise.resolve({
+        json: () => Promise.resolve({}),
+        headers: {
+          get: () => '',
+        },
+      }) as any),
+    });
 
     await new ApiRequester({
       server,
@@ -107,17 +109,19 @@ describe('Calling fetch', () => {
 describe('With a refresh token', () => {
   it('should retry the call with a new token', async () => {
     let calls = 0;
+    Object.defineProperty(global, 'fetch', {
+      value: jest.fn(() => {
+        calls++;
+        return Promise.resolve({
+          headers: {
+            get: () => 'application/json',
+          },
+          status: calls === 1 ? 401 : 200,
+          json: () => Promise.resolve({}),
+        } as any);
+      }),
+    });
 
-    global.fetch = jest.fn(() => {
-      calls++;
-      return Promise.resolve({
-        headers: {
-          get: () => 'application/json',
-        },
-        status: calls === 1 ? 401 : 200,
-        json: () => Promise.resolve({}),
-      });
-    }) as any;
     const requester = new ApiRequester({
       server,
       agent: null,

package/src/web-rtc-client.ts

@@ -565,7 +565,7 @@ export default class WebRTCClient extends Emitter {
         audioOnly,
       },
       earlyMedia: true,
-      options,
+      ...options,
     };
 
     if (audioOnly) {

package/src/websocket-client.ts

@@ -277,7 +277,7 @@ class WebSocketClient extends Emitter {
       return;
     }
 
-    this.socket.close();
+    this.socket.close(1000);
     this.initialized = false;
 
     if (force) {