Fix update bugs

packages/perspective/src/js/api/dispatch.js

@@ -57,7 +57,8 @@ export function subscribe(method, cmd) {
                 resolve = args.splice(i, 1)[0];
             }
         }
-        __CALLBACK_CACHE__.set(resolve, __CALLBACK_INDEX__++);
+        __CALLBACK_INDEX__++;
+        __CALLBACK_CACHE__.set(resolve, __CALLBACK_INDEX__);
         let msg = {
             cmd: cmd || "view_method",
             name: this._name,

packages/perspective/src/js/perspective.js

@@ -1293,6 +1293,8 @@ export default function(Module) {
                 meter(pdata.map(x => x.row_count).reduce((x, y) => x + y));
             }
             is_arrow = true;
+            // make sure implicit index can be read for arrow updates
+            accessor.names = pdata[0].names;
         } else if (typeof data === "string") {
             if (data[0] === ",") {
                 data = "_" + data;

packages/perspective/test/js/delete.js

@@ -8,7 +8,7 @@
  */
 
 module.exports = perspective => {
-    describe.only("Delete", function() {
+    describe("Delete", function() {
         it("calls all delete callbacks registered on table", async function() {
             const table = perspective.table([{x: 1}]);
 

packages/perspective/test/js/updates.js

@@ -1290,4 +1290,48 @@ module.exports = perspective => {
             table.delete();
         });
     });
+
+    describe("Remove update", function() {
+        it("Should remove a single update", function(done) {
+            const cb1 = jest.fn();
+            const cb2 = () => {
+                expect(cb1).toBeCalledTimes(0);
+                setTimeout(() => {
+                    view.delete();
+                    table.delete();
+                    done();
+                }, 0);
+            };
+            const table = perspective.table(meta);
+            const view = table.view();
+            view.on_update(cb1);
+            view.on_update(cb2);
+            view.remove_update(cb1);
+            table.update(data);
+        });
+
+        it("Should remove multiple updates", function(done) {
+            const cb1 = jest.fn();
+            const cb2 = jest.fn();
+            const cb3 = function() {
+                // cb2 should have been called
+                expect(cb1).toBeCalledTimes(0);
+                expect(cb2).toBeCalledTimes(0);
+                setTimeout(() => {
+                    view.delete();
+                    table.delete();
+                    done();
+                }, 0);
+            };
+
+            const table = perspective.table(meta);
+            const view = table.view();
+            view.on_update(cb1);
+            view.on_update(cb2);
+            view.on_update(cb3);
+            view.remove_update(cb1);
+            view.remove_update(cb2);
+            table.update(data);
+        });
+    });
 };