@youversion/platform-react-hooks 1.18.1 → 1.20.0

package/src/context/VerseSelectionProvider.test.tsx

@@ -1,362 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { renderHook, act } from '@testing-library/react';
-import type { ReactNode } from 'react';
-import { VerseSelectionProvider } from './VerseSelectionProvider';
-import { useVerseSelection } from '../useVerseSelection';
-
-// Wrapper for renderHook
-const wrapper = ({ children }: { children: ReactNode }) => (
-  <VerseSelectionProvider>{children}</VerseSelectionProvider>
-);
-
-describe('VerseSelectionProvider', () => {
-  describe('initial state', () => {
-    it('should provide an empty Set instance', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      expect(result.current.selectedVerseUsfms).toBeInstanceOf(Set);
-      expect(result.current.selectedVerseUsfms.size).toBe(0);
-    });
-
-    it('should have selectedCount of 0', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      expect(result.current.selectedCount).toBe(0);
-    });
-
-    it('should return false for isSelected on any verse', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      expect(result.current.isSelected('MAT.1.1')).toBe(false);
-      expect(result.current.isSelected('GEN.1.1')).toBe(false);
-    });
-  });
-
-  describe('toggleVerse', () => {
-    it('should add verse to selection when not present', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      expect(result.current.selectedVerseUsfms.has('MAT.1.1')).toBe(true);
-      expect(result.current.selectedCount).toBe(1);
-    });
-
-    it('should remove verse from selection when already present', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      expect(result.current.selectedCount).toBe(1);
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      expect(result.current.selectedVerseUsfms.has('MAT.1.1')).toBe(false);
-      expect(result.current.selectedCount).toBe(0);
-    });
-
-    it('should support multiple verses selected simultaneously', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-        result.current.toggleVerse('GEN.1.1');
-        result.current.toggleVerse('JHN.3.16');
-      });
-
-      expect(result.current.selectedCount).toBe(3);
-      expect(result.current.selectedVerseUsfms.has('MAT.1.1')).toBe(true);
-      expect(result.current.selectedVerseUsfms.has('GEN.1.1')).toBe(true);
-      expect(result.current.selectedVerseUsfms.has('JHN.3.16')).toBe(true);
-    });
-
-    it('should create new Set reference on each update (immutability)', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const set1 = result.current.selectedVerseUsfms;
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const set2 = result.current.selectedVerseUsfms;
-
-      expect(set1).not.toBe(set2);
-      expect(set1.size).toBe(0);
-      expect(set2.size).toBe(1);
-    });
-
-    it('should create new Set reference when removing a verse', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const setWithVerse = result.current.selectedVerseUsfms;
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const setWithoutVerse = result.current.selectedVerseUsfms;
-
-      expect(setWithVerse).not.toBe(setWithoutVerse);
-      expect(setWithVerse.size).toBe(1);
-      expect(setWithoutVerse.size).toBe(0);
-    });
-
-    it('should not add duplicate verses', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      // Should toggle off, not add twice
-      expect(result.current.selectedCount).toBe(0);
-    });
-  });
-
-  describe('isSelected', () => {
-    it('should return true for selected verses', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      expect(result.current.isSelected('MAT.1.1')).toBe(true);
-    });
-
-    it('should return false for unselected verses', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      expect(result.current.isSelected('GEN.1.1')).toBe(false);
-    });
-
-    it('should work correctly after toggle operations', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      // Initially not selected
-      expect(result.current.isSelected('MAT.1.1')).toBe(false);
-
-      // Add verse - now selected
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-      expect(result.current.isSelected('MAT.1.1')).toBe(true);
-
-      // Remove verse - not selected again
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-      expect(result.current.isSelected('MAT.1.1')).toBe(false);
-    });
-  });
-
-  describe('clearSelection', () => {
-    it('should clear all selected verses', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-        result.current.toggleVerse('GEN.1.1');
-      });
-
-      expect(result.current.selectedCount).toBe(2);
-
-      act(() => {
-        result.current.clearSelection();
-      });
-
-      expect(result.current.selectedCount).toBe(0);
-      expect(result.current.selectedVerseUsfms.size).toBe(0);
-    });
-
-    it('should create new Set reference', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const setBeforeClear = result.current.selectedVerseUsfms;
-
-      act(() => {
-        result.current.clearSelection();
-      });
-
-      const setAfterClear = result.current.selectedVerseUsfms;
-
-      expect(setBeforeClear).not.toBe(setAfterClear);
-      expect(setBeforeClear.size).toBe(1);
-      expect(setAfterClear.size).toBe(0);
-    });
-
-    it('should work when already empty', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const setBeforeClear = result.current.selectedVerseUsfms;
-
-      act(() => {
-        result.current.clearSelection();
-      });
-
-      const setAfterClear = result.current.selectedVerseUsfms;
-
-      expect(setBeforeClear).not.toBe(setAfterClear);
-      expect(setAfterClear.size).toBe(0);
-    });
-  });
-
-  describe('selectedCount', () => {
-    it('should start at 0', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      expect(result.current.selectedCount).toBe(0);
-    });
-
-    it('should increment when verse added', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-      expect(result.current.selectedCount).toBe(1);
-
-      act(() => {
-        result.current.toggleVerse('GEN.1.1');
-      });
-      expect(result.current.selectedCount).toBe(2);
-
-      act(() => {
-        result.current.toggleVerse('JHN.3.16');
-      });
-      expect(result.current.selectedCount).toBe(3);
-    });
-
-    it('should decrement when verse removed', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-        result.current.toggleVerse('GEN.1.1');
-      });
-      expect(result.current.selectedCount).toBe(2);
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-      expect(result.current.selectedCount).toBe(1);
-    });
-
-    it('should return to 0 after clear', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-        result.current.toggleVerse('GEN.1.1');
-        result.current.toggleVerse('JHN.3.16');
-      });
-      expect(result.current.selectedCount).toBe(3);
-
-      act(() => {
-        result.current.clearSelection();
-      });
-      expect(result.current.selectedCount).toBe(0);
-    });
-  });
-
-  describe('callback stability', () => {
-    it('should have stable toggleVerse reference across renders', () => {
-      const { result, rerender } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const toggleRef1 = result.current.toggleVerse;
-
-      rerender();
-
-      const toggleRef2 = result.current.toggleVerse;
-
-      expect(toggleRef1).toBe(toggleRef2);
-    });
-
-    it('should have stable clearSelection reference across renders', () => {
-      const { result, rerender } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const clearRef1 = result.current.clearSelection;
-
-      rerender();
-
-      const clearRef2 = result.current.clearSelection;
-
-      expect(clearRef1).toBe(clearRef2);
-    });
-
-    it('should have stable toggleVerse reference after state changes', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const toggleRef1 = result.current.toggleVerse;
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const toggleRef2 = result.current.toggleVerse;
-
-      expect(toggleRef1).toBe(toggleRef2);
-    });
-
-    it('should have stable clearSelection reference after state changes', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const clearRef1 = result.current.clearSelection;
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const clearRef2 = result.current.clearSelection;
-
-      expect(clearRef1).toBe(clearRef2);
-    });
-
-    it('should update isSelected reference when selectedVerseUsfms changes', () => {
-      const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const isSelectedRef1 = result.current.isSelected;
-
-      act(() => {
-        result.current.toggleVerse('MAT.1.1');
-      });
-
-      const isSelectedRef2 = result.current.isSelected;
-
-      // isSelected has selectedVerseUsfms in its dependency array, so it should change
-      expect(isSelectedRef1).not.toBe(isSelectedRef2);
-    });
-
-    it('should have stable isSelected reference when no state changes', () => {
-      const { result, rerender } = renderHook(() => useVerseSelection(), { wrapper });
-
-      const isSelectedRef1 = result.current.isSelected;
-
-      rerender();
-
-      const isSelectedRef2 = result.current.isSelected;
-
-      expect(isSelectedRef1).toBe(isSelectedRef2);
-    });
-  });
-});

package/src/useChapterNavigation.test.tsx

@@ -1,160 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { renderHook, act } from '@testing-library/react';
-import React from 'react';
-import { useChapterNavigation } from './useChapterNavigation';
-import { useReaderContext } from './context/ReaderContext';
-import { ReaderProvider } from './context/ReaderProvider';
-import { createMockBook, createMockVersion } from './__tests__/mocks/bibles';
-import type { BibleBook, BibleChapter } from '@youversion/platform-core';
-
-function makeChapters(bookId: string, count: number): BibleChapter[] {
-  return Array.from({ length: count }, (_, i) => ({
-    id: (i + 1).toString(),
-    passage_id: `${bookId}.${i + 1}`,
-    title: (i + 1).toString(),
-  }));
-}
-
-const genChapters = makeChapters('GEN', 50);
-const exoChapters = makeChapters('EXO', 40);
-const revChapters = makeChapters('REV', 22);
-
-const mockBooks: BibleBook[] = [
-  createMockBook({
-    id: 'GEN',
-    title: 'Genesis',
-    chapters: genChapters,
-    intro: { id: 'INTRO', passage_id: 'GEN.INTRO', title: 'Intro' },
-  }),
-  createMockBook({
-    id: 'EXO',
-    title: 'Exodus',
-    canon: 'old_testament',
-    chapters: exoChapters,
-  }),
-  createMockBook({
-    id: 'REV',
-    title: 'Revelation',
-    canon: 'new_testament',
-    chapters: revChapters,
-  }),
-];
-
-const mockUseBooks = vi.fn();
-vi.mock('./useBooks', () => ({
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-return
-  useBooks: (...args: unknown[]) => mockUseBooks(...args),
-}));
-
-function useNavWithContext() {
-  const nav = useChapterNavigation();
-  const ctx = useReaderContext();
-  return { nav, ctx };
-}
-
-function wrapper(book: BibleBook, chapter: BibleChapter) {
-  return ({ children }: { children: React.ReactNode }) => (
-    <ReaderProvider
-      currentVersion={createMockVersion()}
-      currentBook={book}
-      currentChapter={chapter}
-      currentVerse={null}
-    >
-      {children}
-    </ReaderProvider>
-  );
-}
-
-const genBook = mockBooks[0]!;
-const exoBook = mockBooks[1]!;
-const revBook = mockBooks[2]!;
-
-describe('useChapterNavigation', () => {
-  beforeEach(() => {
-    mockUseBooks.mockReturnValue({
-      books: { data: mockBooks },
-      loading: false,
-      error: null,
-      refetch: vi.fn(),
-    });
-  });
-
-  it('navigateToNext within same book updates chapter, keeps book', () => {
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(genBook, genChapters[0]!),
-    });
-
-    act(() => result.current.nav.navigateToNext());
-
-    expect(result.current.ctx.currentChapter.id).toBe('2');
-    expect(result.current.ctx.currentBook.id).toBe('GEN');
-  });
-
-  it('navigateToNext cross-book updates both book and chapter', () => {
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(genBook, genChapters.at(-1)!),
-    });
-
-    act(() => result.current.nav.navigateToNext());
-
-    expect(result.current.ctx.currentBook.id).toBe('EXO');
-    expect(result.current.ctx.currentChapter.id).toBe('1');
-  });
-
-  it('navigateToPrevious to intro sets chapter to INTRO, keeps book', () => {
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(genBook, genChapters[0]!),
-    });
-
-    act(() => result.current.nav.navigateToPrevious());
-
-    expect(result.current.ctx.currentChapter.id).toBe('INTRO');
-    expect(result.current.ctx.currentBook.id).toBe('GEN');
-  });
-
-  it('navigateToPrevious cross-book updates both book and chapter', () => {
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(exoBook, exoChapters[0]!),
-    });
-
-    act(() => result.current.nav.navigateToPrevious());
-
-    expect(result.current.ctx.currentBook.id).toBe('GEN');
-    expect(result.current.ctx.currentChapter.id).toBe('50');
-  });
-
-  it('canNavigatePrevious is false at Bible start', () => {
-    const introChapter: BibleChapter = { id: 'INTRO', passage_id: 'GEN.INTRO', title: 'Intro' };
-
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(genBook, introChapter),
-    });
-
-    expect(result.current.nav.canNavigatePrevious).toBe(false);
-  });
-
-  it('canNavigateNext is false at Bible end', () => {
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(revBook, revChapters[21]!),
-    });
-
-    expect(result.current.nav.canNavigateNext).toBe(false);
-  });
-
-  it('both disabled while loading', () => {
-    mockUseBooks.mockReturnValue({
-      books: null,
-      loading: true,
-      error: null,
-      refetch: vi.fn(),
-    });
-
-    const { result } = renderHook(useNavWithContext, {
-      wrapper: wrapper(genBook, genChapters[0]!),
-    });
-
-    expect(result.current.nav.canNavigateNext).toBe(false);
-    expect(result.current.nav.canNavigatePrevious).toBe(false);
-    expect(result.current.nav.isLoading).toBe(true);
-  });
-});

package/src/useVerseSelection.test.tsx

@@ -1,33 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { renderHook } from '@testing-library/react';
-import type { ReactNode } from 'react';
-import { useVerseSelection } from './useVerseSelection';
-import { VerseSelectionProvider } from './context/VerseSelectionProvider';
-
-// Wrapper for renderHook
-const wrapper = ({ children }: { children: ReactNode }) => (
-  <VerseSelectionProvider>{children}</VerseSelectionProvider>
-);
-
-describe('useVerseSelection', () => {
-  it('should throw error when used outside provider', () => {
-    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(vi.fn());
-
-    expect(() => {
-      renderHook(() => useVerseSelection());
-    }).toThrow('useVerseSelection must be used within a VerseSelectionProvider');
-
-    consoleSpy.mockRestore();
-  });
-
-  it('should return expected shape with correct initial values', () => {
-    const { result } = renderHook(() => useVerseSelection(), { wrapper });
-
-    expect(result.current.selectedVerseUsfms).toBeInstanceOf(Set);
-    expect(result.current.selectedVerseUsfms.size).toBe(0);
-    expect(result.current.selectedCount).toBe(0);
-    expect(typeof result.current.toggleVerse).toBe('function');
-    expect(typeof result.current.isSelected).toBe('function');
-    expect(typeof result.current.clearSelection).toBe('function');
-  });
-});

package/vitest.setup.ts

@@ -1 +0,0 @@
-import '@testing-library/jest-dom/vitest';