Closed elia closed 1 year ago
Merging #304 (2979d7e) into main (deb292a) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #304 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 30 30
Lines 546 546
=======================================
Hits 544 544
Misses 2 2
:mega: Weβre building smart automated test selection to slash your CI/CD build times. Learn more
@waiting-for-dev we initially tried that approach but eventually we rolled it back because there was little to no reuse, as the combinations of flash messages and payment statuses were always different. Maybe there's a case for using I18n here, but that also as its own downsides, and is usually better to use raw values rather than dynamic ones in tests to avoid the equivalent of circular definitions.
Summary
The implementation was fine, just had to adjust the expectation and test setup.
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
The following are not always needed: