You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
feat: make NylasClient and its methods open for mocking (#258)
# Make NylasClient and its methods open for mocking
This PR makes the NylasClient class and its methods open to enable
mocking in tests, addressing issue #257.
## Changes
- Added `open` modifier to NylasClient class
- Made all resource accessor methods (messages, calendars, etc.) open
- Made HTTP execution methods open for comprehensive mocking support
## Testing
These changes are purely related to class and method modifiers to enable
mocking. The changes do not affect runtime behavior and maintain all
existing functionality. The modifications only impact compile-time
characteristics to allow for better testing capabilities.
## Notes
- This change is backward compatible
- No runtime behavior changes
- Enables better testing capabilities for SDK users
Fixes#257
Link to Devin run:
https://app.devin.ai/sessions/f6b8eb21945e4b84a4b26f39136760cd
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Aaron de Mello <aaron.d@nylas.com>
Co-authored-by: Aaron de Mello <314152+AaronDDM@users.noreply.github.com>
0 commit comments