simplenft: keep single nft module; demote doc comment to line comment

Author: root28root

escrow/Move.toml

@@ -4,4 +4,8 @@ version = "0.0.1"
 authors = ["Rootmhz"]
 
 [addresses]
+std = "0x1"
 Escrow = "0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd"         
+
+[dependencies]
+MoveStdlib = { local = "../minimarket/vendor/aptos-core/aptos-move/framework/move-stdlib" }

escrow/tests/escrow_tests.move

@@ -1,36 +1,87 @@
-module Escrow::escrow_tests {
-    use Escrow::escrow;
+#[test_only]
+module 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::escrow_tests {
+    use 0x1::signer;
+    use 0x1::vector;
+    use 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::escrow;
 
     const BUYER: address = @0x1;
     const SELLER: address = @0x2;
 
-    #[test]
-    public fun full_success_flow() {
+    // Мусорка, чтобы не было implicit drop для Wallet/Deal
+    struct Trash has key {
+        buyers: vector<escrow::Wallet>,
+        sellers: vector<escrow::Wallet>,
+        deals: vector<escrow::Deal>,
+    }
+
+    public entry fun init_trash(s: &signer) {
+        move_to(s, Trash {
+            buyers: vector::empty<escrow::Wallet>(),
+            sellers: vector::empty<escrow::Wallet>(),
+            deals: vector::empty<escrow::Deal>(),
+        });
+    }
+
+    fun sink_all(s: &signer, bw: escrow::Wallet, sw: escrow::Wallet, d: escrow::Deal) acquires Trash {
+        let t = borrow_global_mut<Trash>(signer::address_of(s));
+        vector::push_back(&mut t.buyers, bw);
+        vector::push_back(&mut t.sellers, sw);
+        vector::push_back(&mut t.deals, d);
+    }
+
+    // === happy path: deposit -> fund -> release ===
+    #[test(admin = @0xAA)]
+    public entry fun test_happy(admin: &signer) acquires Trash {
+        init_trash(admin);
+
         let bw = escrow::new_wallet();
         let sw = escrow::new_wallet();
 
         escrow::deposit(&mut bw, 150);
         let d = escrow::create(BUYER, SELLER, 100);
 
         escrow::fund(BUYER, &mut bw, &mut d);
-        assert!(escrow::balance(&bw) == 50, 0);
-
         escrow::release(SELLER, &mut sw, &mut d);
+
+        assert!(escrow::balance(&bw) == 50, 0);
         assert!(escrow::balance(&sw) == 100, 1);
+
+        sink_all(admin, bw, sw, d);
     }
 
-    #[test]
-    public fun refund_flow() {
+    // Нельзя фондить недостаточную сумму (E_INSUFFICIENT = 4)
+    #[test(admin = @0xAB)]
+    #[expected_failure(abort_code = 4, location = 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::escrow)]
+    public entry fun test_insufficient(admin: &signer) acquires Trash {
+        init_trash(admin);
+
         let bw = escrow::new_wallet();
         let sw = escrow::new_wallet();
 
-        escrow::deposit(&mut bw, 100);
+        escrow::deposit(&mut bw, 50); // меньше, чем нужно
         let d = escrow::create(BUYER, SELLER, 100);
 
+        // упадёт по E_INSUFFICIENT (4)
         escrow::fund(BUYER, &mut bw, &mut d);
-        escrow::refund(BUYER, &mut bw, &mut d);
 
-        assert!(escrow::balance(&bw) == 100, 2);
-        assert!(escrow::balance(&sw) == 0, 3);
+        sink_all(admin, bw, sw, d);
+    }
+
+    // Нельзя фондить не-байеру (E_NOT_BUYER = 5)
+    #[test(admin = @0xAC)]
+    #[expected_failure(abort_code = 5, location = 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::escrow)]
+    public entry fun test_not_buyer(admin: &signer) acquires Trash {
+        init_trash(admin);
+
+        let bw = escrow::new_wallet();
+        let sw = escrow::new_wallet();
+
+        escrow::deposit(&mut bw, 150);
+        let d = escrow::create(BUYER, SELLER, 100);
+
+        // адрес не совпадает с BUYER
+        escrow::fund(@0xDEAD, &mut bw, &mut d);
+
+        sink_all(admin, bw, sw, d);
     }
 }

mytoken/Move.toml

@@ -4,4 +4,8 @@ version = "0.0.1"
 authors = ["Rootmhz"]
 
 [addresses]
+std = "0x1"
 MyToken = "0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd"         
+
+[dependencies]
+MoveStdlib = { local = "../minimarket/vendor/aptos-core/aptos-move/framework/move-stdlib" }

simplenft/sources/simple_nft.move simplenft/sources/nft.movesimplenft/sources/simple_nft.move renamed to simplenft/sources/nft.move

@@ -19,13 +19,12 @@ module SimpleNFT::nft {
         }
     }
 
+    /// Минтим ровно один NFT (#0) для демо
     public fun mint(r: &mut Registry, to: address, name: vector<u8>) {
-        // Разрешаем ровно один минт (#0) для демо
         assert!(!r.has0, 1);
         let id = r.next_id;
         r.next_id = id + 1;
 
-        // Заполняем поля примитивами — никаких ресурсов = нет проблем с drop
         r.id0 = id;
         r.name0 = name;
         r.owner0 = to;
@@ -44,7 +43,7 @@ module SimpleNFT::nft {
         r.owner0
     }
 
-    // «Поглощаем» реестр, чтобы не было implicit drop предупреждений
+    // «Поглощаем» реестр, чтобы не было implicit drop
     fun destroy_registry(r: Registry) {
         let Registry {
             next_id: _,
@@ -55,7 +54,7 @@ module SimpleNFT::nft {
         } = r;
     }
 
-    /// DEMO: создаём реестр, минтим NFT #0 → переводим → проверяем владельца
+    /// DEMO: минт → трансфер → проверка владельца
     public entry fun entry_demo(_s: &signer) {
         let r = new_registry();
         mint(&mut r, @0x1, b"Demo NFT");
@@ -64,4 +63,10 @@ module SimpleNFT::nft {
         assert!(owner == @0x2, 0);
         destroy_registry(r);
     }
+
+// Для юнит-тестов: публичный «съедатель» реестра
+    #[test_only]
+    public fun destroy_for_test(r: Registry) {
+        destroy_registry(r);
+    }
 }

simplenft/tests/simple_nft_tests.move

@@ -1,14 +1,52 @@
-module SimpleNFT::nft_tests {
-    use SimpleNFT::nft;
+#[test_only]
+module 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::simple_nft_tests {
+    use 0xb35962eed27b9a272d82673f2b7a99e7257b7b1a9af02c1a09143dacbaf498bd::nft;
 
     const A1: address = @0x1;
     const A2: address = @0x2;
 
     #[test]
-    public fun mint_and_transfer() {
-        let n = nft::mint(0, A1);
-        assert!(nft::owner_of(&n) == A1, 0);
-        nft::transfer(&mut n, A2);
-        assert!(nft::owner_of(&n) == A2, 1);
+    public entry fun test_mint_transfer_ok() {
+        let r = nft::new_registry();
+
+        nft::mint(&mut r, A1, b"Test NFT");
+        assert!(nft::get_owner(&r, 0) == A1, 0);
+
+        nft::transfer(&mut r, 0, A2);
+        assert!(nft::get_owner(&r, 0) == A2, 1);
+
+        nft::destroy_for_test(r);
+    }
+
+    // Повторный mint должен падать с кодом 1 из SimpleNFT::nft::mint
+    #[test]
+    #[expected_failure(abort_code = 1, location = SimpleNFT::nft)]
+    public entry fun test_mint_twice_fails() {
+        let r = nft::new_registry();
+        nft::mint(&mut r, A1, b"x");
+        // второй mint -> abort(1)
+        nft::mint(&mut r, A1, b"y");
+        // не дойдём, но чтобы удовлетворить типы:
+        nft::destroy_for_test(r);
+    }
+
+    // Неверный id при transfer -> abort(3) в SimpleNFT::nft::transfer
+    #[test]
+    #[expected_failure(abort_code = 3, location = SimpleNFT::nft)]
+    public entry fun test_transfer_wrong_id_fails() {
+        let r = nft::new_registry();
+        nft::mint(&mut r, A1, b"x");
+        nft::transfer(&mut r, /*wrong*/ 999, A2);
+        nft::destroy_for_test(r);
+    }
+
+    // Неверный id при get_owner -> abort(5) в SimpleNFT::nft::get_owner
+    #[test]
+    #[expected_failure(abort_code = 5, location = SimpleNFT::nft)]
+    public entry fun test_get_owner_wrong_id_fails() {
+        let r = nft::new_registry();
+        nft::mint(&mut r, A1, b"x");
+        let _ = nft::get_owner(&r, /*wrong*/ 123);
+        nft::destroy_for_test(r);
     }
 }