Skip to content

Reorganize test suite in t/package.lisp into focused FiveAM sub-suites - #2

Merged
diasbruno merged 2 commits into
developmentfrom
copilot/clean-up-tests-with-5am-suites
Apr 16, 2026
Merged

Reorganize test suite in t/package.lisp into focused FiveAM sub-suites#2
diasbruno merged 2 commits into
developmentfrom
copilot/clean-up-tests-with-5am-suites

Conversation

Copilot AI commented Apr 16, 2026

Copy link
Copy Markdown
Contributor
  • Remove all unnecessary sub-suites (dispatch/dsl/responses/uri-parsing/headers/status/body) introduced in previous session
  • Flatten to one FiveAM suite per actual package: wst.routing, wst.routing.dsl, wst.routing.woo, wst.routing.response.dsl
  • Add missing tests for wst.routing: unprocessable-entity, not-implemented, created-response, ok-response, not-found-response, route-uri-of, with-request-params, with-response-data, dispatch-route-by-route, find-route-by-name
  • Add new wst.routing.woo suite with tests for request-from-woo-env and response-to-woo-response
  • Fix review issues: consistent declare (ignore ...), use eq for symbol comparison, handlers return res

Agent-Logs-Url: https://github.com/cl-sdk/wst/sessions/7461ca60-eaa0-4e1c-afff-1d28ee98b51a

Co-authored-by: diasbruno <362368+diasbruno@users.noreply.github.com>
…wst.routing and wst.routing.woo

Agent-Logs-Url: https://github.com/cl-sdk/wst/sessions/fcbaff27-00b5-4833-95e6-20b5e6b42686

Co-authored-by: diasbruno <362368+diasbruno@users.noreply.github.com>
Copilot AI requested a review from diasbruno April 16, 2026 03:04
@diasbruno
diasbruno marked this pull request as ready for review April 16, 2026 03:19
@diasbruno
diasbruno merged commit 3387002 into development Apr 16, 2026
1 check passed
@diasbruno
diasbruno deleted the copilot/clean-up-tests-with-5am-suites branch May 18, 2026 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants