typescript-rest icon indicating copy to clipboard operation
typescript-rest copied to clipboard

feat: provide controller result to PostProcessor functions

Open Inkdpixels opened this issue 5 years ago • 2 comments
trafficstars

This PR introduces the option that all @PostProcessor decorators can access the result of the controller method.

This is useful as one could validate the returned result against a swagger schema inside a processor function.

Inkdpixels avatar Nov 16 '20 11:11 Inkdpixels

This makes total sense. Why is it not merged yet? @thiagobustamante

IkerZamora avatar Mar 09 '21 15:03 IkerZamora

Codecov Report

Merging #151 (262e676) into master (70a2716) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #151   +/-   ##
=======================================
  Coverage   94.48%   94.48%           
=======================================
  Files          14       14           
  Lines        1070     1070           
  Branches      195      195           
=======================================
  Hits         1011     1011           
  Misses         59       59           
Impacted Files Coverage Δ
src/server/model/metadata.ts 100.00% <ø> (ø)
src/server/model/server-types.ts 100.00% <ø> (ø)
src/decorators/services.ts 100.00% <100.00%> (ø)
src/server/service-invoker.ts 92.35% <100.00%> (ø)

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 70a2716...a11a257. Read the comment docs.

codecov[bot] avatar Mar 10 '21 08:03 codecov[bot]