calcTotalValueLockedForTerm param should be tokenInfo not lists

[jrhea]

it might be better for this method to accept the tokenInfos need rather than passing the list and having it search.

export async function calcTotalValueLockedForTerm(
  trancheInfo: PrincipalTokenInfo,
  balancerVaultAddress: string,
  ptPoolInfo: PoolInfo,
  ytPoolInfo: PoolInfo,
  // etc etc
  assetProxyTokenInfo: AssetProxyTokenInfo,
  signerOrProvider: Signer | Provider
): Promise<Money> {

Originally posted by @jrhea in #37 (comment)

[51bitquant]

Yes, that will be more readable.