Skip to content

fix: hardened the compatibility checks to ensure that we tests that l… #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 26, 2025

Conversation

mridang
Copy link
Collaborator

@mridang mridang commented May 26, 2025

Description

This pull request improves the compatibility tests for the zitadel-client module. Previously, our tests only confirmed the module was installed. Now, they also import the module, which helps catch any errors that might occur when it's actually loaded and used.

Related Issue

None

Motivation and Context

The old testing approach was incomplete; simply installing the module didn't guarantee it would work correctly when imported. This oversight could hide critical issues. By adding an import step to our tests, we can find these problems much earlier, leading to a more robust and reliable integration.

How Has This Been Tested?

These changes have been thoroughly tested using our Continuous Integration (CI) pipeline.

Documentation:

None

Checklist:

  • I have updated the documentation accordingly.
  • I have assigned the correct milestone or created one if non-existent.
  • I have correctly labeled this pull request.
  • I have linked the corresponding issue in this description.
  • I have requested a review from at least 2 reviewers
  • I have checked the base branch of this pull request
  • I have checked my code for any possible security vulnerabilities

@mridang mridang added the enhancement New feature or request label May 26, 2025
@mridang mridang self-assigned this May 26, 2025
@mridang mridang merged commit 77fd03c into main May 26, 2025
1 check passed
github-actions bot pushed a commit that referenced this pull request Jun 4, 2025
## [1.4.4](v1.4.3...v1.4.4) (2025-06-04)

### Bug Fixes

* fixed the workflow as a broken merge was committed ([2674d36](2674d36))
* hardened the compatibility checks to ensure that we tests that loading of the module works ([#34](#34)) ([77fd03c](77fd03c))
* updated the code to ensure compatibility with Python 3.9 ([01b749e](01b749e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant