Make AnnotationsExporter not depend on SidebarStore

src/sidebar/components/ShareDialog/ExportAnnotations.tsx

@@ -84,7 +84,8 @@ function ExportAnnotations({
   );
 
   // User whose annotations are going to be exported.
-  const currentUser = store.profile().userid;
+  const profile = store.profile();
+  const currentUser = profile.userid;
   const allAnnotationsOption: Omit<UserAnnotations, 'userid'> = useMemo(
     () => ({
       annotations: exportableAnnotations,
@@ -131,8 +132,10 @@ function ExportAnnotations({
 
       switch (format) {
         case 'json': {
-          const exportData =
-            annotationsExporter.buildJSONExportContent(annotationsToExport);
+          const exportData = annotationsExporter.buildJSONExportContent(
+            annotationsToExport,
+            { profile },
+          );
           downloadJSONFile(exportData, filename);
           break;
         }

src/sidebar/services/annotations-exporter.ts

 import { trimAndDedent } from '../../shared/trim-and-dedent';
-import type { APIAnnotationData } from '../../types/api';
+import type { APIAnnotationData, Profile } from '../../types/api';
 import {
   documentMetadata,
   isReply,
@@ -9,7 +9,6 @@ import {
 import { annotationDisplayName } from '../helpers/annotation-user';
 import { stripInternalProperties } from '../helpers/strip-internal-properties';
 import { VersionData } from '../helpers/version-data';
-import type { SidebarStore } from '../store';
 
 export type JSONExportContent = {
   export_date: string;
@@ -18,6 +17,11 @@ export type JSONExportContent = {
   annotations: APIAnnotationData[];
 };
 
+export type JSONExportOptions = {
+  profile: Profile;
+  now?: Date;
+};
+
 export type TextExportOptions = {
   defaultAuthority?: string;
   displayNamesEnabled?: boolean;
@@ -31,18 +35,14 @@ export type TextExportOptions = {
  * @inject
  */
 export class AnnotationsExporter {
-  private _store: SidebarStore;
-
-  constructor(store: SidebarStore) {
-    this._store = store;
-  }
-
   buildJSONExportContent(
     annotations: APIAnnotationData[],
+    {
+      profile,
       /* istanbul ignore next - test seam */
       now = new Date(),
+    }: JSONExportOptions,
   ): JSONExportContent {
-    const profile = this._store.profile();
     const versionData = new VersionData(profile, []);
 
     return {

src/sidebar/services/test/annotations-exporter-test.js

@@ -6,20 +6,17 @@ import {
 import { AnnotationsExporter } from '../annotations-exporter';
 
 describe('AnnotationsExporter', () => {
-  let fakeStore;
   let now;
   let exporter;
 
   beforeEach(() => {
-    fakeStore = {
-      profile: sinon.stub().returns({ userid: 'userId' }),
-    };
     now = new Date();
-
-    exporter = new AnnotationsExporter(fakeStore);
+    exporter = new AnnotationsExporter();
   });
 
+  describe('buildJSONExportContent', () => {
     it('generates JSON content with provided annotations', () => {
+      const profile = { userid: 'userId' };
       const firstBaseAnnotation = publicAnnotation();
       const secondBaseAnnotation = publicAnnotation();
       const annotations = [
@@ -33,7 +30,10 @@ describe('AnnotationsExporter', () => {
         },
       ];
 
-    const result = exporter.buildJSONExportContent(annotations, now);
+      const result = exporter.buildJSONExportContent(annotations, {
+        now,
+        profile,
+      });
 
       assert.deepEqual(result, {
         export_date: now.toISOString(),
@@ -42,6 +42,7 @@ describe('AnnotationsExporter', () => {
         annotations: [firstBaseAnnotation, secondBaseAnnotation],
       });
     });
+  });
 
   describe('buildTextExportContent', () => {
     let baseAnnotation;