Usecase should have access to req.user inside ctx

[vitorgamer58]

Feature https://github.com/herbsjs/herbs2rest/issues/41

Proposed Changes

1. Added injection in the defaultController function so that req.user data can be accessed within usecase steps through ctx.di

Readiness Checklist

Author/Contributor

• [x] If documentation is needed for this change, has that been included in this pull request
• [x] Remember to check if code coverage decrease, if so, please implement the necessary tests

Reviewing Maintainer

• [ ] Label as breaking if this is a large fundamental change
• [ ] Label as either automation, bug, documentation, enhancement, infrastructure, or performance

[vitorgamer58]

Is working

[codecov[bot]]

Codecov Report

Merging #42 (b87b441) into main (90cf623) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #42   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          129       132    +3     
=========================================
+ Hits           129       132    +3     

Impacted Files	Coverage Δ
src/defaultController.js	100.00% <100.00%> (ø)
src/generateRoutes.js	100.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

[vitorgamer58]

Good PR @vitorgamer58, but I think the doc about this is missing

I'll work on this.

[vitorgamer58]

@jhomarolo I'm not sure if I'm satisfied with the documentation of my feature, I believe the documentation needs a little more context about the req.user, maybe with some middleware example from express.

[dalssoft]

I dont think this PR makes sense anymore given https://github.com/herbsjs/buchu/pull/85

[vitorgamer58]

I agree