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

Update the WASM compatibility tests for 0.36 release #2271

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

rafal-ch
Copy link
Contributor

@rafal-ch rafal-ch commented Oct 1, 2024

Closes #2198.

Description

This test upgrades the forward compatibility test by replacing the "genesis" nodes (which used v0.26) with the nodes that use v36 of the client.

It also adds one backward compatibility test: latest_binary_is_backward_compatible_and_follows_blocks_created_by_v36_binary()

Before requesting review

  • I have reviewed the code myself

@rafal-ch rafal-ch added the no changelog Skip the CI check of the changelog modification label Oct 1, 2024
@rafal-ch rafal-ch marked this pull request as ready for review October 1, 2024 12:27
@rafal-ch rafal-ch self-assigned this Oct 1, 2024
@rafal-ch rafal-ch requested a review from a team October 1, 2024 12:27
@@ -47,15 +47,14 @@ async fn latest_binary_is_backward_compatible_and_can_load_testnet_config() {
}

#[tokio::test]
async fn latest_binary_is_backward_compatible_and_follows_blocks_created_by_genesis_binary(
) {
async fn latest_binary_is_backward_compatible_and_follows_blocks_created_by_v36_binary() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

It still would be nice to have one more backward compatibility tests that produces blocks by fuel-core 0.26.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Genesis test is restored in 13f2653

@rafal-ch rafal-ch requested a review from a team October 2, 2024 07:12
rymnc
rymnc previously approved these changes Oct 3, 2024
Comment on lines 136 to 137
"--snapshot",
IGNITION_TESTNET_SNAPSHOT,
Copy link
Collaborator

Choose a reason for hiding this comment

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

This snapshot using the state transition function from fuel-core 0.26.0. Maybe we want to add a new snapshot for fuel-core 0.36?=)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Skip the CI check of the changelog modification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants