search

wso2 / product-apim

Welcome to the WSO2 API Manager source code! For info on working with the WSO2 API Manager repository and contributing code, click the link below.
http://wso2.github.io/
Apache License 2.0
821 stars 767 forks source link

Update client secret assertions to support empty client secret #13466

Closed shnrndk closed 2 months ago

shnrndk commented 3 months ago

This PR fixed the test failures caused by the PR https://github.com/wso2/carbon-apimgt/pull/12312.

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 21.59%. Comparing base (a984da2) to head (1060470). Report is 6 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #13466 +/- ## ============================================ - Coverage 24.32% 21.59% -2.74% + Complexity 1243 1107 -136 ============================================ Files 254 254 Lines 11495 11495 Branches 1200 1200 ============================================ - Hits 2796 2482 -314 - Misses 8672 8992 +320 + Partials 27 21 -6 ``` | [Flag](https://app.codecov.io/gh/wso2/product-apim/pull/13466/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/product-apim/pull/13466/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wso2) | `21.59% <ø> (-2.74%)` | :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.