Skip to content

Commit

Permalink
Use Testnet in NU5 tests that temporarily should_panic
Browse files Browse the repository at this point in the history
We've marked these tests as should_panic until there is a NU5 activation
height. But Testnet will have an activation height first, so we should
prefer it in the tests. (Or use both networks.)
  • Loading branch information
teor2345 committed Jun 29, 2021
1 parent 3ee905f commit 09b815c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions zebra-consensus/src/transaction/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ async fn v4_transaction_with_transparent_transfer_is_rejected_by_the_script() {
// defined.
#[should_panic]
async fn v5_transaction_with_transparent_transfer_is_accepted() {
let network = Network::Mainnet;
let network = Network::Testnet;
let network_upgrade = NetworkUpgrade::Nu5;

let nu5_activation_height = network_upgrade
Expand Down Expand Up @@ -446,7 +446,7 @@ async fn v5_transaction_with_transparent_transfer_is_accepted() {
// defined.
#[should_panic]
async fn v5_coinbase_transaction_is_accepted() {
let network = Network::Mainnet;
let network = Network::Testnet;
let network_upgrade = NetworkUpgrade::Nu5;

let nu5_activation_height = network_upgrade
Expand Down Expand Up @@ -498,7 +498,7 @@ async fn v5_coinbase_transaction_is_accepted() {
// defined.
#[should_panic]
async fn v5_transaction_with_transparent_transfer_is_rejected_by_the_script() {
let network = Network::Mainnet;
let network = Network::Testnet;
let network_upgrade = NetworkUpgrade::Nu5;

let nu5_activation_height = network_upgrade
Expand Down

0 comments on commit 09b815c

Please sign in to comment.