fix sui cli split-coin by amounts

crates/sui-faucet/src/faucet/simple_faucet.rs

@@ -567,7 +567,7 @@ mod tests {
             amounts: Some(vec![tiny_value + TRANSFER_SUI_GAS]),
             gas_budget: 50000,
             gas: None,
-            count: 0,
+            count: None,
         }
         .execute(&mut context)
         .await

crates/sui-open-rpc/src/generate_json_rpc_spec.rs

@@ -448,7 +448,7 @@ async fn create_coin_split_response(
     let result = SuiClientCommands::SplitCoin {
         coin_id: coins.first().unwrap().object_id,
         amounts: Some(vec![20, 20, 20, 20, 20]),
-        count: 0,
+        count: None,
         gas: None,
         gas_budget: 1000,
     }

crates/sui/src/client_commands.rs

@@ -250,7 +250,7 @@ pub enum SuiClientCommands {
         amounts: Option<Vec<u64>>,
         /// Count of equal-size coins to split into
         #[clap(long)]
-        count: u64,
+        count: Option<u64>,
         /// ID of the gas object for gas payment, in 20 bytes Hex string
         /// If not provided, a gas object with at least gas_budget value will be selected
         #[clap(long)]
@@ -511,21 +511,27 @@ impl SuiClientCommands {
                 gas_budget,
             } => {
                 let signer = context.get_object_owner(&coin_id).await?;
-                let data = if let Some(amounts) = amounts {
-                    context
-                        .client
-                        .transaction_builder()
-                        .split_coin(signer, coin_id, amounts, gas, gas_budget)
-                        .await?
-                } else {
-                    if count == 0 {
-                        return Err(anyhow!("Coin split count must be greater than 0"));
+                let data = match (amounts, count) {
+                    (Some(amounts), None) => {
+                        context
+                            .client
+                            .transaction_builder()
+                            .split_coin(signer, coin_id, amounts, gas, gas_budget)
+                            .await?
+                    }
+                    (None, Some(count)) => {
+                        if count == 0 {
+                            return Err(anyhow!("Coin split count must be greater than 0"));
+                        }
+                        context
+                            .client
+                            .transaction_builder()
+                            .split_coin_equal(signer, coin_id, count, gas, gas_budget)
+                            .await?
+                    }
+                    _ => {
+                        return Err(anyhow!("Exactly one of `count` and `amounts` must be present for split-coin command."));
                     }
-                    context
-                        .client
-                        .transaction_builder()
-                        .split_coin_equal(signer, coin_id, count, gas, gas_budget)
-                        .await?
                 };
                 let signature = context.config.keystore.sign(&signer, &data.to_bytes())?;
                 let response = context

crates/sui/src/unit_tests/cli_tests.rs

@@ -1017,7 +1017,7 @@ async fn test_split_coin() -> Result<(), anyhow::Error> {
         gas_budget: 1000,
         coin_id: coin,
         amounts: Some(vec![1000, 10]),
-        count: 0,
+        count: None,
     }
     .execute(context)
     .await?;
@@ -1073,7 +1073,7 @@ async fn test_split_coin() -> Result<(), anyhow::Error> {
         gas_budget: 1000,
         coin_id: coin,
         amounts: None,
-        count: 3,
+        count: Some(3),
     }
     .execute(context)
     .await?;
@@ -1132,7 +1132,7 @@ async fn test_split_coin() -> Result<(), anyhow::Error> {
         gas_budget: 1000,
         coin_id: coin,
         amounts: Some(vec![1000, 10]),
-        count: 0,
+        count: None,
     }
     .execute(context)
     .await?;

crates/sui/tests/full_node_tests.rs

@@ -359,7 +359,7 @@ async fn test_full_node_sync_flood() -> Result<(), anyhow::Error> {
                     let context = &mut context.lock().await;
                     SuiClientCommands::SplitCoin {
                         amounts: Some(vec![1]),
-                        count: 0,
+                        count: None,
                         coin_id: object_to_split.0,
                         gas: Some(gas_object_id),
                         gas_budget: 50000,

crates/test-utils/src/transaction.rs

@@ -329,7 +329,7 @@ pub async fn split_coin_with_wallet_context(context: &mut WalletContext, coin_id
     SuiClientCommands::SplitCoin {
         coin_id,
         amounts: None,
-        count: 2,
+        count: Some(2),
         gas: None,
         gas_budget: MAX_GAS,
     }