[codex] Reject zero standard stableswap convergence threshold

contracts/stableswap.clar

@@ -1005,6 +1005,7 @@
         )
         ;; Assert that tx-sender is an admin using is-some & index-of with the admins var
         (asserts! (is-some (index-of current-admins tx-sender)) (err "err-not-admin"))
+        (asserts! (> new-convergence-threshold u0) (err "err-invalid-convergence-threshold"))
 
         (ok (var-set convergence-threshold new-convergence-threshold))
     )
@@ -1032,4 +1033,3 @@
         (ok staking-contract)
     )
 )
-

tests/stableswap_test.ts

@@ -1675,4 +1675,36 @@ Clarinet.test({
         block.receipts[0].result.expectErr()
         console.log(JSON.stringify(block.receipts));
     },
-});
+});
+
+// Test convergence threshold admin controls
+Clarinet.test({
+    name: "Ensure admins cannot set convergence threshold to zero",
+    async fn(chain: Chain, accounts: Map<string, Account>) {
+        const deployer = accounts.get("deployer")!;
+        const wallet_1 = accounts.get("wallet_1")!;
+
+        const block = chain.mineBlock([
+            Tx.contractCall("stableswap", "change-convergence-threshold", [types.uint(0)], deployer.address)
+        ]);
+
+        block.receipts[0].result.expectErr()
+        console.log(JSON.stringify(block.receipts));
+    },
+});
+
+// Test convergence threshold admin controls
+Clarinet.test({
+    name: "Ensure admins can set convergence threshold above zero",
+    async fn(chain: Chain, accounts: Map<string, Account>) {
+        const deployer = accounts.get("deployer")!;
+        const wallet_1 = accounts.get("wallet_1")!;
+
+        const block = chain.mineBlock([
+            Tx.contractCall("stableswap", "change-convergence-threshold", [types.uint(1)], deployer.address)
+        ]);
+
+        block.receipts[0].result.expectOk()
+        console.log(JSON.stringify(block.receipts));
+    },
+});