Renamed fulfill method

contracts/orderbook/src/order.rs

@@ -296,7 +296,7 @@ pub fn resolve_fulfillments(
         // TODO: Add price detection for tick
         let msg = fulfillment
             .order
-            .fulfill(&denom, fulfillment.amount, Decimal::one())?;
+            .fill(&denom, fulfillment.amount, Decimal::one())?;
         msgs.push(msg);
         if fulfillment.order.quantity.is_zero() {
             orders().remove(

contracts/orderbook/src/tests/test_order.rs

@@ -836,7 +836,7 @@ fn test_resolve_fulfillments() {
             // Check message is generated as expected
             let mut order = order.clone();
             let denom = orderbook.get_expected_denom(&order.order_direction);
-            let msg = order.fulfill(denom, *amount, Decimal::one()).unwrap();
+            let msg = order.fill(denom, *amount, Decimal::one()).unwrap();
 
             assert_eq!(response[idx], msg, "{}", format_test_name(test.name));
         }

contracts/orderbook/src/types/order.rs

@@ -40,7 +40,7 @@ impl LimitOrder {
     }
 
     // Transfers the specified quantity of the order's asset to the owner
-    pub fn fulfill(
+    pub fn fill(
         &mut self,
         denom: impl Into<String>,
         quantity: Uint128,
@@ -56,7 +56,7 @@ impl LimitOrder {
                 reason: Some("Order does not have enough funds".to_string())
             }
         );
-        self.quantity = self.quantity.checked_sub(quantity.min(self.quantity))?;
+        self.quantity = self.quantity.checked_sub(quantity)?;
         Ok(BankMsg::Send {
             to_address: self.owner.to_string(),
             amount: vec![coin(