Uh oh!
There was an error while loading. Please reload this page.
Make expect_payment_failed_conditions a function - #1496
Conversation
Codecov Report
@@ Coverage Diff @@## main #1496 +/- ##
==========================================
+ Coverage 90.93% 91.53% +0.60%
==========================================
Files 80 80 Lines 43386 48432 +5046 Branches 43386 48432 +5046 ==========================================
+ Hits 39451 44331 +4880 - Misses 3935 4101 +166
Continue to review full report at Codecov.
|
6df8d12 to
3750650CompareUh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
tnull
commented
Jun 6, 2022
Needs a rebase it seems, other than that LGTM! |
bdcf9e0 to
cdaa0fdCompareTheBlueMatt
commented
Jun 6, 2022
Rebased and fixed a nonsense variable name introduced when this commit wasn't intended for upstream. |
Uh oh!
There was an error while loading. Please reload this page.
8e2b93f to
4671a08Comparetnull
commented
Jun 7, 2022
LGTM, feel free to squash to fix the failing |
This reduces macro generated code in tests a good bit, and moves us one step further away from using macros everywhere when we don't need to.
4671a08 to
70acdf9CompareTheBlueMatt
commented
Jun 9, 2022
Squashed without changes, should be good to go: |
This reduces macro generated code in tests a good bit, and moves us
one step further away from using macros everywhere when we don't
need to.
Did this in an earlier version of #1495 but didn't need it, figured it was still worth upstreaming.