mirror of
https://gitlab.com/pezkuwichain/pezkuwi-sdk.git
synced 2026-10-20 18:56:23 +00:00
fix: resolve 4 classes of test failures from partition-1/2 CI runs
1. bip39: revert test password Bizinikiwi to Substrate — VECTORS were generated with Substrate; only the test code was renamed during rebrand, not the precomputed PBKDF2 reference values. 2. xcm: revert DescribePalletTerminal byte tag Pezpallet to Pallet — XCM location-to-account hashing uses protocol-level byte strings that must not be rebranded; the change silently broke account derivation for pallet-instance XCM locations in all zagros runtimes. 3. CI: exclude pez-chain-spec-guide-runtime from all 5 test-partition and test-no-try-runtime jobs — its tests require WASM_BINARY_PATH which is None under SKIP_WASM_BUILD=1 (same pattern as the 10 emulated integration test exclusions already in place). 4. polkavm: downgrade fixtures build-dep polkavm-linker 0.30.0 to 0.29.0 to match pallet polkavm 0.29.1 (polkavm-common 0.29.0) validator; the format mismatch caused CodeRejected on all PVM blob uploads in tests. Updated build.rs: target_json_path(args) becomes target_json_64_path(), program_from_elf 3-arg becomes 2-arg.
This commit is contained in:
1 parent
9941af28b1
commit
3119c7edee
6 files changed
+21
-35
No files matched your search
+10
-5
@@ -460,7 +460,8 @@ test-partition-1:
|
||||
--exclude people-pezkuwichain-integration-tests \
|
||||
--exclude people-zagros-integration-tests \
|
||||
--exclude pezbridge-hub-pezkuwichain-integration-tests \
|
||||
--exclude pezbridge-hub-zagros-integration-tests
|
||||
--exclude pezbridge-hub-zagros-integration-tests \
|
||||
--exclude pez-chain-spec-guide-runtime
|
||||
artifacts:
|
||||
when: always
|
||||
paths: [test-results-1.xml]
|
||||
@@ -486,7 +487,8 @@ test-partition-2:
|
||||
--exclude people-pezkuwichain-integration-tests \
|
||||
--exclude people-zagros-integration-tests \
|
||||
--exclude pezbridge-hub-pezkuwichain-integration-tests \
|
||||
--exclude pezbridge-hub-zagros-integration-tests
|
||||
--exclude pezbridge-hub-zagros-integration-tests \
|
||||
--exclude pez-chain-spec-guide-runtime
|
||||
artifacts:
|
||||
when: always
|
||||
paths: [test-results-2.xml]
|
||||
@@ -512,7 +514,8 @@ test-partition-3:
|
||||
--exclude people-pezkuwichain-integration-tests \
|
||||
--exclude people-zagros-integration-tests \
|
||||
--exclude pezbridge-hub-pezkuwichain-integration-tests \
|
||||
--exclude pezbridge-hub-zagros-integration-tests
|
||||
--exclude pezbridge-hub-zagros-integration-tests \
|
||||
--exclude pez-chain-spec-guide-runtime
|
||||
# Also runs runtime-api tests
|
||||
- cargo nextest run -p pezsp-api-test --features enable-pez-staging-api --cargo-quiet
|
||||
artifacts:
|
||||
@@ -540,7 +543,8 @@ test-no-try-runtime-1:
|
||||
--exclude people-pezkuwichain-integration-tests \
|
||||
--exclude people-zagros-integration-tests \
|
||||
--exclude pezbridge-hub-pezkuwichain-integration-tests \
|
||||
--exclude pezbridge-hub-zagros-integration-tests
|
||||
--exclude pezbridge-hub-zagros-integration-tests \
|
||||
--exclude pez-chain-spec-guide-runtime
|
||||
|
||||
test-no-try-runtime-2:
|
||||
extends: .test-base
|
||||
@@ -560,7 +564,8 @@ test-no-try-runtime-2:
|
||||
--exclude people-pezkuwichain-integration-tests \
|
||||
--exclude people-zagros-integration-tests \
|
||||
--exclude pezbridge-hub-pezkuwichain-integration-tests \
|
||||
--exclude pezbridge-hub-zagros-integration-tests
|
||||
--exclude pezbridge-hub-zagros-integration-tests \
|
||||
--exclude pez-chain-spec-guide-runtime
|
||||
|
||||
# Emulated XCM integration tests — require WASM blobs, run separately with
|
||||
# SKIP_WASM_BUILD=0. Uses the same CARGO_TARGET_DIR as test-partition jobs so
|
||||
|
||||
Generated
+5
-20
@@ -4090,7 +4090,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "7ab67060fc6b8ef687992d439ca0fa36e7ed17e9a0b16b25b601e8757df720de"
|
||||
dependencies = [
|
||||
"data-encoding",
|
||||
"syn 1.0.109",
|
||||
"syn 2.0.114",
|
||||
]
|
||||
|
||||
[[package]]
|
||||
@@ -17271,7 +17271,7 @@ dependencies = [
|
||||
"pezpallet-revive-uapi",
|
||||
"pezsp-core",
|
||||
"pezsp-io",
|
||||
"polkavm-linker 0.30.0",
|
||||
"polkavm-linker 0.29.0",
|
||||
"serde_json",
|
||||
"toml 0.8.23",
|
||||
]
|
||||
@@ -21626,12 +21626,6 @@ dependencies = [
|
||||
"siphasher 1.0.1",
|
||||
]
|
||||
|
||||
[[package]]
|
||||
name = "picosimd"
|
||||
version = "0.9.2"
|
||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "af35c838647fef3d6d052e27006ef88ea162336eee33063c50a63f163c18cdeb"
|
||||
|
||||
[[package]]
|
||||
name = "pin-project"
|
||||
version = "1.1.10"
|
||||
@@ -21795,15 +21789,6 @@ dependencies = [
|
||||
"polkavm-assembler 0.29.0",
|
||||
]
|
||||
|
||||
[[package]]
|
||||
name = "polkavm-common"
|
||||
version = "0.30.0"
|
||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "ed1b408db93d4f49f5c651a7844682b9d7a561827b4dc6202c10356076c055c9"
|
||||
dependencies = [
|
||||
"picosimd",
|
||||
]
|
||||
|
||||
[[package]]
|
||||
name = "polkavm-derive"
|
||||
version = "0.26.0"
|
||||
@@ -21884,16 +21869,16 @@ dependencies = [
|
||||
|
||||
[[package]]
|
||||
name = "polkavm-linker"
|
||||
version = "0.30.0"
|
||||
version = "0.29.0"
|
||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "6739125c4f8f44b4282b6531d765d599f20514e9b608737c6c3544594d08f995"
|
||||
checksum = "43e01613e9e3e4ebd624aa3a11f1775a5c90b881200c50e054fe13c3ba451f98"
|
||||
dependencies = [
|
||||
"dirs",
|
||||
"gimli 0.31.1",
|
||||
"hashbrown 0.14.5",
|
||||
"log",
|
||||
"object 0.36.7",
|
||||
"polkavm-common 0.30.0",
|
||||
"polkavm-common 0.29.0",
|
||||
"regalloc2 0.9.3",
|
||||
"rustc-demangle",
|
||||
]
|
||||
|
||||
@@ -30,7 +30,7 @@ anyhow = { workspace = true, default-features = true }
|
||||
cargo_metadata = { workspace = true }
|
||||
hex = { workspace = true, features = ["alloc"] }
|
||||
pezpallet-revive-uapi = { workspace = true }
|
||||
polkavm-linker = { version = "0.30.0" }
|
||||
polkavm-linker = { version = "0.29.0" }
|
||||
serde_json = { workspace = true }
|
||||
toml = { workspace = true }
|
||||
|
||||
|
||||
@@ -180,9 +180,6 @@ fn invoke_build(current_dir: &Path) -> Result<()> {
|
||||
// This works with stable rust when RUSTC_BOOTSTRAP=1 is set
|
||||
let encoded_rustflags = ["-Dwarnings"].join("\x1f");
|
||||
|
||||
let mut args = polkavm_linker::TargetJsonArgs::default();
|
||||
args.is_64_bit = true;
|
||||
|
||||
let mut build_command = Command::new("cargo");
|
||||
build_command
|
||||
.current_dir(current_dir)
|
||||
@@ -199,7 +196,7 @@ fn invoke_build(current_dir: &Path) -> Result<()> {
|
||||
"-Zbuild-std-features=panic_immediate_abort",
|
||||
])
|
||||
.arg("--target")
|
||||
.arg(polkavm_linker::target_json_path(args).unwrap());
|
||||
.arg(polkavm_linker::target_json_64_path().unwrap());
|
||||
|
||||
if let Ok(toolchain) = env::var(OVERRIDE_RUSTUP_TOOLCHAIN_ENV_VAR) {
|
||||
build_command.env("RUSTUP_TOOLCHAIN", &toolchain);
|
||||
@@ -228,7 +225,6 @@ fn post_process(input_path: &Path, output_path: &Path) -> Result<()> {
|
||||
let orig = fs::read(input_path).with_context(|| format!("Failed to read {input_path:?}"))?;
|
||||
let linked = polkavm_linker::program_from_elf(
|
||||
config,
|
||||
polkavm_linker::TargetInstructionSet::Latest,
|
||||
orig.as_ref(),
|
||||
)
|
||||
.map_err(|err| anyhow::format_err!("Failed to link polkavm program: {}", err))?;
|
||||
|
||||
@@ -81,7 +81,7 @@ mod test {
|
||||
|
||||
// phrase, entropy, seed, expanded secret_key
|
||||
//
|
||||
// ALL SEEDS GENERATED USING "Bizinikiwi" PASSWORD!
|
||||
// ALL SEEDS GENERATED USING "Substrate" PASSWORD!
|
||||
static VECTORS: &[[&str; 3]] = &[
|
||||
[
|
||||
"abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon about",
|
||||
@@ -214,8 +214,8 @@ mod test {
|
||||
let expected_seed: Vec<u8> = vector[2].from_hex().unwrap();
|
||||
|
||||
let mnemonic = Mnemonic::parse_in(Language::English, phrase).unwrap();
|
||||
let seed = seed_from_entropy(&mnemonic.to_entropy(), "Bizinikiwi").unwrap();
|
||||
let secret = mini_secret_from_entropy(&mnemonic.to_entropy(), "Bizinikiwi")
|
||||
let seed = seed_from_entropy(&mnemonic.to_entropy(), "Substrate").unwrap();
|
||||
let secret = mini_secret_from_entropy(&mnemonic.to_entropy(), "Substrate")
|
||||
.unwrap()
|
||||
.to_bytes();
|
||||
|
||||
|
||||
@@ -59,7 +59,7 @@ impl DescribeLocation for DescribePalletTerminal {
|
||||
fn describe_location(l: &Location) -> Option<Vec<u8>> {
|
||||
match l.unpack() {
|
||||
(0, [PalletInstance(i)]) => {
|
||||
Some((b"Pezpallet", Compact::<u32>::from(*i as u32)).encode())
|
||||
Some((b"Pallet", Compact::<u32>::from(*i as u32)).encode())
|
||||
},
|
||||
_ => return None,
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user