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

test: backwards compatibility test for the serialization feature #4548

Merged
merged 17 commits into from May 20, 2024

Conversation

maddeleine
Copy link
Contributor

Resolved issues:

N/A

Description of changes:

This adds an integration test that checks that new changes are backwards compatible with older versions of the serialization feature.

Testing:

Integ test
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the s2n-core team label May 8, 2024

from configuration import available_ports
from common import ProviderOptions, Protocols
from fixtures import managed_process # lgtm [py/unused-import]

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'managed_process' is not used.
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
tests/integrationv2/test_serialization.py Outdated Show resolved Hide resolved
@maddeleine maddeleine requested a review from lrstewart May 15, 2024 17:47
@maddeleine maddeleine requested a review from lrstewart May 16, 2024 18:29
@maddeleine maddeleine enabled auto-merge (squash) May 16, 2024 21:03
@maddeleine maddeleine merged commit 61263b0 into main May 20, 2024
36 checks passed
@maddeleine maddeleine deleted the serialize_integ_test branch May 20, 2024 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants