chore: format noir stuff

noir-projects/aztec-nr/authwit/src/auth_witness.nr

@@ -3,7 +3,7 @@ unconstrained fn get_auth_witness_oracle<let N: u32>(_message_hash: Field) -> [F
 
 /**
  * Oracle wrapper to fetch an `auth_witness` for a given `message_hash` from the PXE.
- * 
+ *
  * @param message_hash The hash of the message for which the `auth_witness` is to be fetched.
  * @return The `auth_witness` for the given `message_hash` as Field array.
  */

noir-projects/aztec-nr/aztec/src/keys/getters/test.nr

@@ -12,7 +12,7 @@ unconstrained fn setup() -> TestAccount {
     account
 }
 
-#[test(should_fail_with="Invalid public keys hint for address")]
+#[test(should_fail_with = "Invalid public keys hint for address")]
 unconstrained fn test_get_public_keys_unknown() {
     let account = setup();
 

noir-projects/aztec-nr/aztec/src/note/note_getter/test.nr

@@ -106,7 +106,7 @@ unconstrained fn can_return_zero_notes() {
     assert_eq(returned.len(), 0);
 }
 
-#[test(should_fail_with="Got more notes than limit.")]
+#[test(should_fail_with = "Got more notes than limit.")]
 unconstrained fn rejects_mote_notes_than_limit() {
     let mut env = setup();
     let mut context = env.private();
@@ -155,7 +155,7 @@ unconstrained fn applies_filter_before_constraining() {
     assert_eq(context.note_hash_read_requests.len(), 1);
 }
 
-#[test(should_fail_with="Mismatch note header contract address.")]
+#[test(should_fail_with = "Mismatch note header contract address.")]
 unconstrained fn rejects_mismatched_address() {
     let mut env = setup();
     let mut context = env.private();
@@ -169,7 +169,7 @@ unconstrained fn rejects_mismatched_address() {
     let _ = constrain_get_notes_internal(&mut context, storage_slot, opt_notes, options);
 }
 
-#[test(should_fail_with="Mismatch note header storage slot.")]
+#[test(should_fail_with = "Mismatch note header storage slot.")]
 unconstrained fn rejects_mismatched_storage_slot() {
     let mut env = setup();
     let mut context = env.private();
@@ -183,7 +183,7 @@ unconstrained fn rejects_mismatched_storage_slot() {
     let _ = constrain_get_notes_internal(&mut context, storage_slot, opt_notes, options);
 }
 
-#[test(should_fail_with="Mismatch return note field.")]
+#[test(should_fail_with = "Mismatch return note field.")]
 unconstrained fn rejects_mismatched_selector() {
     let mut env = setup();
     let mut context = env.private();
@@ -204,7 +204,7 @@ unconstrained fn rejects_mismatched_selector() {
     let _ = constrain_get_notes_internal(&mut context, storage_slot, opt_notes, options);
 }
 
-#[test(should_fail_with="Return notes not sorted in descending order.")]
+#[test(should_fail_with = "Return notes not sorted in descending order.")]
 unconstrained fn rejects_mismatched_desc_sort_order() {
     let mut env = setup();
     let mut context = env.private();
@@ -222,7 +222,7 @@ unconstrained fn rejects_mismatched_desc_sort_order() {
     let _ = constrain_get_notes_internal(&mut context, storage_slot, opt_notes, options);
 }
 
-#[test(should_fail_with="Return notes not sorted in ascending order.")]
+#[test(should_fail_with = "Return notes not sorted in ascending order.")]
 unconstrained fn rejects_mismatched_asc_sort_order() {
     let mut env = setup();
     let mut context = env.private();

noir-projects/aztec-nr/aztec/src/state_vars/public_immutable/test.nr

@@ -33,7 +33,7 @@ fn test_initialize_uninitialized() {
     assert(state_var.read() == value);
 }
 
-#[test(should_fail_with="PublicImmutable already initialized")]
+#[test(should_fail_with = "PublicImmutable already initialized")]
 fn test_initialize_already_initialized() {
     let env = setup();
     let state_var = in_public(env);
@@ -44,7 +44,7 @@ fn test_initialize_already_initialized() {
     state_var.initialize(value);
 }
 
-#[test(should_fail_with="PublicImmutable not initialized")]
+#[test(should_fail_with = "PublicImmutable not initialized")]
 fn test_read_uninitialized() {
     let env = setup();
     let state_var = in_public(env);

noir-projects/aztec-nr/aztec/src/state_vars/shared_mutable/test.nr

@@ -267,7 +267,7 @@ unconstrained fn test_get_current_value_in_private_with_non_initial_delay() {
     assert_eq(private_state_var.context.max_block_number.unwrap(), historical_block_number + new_delay);
 }
 
-#[test(should_fail_with="Hint values do not match hash")]
+#[test(should_fail_with = "Hint values do not match hash")]
 unconstrained fn test_get_current_value_in_private_bad_value_hints() {
     let mut env = setup();
 
@@ -287,7 +287,7 @@ unconstrained fn test_get_current_value_in_private_bad_value_hints() {
     let _ = private_state_var.get_current_value_in_private();
 }
 
-#[test(should_fail_with="Hint values do not match hash")]
+#[test(should_fail_with = "Hint values do not match hash")]
 unconstrained fn test_get_current_value_in_private_bad_delay_hints() {
     let mut env = setup();
 
@@ -307,7 +307,7 @@ unconstrained fn test_get_current_value_in_private_bad_delay_hints() {
     let _ = private_state_var.get_current_value_in_private();
 }
 
-#[test(should_fail_with="Non-zero value change for zero hash")]
+#[test(should_fail_with = "Non-zero value change for zero hash")]
 unconstrained fn test_get_current_value_in_private_bad_zero_hash_value_hints() {
     let mut env = setup();
 
@@ -324,7 +324,7 @@ unconstrained fn test_get_current_value_in_private_bad_zero_hash_value_hints() {
     let _ = state_var.get_current_value_in_private();
 }
 
-#[test(should_fail_with="Non-zero delay change for zero hash")]
+#[test(should_fail_with = "Non-zero delay change for zero hash")]
 unconstrained fn test_get_current_value_in_private_bad_zero_hash_delay_hints() {
     let mut env = setup();
 

noir-projects/aztec-nr/aztec/src/utils/test.nr

@@ -57,7 +57,7 @@ unconstrained fn verify_collapse_hints_good_hints() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Wrong collapsed vec length")]
+#[test(should_fail_with = "Wrong collapsed vec length")]
 unconstrained fn verify_collapse_hints_wrong_length() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
     let collapsed = BoundedVec::from_array([7]);
@@ -66,7 +66,7 @@ unconstrained fn verify_collapse_hints_wrong_length() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Collapse hint vec length mismatch")]
+#[test(should_fail_with = "Collapse hint vec length mismatch")]
 unconstrained fn verify_collapse_hints_hint_length_mismatch() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
     let collapsed = BoundedVec::from_array([7, 3]);
@@ -75,7 +75,7 @@ unconstrained fn verify_collapse_hints_hint_length_mismatch() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Out of bounds index hint")]
+#[test(should_fail_with = "Out of bounds index hint")]
 unconstrained fn verify_collapse_hints_out_of_bounds_index_hint() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
     let collapsed = BoundedVec::from_array([7, 3]);
@@ -93,7 +93,7 @@ unconstrained fn verify_collapse_hints_hint_to_none() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Wrong collapsed vec content")]
+#[test(should_fail_with = "Wrong collapsed vec content")]
 unconstrained fn verify_collapse_hints_wrong_vec_content() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
     let collapsed = BoundedVec::from_array([7, 42]);
@@ -102,7 +102,7 @@ unconstrained fn verify_collapse_hints_wrong_vec_content() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Wrong collapsed vec order")]
+#[test(should_fail_with = "Wrong collapsed vec order")]
 unconstrained fn verify_collapse_hints_wrong_vec_order() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
     let collapsed = BoundedVec::from_array([3, 7]);
@@ -111,7 +111,7 @@ unconstrained fn verify_collapse_hints_wrong_vec_order() {
     verify_collapse_hints(original, collapsed, collapsed_to_input_index_mapping);
 }
 
-#[test(should_fail_with="Dirty collapsed vec storage")]
+#[test(should_fail_with = "Dirty collapsed vec storage")]
 unconstrained fn verify_collapse_hints_dirty_storage() {
     let original = [Option::some(7), Option::none(), Option::some(3)];
 

noir-projects/noir-contracts/contracts/easy_private_voting_contract/src/main.nr

@@ -21,7 +21,8 @@ contract EasyPrivateVoting {
 
     // docs:start:constructor
     #[public]
-    #[initializer] // annotation to mark function as a constructor
+    #[initializer]
+    // annotation to mark function as a constructor
     fn constructor(admin: AztecAddress) {
         storage.admin.write(admin);
         storage.vote_ended.write(false);
@@ -30,7 +31,8 @@ contract EasyPrivateVoting {
     // docs:end:constructor
 
     // docs:start:cast_vote
-    #[private] // annotation to mark function as private and expose private context
+    #[private]
+    // annotation to mark function as private and expose private context
     fn cast_vote(candidate: Field) {
         let msg_sender_npk_m_hash = get_public_keys(context.msg_sender()).npk_m.hash();
 

noir-projects/noir-contracts/contracts/nft_contract/src/test/transfer_in_private.nr

@@ -59,7 +59,7 @@ unconstrained fn transfer_in_private_on_behalf_of_other() {
     utils::assert_owns_private_nft(nft_contract_address, recipient, token_id);
 }
 
-#[test(should_fail_with="NFT not found when transferring")]
+#[test(should_fail_with = "NFT not found when transferring")]
 unconstrained fn transfer_in_private_failure_not_an_owner() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, owner, not_owner, token_id) = utils::setup_mint_and_transfer_to_private(/* with_account_contracts */ false);
@@ -68,7 +68,7 @@ unconstrained fn transfer_in_private_failure_not_an_owner() {
     env.call_private_void(NFT::at(nft_contract_address).transfer_in_private(not_owner, owner, token_id, 0));
 }
 
-#[test(should_fail_with="invalid nonce")]
+#[test(should_fail_with = "invalid nonce")]
 unconstrained fn transfer_in_private_failure_on_behalf_of_self_non_zero_nonce() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough.
     // The nonce check is in the beginning so we don't need to waste time on minting the NFT and transferring
@@ -82,7 +82,7 @@ unconstrained fn transfer_in_private_failure_on_behalf_of_self_non_zero_nonce()
     env.call_private_void(NFT::at(nft_contract_address).transfer_in_private(sender, recipient, token_id, 1));
 }
 
-#[test(should_fail_with="Authorization not found for message hash")]
+#[test(should_fail_with = "Authorization not found for message hash")]
 unconstrained fn transfer_in_private_failure_on_behalf_of_other_without_approval() {
     // Setup with account contracts. Slower since we actually deploy them, but needed for authwits.
     // The authwit check is in the beginning so we don't need to waste time on minting the NFT and transferring
@@ -98,7 +98,7 @@ unconstrained fn transfer_in_private_failure_on_behalf_of_other_without_approval
     env.call_private_void(NFT::at(nft_contract_address).transfer_in_private(sender, recipient, token_id, 1));
 }
 
-#[test(should_fail_with="Authorization not found for message hash")]
+#[test(should_fail_with = "Authorization not found for message hash")]
 unconstrained fn transfer_in_private_failure_on_behalf_of_other_wrong_caller() {
     // Setup with account contracts. Slower since we actually deploy them, but needed for authwits.
     // The authwit check is in the beginning so we don't need to waste time on minting the NFT and transferring

noir-projects/noir-contracts/contracts/nft_contract/src/test/transfer_in_public.nr

@@ -42,7 +42,7 @@ unconstrained fn transfer_in_public_on_behalf_of_other() {
     utils::assert_owns_public_nft(env, nft_contract_address, recipient, token_id);
 }
 
-#[test(should_fail_with="invalid nonce")]
+#[test(should_fail_with = "invalid nonce")]
 unconstrained fn transfer_in_public_failure_on_behalf_of_self_non_zero_nonce() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough.
     // The authwit check is in the beginning so we don't need to waste time on minting the NFT and transferring
@@ -56,7 +56,7 @@ unconstrained fn transfer_in_public_failure_on_behalf_of_self_non_zero_nonce() {
     env.call_public(NFT::at(nft_contract_address).transfer_in_public(sender, recipient, token_id, random()));
 }
 
-#[test(should_fail_with="invalid owner")]
+#[test(should_fail_with = "invalid owner")]
 unconstrained fn transfer_in_public_non_existent_nft() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, sender, recipient) = utils::setup(/* with_account_contracts */ false);

noir-projects/noir-contracts/contracts/nft_contract/src/test/transfer_to_private.nr

@@ -64,7 +64,7 @@ unconstrained fn transfer_to_private_to_a_different_account() {
     utils::assert_owns_public_nft(env, nft_contract_address, AztecAddress::zero(), token_id);
 }
 
-#[test(should_fail_with="transfer not prepared")]
+#[test(should_fail_with = "transfer not prepared")]
 unconstrained fn transfer_to_private_to_self_transfer_not_prepared() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, _, _, token_id) = utils::setup_and_mint(/* with_account_contracts */ false);
@@ -78,7 +78,7 @@ unconstrained fn transfer_to_private_to_self_transfer_not_prepared() {
     );
 }
 
-#[test(should_fail_with="transfer not prepared")]
+#[test(should_fail_with = "transfer not prepared")]
 unconstrained fn transfer_to_private_finalizing_from_incorrect_sender() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, incorrect_sender, recipient, token_id) = utils::setup_and_mint(/* with_account_contracts */ false);
@@ -106,7 +106,7 @@ unconstrained fn transfer_to_private_finalizing_from_incorrect_sender() {
     );
 }
 
-#[test(should_fail_with="invalid NFT owner")]
+#[test(should_fail_with = "invalid NFT owner")]
 unconstrained fn transfer_to_private_failure_not_an_owner() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, _, not_owner, token_id) = utils::setup_and_mint(/* with_account_contracts */ false);

noir-projects/noir-contracts/contracts/nft_contract/src/test/transfer_to_public.nr

@@ -40,7 +40,7 @@ unconstrained fn transfer_to_public_on_behalf_of_other() {
     utils::assert_owns_public_nft(env, nft_contract_address, recipient, token_id);
 }
 
-#[test(should_fail_with="NFT not found when transferring to public")]
+#[test(should_fail_with = "NFT not found when transferring to public")]
 unconstrained fn transfer_to_public_failure_not_an_owner() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, _, not_owner, token_id) = utils::setup_mint_and_transfer_to_private(/* with_account_contracts */ false);
@@ -49,15 +49,15 @@ unconstrained fn transfer_to_public_failure_not_an_owner() {
     env.call_private_void(NFT::at(nft_contract_address).transfer_to_public(not_owner, not_owner, token_id, 0));
 }
 
-#[test(should_fail_with="invalid nonce")]
+#[test(should_fail_with = "invalid nonce")]
 unconstrained fn transfer_to_public_failure_on_behalf_of_self_non_zero_nonce() {
     // Setup without account contracts. We are not using authwits here, so dummy accounts are enough
     let (env, nft_contract_address, user, _, token_id) = utils::setup_mint_and_transfer_to_private(/* with_account_contracts */ false);
 
     env.call_private_void(NFT::at(nft_contract_address).transfer_to_public(user, user, token_id, random()));
 }
 
-#[test(should_fail_with="Authorization not found for message hash")]
+#[test(should_fail_with = "Authorization not found for message hash")]
 unconstrained fn transfer_to_public_failure_on_behalf_of_other_invalid_designated_caller() {
     let (env, nft_contract_address, sender, recipient, token_id) = utils::setup_mint_and_transfer_to_private(/* with_account_contracts */ true);
 
@@ -69,7 +69,7 @@ unconstrained fn transfer_to_public_failure_on_behalf_of_other_invalid_designate
     env.call_private_void(transfer_to_public_call_interface);
 }
 
-#[test(should_fail_with="Authorization not found for message hash")]
+#[test(should_fail_with = "Authorization not found for message hash")]
 unconstrained fn transfer_to_public_failure_on_behalf_of_other_no_approval() {
     let (env, nft_contract_address, sender, recipient, token_id) = utils::setup_mint_and_transfer_to_private(/* with_account_contracts */ true);
 

noir-projects/noir-contracts/contracts/token_blacklist_contract/src/main.nr

@@ -249,7 +249,6 @@ contract TokenBlacklist {
     }
 
     /// Internal ///
-
     #[public]
     #[internal]
     fn _increase_public_balance(to: AztecAddress, amount: Field) {
@@ -266,7 +265,6 @@ contract TokenBlacklist {
     }
 
     /// Unconstrained ///
-
     unconstrained fn balance_of_private(owner: AztecAddress) -> pub Field {
         storage.balances.balance_of(owner).to_field()
     }

noir-projects/noir-contracts/contracts/token_bridge_contract/src/main.nr

@@ -132,7 +132,6 @@ contract TokenBridge {
         Token::at(token).burn(context.msg_sender(), amount, nonce).call(&mut context);
     }
     /// docs:end:exit_to_l1_private
-
     // docs:start:get_token
     #[public]
     #[view]

noir-projects/noir-contracts/contracts/token_contract/src/test/burn.nr

@@ -112,7 +112,7 @@ unconstrained fn burn_private_on_behalf_of_other() {
     utils::check_private_balance(token_contract_address, owner, mint_amount - burn_amount);
 }
 
-#[test(should_fail_with="Balance too low")]
+#[test(should_fail_with = "Balance too low")]
 unconstrained fn burn_private_failure_more_than_balance() {
     let (env, token_contract_address, owner, _, mint_amount) = utils::setup_and_mint(/* with_account_contracts */ false);
 
@@ -123,7 +123,7 @@ unconstrained fn burn_private_failure_more_than_balance() {
     // Private doesnt revert, so we cannot check balances here since notes have already been nullified. Test is done.
 }
 
-#[test(should_fail_with="invalid nonce")]
+#[test(should_fail_with = "invalid nonce")]
 unconstrained fn burn_private_failure_on_behalf_of_self_non_zero_nonce() {
     let (env, token_contract_address, owner, _, mint_amount) = utils::setup_and_mint(/* with_account_contracts */ false);