Merge pull request #2160 from CosmWasm/add-message_info

Add message_info and deprecate mock_info

Author: webmaster128

CHANGELOG.md

@@ -80,6 +80,7 @@ and this project adheres to
 - cosmwasm-vm: Update wasmer to 4.3.1 ([#2147], [#2153])
 - cosmwasm-vm: Rebalance gas costs for cryptographic functions and wasm
   instructions. ([#2152])
+- cosmwasm-std: Add message_info and deprecate mock_info ([#2160])
 
 [#2044]: https://github.com/CosmWasm/cosmwasm/pull/2044
 [#2051]: https://github.com/CosmWasm/cosmwasm/pull/2051
@@ -92,6 +93,7 @@ and this project adheres to
 [#2147]: https://github.com/CosmWasm/cosmwasm/pull/2147
 [#2152]: https://github.com/CosmWasm/cosmwasm/pull/2152
 [#2153]: https://github.com/CosmWasm/cosmwasm/pull/2153
+[#2160]: https://github.com/CosmWasm/cosmwasm/pull/2160
 
 ## [2.0.1] - 2024-04-03
 

MIGRATING.md

@@ -4,7 +4,7 @@ This guide explains what is needed to upgrade contracts when migrating over
 major releases of `cosmwasm`. Note that you can also view the
 [complete CHANGELOG](./CHANGELOG.md) to understand the differences.
 
-## 1.5.x -> 2.0.0
+## 1.5.x -> 2.0.x
 
 - Update `cosmwasm-*` dependencies in Cargo.toml (skip the ones you don't use):
 
@@ -262,6 +262,10 @@ major releases of `cosmwasm`. Note that you can also view the
   in 2.0. To keep the CosmWasm 1.x behaviour, just set payload to
   `Binary::default()`.
 
+- In test code, replace calls to `mock_info` with `message_info`. This takes a
+  `&Addr` as the first argument which you get by using owned `Addr` in the test
+  bodies.
+
 ## 1.4.x -> 1.5.0
 
 - Update `cosmwasm-*` dependencies in Cargo.toml (skip the ones you don't use):

contracts/burner/src/contract.rs

@@ -83,7 +83,7 @@ fn cleanup(storage: &mut dyn Storage, mut limit: usize) -> usize {
 mod tests {
     use super::*;
     use cosmwasm_std::testing::{
-        mock_dependencies, mock_dependencies_with_balance, mock_env, mock_info,
+        message_info, mock_dependencies, mock_dependencies_with_balance, mock_env,
     };
     use cosmwasm_std::{coins, Attribute, StdError, Storage, SubMsg};
 
@@ -102,8 +102,10 @@ mod tests {
     fn instantiate_fails() {
         let mut deps = mock_dependencies();
 
+        let creator = deps.api.addr_make("creator");
+
         let msg = InstantiateMsg {};
-        let info = mock_info("creator", &coins(1000, "earth"));
+        let info = message_info(&creator, &coins(1000, "earth"));
         // we can just call .unwrap() to assert this was a success
         let res = instantiate(deps.as_mut(), mock_env(), info, msg);
         match res.unwrap_err() {
@@ -164,6 +166,8 @@ mod tests {
     fn execute_cleans_up_data() {
         let mut deps = mock_dependencies_with_balance(&coins(123456, "gold"));
 
+        let anon = deps.api.addr_make("anon");
+
         // store some sample data
         deps.storage.set(b"foo", b"bar");
         deps.storage.set(b"key2", b"data2");
@@ -179,7 +183,7 @@ mod tests {
         let res = execute(
             deps.as_mut(),
             mock_env(),
-            mock_info("anon", &[]),
+            message_info(&anon, &[]),
             ExecuteMsg::Cleanup { limit: Some(2) },
         )
         .unwrap();
@@ -192,7 +196,7 @@ mod tests {
         let res = execute(
             deps.as_mut(),
             mock_env(),
-            mock_info("anon", &[]),
+            message_info(&anon, &[]),
             ExecuteMsg::Cleanup { limit: Some(2) },
         )
         .unwrap();

contracts/crypto-verify/src/contract.rs

@@ -324,7 +324,7 @@ pub fn query_verify_bls12_pairing_g2(
 mod tests {
     use super::*;
     use cosmwasm_std::testing::{
-        mock_dependencies, mock_env, mock_info, MockApi, MockQuerier, MockStorage,
+        message_info, mock_dependencies, mock_env, MockApi, MockQuerier, MockStorage,
     };
     use cosmwasm_std::{from_json, OwnedDeps, RecoverPubkeyError, VerificationError};
     use hex_literal::hex;
@@ -354,8 +354,9 @@ mod tests {
 
     fn setup() -> OwnedDeps<MockStorage, MockApi, MockQuerier> {
         let mut deps = mock_dependencies();
+        let creator = deps.api.addr_make(CREATOR);
         let msg = InstantiateMsg {};
-        let info = mock_info(CREATOR, &[]);
+        let info = message_info(&creator, &[]);
         let res = instantiate(deps.as_mut(), mock_env(), info, msg).unwrap();
         assert_eq!(0, res.messages.len());
         deps

contracts/cyberpunk/src/contract.rs

@@ -227,14 +227,15 @@ fn query_denom(deps: Deps, denom: String) -> StdResult<DenomMetadata> {
 mod tests {
     use super::*;
     use cosmwasm_std::testing::{
-        mock_dependencies, mock_env, mock_info, MockApi, MockQuerier, MockStorage,
+        message_info, mock_dependencies, mock_env, MockApi, MockQuerier, MockStorage,
     };
     use cosmwasm_std::{from_json, DenomMetadata, DenomUnit, OwnedDeps};
 
     fn setup() -> OwnedDeps<MockStorage, MockApi, MockQuerier> {
         let mut deps = mock_dependencies();
+        let creator = deps.api.addr_make("creator");
         let msg = Empty {};
-        let info = mock_info("creator", &[]);
+        let info = message_info(&creator, &[]);
         let res = instantiate(deps.as_mut(), mock_env(), info, msg).unwrap();
         assert_eq!(0, res.messages.len());
         deps
@@ -248,9 +249,10 @@ mod tests {
     #[test]
     fn debug_works() {
         let mut deps = setup();
+        let caller = deps.api.addr_make("caller");
 
         let msg = ExecuteMsg::Debug {};
-        execute(deps.as_mut(), mock_env(), mock_info("caller", &[]), msg).unwrap();
+        execute(deps.as_mut(), mock_env(), message_info(&caller, &[]), msg).unwrap();
     }
 
     #[test]