suidouble 1.0.4-5 → 1.0.4-7

package/lib/SuiCoin.js

@@ -1,4 +1,4 @@
-const { Commands } = require('@mysten/sui/transactions');
+const { Commands, Transaction } = require('@mysten/sui/transactions');
 const { bcs } = require('@mysten/sui/bcs');
 // console.log(bcs);
 
@@ -102,6 +102,15 @@ class SuiCoin {
         return '0x'+this._coinType;
     }
 
+    /**
+     * Move type for the Coin object of this coin type
+     * 
+     * @type {string}
+     */
+    get coinObjectType() {
+        return '0x2::coin::Coin<'+this.coinType+'>';
+    }
+
     get decimals() {
         if (this.metadata) {
             return this.metadata.decimals;
@@ -202,6 +211,15 @@ class SuiCoin {
         return totalAmount;
     }
 
+    /**
+     * Returns TransactionObjectArgument with Coin of amount to be used in tranasctions
+     * 
+     * @param {import('@mysten/sui/transactions').Transaction} txb - Native SUI SDK Transaction
+     * @param {string} owner - address of the owner
+     * @param {BigInt|string} amount - amount of coin. BigIng or String to be normalized via Coin decimals, "0.05" for 0.05 sui
+     * @param {boolean} addEmptyCoins - attach coins == 0 to the list
+     * @returns {import('@mysten/sui/transactions').TransactionObjectArgument}
+     */
     async coinOfAmountToTxCoin(txb, owner, amount, addEmptyCoins = false) {
         const normalizedAmount = await this.lazyNormalizeAmount(amount);
 
@@ -215,12 +233,6 @@ class SuiCoin {
         if (coinIds.length == 1) {
             // only one coin object enough to cover the expense
             if (this.isSUI()) {
-                // console.log(txb.gas);
-                // console.log(bcs);
-                // console.log(txb.pure(bcs.vector(bcs.u64).serialize([expectedAmountAsBigInt])));
-                // console.log(txb.pure.u64(expectedAmountAsBigInt));
-                // txb.pure(bcs.vector(bcs.u64).serialize(['0x123']))
-
                 const coinInput = txb.add(Commands.SplitCoins(txb.gas, [txb.pure.u64(expectedAmountAsBigInt)]));
                 return coinInput;
             } else {

package/lib/SuiCoins.js

@@ -97,6 +97,12 @@ class SuiCoins extends SuiCommonMethods {
         return coinType;
     }
 
+    /**
+     * Return instance of SuiCoin of specific type
+     * 
+     * @param {string} coinType - MoveType, or 'SUI' as helper
+     * @returns {SuiCoin}
+     */
     get(coinType) {
         const normalizedCoinType = this.normalizeCoinType(coinType); 
         let suiCoin = this._coins[normalizedCoinType];

package/lib/SuiMaster.js

@@ -102,6 +102,11 @@ class SuiMaster extends SuiCommonMethods {
         return SuiUtils;
     }
 
+    /**
+     * Instance of SuiCoins class connected to this SuiMaster
+     * 
+     * @type {SuiCoins}
+     */
     get suiCoins() {
         return this._suiCoins;
     }

package/lib/SuiPackageModule.js

@@ -7,8 +7,18 @@ const SuiEvent = require('./SuiEvent.js');
 
 const { Transaction } = require('@mysten/sui/transactions');
 const { normalizeSuiAddress } = require('@mysten/sui/utils');
+const SuiMaster = require('./SuiMaster.js');
+const SuiPackage = require('./SuiPackage.js');
 
 class SuiPackageModule extends SuiCommonMethods {
+    
+    /**
+     * SuiPackageModule constructor
+     * @param {Object} params - Initialization parameters
+     * @param {SuiPackage} params.package - instance of SuiPackage this module is part of
+     * @param {SuiMaster} params.suiMaster - instance of SuiMaster
+     * @param {string} params.moduleName - name of the Move module
+     */
     constructor(params = {}) {
         super(params);
 
@@ -16,6 +26,7 @@ class SuiPackageModule extends SuiCommonMethods {
         if (!this._package) {
             throw new Error('package is required for SuiPackageModule');
         }
+
         this._suiMaster = params.suiMaster;
         if (!this._suiMaster) {
             throw new Error('suiMaster is requried for SuiPackageModule');
@@ -25,9 +36,6 @@ class SuiPackageModule extends SuiCommonMethods {
             throw new Error('moduleName is required for SuiPackageModule');
         }
 
-        // this._objects = {};
-        // this._objectsArray = [];
-
         // we need to get very first version's address of this package to use for types, so we are doing this in separate call
         this._checkedOnChain = false;
         this._normalizedMoveModule = {};
@@ -154,10 +162,10 @@ class SuiPackageModule extends SuiCommonMethods {
                     // vector<Coin<SUI>>
                     const ownerAddress = this._suiMaster.address;
 
-                    const suiCoin = await this._suiMaster.suiCoins.get(param.type);
-                    const txCoinToSend = await suiCoin.coinOfAmountToTxCoin(tx, ownerAddress, param.amount);
+                    const suiCoin = await this._suiMaster.suiCoins.get(param[0].type);
+                    const txCoinToSend = await suiCoin.coinOfAmountToTxCoin(tx, ownerAddress, param[0].amount);
 
-                    callArgs.push(tx.makeMoveVec({ objects: [txCoinToSend]}));
+                    callArgs.push(tx.makeMoveVec({ type: suiCoin.coinObjectType, elements: [txCoinToSend]}));
                 } else {
                     callArgs.push(tx.pure(param));
                 }

package/lib/SuiUtils.js

@@ -128,7 +128,7 @@ class SuiUtils extends SuiCommonMethods {
                 client = SuiUtils.suiClientFor('mainnet');
                 providerName = 'sui:mainnet';
             } else {
-                if (clientParam && clientParam.constructor && clientParam.constructor.name && clientParam.constructor.name == 'SuiClient') {
+                if (clientParam && clientParam.constructor && (clientParam.endpoint || clientParam.transport)) {
                     client = clientParam;
                     let url = '';
                     if (clientParam.endpoint) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "suidouble",
-  "version": "1.0.4-5",
+  "version": "1.0.4-7",
   "description": "Set of provider, package and object classes for javascript representation of Sui Move smart contracts. Use same code for publishing, upgrading, integration testing, interaction with smart contracts and integration in browser web3 dapps",
   "main": "index.js",
   "scripts": {

package/test/sui_master_onlocal.test.js

@@ -346,9 +346,7 @@ test('testing move call with coins', async t => {
 
     t.ok(foundChatTopMessage);
     t.ok(foundChatResponse);
-    
-    // messageTextAsBytes = [].slice.call(new TextEncoder().encode(messageText)); // regular array with utf data
-    // suidouble_chat contract store text a bytes (easier to work with unicode things), let's convert it back to js string
+
     foundText = new TextDecoder().decode(new Uint8Array(foundText));
 
     t.equal(foundText, longMessageYouCanNotPostForFree);
@@ -358,6 +356,19 @@ test('testing move call with coins', async t => {
     t.ok( balanceNow <= (balanceWas - 400000000000n) );
 });
 
+test('testing move call with vector<Coin<..>>', async t => {
+    const balanceWas = await suiMaster.getBalance();
+    const longMessageYouCanNotPostForFree = ('message ').padEnd(500, 'test');
+
+    // you can pass vector of coin, wrapping it's definition in array
+    const moveCallResult = await contract.moveCall('suidouble_chat', 'post_pay_with_coin_vector', [chatShopObjectId, [{type: 'SUI', amount: 400000000000n}], contract.arg('string', longMessageYouCanNotPostForFree), contract.arg('string', 'metadata')]);
+    // it's the wrapper over the same move function we've already tested, so lets keep the unit simple:
+    t.ok(moveCallResult.created.length > 0);
+
+    const balanceNow = await suiMaster.getBalance();
+    t.ok( balanceNow <= (balanceWas - 400000000000n) ); // vector<Coin<SUI>> paid
+});
+
 test('testing move call deleting object', async t => {
 
     console.error('chatResponseToDelete', chatResponseToDelete);

package/test/test_move_contracts/suidouble_chat/sources/suidouble_chat.move

@@ -3,7 +3,7 @@ module suidouble_chat::suidouble_chat {
     use sui::object::{Self, UID, ID};
     use sui::transfer;
     use sui::tx_context::{Self, TxContext};
-    use std::vector::length;
+    use std::vector::{Self, length};
 
     use sui::dynamic_object_field::{Self};
 
@@ -12,6 +12,7 @@ module suidouble_chat::suidouble_chat {
     use sui::balance::{Self, Balance};
     
     use std::debug;
+    use sui::pay;
 
     use sui::event::emit;
 
@@ -168,6 +169,17 @@ module suidouble_chat::suidouble_chat {
         transfer::share_object(chat_top_message);
     }
 
+    public entry fun post_pay_with_coin_vector(
+        chat_shop: &mut ChatShop,
+        coins: vector<Coin<SUI>>,
+        text: vector<u8>,
+        metadata: vector<u8>,
+        ctx: &mut TxContext
+    ) {
+        let base = vector::pop_back(&mut coins);
+        pay::join_vec(&mut base, coins);
+        post_pay(chat_shop, base, text, metadata, ctx);
+    }
 
     /// Mint (post) a ChatResponse object 
     public entry fun reply(