Alan-Jowett / CoPilot-For-Consensus / 22115451450

78%

push

github

Add thread-centric Discussions page with enhanced filtering and sorting (#1172)

* Initial plan

* Add Discussions page with enhanced thread filtering (backend + frontend)

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add comprehensive backend unit and API tests for threads filtering

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add comprehensive frontend tests for Discussions page and API

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix existing tests to account for new get_threads signature and behavior

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix frontend API test imports - all API tests passing

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback - fix type safety and simplify logic

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix CSS classes to use existing styles and add integration tests

- Replace undefined CSS classes with existing ones from styles.css
- Use .filters, .filter-section, .filter-row, .reports-table, .pagination
- Add archive_source field to Thread interface (TypeScript)
- Add 2 Azure Cosmos integration tests for threads filtering and sorting

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix TypeScript errors in DiscussionsList pagination

Add nullish coalescing for q.skip and q.limit which are possibly
undefined per the ThreadsQuery interface.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alanjo@microsoft.com>

* Fix DiscussionsList test failures

- Use getAllByText for source names that appear in both dropdown and table
- Use getByLabelText instead of getByPlaceholderText for advanced filters
- Provide 20 mock threads in pagination test so Next button is enabled

Co-authored-by: Copilot <223556219+Copilot@users.nor... (continued)

51 of 53 new or added lines in 1 file covered. (96.23%)

8836 of 11342 relevant lines covered (77.91%)

0.88 hits per line