Add testcases #44

joii2020 · 2024-03-05T02:15:47Z

Fix incorrect shrinking: basic.mol
Testcase: The witnesses is none
Testcase: The output data is big enough(700k)
Testcase: Add a 700k witness
Testcase: Args is big enough (700k)

* The witnesses is none * The output data is big enough(16M) * Add a 16M witness * Args is big enough (16M)

XuJiandong · 2024-03-05T05:20:46Z

tests/omni_lock_rust/tests/test_omni_lock.rs

+        .set_chain_config(Box::new(BitcoinConfig { sign_vtype: BITCOIN_V_TYPE_P2PKHUNCOMPRESSED, pubkey_err: false }));
+
+    let tx = gen_tx(&mut data_loader, &mut config);
+    let tx = tx.as_advanced_builder().set_outputs_data(vec![[0u8; 1024 * 1024 * 16].pack()]).build();


700 K is enough. 4M is too large and the cycles will be large too.

joii2020 added 2 commits March 4, 2024 18:09

Fix incorrect shrinking

3774f40

Add testcase:

75eab54

* The witnesses is none * The output data is big enough(16M) * Add a 16M witness * Args is big enough (16M)

XuJiandong requested changes Mar 5, 2024

View reviewed changes

Testdata length changed from 16M to 700k

072b858

joii2020 requested a review from XuJiandong March 5, 2024 08:47

XuJiandong approved these changes Mar 6, 2024

View reviewed changes

XuJiandong merged commit 45ef4e5 into XuJiandong:cobuild Mar 6, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add testcases #44

Add testcases #44

joii2020 commented Mar 5, 2024 •

edited

Loading

XuJiandong Mar 5, 2024

Add testcases #44

Add testcases #44

Conversation

joii2020 commented Mar 5, 2024 • edited Loading

XuJiandong Mar 5, 2024

Choose a reason for hiding this comment

joii2020 commented Mar 5, 2024 •

edited

Loading