refactor: use valueOrNotAvailableIntIfReverted in updateMarket contra…

…ct calls

subgraphs/isolated-pools/src/constants/index.ts

@@ -2,6 +2,9 @@ import { BigDecimal, BigInt } from '@graphprotocol/graph-ts';
 
 import exponentToBigDecimal from '../utilities/exponentToBigDecimal';
 
+export const NOT_AVAILABLE_BIG_INT = BigInt.fromString('-1');
+export const NOT_AVAILABLE_BIG_DECIMAL = BigDecimal.fromString('-1');
+
 export const zeroBigDecimal = BigDecimal.fromString('0');
 export const zeroBigInt32 = BigInt.fromString('0');
 export const oneBigInt = BigInt.fromString('1');

subgraphs/isolated-pools/src/utilities/getTokenPriceInUsd.ts

@@ -1,8 +1,10 @@
 import { Address, BigDecimal } from '@graphprotocol/graph-ts';
 
 import { PriceOracle } from '../../generated/templates/VToken/PriceOracle';
+import { NOT_AVAILABLE_BIG_DECIMAL } from '../constants';
 import { getPool } from '../operations/get';
 import exponentToBigDecimal from '../utilities/exponentToBigDecimal';
+import valueOrNotAvailableIntIfReverted from './valueOrNotAvailableIntIfReverted';
 
 // Used for all vBEP20 contracts
 const getTokenPrice = (
@@ -11,7 +13,8 @@ const getTokenPrice = (
   underlyingDecimals: i32,
 ): BigDecimal => {
   const pool = getPool(poolAddress);
-  let underlyingPrice = BigDecimal.zero();
+  // will return NOT_AVAILABLE if the price cannot be fetched
+  let underlyingPrice = NOT_AVAILABLE_BIG_DECIMAL;
   if (pool && pool.priceOracleAddress) {
     const oracleAddress = Address.fromBytes(pool.priceOracleAddress);
     /* PriceOracle2 is used from starting of Comptroller.
@@ -22,12 +25,11 @@ const getTokenPrice = (
      */
     const mantissaDecimalFactor = exponentToBigDecimal(36 - underlyingDecimals);
     const priceOracle = PriceOracle.bind(oracleAddress);
-    // Calling getUnderlyingPrice might revert if the pyth price pusher is unfunded
-    // On revert we will return 0
-    const underlyingPriceResult = priceOracle.try_getUnderlyingPrice(eventAddress);
-    if (!underlyingPriceResult.reverted) {
-      underlyingPrice = underlyingPriceResult.value.toBigDecimal().div(mantissaDecimalFactor);
-    }
+
+    const underlyingPriceBigInt = valueOrNotAvailableIntIfReverted(
+      priceOracle.try_getUnderlyingPrice(eventAddress),
+    );
+    underlyingPrice = underlyingPriceBigInt.toBigDecimal().div(mantissaDecimalFactor);
   }
 
   return underlyingPrice;

subgraphs/isolated-pools/src/utilities/index.ts

@@ -1,4 +1,4 @@
 export { default as getExchangeRateBigDecimal } from './getExchangeRateBigDecimal';
 export { default as getTokenPriceInUsd } from './getTokenPriceInUsd';
 export { default as exponentToBigDecimal } from './exponentToBigDecimal';
-export { default as valueOrBigIntZeroIfReverted } from './valueOrBigIntZeroIfReverted';
+export { default as valueOrNotAvailableIntIfReverted } from './valueOrNotAvailableIntIfReverted';

subgraphs/isolated-pools/src/utilities/valueOrNotAvailableIntIfReverted.ts

@@ -0,0 +1,10 @@
+import { BigInt, ethereum } from '@graphprotocol/graph-ts';
+
+import { NOT_AVAILABLE_BIG_INT } from '../constants';
+
+// checks if a call reverted, in case it is we return -1 to indicate the wanted value is not available
+function valueOrNotAvailableIntIfReverted(callResult: ethereum.CallResult<BigInt>): BigInt {
+  return callResult.reverted ? NOT_AVAILABLE_BIG_INT : callResult.value;
+}
+
+export default valueOrNotAvailableIntIfReverted;