Update api beta test guide: add 'editoffer'

apitest/docs/api-beta-test-guide.md

@@ -408,8 +408,118 @@ The offer will be removed from other Bisq users' offer views, and paid transacti
 
 ### Editing an Existing Offer
 
-Editing existing offers is not yet supported.  You can cancel and re-create an offer, but paid transaction fees
-for the canceled offer will be forfeited.
+Offers you create can be edited in various ways:
+
+- Disable or re-enable an offer.
+- Change an offer's price model and disable (or re-enable) it.
+- Change a market price margin based offer to a fixed price offer.
+- Change a market price margin based offer's price margin.
+- Change, set, or remove a trigger price on a market price margin based offer.
+- Change a market price margin based offer's price margin and trigger price.
+- Change a market price margin based offer's price margin and remove its trigger price.
+- Change a fixed price offer to a market price margin based offer.
+- Change a fixed price offer's fixed price.
+
+_Note: the API does not support editing an offer's payment account._
+
+The subsections below contain examples related to specific use cases.
+
+#### Enable and Disable Offer
+
+Existing offers you create can be disabled (removed from offer book) and re-enabled (re-published to offer book).
+
+To disable an offer:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --enable=false
+```
+
+To enable an offer:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --enable=true
+```
+
+#### Change Offer Pricing Model
+The `editoffer` command can be used to change an existing market price margin based offer to a fixed price offer,
+and vice-versa.
+
+##### Change Market Price Margin Based to Fixed Price Offer
+Suppose you used `createoffer` to create a market price margin based offer as follows:
+```
+$ ./bisq-cli --password=xyz --port=9998 createoffer \
+    --payment-account=f3c1ec8b-9761-458d-b13d-9039c6892413 \
+    --direction=SELL \
+    --currency-code=JPY \
+    --amount=0.125 \
+    --market-price-margin=0.5 \
+    --security-deposit=15.0 \
+    --fee-currency=BSQ
+```
+To change the market price margin based offer to a fixed price offer:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --fixed-price=3960000.5555
+```
+
+##### Change Fixed Price Offer to Market Price Margin Based Offer
+Suppose you used `createoffer` to create a fixed price offer as follows:
+```
+$ ./bisq-cli --password=xyz --port=9998 createoffer \
+    --payment-account=f3c1ec8b-9761-458d-b13d-9039c6892413 \
+    --direction=SELL \
+    --currency-code=JPY \
+    --amount=0.125 \
+    --fixed-price=3960000.0000 \
+    --security-deposit=15.0 \
+    --fee-currency=BSQ
+```
+To change the fixed price offer to a market price margin based offer:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --market-price-margin=0.5
+```
+Alternatively, you can also set a trigger price on the re-published, market price margin based offer.
+A trigger price on a SELL offer causes the offer to be automatically disabled when the market price
+falls below the trigger price.  In the `editoffer` example below, the SELL offer will be disabled when
+the JPY market price falls below 3960000.0000.
+
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --market-price-margin=0.5 \
+    --trigger-price=3960000.0000
+```
+On a BUY offer, a trigger price causes the BUY offer to be automatically disabled when the market price
+rises above the trigger price.
+
+_Note: Disabled offers never automatically re-enable; they can only be manually re-enabled via
+`editoffer --offer-id=<id> --enable=true`._
+
+#### Remove Trigger Price
+To remove a trigger price on a market price margin based offer, set the trigger price to 0:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --trigger-price=0
+```
+
+#### Change Disabled Offer's Pricing Model and Enable It
+You can use `editoffer` to simultaneously change an offer's price details and disable or re-enable it.
+
+Suppose you have a disabled, fixed price offer, and want to change it to a market price margin based offer, set
+a trigger price, and re-enable it:
+```
+./bisq-cli --password=xyz --port=9998 editoffer \
+    --offer-id=83e8b2e2-51b6-4f39-a748-3ebd29c22aea \
+    --market-price-margin=0.5 \
+    --trigger-price=3960000.0000 \
+    --enable=true
+```
 
 ### Taking Offers
 

apitest/src/test/java/bisq/apitest/method/offer/AbstractOfferTest.java

@@ -17,14 +17,13 @@
 
 package bisq.apitest.method.offer;
 
-import bisq.core.monetary.Altcoin;
-
 import protobuf.PaymentAccount;
 
-import org.bitcoinj.utils.Fiat;
-
 import java.math.BigDecimal;
 
+import java.util.function.BiFunction;
+import java.util.function.Function;
+
 import lombok.Setter;
 import lombok.extern.slf4j.Slf4j;
 
@@ -37,10 +36,7 @@
 import static bisq.apitest.config.BisqAppConfig.arbdaemon;
 import static bisq.apitest.config.BisqAppConfig.bobdaemon;
 import static bisq.apitest.config.BisqAppConfig.seednode;
-import static bisq.common.util.MathUtils.roundDouble;
-import static bisq.common.util.MathUtils.scaleDownByPowerOf10;
-import static bisq.core.locale.CurrencyUtil.isCryptoCurrency;
-import static java.math.RoundingMode.HALF_UP;
+import static bisq.common.util.MathUtils.exactMultiply;
 
 
 
@@ -49,6 +45,10 @@
 @Slf4j
 public abstract class AbstractOfferTest extends MethodTest {
 
+    protected static final int ACTIVATE_OFFER = 1;
+    protected static final int DEACTIVATE_OFFER = 0;
+    protected static final long NO_TRIGGER_PRICE = 0;
+
     @Setter
     protected static boolean isLongRunningTest;
 
@@ -67,6 +67,35 @@ public static void setUp() {
     }
 
 
+    // Mkt Price Margin value of offer returned from server is scaled down by 10^-2.
+    protected final Function<Double, Double> scaledDownMktPriceMargin = (mktPriceMargin) ->
+            exactMultiply(mktPriceMargin, 0.01);
+
+    // Price value of fiat offer returned from server will be scaled up by 10^4.
+    protected final Function<BigDecimal, Long> scaledUpFiatOfferPrice = (price) -> {
+        BigDecimal factor = new BigDecimal(10).pow(4);
+        return price.multiply(factor).longValue();
+    };
+
+    // Price value of altcoin offer returned from server will be scaled up by 10^8.
+    protected final Function<String, Long> scaledUpAltcoinOfferPrice = (altcoinPriceAsString) -> {
+        BigDecimal factor = new BigDecimal(10).pow(8);
+        BigDecimal priceAsBigDecimal = new BigDecimal(altcoinPriceAsString);
+        return priceAsBigDecimal.multiply(factor).longValue();
+    };
+
+    protected final BiFunction<Double, Double, Long> calcPriceAsLong = (base, delta) -> {
+        var priceAsDouble = new BigDecimal(base).add(new BigDecimal(delta)).doubleValue();
+        return Double.valueOf(exactMultiply(priceAsDouble, 10_000)).longValue();
+    };
+
+    protected final BiFunction<Double, Double, String> calcPriceAsString = (base, delta) -> {
+        var priceAsBigDecimal = new BigDecimal(Double.toString(base))
+                .add(new BigDecimal(Double.toString(delta)));
+        return priceAsBigDecimal.toPlainString();
+    };
+
+    @SuppressWarnings("ConstantConditions")
     public static void createBsqPaymentAccounts() {
         alicesBsqAcct = aliceClient.createCryptoCurrencyPaymentAccount("Alice's BSQ Account",
                 BSQ,
@@ -78,17 +107,6 @@ public static void createBsqPaymentAccounts() {
                 false);
     }
 
-    protected double getScaledOfferPrice(double offerPrice, String currencyCode) {
-        int precision = isCryptoCurrency(currencyCode) ? Altcoin.SMALLEST_UNIT_EXPONENT : Fiat.SMALLEST_UNIT_EXPONENT;
-        return scaleDownByPowerOf10(offerPrice, precision);
-    }
-
-    protected final double getPercentageDifference(double price1, double price2) {
-        return BigDecimal.valueOf(roundDouble((1 - (price1 / price2)), 5))
-                .setScale(4, HALF_UP)
-                .doubleValue();
-    }
-
     @AfterAll
     public static void tearDown() {
         tearDownScaffold();

apitest/src/test/java/bisq/apitest/method/offer/CancelOfferTest.java

@@ -54,7 +54,8 @@ public class CancelOfferTest extends AbstractOfferTest {
                 0.00,
                 getDefaultBuyerSecurityDepositAsPercent(),
                 paymentAccountId,
-                BSQ);
+                BSQ,
+                NO_TRIGGER_PRICE);
     };
 
     @Test

apitest/src/test/java/bisq/apitest/method/offer/CreateOfferUsingMarketPriceMarginTest.java

@@ -17,12 +17,18 @@
 
 package bisq.apitest.method.offer;
 
+import bisq.core.monetary.Altcoin;
+import bisq.core.monetary.Price;
 import bisq.core.payment.PaymentAccount;
 
 import bisq.proto.grpc.OfferInfo;
 
+import org.bitcoinj.utils.Fiat;
+
 import java.text.DecimalFormat;
 
+import java.math.BigDecimal;
+
 import lombok.extern.slf4j.Slf4j;
 
 import org.junit.jupiter.api.Disabled;
@@ -33,18 +39,22 @@
 
 import static bisq.apitest.config.ApiTestConfig.BTC;
 import static bisq.cli.TableFormat.formatOfferTable;
+import static bisq.common.util.MathUtils.roundDouble;
 import static bisq.common.util.MathUtils.scaleDownByPowerOf10;
 import static bisq.common.util.MathUtils.scaleUpByPowerOf10;
 import static bisq.core.btc.wallet.Restrictions.getDefaultBuyerSecurityDepositAsPercent;
+import static bisq.core.locale.CurrencyUtil.isCryptoCurrency;
 import static java.lang.Math.abs;
 import static java.lang.String.format;
+import static java.math.RoundingMode.HALF_UP;
 import static java.util.Collections.singletonList;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static protobuf.OfferPayload.Direction.BUY;
 import static protobuf.OfferPayload.Direction.SELL;
 
+@SuppressWarnings("ConstantConditions")
 @Disabled
 @Slf4j
 @TestMethodOrder(MethodOrderer.OrderAnnotation.class)
@@ -68,7 +78,8 @@ public void testCreateUSDBTCBuyOffer5PctPriceMargin() {
                 priceMarginPctInput,
                 getDefaultBuyerSecurityDepositAsPercent(),
                 usdAccount.getId(),
-                MAKER_FEE_CURRENCY_CODE);
+                MAKER_FEE_CURRENCY_CODE,
+                NO_TRIGGER_PRICE);
         log.info("OFFER #1:\n{}", formatOfferTable(singletonList(newOffer), "usd"));
         String newOfferId = newOffer.getId();
         assertNotEquals("", newOfferId);
@@ -109,7 +120,8 @@ public void testCreateNZDBTCBuyOfferMinus2PctPriceMargin() {
                 priceMarginPctInput,
                 getDefaultBuyerSecurityDepositAsPercent(),
                 nzdAccount.getId(),
-                MAKER_FEE_CURRENCY_CODE);
+                MAKER_FEE_CURRENCY_CODE,
+                NO_TRIGGER_PRICE);
         log.info("OFFER #2:\n{}", formatOfferTable(singletonList(newOffer), "nzd"));
         String newOfferId = newOffer.getId();
         assertNotEquals("", newOfferId);
@@ -150,7 +162,8 @@ public void testCreateGBPBTCSellOfferMinus1Point5PctPriceMargin() {
                 priceMarginPctInput,
                 getDefaultBuyerSecurityDepositAsPercent(),
                 gbpAccount.getId(),
-                MAKER_FEE_CURRENCY_CODE);
+                MAKER_FEE_CURRENCY_CODE,
+                NO_TRIGGER_PRICE);
         log.info("OFFER #3:\n{}", formatOfferTable(singletonList(newOffer), "gbp"));
         String newOfferId = newOffer.getId();
         assertNotEquals("", newOfferId);
@@ -191,7 +204,8 @@ public void testCreateBRLBTCSellOffer6Point55PctPriceMargin() {
                 priceMarginPctInput,
                 getDefaultBuyerSecurityDepositAsPercent(),
                 brlAccount.getId(),
-                MAKER_FEE_CURRENCY_CODE);
+                MAKER_FEE_CURRENCY_CODE,
+                NO_TRIGGER_PRICE);
         log.info("OFFER #4:\n{}", formatOfferTable(singletonList(newOffer), "brl"));
         String newOfferId = newOffer.getId();
         assertNotEquals("", newOfferId);
@@ -220,6 +234,30 @@ public void testCreateBRLBTCSellOffer6Point55PctPriceMargin() {
         assertCalculatedPriceIsCorrect(newOffer, priceMarginPctInput);
     }
 
+    @Test
+    @Order(5)
+    public void testCreateUSDBTCBuyOfferWithTriggerPrice() {
+        PaymentAccount usdAccount = createDummyF2FAccount(aliceClient, "US");
+        double mktPriceAsDouble = aliceClient.getBtcPrice("usd");
+        BigDecimal mktPrice = new BigDecimal(Double.toString(mktPriceAsDouble));
+        BigDecimal triggerPrice = mktPrice.add(new BigDecimal("1000.9999"));
+        long triggerPriceAsLong = Price.parse("USD", triggerPrice.toString()).getValue();
+
+        var newOffer = aliceClient.createMarketBasedPricedOffer(BUY.name(),
+                "usd",
+                10_000_000L,
+                5_000_000L,
+                0.0,
+                getDefaultBuyerSecurityDepositAsPercent(),
+                usdAccount.getId(),
+                MAKER_FEE_CURRENCY_CODE,
+                triggerPriceAsLong);
+        genBtcBlocksThenWait(1, 4000); // give time to add to offer book
+        newOffer = aliceClient.getMyOffer(newOffer.getId());
+        log.info("OFFER #5:\n{}", formatOfferTable(singletonList(newOffer), "usd"));
+        assertEquals(triggerPriceAsLong, newOffer.getTriggerPrice());
+    }
+
     private void assertCalculatedPriceIsCorrect(OfferInfo offer, double priceMarginPctInput) {
         assertTrue(() -> {
             String counterCurrencyCode = offer.getCounterCurrencyCode();
@@ -239,6 +277,17 @@ private void assertCalculatedPriceIsCorrect(OfferInfo offer, double priceMarginP
         });
     }
 
+    private double getPercentageDifference(double price1, double price2) {
+        return BigDecimal.valueOf(roundDouble((1 - (price1 / price2)), 5))
+                .setScale(4, HALF_UP)
+                .doubleValue();
+    }
+
+    private double getScaledOfferPrice(double offerPrice, String currencyCode) {
+        int precision = isCryptoCurrency(currencyCode) ? Altcoin.SMALLEST_UNIT_EXPONENT : Fiat.SMALLEST_UNIT_EXPONENT;
+        return scaleDownByPowerOf10(offerPrice, precision);
+    }
+
     private boolean isCalculatedPriceWithinErrorTolerance(double delta,
                                                           double expectedDiffPct,
                                                           double actualDiffPct,