add is_position_liquidable in reader

keep-starknet-strange · Jul 3, 2024 · c79897a · c79897a
1 parent 60d3d1e
commit c79897a
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 6 deletions.
diff --git a/src/reader/reader.cairo b/src/reader/reader.cairo
@@ -425,6 +425,16 @@ trait IReader<TContractState> {
         is_long: bool,
         prices: MarketPrices
     ) -> (u64, bool, i256, u256);
+
+    fn is_position_liquidable(
+        self: @TContractState,
+        data_store: IDataStoreDispatcher,
+        referral_storage: IReferralStorageDispatcher,
+        position: Position,
+        market: Market,
+        prices: MarketPrices,
+        should_validate_min_collateral_usd: bool,
+    ) -> (bool, felt252);
 }
 
 #[starknet::contract]
@@ -856,6 +866,20 @@ mod Reader {
             );
             (latest_adl_block, should_enabled_ald, pnl_to_pool_factor, max_pnl_factor)
         }
+
+        fn is_position_liquidable(
+            self: @ContractState,
+            data_store: IDataStoreDispatcher,
+            referral_storage: IReferralStorageDispatcher,
+            position: Position,
+            market: Market,
+            prices: MarketPrices,
+            should_validate_min_collateral_usd: bool
+        ) -> (bool, felt252) {
+            position_utils::is_position_liquiditable(
+                data_store, referral_storage, position, market, prices, should_validate_min_collateral_usd
+            )
+        }
     }
 }
 
diff --git a/tests/integration/test_long_integration.cairo b/tests/integration/test_long_integration.cairo
@@ -3713,8 +3713,8 @@ fn test_long_liquidation() {
         short_token_price: Price { min: 1, max: 1, },
     };
 
-    let (is_liquiditable, reason) = position_utils::is_position_liquiditable(
-        data_store, referal_storage, first_position, market, market_prices, false
+    let (is_liquiditable, reason) = reader.is_position_liquidable(
+        data_store, referal_storage, first_position, market, market_prices, true
     );
 
     assert(is_liquiditable == true, 'Position is liquidable');
@@ -3727,8 +3727,8 @@ fn test_long_liquidation() {
         short_token_price: Price { min: 1, max: 1, },
     };
 
-    let (is_liquiditable, reason) = position_utils::is_position_liquiditable(
-        data_store, referal_storage, first_position, market, market_prices, false
+    let (is_liquiditable, reason) = reader.is_position_liquidable(
+        data_store, referal_storage, first_position, market, market_prices, true
     );
 
     assert(is_liquiditable == false, 'Position is not liquidable');
@@ -4637,8 +4637,8 @@ fn test_long_leverage_liquidation() {
         short_token_price: Price { min: 1, max: 1, },
     };
 
-    let (is_liquiditable, reason) = position_utils::is_position_liquiditable(
-        data_store, referal_storage, first_position, market, market_prices, false
+    let (is_liquiditable, reason) = reader.is_position_liquidable(
+        data_store, referal_storage, first_position, market, market_prices, true
     );
     // position x10 leverage is liquidable at 3000$, position x1 leverage is not liquidable at 3000$
     assert(is_liquiditable == true, 'Position is liquidable');