Skip to content

Added more unit tests#167

Open
jlouvel wants to merge 1 commit intomainfrom
chore/increase-test-coverage3
Open

Added more unit tests#167
jlouvel wants to merge 1 commit intomainfrom
chore/increase-test-coverage3

Conversation

@jlouvel
Copy link
Contributor

@jlouvel jlouvel commented Mar 21, 2026

Related Issue

Closes #139


What does this PR do?

  • add focused tests for skill contents and download endpoints
  • make endpoint assertions environment-agnostic by accepting 404 when fixture location is absent
  • keep strict payload and ZIP validation when 200 is returned
  • prevent false negatives across OS-specific local path setups

Checklist

  • CI is green (build, tests, schema validation, security scans)
  • Rebased on latest main
  • Small and focused — one concern per PR
  • Commit messages follow Conventional Commits

Agent Context (optional)

GTP 5.3-Codex

@jlouvel jlouvel requested a review from eskenazit March 21, 2026 22:30
@jlouvel jlouvel self-assigned this Mar 21, 2026
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.

Increase Framework test coverage

1 participant