Add tests for when a frame is destroyed

src/sidebar/reducers/test/frames-test.js

@@ -21,6 +21,19 @@ describe('frames reducer', function () {
     });
   });
 
+  describe('#destroyFrame', function () {
+    it('removes the frame from the list of connected frames', function () {
+      var frameList = [{uri: 'http://example.com'}, {uri: 'http://example.org'}];
+      var state = init();
+      frameList.forEach(function (frame) {
+        state = update(state, actions.connectFrame(frame));
+      });
+      assert.deepEqual(frames.frames(state), frameList);
+      var updatedState = update(state, actions.destroyFrame(frameList[0]));
+      assert.deepEqual(frames.frames(updatedState), [frameList[1]]);
+    });
+  });
+
   describe('#updateFrameAnnotationFetchStatus', function () {
     it('updates the isAnnotationFetchComplete status of the frame', function () {
       var frame = {

src/sidebar/test/frame-sync-test.js

@@ -39,6 +39,12 @@ var fixtures = {
       link: [{href: 'http://example.org/paper.pdf'}, {href:'urn:1234'}],
     },
   },
+
+  // The entry in the list of frames currently connected
+  framesListEntry: {
+    uri: 'http://example.com',
+    isAnnotationFetchComplete: true,
+  },
 };
 
 describe('FrameSync', function () {
@@ -55,9 +61,10 @@ describe('FrameSync', function () {
   beforeEach(function () {
     fakeAnnotationUI = fakeStore({annotations: []}, {
       connectFrame: sinon.stub(),
+      destroyFrame: sinon.stub(),
       findIDsForTags: sinon.stub(),
       focusAnnotations: sinon.stub(),
-      frames: sinon.stub().returns([{uri: 'http://example.com', isAnnotationFetchComplete: true }]),
+      frames: sinon.stub().returns([fixtures.framesListEntry]),
       selectAnnotations: sinon.stub(),
       selectTab: sinon.stub(),
       toggleSelectedAnnotations: sinon.stub(),
@@ -214,6 +221,16 @@ describe('FrameSync', function () {
     });
   });
 
+  context('when a frame is destroyed', function () {
+    var frameUri = fixtures.framesListEntry.uri;
+
+    it("adds the page's metadata to the frames list", function () {
+      fakeBridge.emit('destroyFrame', frameUri);
+
+      assert.calledWith(fakeAnnotationUI.destroyFrame, fixtures.framesListEntry);
+    });
+  });
+
   describe('on "showAnnotations" message', function () {
     it('selects annotations which have an ID', function () {
       fakeAnnotationUI.findIDsForTags.returns(['id1','id2','id3']);