test: add tests for modified note indicators

- NoteList: 4 tests for modified indicator dot visibility
- TabBar: 3 tests for modified indicator on tabs
- StatusBar: 3 tests for pending count display
- useEditorSave: 2 tests for onAfterSave callback

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

src/components/NoteList.test.tsx

@@ -716,6 +716,47 @@ describe('filterEntries — trash', () => {
   })
 })
 
+describe('NoteList — modified indicators', () => {
+  it('shows modified indicator dot for entries in modifiedFiles', () => {
+    const modifiedFiles = [
+      { path: mockEntries[0].path, relativePath: 'project/26q1-laputa-app.md', status: 'modified' as const },
+    ]
+    render(
+      <NoteList entries={mockEntries} selection={allSelection} selectedNote={null} onSelectNote={noopSelect} onReplaceActiveTab={noopReplace} allContent={{}} modifiedFiles={modifiedFiles} onCreateNote={vi.fn()} />
+    )
+    const indicators = screen.getAllByTestId('modified-indicator')
+    expect(indicators).toHaveLength(1)
+    // The indicator's parent div contains the title text
+    const noteRow = indicators[0].closest('[data-testid="modified-indicator"]')!.parentElement!.parentElement!
+    expect(noteRow.textContent).toContain('Build Laputa App')
+  })
+
+  it('does not show modified indicator when modifiedFiles is empty', () => {
+    render(
+      <NoteList entries={mockEntries} selection={allSelection} selectedNote={null} onSelectNote={noopSelect} onReplaceActiveTab={noopReplace} allContent={{}} modifiedFiles={[]} onCreateNote={vi.fn()} />
+    )
+    expect(screen.queryByTestId('modified-indicator')).not.toBeInTheDocument()
+  })
+
+  it('shows multiple modified indicators for multiple modified files', () => {
+    const modifiedFiles = [
+      { path: mockEntries[0].path, relativePath: 'project/26q1-laputa-app.md', status: 'modified' as const },
+      { path: mockEntries[1].path, relativePath: 'note/facebook-ads-strategy.md', status: 'modified' as const },
+    ]
+    render(
+      <NoteList entries={mockEntries} selection={allSelection} selectedNote={null} onSelectNote={noopSelect} onReplaceActiveTab={noopReplace} allContent={{}} modifiedFiles={modifiedFiles} onCreateNote={vi.fn()} />
+    )
+    expect(screen.getAllByTestId('modified-indicator')).toHaveLength(2)
+  })
+
+  it('does not show modified indicator when modifiedFiles prop is undefined', () => {
+    render(
+      <NoteList entries={mockEntries} selection={allSelection} selectedNote={null} onSelectNote={noopSelect} onReplaceActiveTab={noopReplace} allContent={{}} onCreateNote={vi.fn()} />
+    )
+    expect(screen.queryByTestId('modified-indicator')).not.toBeInTheDocument()
+  })
+})
+
 describe('NoteList — trash view', () => {
   const trashSelection: SidebarSelection = { kind: 'filter', filter: 'trash' }
 

src/components/StatusBar.test.tsx

@@ -128,6 +128,22 @@ describe('StatusBar', () => {
     expect(screen.getByText('Connect GitHub repo')).toBeInTheDocument()
   })
 
+  it('shows modified count when modifiedCount is > 0', () => {
+    render(<StatusBar noteCount={100} modifiedCount={3} vaultPath="/Users/luca/Laputa" vaults={vaults} onSwitchVault={vi.fn()} />)
+    expect(screen.getByTestId('status-modified-count')).toBeInTheDocument()
+    expect(screen.getByText('3 pending')).toBeInTheDocument()
+  })
+
+  it('does not show modified count when modifiedCount is 0', () => {
+    render(<StatusBar noteCount={100} modifiedCount={0} vaultPath="/Users/luca/Laputa" vaults={vaults} onSwitchVault={vi.fn()} />)
+    expect(screen.queryByTestId('status-modified-count')).not.toBeInTheDocument()
+  })
+
+  it('does not show modified count when modifiedCount is not provided', () => {
+    render(<StatusBar noteCount={100} vaultPath="/Users/luca/Laputa" vaults={vaults} onSwitchVault={vi.fn()} />)
+    expect(screen.queryByTestId('status-modified-count')).not.toBeInTheDocument()
+  })
+
   it('closes menu after clicking "Open local folder"', () => {
     render(
       <StatusBar noteCount={100} vaultPath="/Users/luca/Laputa" vaults={vaults} onSwitchVault={vi.fn()} onOpenLocalFolder={vi.fn()} />

src/components/TabBar.test.tsx

@@ -110,6 +110,28 @@ describe('TabBar', () => {
     expect(onReorderTabs).not.toHaveBeenCalled()
   })
 
+  it('shows modified indicator dot on modified tabs', () => {
+    const tabs = makeTabs(['Alpha', 'Beta'])
+    const isModified = (path: string) => path === tabs[0].entry.path
+    render(<TabBar tabs={tabs} activeTabPath={tabs[0].entry.path} isModified={isModified} {...defaultProps} />)
+    const indicators = screen.getAllByTestId('tab-modified-indicator')
+    expect(indicators).toHaveLength(1)
+  })
+
+  it('does not show modified indicator when no tabs are modified', () => {
+    const tabs = makeTabs(['Alpha', 'Beta'])
+    const isModified = () => false
+    render(<TabBar tabs={tabs} activeTabPath={tabs[0].entry.path} isModified={isModified} {...defaultProps} />)
+    expect(screen.queryByTestId('tab-modified-indicator')).not.toBeInTheDocument()
+  })
+
+  it('shows modified indicator on multiple tabs', () => {
+    const tabs = makeTabs(['Alpha', 'Beta', 'Gamma'])
+    const isModified = () => true
+    render(<TabBar tabs={tabs} activeTabPath={tabs[0].entry.path} isModified={isModified} {...defaultProps} />)
+    expect(screen.getAllByTestId('tab-modified-indicator')).toHaveLength(3)
+  })
+
   it('switches tab on click', () => {
     const onSwitchTab = vi.fn()
     const tabs = makeTabs(['Alpha', 'Beta'])