Skip to content
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

tests: multi consul version tests #53

Merged
merged 1 commit into from
Mar 18, 2024
Merged

tests: multi consul version tests #53

merged 1 commit into from
Mar 18, 2024

Conversation

mbrulatout
Copy link
Contributor

@mbrulatout mbrulatout commented Aug 28, 2023

This PR introduces multi-version tests.
Due to the fact that tests were running on consul 1.1.0,
some tests have been temporarily disabled due to
issues with newer API.

Will be fixed upcoming PRs dropping consul 1.1.0 support

This commit introduces multi-version tests.
Due to the fact that tests were running on consul 1.1.0,
some tests have been temporarily disabled due to
issues with newer API.

Will be fixed other commits
@mbrulatout mbrulatout changed the base branch from cleanup_async to master March 15, 2024 14:13
@mbrulatout mbrulatout changed the title [WIP] tests: multi version tests tests: multi consul version tests Mar 15, 2024
tests/test_aio.py Show resolved Hide resolved
tests/test_aio.py Show resolved Hide resolved
tests/test_aio.py Show resolved Hide resolved
tests/test_aio.py Show resolved Hide resolved
@mbrulatout mbrulatout requested a review from mougams March 18, 2024 12:13
Copy link
Contributor

@mougams mougams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FAR: Fix the discussion I've opened

@mbrulatout mbrulatout merged commit 45c8d6a into master Mar 18, 2024
4 checks passed
@manu-ns manu-ns deleted the multiversion branch March 29, 2024 08:56
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.

3 participants