wylab/nanobot
1
0
Fork 0
Code Issues 5 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

Add matrix optional dependencies and fix tests #25

Closed
code-server wants to merge 0 commits from fix/matrix-test-optional-deps into main
pull from: fix/matrix-test-optional-deps
merge into: wylab:main
Conversation 0 Commits - Files Changed -
+9 -5
code-server commented 2026-03-05 18:29:06 +01:00
Collaborator
Copy Link
Copy Source

Fixes matrix channel test collection error and 5 failing tests.

Changes

  • Add [matrix] optional dependencies section to pyproject.toml (matrix-nio, mistune, nh3)
    • Fixes ImportError during test collection
    • Matches the error message guidance: pip install nanobot-ai[matrix]
  • Fix test mock function signature to accept positional args (removed *,)
  • Fix test assertions to handle optional metadata keys using .get("attachments", [])

Result

All 45 matrix channel tests now pass (was: 5 failed, 40 passed).

🤖 Generated with Claude Code

Fixes matrix channel test collection error and 5 failing tests. ## Changes - **Add [matrix] optional dependencies section** to pyproject.toml (matrix-nio, mistune, nh3) - Fixes ImportError during test collection - Matches the error message guidance: `pip install nanobot-ai[matrix]` - **Fix test mock function signature** to accept positional args (removed `*,`) - **Fix test assertions** to handle optional metadata keys using `.get("attachments", [])` ## Result All 45 matrix channel tests now pass (was: 5 failed, 40 passed). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
code-server added 1 commit 2026-03-05 18:29:06 +01:00
add matrix optional dependencies and fix tests
Build Nanobot OAuth / build (pull_request) Failing after 53s
Details
Build Nanobot OAuth / cleanup (pull_request) Has been skipped
Details
34584c3a2e 
- Add [matrix] optional dependencies section to pyproject.toml
  (matrix-nio, mistune, nh3) to match error message guidance
- Fix test mock function signature to accept positional args
  instead of keyword-only args (removed *,)
- Fix test assertions to handle optional metadata keys
  using .get("attachments", []) instead of ["attachments"]

All 45 matrix channel tests now pass.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
code-server closed this pull request 2026-03-05 18:32:18 +01:00
code-server deleted branch fix/matrix-test-optional-deps 2026-03-05 18:32:18 +01:00
Some required checks failed
Build Nanobot OAuth / build (pull_request) Failing after 53s
Required
Details
Build Nanobot OAuth / cleanup (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
code-server (Code Server) nanobot (Nanobot Agent) wylab
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wylab/nanobot#25
Delete Branch "fix/matrix-test-optional-deps"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?