Skip to content

Comments

[WIP] Add unit tests for chain method in BasicCrawler#3436

Closed
Copilot wants to merge 2 commits intochore/more-context-pipelinefrom
copilot/sub-pr-3388
Closed

[WIP] Add unit tests for chain method in BasicCrawler#3436
Copilot wants to merge 2 commits intochore/more-context-pipelinefrom
copilot/sub-pr-3388

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

  • Add unit tests for the chain method in ContextPipeline
    • Test that middlewares from both pipelines run in order and context accumulates correctly
    • Test that cleanup routines from both pipelines are called in the correct reverse order
  • All existing and new tests pass

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: barjin <61918049+barjin@users.noreply.github.com>
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