Skip to content

test: cover admin plugins list#25

Merged
MitulShah1 merged 1 commit into
ferro-labs:developmentfrom
tine1117:test-admin-plugins-list
Jun 2, 2026
Merged

test: cover admin plugins list#25
MitulShah1 merged 1 commit into
ferro-labs:developmentfrom
tine1117:test-admin-plugins-list

Conversation

@tine1117

@tine1117 tine1117 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add coverage for client.admin.plugins.list()
  • verify both gateway response shapes: a bare JSON array and a { "data": [...] } wrapper

Closes #11.

Tests

  • python -m pytest tests/test_sdk.py -q -k plugins
  • python -m ruff format --check tests/test_sdk.py
  • python -m ruff check tests/test_sdk.py
  • python -m pytest tests/test_sdk.py -q

@MitulShah1

Copy link
Copy Markdown
Contributor

Hey @tine1117 Thanks for contribution. if you like please star repo much appreciate.

@MitulShah1
MitulShah1 merged commit 3965b29 into ferro-labs:development Jun 2, 2026
5 checks passed
MitulShah1 pushed a commit that referenced this pull request Jun 3, 2026
MitulShah1 added a commit that referenced this pull request Jun 12, 2026
* Development (#4) (#6)

* feat: ship langchain adapter
* fix: support langchain integration on python 3.9

* test: cover admin plugins list (#25)

* test: cover admin dashboard (#23)

* fix: raise FerroStreamError on malformed streaming chunks

Both the sync and async _stream() paths swallowed JSONDecodeError with a
bare `continue`, silently dropping malformed SSE chunks and losing data
with no error or log. They now raise FerroStreamError instead — the first
code path that actually raises this previously-unused exception.

Also adds the {"plugins": [...]} wrapper coverage for admin.plugins.list()
and streaming malformed-chunk regression tests for both clients.

Closes #15
Closes #16

* ci: add release-notes config to credit contributors on tagged releases

Add .github/release.yml so GitHub's auto-generated release notes (the
publish job already runs softprops/action-gh-release with
generate_release_notes: true) group merged PRs by label and credit every
contributor by @handle, plus a "New Contributors" section. Contributor
credit lives in the release notes only, not in CHANGELOG.md.

---------

Co-authored-by: Tine <119507611+tine1117@users.noreply.github.com>
Co-authored-by: AmitabhainArunachala <79852583+AmitabhainArunachala@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