Fix sendMessage declaration and usage

ivantsov · Jan 9, 2017 · 688020e · 688020e
1 parent ee7311a
commit 688020e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/decls/vars.js b/decls/vars.js
@@ -3,7 +3,6 @@ declare var chrome: {
         connect: (options: {name: string}) => Connection,
         sendMessage: (
             msg: any,
-            sender?: ?string,
             cb?: (res: Object) => void
         ) => void,
         onConnect: {

diff --git a/src/ui-store.js b/src/ui-store.js
@@ -47,7 +47,7 @@ export default function (): Promise<Store> {
 
     // return promise to allow getting current state of "background" store
     return new Promise(resolve => {
-        chrome.runtime.sendMessage({type: UPDATE_STATE}, null, res => {
+        chrome.runtime.sendMessage({type: UPDATE_STATE}, res => {
             state = res;
 
             // return an object with equivalent to Redux store interface

diff --git a/tests/ui-store.js b/tests/ui-store.js
@@ -23,9 +23,9 @@ async function createStore(state) {
 
     expect(chrome.runtime.connect).lastCalledWith({name: CONNECTION_NAME});
     expect(connection.onMessage.addListener).lastCalledWith(jasmine.any(Function));
-    expect(chrome.runtime.sendMessage).lastCalledWith({type: UPDATE_STATE}, null, jasmine.any(Function));
+    expect(chrome.runtime.sendMessage).lastCalledWith({type: UPDATE_STATE}, jasmine.any(Function));
 
-    chrome.runtime.sendMessage.mock.calls[0][2](state);
+    chrome.runtime.sendMessage.mock.calls[0][1](state);
 
     const store = await promise;