fix: Update auth_app params.

After 0.6.0 safe-nodejs update, port is no longer required (and should be a string anyway). This fixes #1119
maidsafe · Dec 17, 2019 · 611e7d5 · 611e7d5
1 parent afc5f2a
commit 611e7d5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/extensions/safe/backgroundProcess/safeBrowserApplication/init/initAuthed.ts b/app/extensions/safe/backgroundProcess/safeBrowserApplication/init/initAuthed.ts
@@ -23,8 +23,7 @@ export const initAuthed = async (): Safe => {
         const authCredentials = await safeBrowserAppObject.auth_app(
             APP_ID,
             APP_NAME,
-            APP_VENDOR,
-            41805
+            APP_VENDOR
         );
 
         await safeBrowserAppObject.connect( APP_ID, authCredentials );

diff --git a/app/extensions/safe/test/utils/requestManagement.spec.ts b/app/extensions/safe/test/utils/requestManagement.spec.ts
@@ -13,7 +13,9 @@ jest.mock( 'electron', () => {
     const app = {
         getPath: jest.fn( () => {
             return '/Somewhere/Electron.app';
-        } )
+        } ),
+        getName: jest.fn( () => 'Safe Browser' ),
+        getVersion: jest.fn( () => 'v0.42.0' )
     };
 
     return {