Skip to content

test: update unit test code for test_connection #578

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

anencore94
Copy link

fixes #542

This commit fixes failed unit tests on "tests/unittests/tools/application_integration_tool/clients/test_connections_client.py"
and add another test cases for changed interface "test_get_connection_details_without_name"

Notes
These changes do not affect the main logic or features, but ensure consistency and reliability of the test suite.
Please let me know if you have any questions or need further adjustments.

@anencore94 anencore94 force-pushed the bugfix/failed_unittests branch from eacf74c to 25979fe Compare May 6, 2025 07:39
@hangfei
Copy link
Collaborator

hangfei commented May 7, 2025

It's fixed by d5b3a89

Thanks for your contribution though.

@hangfei
Copy link
Collaborator

hangfei commented May 8, 2025

@anencore94 Could you resolve conflicts to see if the new tests are covered or not?

@anencore94 anencore94 force-pushed the bugfix/failed_unittests branch from 25979fe to 8772b3d Compare May 8, 2025 04:04
@anencore94
Copy link
Author

@anencore94 Could you resolve conflicts to see if the new tests are covered or not?

Thanks for the suggestion. PTAL @hangfei

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.

Looking for community contribution: Improve unit tests test coverage
2 participants