Refactor unit tests for extracted getAllowedTabs() function

[Copilot]

Description

Refactored unit tests to test the newly exported getAllowedTabs(state, self) pure function instead of the TermTypeSearch.addTabsAllowed(state) instance method.

Changes

• Import: getAllowedTabs function instead of TermTypeSearch class and d3-selection
• Test helper: getMockSelf() creates minimal mock with only required properties (types, app.vocabApi.termdbConfig, useCasesExcluded, setTermTypeGroup)
• State structure: usecase at top level to match getState() output (was tree.usecase in appState)
• Test pattern: Synchronous calls to getAllowedTabs(state, self) returning tabs array
• Removed: DOM manipulation, async/await, class instantiation, cleanup code

Impact

• 116 lines removed (18% reduction)
• Tests are now true unit tests: pure function with explicit inputs/outputs
• No DOM overhead, no async complexity, no shared state between tests
• All 18 test cases preserved with identical assertions

Checklist

• Tests: Added and/or passed unit and integration tests, or N/A
• Todos: Commented or documented, or N/A
• Notable Changes: updated release.txt, prefixed a commit message with "fix:" or "feat:", added to an internal tracking document, or N/A
• Rust: Checked to see whether Rust needs to be re-compiled because of this PR, or N/A

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[Copilot]

Pull request overview

This PR refactors TermTypeSearch to extract tab-selection logic into an exported pure function (getAllowedTabs(state, self)) and updates the unit tests to directly exercise that function, removing DOM- and class-instantiation dependencies.

Changes:

• Export useCasesExcluded and introduce getAllowedTabs() to generate the allowed tab definitions based on state + self.
• Update TermTypeSearch.init() to use getAllowedTabs() and then load handlers for the resulting tabs.
• Refactor TermTypeSearch.unit.spec.ts to unit-test getAllowedTabs() with minimal mocks and synchronous assertions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
client/termdb/test/TermTypeSearch.unit.spec.ts	Rewrites tests to target getAllowedTabs() directly with minimal mocked self and updated state shape.
client/termdb/TermTypeSearch.ts	Exports useCasesExcluded, adds Tab type + getAllowedTabs(), and updates initialization to build tabs via the extracted function.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.