Restore all sinon stubs after each test

Author: acelaya

src/annotator/test/sidebar-test.js

@@ -30,16 +30,6 @@ describe('Sidebar', () => {
   let fakeSendErrorsTo;
   let fakeEmitter;
 
-  beforeAll(() => {
-    // Make `requestAnimationFrame` invoke its callback synchronously. rAF is
-    // used to debounce some internal actions.
-    sinon.stub(window, 'requestAnimationFrame').yields();
-  });
-
-  afterAll(() => {
-    window.requestAnimationFrame.restore();
-  });
-
   // Helpers for getting the channels used for host <-> guest/sidebar communication.
   // These currently rely on knowing the implementation detail of which order
   // the channels are created in.
@@ -132,6 +122,10 @@ describe('Sidebar', () => {
   };
 
   beforeEach(() => {
+    // Make `requestAnimationFrame` invoke its callback synchronously. rAF is
+    // used to debounce some internal actions.
+    sinon.stub(window, 'requestAnimationFrame').yields();
+
     sidebars = [];
     containers = [];
 

src/sidebar/test/bootstrap.js

@@ -19,7 +19,9 @@ globalThis.context ??= globalThis.describe;
 // Configure Enzyme for UI tests.
 configure({ adapter: new Adapter() });
 
-// Unmount all UI components after each test.
 afterEach(() => {
+  // Unmount all UI components after each test.
   unmountAll();
+  // Restore all created mocks during a test, to free-up memory
+  sinon.restore();
 });