Use non-default book for test; Fix capitalization

Author: imnasnainaec

README.md

@@ -99,7 +99,7 @@ The general file structure for an extension is as follows:
   - `*.web-view.tsx` files will be treated as React WebViews
   - `*.web-view.scss` files provide styles for WebViews
   - `*.web-view.html` files are a conventional way to provide HTML WebViews (no special functionality)
-  - `src/__tests__/` contains unit tests (Jest) for the extension, including parser tests (valid and invalid XML, edge cases) and web-view tests
+  - `src/__tests__/` contains unit tests (Jest) for the extension, including parser tests (valid and invalid XML, edge cases) and WebView tests
 - `__mocks__/` contains Jest mocks for the PAPI, file modules, and test fixtures used by tests in `src/__tests__/`. The `@papi/backend` and `@papi/frontend` mocks are used mutually exclusively (backend for main.ts tests, frontend for WebView tests); each mock file ends with `export {}` so TypeScript treats it as a module.
 - `assets/` contains asset files the extension and its WebViews can retrieve using the `papi-extension:` protocol, as well as textual descriptions in various languages. It is copied into the build folder
   - `assets/displayData.json` contains (optionally) a path to the extension's icon file as well as text for the extension's display name, short summary, and path to the full description file

src/__tests__/interlinearizer.web-view.test.tsx

@@ -68,11 +68,7 @@ describe('InterlinearizerWebView', () => {
   });
 
   it('shows the book and projectId when a project is linked', () => {
-    mockBookData({
-      type: 'USJ',
-      version: '3.1',
-      content: [{ type: 'book', marker: 'id', code: 'GEN' }],
-    });
+    mockBookData({ type: 'USJ', version: '3.1', content: [] });
     render(<InterlinearizerWebView {...makeProps('test-project-id')} />);
 
     expect(screen.getByText(/test-project-id/)).toBeInTheDocument();
@@ -100,11 +96,11 @@ describe('InterlinearizerWebView', () => {
     mockBookData({
       type: 'USJ',
       version: '3.1',
-      content: [{ type: 'book', marker: 'id', code: 'GEN' }],
+      content: [{ type: 'book', marker: 'id', code: 'EXO' }],
     });
     render(<InterlinearizerWebView {...makeProps('test-project-id')} />);
 
-    expect(screen.getByText(/"code": "GEN"/)).toBeInTheDocument();
+    expect(screen.getByText(/"code": "EXO"/)).toBeInTheDocument();
   });
 
   it('shows an error heading and message when book data is a PlatformError', () => {

src/__tests__/main.test.ts

@@ -264,7 +264,7 @@ describe('main', () => {
       });
     });
 
-    it('reuses the webview ID on subsequent opens of the same project', async () => {
+    it('reuses the WebView ID on subsequent opens of the same project', async () => {
       __mockOpenWebView.mockResolvedValue('saved-webview-id');
       const open = await getOpenHandler();
 
@@ -332,7 +332,7 @@ describe('main', () => {
       );
     });
 
-    it('returns the webview ID when the webview opens successfully', async () => {
+    it('returns the WebView ID when the WebView opens successfully', async () => {
       __mockOpenWebView.mockResolvedValue('new-webview-id');
       const open = await getOpenHandler();
 
@@ -356,7 +356,7 @@ describe('main', () => {
       };
     }
 
-    it('looks up the projectId from the given webview and opens the interlinearizer', async () => {
+    it('looks up the projectId from the given WebView and opens the Interlinearizer', async () => {
       __mockGetOpenWebViewDefinition.mockResolvedValue({
         id: 'some-webview',
         webViewType: 'someExtension.view',
@@ -375,7 +375,7 @@ describe('main', () => {
       );
     });
 
-    it('shows a project picker when the webview has no projectId', async () => {
+    it('shows a project picker when the WebView has no projectId', async () => {
       __mockGetOpenWebViewDefinition.mockResolvedValue({
         id: 'some-webview',
         webViewType: 'someExtension.view',
@@ -393,7 +393,7 @@ describe('main', () => {
       );
     });
 
-    it('shows a project picker when the webview definition is not found', async () => {
+    it('shows a project picker when the WebView definition is not found', async () => {
       __mockGetOpenWebViewDefinition.mockResolvedValue(undefined);
       __mockSelectProject.mockResolvedValue('picker-project');
       const openForWebView = await getOpenForWebViewHandler();
@@ -429,7 +429,7 @@ describe('main', () => {
       expect(__mockOpenWebView).not.toHaveBeenCalled();
     });
 
-    it('returns the webview ID when the interlinearizer opens successfully', async () => {
+    it('returns the WebView ID when the Interlinearizer opens successfully', async () => {
       __mockGetOpenWebViewDefinition.mockResolvedValue({
         id: 'src-webview',
         webViewType: 'someExtension.view',
@@ -444,7 +444,7 @@ describe('main', () => {
     });
   });
 
-  describe('webview lifecycle event subscriptions', () => {
+  describe('WebView lifecycle event subscriptions', () => {
     type WebViewEvent = (event: {
       webView: { webViewType: string; id: string; projectId?: string };
     }) => void;
@@ -458,7 +458,7 @@ describe('main', () => {
       expect(__mockOnDidCloseWebView).toHaveBeenCalledTimes(1);
     });
 
-    it('populates openWebViewsByProject when a matching webview opens', async () => {
+    it('populates openWebViewsByProject when a matching WebView opens', async () => {
       const context = createTestActivationContext();
       await activate(context);
       const onOpen: WebViewEvent = __mockOnDidOpenWebView.mock.calls[0][0];
@@ -476,7 +476,7 @@ describe('main', () => {
       );
     });
 
-    it('ignores onDidOpenWebView events for other webview types', async () => {
+    it('ignores onDidOpenWebView events for other WebView types', async () => {
       const context = createTestActivationContext();
       await activate(context);
       const onOpen: WebViewEvent = __mockOnDidOpenWebView.mock.calls[0][0];
@@ -512,7 +512,7 @@ describe('main', () => {
       );
     });
 
-    it('removes the entry from the map when the matching webview closes', async () => {
+    it('removes the entry from the map when the matching WebView closes', async () => {
       __mockOpenWebView.mockResolvedValue('wv-close');
       const context = createTestActivationContext();
       await activate(context);
@@ -536,7 +536,7 @@ describe('main', () => {
       );
     });
 
-    it('does not remove the entry when a different webview ID closes for the same project', async () => {
+    it('does not remove the entry when a different WebView ID closes for the same project', async () => {
       __mockOpenWebView.mockResolvedValue('wv-current');
       const context = createTestActivationContext();
       await activate(context);
@@ -559,7 +559,7 @@ describe('main', () => {
       );
     });
 
-    it('ignores onDidCloseWebView events for other webview types', async () => {
+    it('ignores onDidCloseWebView events for other WebView types', async () => {
       __mockOpenWebView.mockResolvedValue('wv-other-type');
       const context = createTestActivationContext();
       await activate(context);

src/interlinearizer.web-view.tsx

@@ -63,7 +63,7 @@ globalThis.webViewComponent = function InterlinearizerWebView({
         <ProjectBookFetcher projectId={projectId} scrRef={scrRef} />
       ) : (
         <p className="tw-text-sm tw-text-muted-foreground">
-          Open this webview from a Paratext project to load its source book.
+          Open this WebView from a Paratext project to load its source book.
         </p>
       )}
     </div>

src/main.ts

@@ -10,7 +10,7 @@ import interlinearizerReact from './interlinearizer.web-view?inline';
 import interlinearizerStyles from './interlinearizer.web-view.scss?inline';
 
 /**
- * WebView type identifier for the interlinearizer. Used when registering the provider and when
+ * WebView type identifier for the Interlinearizer. Used when registering the provider and when
  * opening the WebView from the platform.
  */
 const mainWebViewType = 'interlinearizer.mainWebView';
@@ -20,16 +20,16 @@ export interface InterlinearizerOpenOptions extends OpenWebViewOptions {
   projectId?: string;
 }
 
-/** WebView provider that provides the interlinearizer React WebView when Platform.Bible requests it. */
+/** WebView provider that provides the Interlinearizer React WebView when Platform.Bible requests it. */
 const mainWebViewProvider: IWebViewProvider = {
   /**
-   * Returns the interlinearizer WebView definition (React component + styles) for the given saved
+   * Returns the Interlinearizer WebView definition (React component + styles) for the given saved
    * definition. Rejects if the requested webViewType does not match this provider's type.
    *
    * @param savedWebView - Platform-provided definition (webViewType, etc.).
    * @param openWebViewOptions - Options passed by the caller; may include a projectId to link.
    * @returns WebView definition with title, content, and styles, or undefined.
-   * @throws {Error} When savedWebView.webViewType is not the interlinearizer type.
+   * @throws {Error} When savedWebView.webViewType is not the Interlinearizer type.
    */
   async getWebView(
     savedWebView: SavedWebViewDefinition,
@@ -85,7 +85,7 @@ async function openInterlinearizer(projectId?: string): Promise<string | undefin
 
 /**
  * Opens the Interlinearizer for the project associated with the given WebView. Called from the
- * webview context menu, which passes the tab's webView ID as the argument.
+ * WebView context menu, which passes the tab's WebView ID as the argument.
  */
 async function openInterlinearizerForWebView(webViewId?: string): Promise<string | undefined> {
   if (!webViewId) return openInterlinearizer();
@@ -94,7 +94,7 @@ async function openInterlinearizerForWebView(webViewId?: string): Promise<string
 }
 
 /**
- * Extension entry point. Registers the interlinearizer WebView provider and the open command.
+ * Extension entry point. Registers the Interlinearizer WebView provider and the open command.
  * Called by the platform when the extension is loaded.
  *
  * @param context - Activation context; used to register disposables so the platform can clean them

src/types/interlinearizer.d.ts

@@ -12,8 +12,8 @@ declare module 'papi-shared-types' {
     'interlinearizer.open': (projectId?: string) => Promise<string | undefined>;
     /**
      * Opens the Interlinearizer for the project associated with the given WebView ID. Called from
-     * webview context menus, which pass the tab's webView ID as the argument. Falls back to a
-     * project picker dialog if the webView has no project or no ID is given.
+     * WebView context menus, which pass the tab's WebView ID as the argument. Falls back to a
+     * project picker dialog if the WebView has no project or no ID is given.
      */
     'interlinearizer.openForWebView': (webViewId?: string) => Promise<string | undefined>;
   }