chore: update to 2024 edition, take 2 (#1678)

* chore: update the rust edition in a few remaining test crates

I missed these in the initial pass as they didn't use the workspace
edition, fix that.

* chore: format for edition changes

Author: Stebalien

integration_tests/Cargo.toml

@@ -4,7 +4,7 @@ description = "Portable integration tests for FVM targets"
 version = "1.0.0"
 license = "MIT OR Apache-2.0"
 authors = ["Protocol Labs", "Filecoin Core Devs"]
-edition = "2021"
+edition.workspace = true
 keywords = ["filecoin", "web3", "wasm"]
 publish = false
 

integration_tests/macro/Cargo.toml

@@ -3,7 +3,7 @@ name = "export_macro"
 description = "Macro to decorate integration tests"
 version = "1.0.0"
 license = "MIT OR Apache-2.0"
-edition = "2021"
+edition.workspace = true
 publish = false
 
 [lib]

integration_tests/src/deals.rs

@@ -2,19 +2,19 @@ use fil_actor_market::{
     ClientDealProposal, DealProposal, Label, Method as MarketMethod, PublishStorageDealsParams,
     PublishStorageDealsReturn,
 };
+use fil_actors_runtime::STORAGE_MARKET_ACTOR_ADDR;
 use fil_actors_runtime::cbor::serialize;
 use fil_actors_runtime::runtime::Policy;
 use fil_actors_runtime::test_utils::make_piece_cid;
-use fil_actors_runtime::STORAGE_MARKET_ACTOR_ADDR;
 use fvm_shared::address::Address;
 use fvm_shared::clock::ChainEpoch;
 use fvm_shared::crypto::signature::{Signature, SignatureType};
 use fvm_shared::econ::TokenAmount;
 use fvm_shared::error::ExitCode;
 use fvm_shared::piece::PaddedPieceSize;
 use num_traits::Zero;
-use vm_api::util::{apply_ok, serialize_ok};
 use vm_api::VM;
+use vm_api::util::{apply_ok, serialize_ok};
 
 #[derive(Clone)]
 pub struct DealOptions {

integration_tests/src/expects.rs

@@ -1,10 +1,10 @@
 use cid::Cid;
-use frc46_token::receiver::{FRC46TokenReceived, FRC46_TOKEN_TYPE};
+use frc46_token::receiver::{FRC46_TOKEN_TYPE, FRC46TokenReceived};
 use frc46_token::token::types::BurnParams;
 use fvm_actor_utils::receiver::UniversalReceiverParams;
 use fvm_ipld_bitfield::BitField;
-use fvm_ipld_encoding::ipld_block::IpldBlock;
 use fvm_ipld_encoding::RawBytes;
+use fvm_ipld_encoding::ipld_block::IpldBlock;
 use fvm_shared::address::Address;
 use fvm_shared::clock::ChainEpoch;
 use fvm_shared::deal::DealID;
@@ -24,7 +24,7 @@ use fil_actor_miner::{IsControllingAddressParam, PowerPair};
 use fil_actor_power::{UpdateClaimedPowerParams, UpdatePledgeTotalParams};
 use fil_actor_verifreg::GetClaimsParams;
 use fil_actors_runtime::{
-    EventBuilder, BURNT_FUNDS_ACTOR_ADDR, DATACAP_TOKEN_ACTOR_ADDR, DATACAP_TOKEN_ACTOR_ID,
+    BURNT_FUNDS_ACTOR_ADDR, DATACAP_TOKEN_ACTOR_ADDR, DATACAP_TOKEN_ACTOR_ID, EventBuilder,
     REWARD_ACTOR_ADDR, STORAGE_MARKET_ACTOR_ADDR, STORAGE_MARKET_ACTOR_ID,
     STORAGE_POWER_ACTOR_ADDR, STORAGE_POWER_ACTOR_ID, VERIFIED_REGISTRY_ACTOR_ADDR,
     VERIFIED_REGISTRY_ACTOR_ID,

integration_tests/src/lib.rs

@@ -1,9 +1,9 @@
 use fil_actors_runtime::reward::FilterEstimate;
 use fvm_shared::{
+    ActorID,
     address::{Address, FIRST_NON_SINGLETON_ADDR},
     econ::TokenAmount,
     sector::StoragePower,
-    ActorID,
 };
 use lazy_static::lazy_static;
 use std::collections::BTreeMap;

integration_tests/src/tests/authenticate_message_test.rs

@@ -1,14 +1,14 @@
 use export_macro::vm_test;
-use fil_actor_account::types::AuthenticateMessageParams;
 use fil_actor_account::Method::AuthenticateMessageExported;
+use fil_actor_account::types::AuthenticateMessageParams;
 use fvm_ipld_encoding::RawBytes;
 use fvm_shared::bigint::Zero;
 use fvm_shared::econ::TokenAmount;
 use fvm_shared::error::ExitCode;
 
 use crate::util::{create_accounts, generate_deal_proposal};
-use vm_api::util::{apply_code, apply_ok};
 use vm_api::VM;
+use vm_api::util::{apply_code, apply_ok};
 
 /// Using a deal proposal as a serialized message, we confirm that:
 /// - calls to Account::authenticate_message with valid signatures succeed

integration_tests/src/tests/batch_onboarding.rs

@@ -1,14 +1,14 @@
 use export_macro::vm_test;
 use fil_actor_miner::SectorPreCommitOnChainInfo;
-use fil_actor_miner::{power_for_sector, State as MinerState};
+use fil_actor_miner::{State as MinerState, power_for_sector};
+use fil_actors_runtime::runtime::Policy;
 use fil_actors_runtime::runtime::policy::policy_constants::PRE_COMMIT_CHALLENGE_DELAY;
 use fil_actors_runtime::runtime::policy_constants::MAX_AGGREGATED_SECTORS;
-use fil_actors_runtime::runtime::Policy;
 use fvm_shared::bigint::BigInt;
 use fvm_shared::econ::TokenAmount;
 use fvm_shared::sector::{RegisteredSealProof, SectorNumber};
-use vm_api::util::{get_state, DynBlockstore};
 use vm_api::VM;
+use vm_api::util::{DynBlockstore, get_state};
 
 use crate::util::{
     advance_to_proving_deadline, create_accounts, create_miner, cron_tick, expect_invariants,

integration_tests/src/tests/batch_onboarding_deals_test.rs

@@ -1,11 +1,11 @@
 use fil_actor_market::DealProposal;
 use fil_actor_miner::{
-    max_prove_commit_duration, power_for_sector, CompactCommD, SectorPreCommitOnChainInfo,
-    State as MinerState,
+    CompactCommD, SectorPreCommitOnChainInfo, State as MinerState, max_prove_commit_duration,
+    power_for_sector,
 };
 use fil_actor_miner::{Method as MinerMethod, ProveCommitAggregateParams};
-use fil_actors_runtime::runtime::policy::policy_constants::PRE_COMMIT_CHALLENGE_DELAY;
 use fil_actors_runtime::runtime::Policy;
+use fil_actors_runtime::runtime::policy::policy_constants::PRE_COMMIT_CHALLENGE_DELAY;
 use fil_actors_runtime::test_utils::make_piece_cid;
 use fvm_shared::address::Address;
 use fvm_shared::bigint::BigInt;
@@ -18,15 +18,16 @@ use fvm_shared::sector::{RegisteredSealProof, StoragePower};
 use num_traits::Zero;
 
 use export_macro::vm_test;
-use vm_api::util::{apply_ok, get_state, DynBlockstore};
 use vm_api::VM;
+use vm_api::util::{DynBlockstore, apply_ok, get_state};
 
 use crate::deals::{DealBatcher, DealOptions};
 use crate::util::{
-    advance_to_proving_deadline, bf_all, create_accounts, create_miner, get_network_stats,
-    make_bitfield, market_add_balance, market_pending_deal_allocations, miner_balance,
-    precommit_meta_data_from_deals, precommit_sectors_v2, precommit_sectors_v2_expect_code,
-    submit_windowed_post, verifreg_add_client, verifreg_add_verifier, PrecommitMetadata,
+    PrecommitMetadata, advance_to_proving_deadline, bf_all, create_accounts, create_miner,
+    get_network_stats, make_bitfield, market_add_balance, market_pending_deal_allocations,
+    miner_balance, precommit_meta_data_from_deals, precommit_sectors_v2,
+    precommit_sectors_v2_expect_code, submit_windowed_post, verifreg_add_client,
+    verifreg_add_verifier,
 };
 
 const BATCH_SIZE: usize = 8;

integration_tests/src/tests/change_beneficiary_test.rs

@@ -6,8 +6,8 @@ use fil_actors_runtime::runtime::Policy;
 use fvm_shared::econ::TokenAmount;
 use fvm_shared::error::ExitCode;
 use fvm_shared::{bigint::Zero, sector::RegisteredSealProof};
-use vm_api::util::apply_code;
 use vm_api::VM;
+use vm_api::util::apply_code;
 
 use crate::util::{
     assert_invariants, change_beneficiary, create_accounts, create_miner, get_beneficiary,

integration_tests/src/tests/change_owner_test.rs

@@ -5,8 +5,8 @@ use fvm_shared::bigint::Zero;
 use fvm_shared::econ::TokenAmount;
 use fvm_shared::error::ExitCode;
 use fvm_shared::sector::RegisteredSealProof;
-use vm_api::util::apply_code;
 use vm_api::VM;
+use vm_api::util::apply_code;
 
 use crate::util::{
     assert_invariants, change_beneficiary, change_owner_address, create_accounts, create_miner,