test: add flowsheet hooks tests#154
Merged
jakebromberg merged 2 commits intomainfrom Apr 21, 2026
Merged
Conversation
jakebromberg
force-pushed
the
test/09-flowsheet-hooks
branch
from
89099b8 to
ef59eac
Compare
Deploying wxyc-dj with
|
| Latest commit: |
601b06e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3bdf12ef.dj-site.pages.dev |
| Branch Preview URL: | https://test-09-flowsheet-hooks.dj-site.pages.dev |
jakebromberg
force-pushed
the
test/09-flowsheet-hooks
branch
from
ef59eac to
dccdebb
Compare
1 task
11 tasks
This was referenced Apr 3, 2026
- Add tests for useShowControl hook - Add tests for useFlowsheetSearch hook - Add tests for useFlowsheet hook - Add tests for useQueue hook - Add tests for useFlowsheetSubmit hook
jakebromberg
force-pushed
the
test/09-flowsheet-hooks
branch
from
dccdebb to
403704c
Compare
Add useGetInfiniteEntriesInfiniteQuery mock to the API mock factory, type mockUseRegistry to accept null for info, type mockUseCatalogFlowsheetSearch to accept AlbumEntry[], provide preventDefault in FormEvent mocks, and remove nonexistent play_freq field from test data.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #254
Summary
Add ~87 tests covering the flowsheet hooks — the primary interface between flowsheet components and the Redux store/API layer:
useShowControl— Join/leave show lifecycle, show state managementuseFlowsheetSearch— Search query dispatching and result handlinguseFlowsheet— Core flowsheet data access, entry ordering, paginationuseQueue— Queue management (add, remove, reorder)useFlowsheetSubmit— Submission workflow from queue/catalog/bin to the live flowsheetAll tests in a single
flowsheetHooks.test.tsxfile since the hooks share interconnected store state.Test plan
npm test src/hooks/flowsheetHooks.test.tsxPart 9 of 26