test: fix test suite compatibility with upgraded test dependencies#98
Merged
alexsander-souza merged 2 commits intocanonical:resolute_supportfrom Apr 2, 2026
Conversation
r00ta
reviewed
Apr 1, 2026
|
|
||
| class TestUUIDToTimestamp(MAASTestCase): | ||
| def test_round_trip_preserves_timestamp(self): | ||
| self.skipTest("UUID1 timestamps offset due to libuuid bug (LP#2145734)") |
Contributor
There was a problem hiding this comment.
we use uuid_to_timestamp in many places and the bug mentioned is changing the behavior of the function. Is this going to break existing functionalities? Instead of skipping the test, do we have to fix the function somehow?
Contributor
Author
There was a problem hiding this comment.
It breaks the beacon only. My suggestion is to skip these tests for now, enabling us to merge the migration to Resolute into master. The timeline to fix the issue is uncertain, as it might involve the upstream, but we should look for a solution before releasing 3.8
Contributor
There was a problem hiding this comment.
can you please open a ticket in jira to keep track of it?
Contributor
Author
There was a problem hiding this comment.
Fix test failures introduced by the dependency version bumps in PR-03. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
Remove HTTP endpoint delimiters and trivial comments that add no value to code comprehension. Improve clarity through self-documenting variable names (e.g., state_with_redirect) and extract the JSON dump mock helper to module level for reusability.
alexsander-souza
force-pushed
the
pr/10-test-framework
branch
from
d00fb5e to
19e9696
Compare
r00ta
approved these changes
Apr 2, 2026
alexsander-souza
merged commit cc2a94c
into
canonical:resolute_support
1 of 2 checks passed
alexsander-souza
added a commit
that referenced
this pull request
Apr 8, 2026
Fix test failures introduced by the dependency version bumps. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
alexsander-souza
added a commit
that referenced
this pull request
Apr 8, 2026
Fix test failures introduced by the dependency version bumps. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
alexsander-souza
added a commit
that referenced
this pull request
Apr 9, 2026
Fix test failures introduced by the dependency version bumps. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
alexsander-souza
added a commit
that referenced
this pull request
Apr 17, 2026
Fix test failures introduced by the dependency version bumps. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
alexsander-souza
added a commit
that referenced
this pull request
Apr 17, 2026
Fix test failures introduced by the dependency version bumps. No production code changes. - pytest-asyncio 1.x: add explicit asyncio reactor event loop setup in maastesting - hypothesis 6.151: replace timing-sensitive Hypothesis tests with deterministic equivalents; reduce max_examples in parallel tests - httpx 0.28: wrap app in ASGITransport; move cookie setup to client instance (AsyncClient(app=...) constructor deprecated) - testtools 2.8: add assertCountEqual shim to MAASTestCase - TLS test fixtures: regenerate certificates with subjectAltName extension required by modern SSL contexts; update data_test_main/generate.sh - libuuid: disable affected test cases with upstream bug reference
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix test failures introduced by the dependency version bumps. No production code changes.