Use VNode approach in a11y tests where we were wrapping component under test in div

src/sidebar/components/ShareDialog/test/ImportAnnotations-test.js

@@ -339,7 +339,6 @@ describe('ImportAnnotations', () => {
   it(
     'should pass a11y checks',
     checkAccessibility({
-      // re. not using `mount`, see https://github.com/hypothesis/client/issues/5690
       content: () => (
         <ImportAnnotations
           store={fakeStore}

src/sidebar/components/ShareDialog/test/ShareDialog-test.js

@@ -126,10 +126,6 @@ describe('ShareDialog', () => {
     it(
       'should pass a11y checks',
       checkAccessibility({
-        // ShareDialog renders a Fragment as its top-level component when
-        // it has import and/or export tabs.
-        // Returning the VNode verbatim ensures no children are discarded.
-        // See https://github.com/hypothesis/client/issues/5671
         content: () => <ShareDialog shareTab exportTab importTab />,
       }),
     );