Closed JacobOaks closed 1 year ago
Merging #393 (cc4ce88) into master (a30081d) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #393 +/- ##
=======================================
Coverage 98.39% 98.39%
=======================================
Files 22 22
Lines 1492 1492
=======================================
Hits 1468 1468
Misses 15 15
Partials 9 9
Files Changed | Coverage Δ | |
---|---|---|
param.go | 97.34% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
We call simple providers with their original scope. (ref) This allows decorators to be applied even if the constructor is exported via
dig.Export(true)
.However, we call value group value providers with whatever scope we found them in: (ref)
This has the consequence of causing exported value group value providers within a module to not be able to be decorated by decorators within the same module, when using
dig.Export(true)
, unlike their simple provider counterpart.For example (playground):
Since we use
dig.Export(true)
by default in Fx, this is the default behavior for value group providers, see https://github.com/uber-go/fx/issues/1104This commit changes
callGroupProviders
to use the value group provider's original scope as well, and adds a test to verify the behavior is fixed.