Update instance tracker tests.

test/src/common/instancetracker.spec.ts

@@ -12,6 +12,16 @@ import {
 } from '../../../lib/common/instancetracker';
 
 
+class TestTracker<T extends Widget> extends InstanceTracker<T> {
+  methods: string[] = [];
+
+  protected onCurrentChanged(): void {
+    super.onCurrentChanged();
+    this.methods.push('onCurrentChanged');
+  }
+}
+
+
 describe('common/instancetracker', () => {
 
   describe('InstanceTracker', () => {
@@ -31,7 +41,6 @@ describe('common/instancetracker', () => {
         let tracker = new InstanceTracker<Widget>();
         let widget = new Widget();
         let called = false;
-        widget.id = 'test';
         tracker.currentChanged.connect(() => { called = true; });
         tracker.add(widget);
         expect(called).to.be(false);
@@ -51,7 +60,6 @@ describe('common/instancetracker', () => {
       it('should be updated by sync if the tracker has the widget', () => {
         let tracker = new InstanceTracker<Widget>();
         let widget = new Widget();
-        widget.id = 'test';
         tracker.add(widget);
         expect(tracker.currentWidget).to.be(null);
         tracker.sync(widget);
@@ -60,6 +68,148 @@ describe('common/instancetracker', () => {
 
     });
 
+    describe('#isDisposed', () => {
+
+      it('should test whether the tracker is disposed', () => {
+        let tracker = new InstanceTracker<Widget>();
+        expect(tracker.isDisposed).to.be(false);
+        tracker.dispose();
+        expect(tracker.isDisposed).to.be(true);
+      });
+
+    });
+
+    describe('#add()', () => {
+
+      it('should add a widget to the tracker', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widget = new Widget();
+        expect(tracker.has(widget)).to.be(false);
+        tracker.add(widget);
+        expect(tracker.has(widget)).to.be(true);
+      });
+
+      it('should remove an added widget if it is disposed', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widget = new Widget();
+        tracker.add(widget);
+        expect(tracker.has(widget)).to.be(true);
+        widget.dispose();
+        expect(tracker.has(widget)).to.be(false);
+      });
+
+    });
+
+    describe('#dispose()', () => {
+
+      it('should dispose of the resources used by the tracker', () => {
+        let tracker = new InstanceTracker<Widget>();
+        expect(tracker.isDisposed).to.be(false);
+        tracker.dispose();
+        expect(tracker.isDisposed).to.be(true);
+      });
+
+      it('should be safe to call multiple times', () => {
+        let tracker = new InstanceTracker<Widget>();
+        expect(tracker.isDisposed).to.be(false);
+        tracker.dispose();
+        tracker.dispose();
+        expect(tracker.isDisposed).to.be(true);
+      });
+
+    });
+
+    describe('#find()', () => {
+
+      it('should find a tracked item that matches a filter function', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widgetA = new Widget();
+        let widgetB = new Widget();
+        let widgetC = new Widget();
+        widgetA.id = 'A';
+        widgetB.id = 'B';
+        widgetC.id = 'C';
+        tracker.add(widgetA);
+        tracker.add(widgetB);
+        tracker.add(widgetC);
+        expect(tracker.find(widget => widget.id === 'B')).to.be(widgetB);
+      });
+
+      it('should return `null` if no item is found', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widgetA = new Widget();
+        let widgetB = new Widget();
+        let widgetC = new Widget();
+        widgetA.id = 'A';
+        widgetB.id = 'B';
+        widgetC.id = 'C';
+        tracker.add(widgetA);
+        tracker.add(widgetB);
+        tracker.add(widgetC);
+        expect(tracker.find(widget => widget.id === 'D')).to.be(null);
+      });
+
+    });
+
+    describe('#forEach()', () => {
+
+      it('should iterate through all the tracked items', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widgetA = new Widget();
+        let widgetB = new Widget();
+        let widgetC = new Widget();
+        let visited = '';
+        widgetA.id = 'A';
+        widgetB.id = 'B';
+        widgetC.id = 'C';
+        tracker.add(widgetA);
+        tracker.add(widgetB);
+        tracker.add(widgetC);
+        tracker.forEach(widget => { visited += widget.id; });
+        expect(visited).to.be('ABC');
+      });
+
+    });
+
+    describe('#has()', () => {
+
+      it('should return `true` if an item exists in the tracker', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widget = new Widget();
+        expect(tracker.has(widget)).to.be(false);
+        tracker.add(widget);
+        expect(tracker.has(widget)).to.be(true);
+      });
+
+    });
+
+    describe('#sync()', () => {
+
+      it('should emit a signal when the current widget is updated', () => {
+        let tracker = new InstanceTracker<Widget>();
+        let widget = new Widget();
+        let called = false;
+        tracker.currentChanged.connect(() => { called = true; });
+        tracker.add(widget);
+        expect(called).to.be(false);
+        tracker.sync(widget);
+        expect(called).to.be(true);
+      });
+
+    });
+
+    describe('#onCurrentChanged()', () => {
+
+      it('should be called when the current widget is changed', () => {
+        let tracker = new TestTracker<Widget>();
+        let widget = new Widget();
+        tracker.add(widget);
+        tracker.sync(widget);
+        expect(tracker.methods).to.contain('onCurrentChanged');
+      });
+
+    });
+
   });
 
 });