Skip to content

fix: update test to use new SecureFabricClient API#29

Merged
sasa-ken merged 1 commit into
mainfrom
claude/setup-packaging-pypi-deb-01EnJCqHAX975gfku3jqTrFp
Nov 18, 2025
Merged

fix: update test to use new SecureFabricClient API#29
sasa-ken merged 1 commit into
mainfrom
claude/setup-packaging-pypi-deb-01EnJCqHAX975gfku3jqTrFp

Conversation

@sasa-ken
Copy link
Copy Markdown
Contributor

Replace deprecated 'bearer' parameter with 'bearer_token' and update TLS configuration to use individual parameters (ca_cert, client_cert, client_key) instead of tls dict.

Description

Fixes #

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update
  • Protocol specification change
  • Refactoring (no functional changes)
  • CI/tooling improvement

Changes Made

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • All existing tests pass

Test results:

<!-- Paste test output here -->

Documentation

  • Documentation updated (if needed)
  • Examples updated (if API changed)
  • CHANGELOG.md updated (for user-facing changes)
  • API documentation generated/updated

Protocol Changes

  • Protocol version bumped (if breaking)
  • Backward compatibility analyzed
  • All SDKs updated to match
  • Migration guide provided (if breaking)

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Comments added for complex logic
  • No new warnings introduced
  • Tests added/updated and passing
  • Documentation updated (if needed)
  • No server/node code added (SDK repo only!)
  • SPDX headers present in new files
  • Commit messages follow Conventional Commits
  • Ready for review

SDK-Specific Checks

Rust SDK

  • cargo fmt --check passes
  • cargo clippy passes (no warnings)
  • cargo test passes
  • Cargo.toml version updated (if needed)

JavaScript/TypeScript SDK

  • npm run lint passes
  • npm run build succeeds
  • npm test passes
  • Type definitions updated

Python SDK

  • black --check . passes
  • mypy type checking passes
  • pytest passes
  • Version updated in pyproject.toml (if needed)

Breaking Changes

Additional Notes

Screenshots/Logs

Replace deprecated 'bearer' parameter with 'bearer_token' and update
TLS configuration to use individual parameters (ca_cert, client_cert,
client_key) instead of tls dict.
@sasa-ken sasa-ken merged commit 07fb021 into main Nov 18, 2025
22 checks passed
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