wso2 / carbon-apimgt

Apache License 2.0
166 stars 626 forks source link

Mark OAuthResponseMediator as content-not-aware #12348

Closed msm1992 closed 6 months ago

msm1992 commented 6 months ago

Fixes https://github.com/wso2/api-manager/issues/2501.

With this PR OAuthResponseMediator is marked as content not aware.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.56%. Comparing base (1531e1d) to head (fc6c890). Report is 16 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #12348 +/- ## ============================================= + Coverage 20.16% 47.56% +27.40% - Complexity 0 4189 +4189 ============================================= Files 1223 1840 +617 Lines 105262 136521 +31259 Branches 14769 19765 +4996 ============================================= + Hits 21223 64936 +43713 + Misses 81242 63927 -17315 - Partials 2797 7658 +4861 ``` | [Flag](https://app.codecov.io/gh/wso2/carbon-apimgt/pull/12348/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | Coverage Δ | | |---|---|---| | [integration_tests](https://app.codecov.io/gh/wso2/carbon-apimgt/pull/12348/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | `42.38% <100.00%> (?)` | | | [unit_tests](https://app.codecov.io/gh/wso2/carbon-apimgt/pull/12348/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | `20.14% <0.00%> (-0.02%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.