Bug
contracts/resolution/tests/gas_snap.rs imports from predictify_hybrid but contracts/resolution/Cargo.toml does not declare it as a dev-dependency.
error[E0433]: cannot find module or crate `predictify_hybrid` in this scope
--> contracts/resolution/tests/gas_snap.rs:40:5
|
40 | use predictify_hybrid::types::{OracleConfig, OracleProvider};
| ^^^^^^^^^^^^^^^^^ use of unresolved module or unlinked crate `predictify_hybrid`
error[E0432]: unresolved import `predictify_hybrid`
--> contracts/resolution/tests/gas_snap.rs:39:5
|
39 | use predictify_hybrid::{PredictifyHybrid, PredictifyHybridClient};
| ^^^^^^^^^^^^^^^^^ use of unresolved module or unlinked crate `predictify_hybrid`
Impact
cargo test -p resolution fails to compile any test in the resolution crate, including the pre-existing error_tests.rs and the recently added err_stab.rs (issue #933). The gas_snap.rs file is currently unrunnable in CI.
Root cause
gas_snap.rs was introduced by PR #1143 ("Add contract-error enum expansion for resolution") to snapshot the resolution entrypoint CPU/memory budgets. It exercises the entrypoints via PredictifyHybrid clients, but predictify-hybrid was never added as a dev-dependency of the resolution crate.
Suggested fix
Add to contracts/resolution/Cargo.toml under [dev-dependencies]:
predictify-hybrid = { path = "../predictify-hybrid" }
Plus an explicit [[test]] registration for gas_snap so cargo discovers it explicitly:
[[test]]
name = "gas_snap"
path = "tests/gas_snap.rs"
Acceptance criteria
cargo test -p resolution compiles and runs all test targets (error_tests, err_stab, gas_snap).
- CI green.
Bug
contracts/resolution/tests/gas_snap.rsimports frompredictify_hybridbutcontracts/resolution/Cargo.tomldoes not declare it as a dev-dependency.Impact
cargo test -p resolutionfails to compile any test in theresolutioncrate, including the pre-existingerror_tests.rsand the recently addederr_stab.rs(issue #933). Thegas_snap.rsfile is currently unrunnable in CI.Root cause
gas_snap.rswas introduced by PR #1143 ("Add contract-error enum expansion for resolution") to snapshot the resolution entrypoint CPU/memory budgets. It exercises the entrypoints viaPredictifyHybridclients, butpredictify-hybridwas never added as a dev-dependency of theresolutioncrate.Suggested fix
Add to
contracts/resolution/Cargo.tomlunder[dev-dependencies]:Plus an explicit
[[test]]registration forgas_snapso cargo discovers it explicitly:Acceptance criteria
cargo test -p resolutioncompiles and runs all test targets (error_tests, err_stab, gas_snap).