[FIX] store: properly call patch on connected components

A previous refactoring broke it by adding a [] arguments to updateprops.

src/component.ts

@@ -332,7 +332,7 @@ export class Component<
   async _updateProps(
     nextProps: Props,
     forceUpdate: boolean = false,
-    patchQueue: any[]
+    patchQueue?: any[]
   ): Promise<void> {
     const shouldUpdate = forceUpdate || this.shouldUpdate(nextProps);
     if (shouldUpdate) {

src/store.ts

@@ -212,10 +212,10 @@ export function connect(mapStateToProps, options: any = {}) {
         }
         if (didChange) {
           (<any>this.__owl__).currentStoreProps = storeProps;
-          this._updateProps(ownProps, false, []);
+          this._updateProps(ownProps, false);
         }
       }
-      _updateProps(nextProps, forceUpdate, patchQueue: any[]) {
+      _updateProps(nextProps, forceUpdate, patchQueue?: any[]) {
         if ((<any>this.__owl__).ownProps !== nextProps) {
           (<any>this.__owl__).currentStoreProps = mapStateToProps(
             this.env.store.state,

tests/store.test.ts

@@ -714,4 +714,39 @@ describe("connecting a component to store", () => {
     await nextTick();
     expect(steps).toEqual(["parent", "child", "parent", "child", "child"]);
   });
+
+  test("connected component willpatch/patch hooks are called on store updates", async () => {
+    const steps: string[] = [];
+    class App extends Component<any, any, any> {
+      inlineTemplate = `<div><t t-esc="props.msg"/></div>`;
+      willPatch() {
+        steps.push("willpatch");
+      }
+      patched() {
+        steps.push("patched");
+      }
+    }
+    const ConnectedApp = connect(function(s) {
+      return { msg: s.msg };
+    })(App);
+
+    const state = { msg: "a" };
+    const mutations = {
+      setMsg({ state }, c) {
+        state.msg = c;
+      }
+    };
+
+    const store = new Store({ state, mutations });
+    (<any>env).store = store;
+    const app = new ConnectedApp(env);
+
+    await app.mount(fixture);
+    expect(fixture.innerHTML).toBe("<div>a</div>");
+
+    store.commit("setMsg", "b");
+    await nextTick();
+    expect(fixture.innerHTML).toBe("<div>b</div>");
+    expect(steps).toEqual(["willpatch", "patched"]);
+  });
 });