[Bugfix] Fix two tests in coin module

[fEst1ck]

We fix two tests in the coin module. In short, the tests failed due to the unconsistenty: we updated the two tests as in framework v1.34, but did not update some of the functions, which the tests depend on, to v1.34 exactly.

1. test_migration_with_existing_primary_fungible_store. We change

   assert!(can_receive_paired_fungible_asset(account_addr, ensure_paired_metadata<FakeMoney>()), 0);
   

   back to

   assert!(!can_receive_paired_fungible_asset(account_addr, ensure_paired_metadata<FakeMoney>()), 0);
   

   as before the framework update, which is the expected behavior since we did not enable NEW_ACCOUNTS_DEFAULT_TO_FA_SUPRA_STORE as in the updated framework.

2. test_migration_coin_store_with_non_coin_type. We change the expected behavior back to

   #[expected_failure(abort_code = 0x10003, location = Self)]
   

   and revert the deleted

   assert!(is_coin_initialized<CoinType>(), error::invalid_argument(ECOIN_INFO_NOT_PUBLISHED));
   

   as before the framework update.
   The test does not abort in v1.34 because the aborting line

   let metadata = ensure_paired_metadata<CoinType>();
   

   is guarded by

   if (is_coin_initialized<CoinType>())
   

[dhaval-supraoracles]

Only have 1 comment.

[dhaval-supraoracles]

So it should fail right?
I thought it should be passed without abort.

[fEst1ck]

Yes. We can update to the new behaviour in 1.34 when we eventually update maybe_convert_to_fungible_store.