midwayjs / injection

Injection is a powerful inversion of control container that is widely used in the midway framework and brings good user experience.
MIT License
160 stars 20 forks source link

fix: Cannot redefine property: _req_ctx #20

Closed Lellansin closed 5 years ago

Lellansin commented 5 years ago

通过 wrapperProvide 提供的 function 返回的对象不需要 req ctx

codecov-io commented 5 years ago

Codecov Report

Merging #20 into master will increase coverage by 44.05%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #20       +/-   ##
===========================================
+ Coverage   43.46%   87.52%   +44.05%     
===========================================
  Files          32       32               
  Lines        1231     1234        +3     
  Branches       50       52        +2     
===========================================
+ Hits          535     1080      +545     
+ Misses        687      144      -543     
- Partials        9       10        +1
Impacted Files Coverage Δ
src/factory/common/managedResolverFactory.ts 83.43% <100%> (+49.69%) :arrow_up:
src/utils/reflectTool.ts 86.2% <0%> (+3.44%) :arrow_up:
src/base/decoratorManager.ts 96.11% <0%> (+25.24%) :arrow_up:
src/base/objectDefinition.ts 100% <0%> (+27.02%) :arrow_up:
src/factory/container.ts 90.78% <0%> (+32.89%) :arrow_up:
src/base/objectCreator.ts 55.31% <0%> (+34.04%) :arrow_up:
src/factory/requestContainer.ts 100% <0%> (+35%) :arrow_up:
src/factory/common/managed.ts 100% <0%> (+45.45%) :arrow_up:
src/factory/applicationContext.ts 83.01% <0%> (+48.11%) :arrow_up:
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update dba0973...96ee305. Read the comment docs.