feat(NFT): add events to standard

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Changelog
 
 ## [unreleased]
+- Added event logs to `near-contract-standards`. Currently only NFT related events. [PR 627](https://github.com/near/near-sdk-rs/pull/627)
 
 ## `4.0.0-pre.6` [01-21-2021]
 

examples/non-fungible-token/test-token-receiver/Cargo.toml

@@ -10,4 +10,3 @@ crate-type = ["cdylib", "rlib"]
 [dependencies]
 near-sdk = { path = "../../../near-sdk" }
 near-contract-standards = { path = "../../../near-contract-standards" }
-

examples/non-fungible-token/test-token-receiver/src/lib.rs

@@ -73,13 +73,8 @@ impl NonFungibleTokenReceiver for TokenReceiver {
             "return-it-later" => {
                 let prepaid_gas = env::prepaid_gas();
                 let account_id = env::current_account_id();
-                ext_self::ok_go(
-                    true,
-                    account_id,
-                    NO_DEPOSIT,
-                    prepaid_gas - GAS_FOR_NFT_ON_TRANSFER,
-                )
-                .into()
+                ext_self::ok_go(true, account_id, NO_DEPOSIT, prepaid_gas - GAS_FOR_NFT_ON_TRANSFER)
+                    .into()
             }
             "keep-it-now" => PromiseOrValue::Value(false),
             "keep-it-later" => {

examples/non-fungible-token/tests/sim/test_approval.rs

@@ -22,14 +22,12 @@ fn simulate_simple_approve() {
 
     // check nft_is_approved, with approval_id=1
     let alice_approval_id_is_1: bool =
-        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(1)))
-            .unwrap_json();
+        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(1))).unwrap_json();
     assert!(alice_approval_id_is_1);
 
     // check nft_is_approved, with approval_id=2
     let alice_approval_id_is_2: bool =
-        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(2)))
-            .unwrap_json();
+        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(2))).unwrap_json();
     assert!(!alice_approval_id_is_2);
 
     // alternatively, one could check the data returned by nft_token
@@ -43,8 +41,7 @@ fn simulate_simple_approve() {
         .assert_success();
 
     let alice_approval_id_is_2: bool =
-        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(2)))
-            .unwrap_json();
+        view!(nft.nft_is_approved(TOKEN_ID.into(), alice.account_id(), Some(2))).unwrap_json();
     assert!(alice_approval_id_is_2);
 
     // approving another account gives different approval_id
@@ -145,8 +142,7 @@ fn simulate_revoke() {
     .assert_success();
 
     // root revokes alice
-    call!(root, nft.nft_revoke(TOKEN_ID.into(), alice.account_id()), deposit = 1)
-        .assert_success();
+    call!(root, nft.nft_revoke(TOKEN_ID.into(), alice.account_id()), deposit = 1).assert_success();
 
     // alice is revoked...
     let alice_approved: bool =

examples/non-fungible-token/tests/sim/test_core.rs

@@ -1,5 +1,6 @@
 use crate::utils::{init, TOKEN_ID};
 use near_contract_standards::non_fungible_token::Token;
+use near_sdk::serde_json::{self, json};
 use near_sdk_sim::{call, view};
 
 #[test]
@@ -124,3 +125,120 @@ fn simulate_transfer_call_receiver_panics() {
     let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
     assert_eq!(token.owner_id, root.account_id());
 }
+
+#[test]
+fn simulate_transfer_call_receiver_panics_and_nft_resolve_transfer_produces_log() {
+    let (root, nft, _, receiver, _) = init();
+    let args = json!({
+      "receiver_id": receiver.account_id(),
+      "token_id": TOKEN_ID,
+      "memo": Some("transfer & call"),
+      "msg": "incorrect message"
+
+    });
+    let args = serde_json::to_vec(&args).unwrap();
+    let res = root
+        .create_transaction(nft.account_id())
+        .function_call("nft_transfer_call".to_string(), args, 35_000_000_000_000 + 1, 1)
+        .submit();
+
+    // Prints final log
+    assert_eq!(res.logs().len(), 1);
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, root.account_id());
+}
+
+#[test]
+fn simulate_transfer_call_receiver_panics_and_nft_resolve_transfer_produces_no_log_if_not_enough_gas() {
+    let (root, nft, _, receiver, _) = init();
+    let args = json!({
+      "receiver_id": receiver.account_id(),
+      "token_id": TOKEN_ID,
+      "memo": Some("transfer & call"),
+      "msg": "incorrect message"
+
+    });
+    let args = serde_json::to_vec(&args).unwrap();
+    let res = root
+        .create_transaction(nft.account_id())
+        .function_call("nft_transfer_call".to_string(), args, 35_000_000_000_000, 1)
+        .submit();
+
+    // Prints final log
+    assert_eq!(res.logs().len(), 0);
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, root.account_id());
+}
+
+#[test]
+fn simulate_transfer_call_no_extra_log() {
+    let (root, nft, _, receiver, _) = init();
+
+    let args = json!({
+      "receiver_id": receiver.account_id(),
+      "token_id": TOKEN_ID,
+      "memo": Some("transfer & call"),
+      "msg": "keep-it-now"
+
+    });
+    let args = serde_json::to_vec(&args).unwrap();
+    let res = root
+        .create_transaction(nft.account_id())
+        .function_call("nft_transfer_call".to_string(), args, 200_000_000_000_000, 1)
+        .submit();
+
+    assert_eq!(res.logs().len(), 0);
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, receiver.account_id());
+}
+
+#[test]
+fn simulate_simple_transfer_logs() {
+    let (root, nft, alice, _, _) = init();
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, root.account_id());
+
+    let args = json!({
+      "receiver_id": alice.account_id(),
+      "token_id": TOKEN_ID,
+      "memo": Some("simple transfer"),
+    });
+    let args = serde_json::to_vec(&args).unwrap();
+    let res = root
+        .create_transaction(nft.account_id())
+        .function_call("nft_transfer".to_string(), args, 200_000_000_000_000, 1)
+        .submit();
+
+    assert_eq!(res.logs().len(), 1);
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, alice.account_id());
+}
+
+#[test]
+fn simulate_simple_transfer_no_logs_on_failure() {
+    let (root, nft, _, _, _) = init();
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, root.account_id());
+
+    let args = json!({
+      // transfer to the current owner should fail and not print log
+      "receiver_id": root.account_id(),
+      "token_id": TOKEN_ID,
+      "memo": Some("simple transfer"),
+    });
+    let args = serde_json::to_vec(&args).unwrap();
+    let res = root
+        .create_transaction(nft.account_id())
+        .function_call("nft_transfer".to_string(), args, 200_000_000_000_000, 1)
+        .submit();
+
+    assert_eq!(res.logs().len(), 0);
+
+    let token: Token = view!(nft.nft_token(TOKEN_ID.into())).unwrap_json();
+    assert_eq!(token.owner_id, root.account_id());
+}