PR feedback

Co-authored-by: Mauro Lacy <maurolacy@users.noreply.github.com>
CosmWasm · Dec 8, 2020 · e9a2300 · e9a2300
1 parent 9b92138
commit e9a2300
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/contracts/cw3-flex-multisig/README.md b/contracts/cw3-flex-multisig/README.md
@@ -33,7 +33,7 @@ to modify it's own group, do the following in multiple transactions:
 
   * init cw4-group, with your personal key as admin
   * init a multisig pointing to the group
-  * `AddHook{multisig}` on the group contract 
+  * `AddHook{multisig}` on the group contract
   * `UpdateAdmin{multisig}` on the group contract
 
 This is the current practice to create such circular dependencies,

diff --git a/contracts/cw3-flex-multisig/src/contract.rs b/contracts/cw3-flex-multisig/src/contract.rs
@@ -698,11 +698,12 @@ mod tests {
         );
 
         let proposal = pay_somebody_proposal(&flex_addr);
+        // Only voters can propose
         let res = app.execute_contract(SOMEBODY, &flex_addr, &proposal, &[]);
         assert_eq!(res.unwrap_err(), ContractError::Unauthorized {}.to_string());
 
         // Wrong expiration option fails
-        let msgs = match pay_somebody_proposal(&flex_addr) {
+        let msgs = match proposal.clone() {
             HandleMsg::Propose { msgs, .. } => msgs,
             _ => panic!("Wrong variant"),
         };