fix(Relayer): Fix Relayer realizedLpFeePct computation (#783)

Rather than the SpokePool block mapped to a HubPool block, the relevant
block for the LP fee is resolved via the quoteTimestamp on the deposit.

Spotted by Nick.

Co-authored-by: nicholaspai <9457025+nicholaspai@users.noreply.github.com>

src/relayer/Relayer.ts

@@ -375,30 +375,26 @@ export class Relayer {
 
   protected async computeRealizedLpFeePct(
     version: number,
-    deposit: Deposit,
+    deposit: DepositWithBlock,
     symbol: string,
-    hubPoolTokenAddress: string,
-    hubPoolBlockNumber?: number
+    hubPoolTokenAddress: string
   ): Promise<BigNumber> {
     if (!sdkUtils.isUBA(version)) {
       return deposit.realizedLpFeePct;
     }
 
-    const { hubPoolClient, ubaClient } = this.clients;
-    hubPoolBlockNumber ??= hubPoolClient.latestBlockNumber;
-
-    const { originChainId, depositId, destinationChainId, amount } = deposit;
+    const { originChainId, depositId, destinationChainId, amount, quoteBlockNumber } = deposit;
     const {
       lpFee,
       depositBalancingFee: depositFee,
       systemFee: realizedLpFeePct,
-    } = await ubaClient.computeSystemFee(
+    } = await this.clients.ubaClient.computeSystemFee(
       originChainId,
       destinationChainId,
       symbol,
       hubPoolTokenAddress,
       amount,
-      hubPoolBlockNumber
+      quoteBlockNumber
     );
 
     const chain = getNetworkName(deposit.originChainId);