-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
improve some nits and refactor foundry tests #12522
Conversation
I see you updated files related to |
I see you updated files related to |
PayoutMode mode = s_payoutMode; | ||
for (uint256 i = 0; i < billingTokens.length; i++) { | ||
IERC20 token = billingTokens[i]; | ||
BillingConfig memory config = billingConfigs[i]; | ||
|
||
// if LINK is a billing option, payout mode must be ON_CHAIN | ||
if (address(token) == address(i_link) && s_payoutMode == PayoutMode.OFF_CHAIN) { | ||
if (address(token) == address(i_link) && mode == PayoutMode.OFF_CHAIN) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this change probably increases gas, fwiw, since we read the thing into memory now
move addFunds() tests to foundry for auto v2.3 add tests for onTokenTransfer()
Quality Gate passedIssues Measures |
* improve some nits and refactor foundry tests * add changesets * fix * fix tests * write registerUpkeep() foundry tests for auto v2.3 (#12535) move addFunds() tests to foundry for auto v2.3 add tests for onTokenTransfer() --------- Co-authored-by: Ryan Hall <RyanRHall@users.noreply.github.com>
…ator (#12534) * Applied LimitMultiplier to chainSpecificGasLimit in all gas estimators * Added changeset * altered tests to use a limit multiplier of 1.5 rather than 1 * improve some nits and refactor foundry tests (#12522) * improve some nits and refactor foundry tests * add changesets * fix * fix tests * write registerUpkeep() foundry tests for auto v2.3 (#12535) move addFunds() tests to foundry for auto v2.3 add tests for onTokenTransfer() --------- Co-authored-by: Ryan Hall <RyanRHall@users.noreply.github.com> * Added error checks * updated existing tests * Extracted fee limit multiplier out of indivudal gas estimators, into WrappedEvmEstimator * Cleaned up unused LimitMultipliers * Removed unused limit multipliers in tests and cleaned up * removed chainSpecificGasLimit from BumpDynamicFee, GetDynamicFee * Fixed failing tests --------- Co-authored-by: FelixFan1992 <fankejin@gmail.com> Co-authored-by: Ryan Hall <RyanRHall@users.noreply.github.com>
No description provided.