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

entropy-test-cli - When storing program, configuration interface files must be provided #1085

Open
ameba23 opened this issue Oct 1, 2024 · 0 comments

Comments

@ameba23
Copy link
Contributor

ameba23 commented Oct 1, 2024

The documentation for the test-cli says we do not have to give configuration interface or aux data interface files when storing a program:

        /// The path to a file containing the program config interface (defaults to empty)
        config_interface_file: Option<PathBuf>,
        /// The path to a file containing the program aux interface (defaults to empty)
        aux_data_interface_file: Option<PathBuf>,

But if we don't provide them, we get an error:

~/r/s/e/e/c/test-cli (master)$ cargo run --release -- store-program -m //Alice ../testing-utils/template_barebones.wasm
    Finished `release` profile [optimized] target(s) in 2.60s
     Running `/home/turnip/radish/src/entropy/entropy-core/target/release/entropy-test-cli store-program -m //Alice ../testing-utils/template_barebones.wasm`
Storing program using account: 5GrwvaEF5zXb26Fz9rcQpDWS57CtERHpNehXCPcNoHGKutQY
thread 'main' panicked at /home/turnip/radish/src/entropy/entropy-core/crates/test-cli/src/lib.rs:287:50:
No config interface file passed

This is because rather than assign them to vector of length 0, we now check for the Option that is passed in to the run function.

I think that it should be possible to store a program from the CLI without providing these files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants