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

Proof that #6324 is not a bug. #6573

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Commits on Sep 26, 2024

  1. Proof that #6324 is not a bug.

    The reproduction presented in #6324 was missing loading the storage
    configuration properly.
    
    To do that we have to use the following in test/src/sdk-harness/test_projects/run_external_proxy_with_storage/mod.rs:
        StorageConfiguration::default().add_slot_overrides_from_file("test_projects/run_external_target_with_storage/out/release/run_external_target_with_storage-storage_slots.json").unwrap();
    
    This commit changes the presented reproduction to load the storage slots
    and the test runs without reverting.
    
    Closes #6324
    esdrubal committed Sep 26, 2024
    Configuration menu
    Copy the full SHA
    e7fdb8e View commit details
    Browse the repository at this point in the history