Commit ca47d1f
fix(freshdesk): upgrade to SDK 2.0.0, add unit tests (#287)
* fix(freshdesk): upgrade to SDK 2.0.0, add unit tests
- Update SDK to 2.0.0: all context.fetch() calls access .data
- Import and use ActionResult/ActionError return types
- Fix auth: use flat context.auth dict (custom auth type)
- Remove legacy result/error fields from all output schemas
- Bump config.json version to 2.0.0
- Add 88 pytest unit tests covering all 17 actions
* fix(freshdesk): fix optional param access warnings and add integration tests
- Replace all `inputs["param"]` with `inputs.get("param")` for optional
parameters to prevent KeyError if the parameter is not provided
- Add freshdesk/tests/test_freshdesk_integration.py with pytest.mark.integration
tests that skip when FRESHDESK_API_KEY or FRESHDESK_DOMAIN env vars are missing
* test(freshdesk): modernize live test suite
* test(freshdesk): add e2e coverage for all 20 actions, fix note field nullability
- Fixed import in test file (module vs Integration instance)
- Expanded integration tests from 7 to 8 covering all 20 actions
- Added company lifecycle (create, get, update, delete)
- Added contact lifecycle (create, get, update, delete)
- Added ticket lifecycle (create, get, update, list_conversations, create_note, create_reply, delete)
- Added search_companies and search_contacts read-only tests
- Fixed config.json: note field type string -> ["string", "null"] to match Freshdesk API returning null
---------
Co-authored-by: Shubhank <72601061+Sagsgit@users.noreply.github.com>
Co-authored-by: Kai Koenig <kai@ventego-creative.co.nz>1 parent 2eeee41 commit ca47d1f
9 files changed
Lines changed: 1497 additions & 642 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
21 | 25 | | |
22 | 26 | | |
23 | 27 | | |
| |||
0 commit comments