Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration test for audience validation with API and API products #13520

Merged
merged 2 commits into from
Aug 24, 2024

Conversation

BiyonFernando
Copy link
Contributor

Purpose

To add integration tests for audience validation support feature described in [1].

[1] - wso2/api-manager#2784

Approach

The audience property is included to the schemas in the publish-api.yaml. Integration tests are written for the following scenarios to test the functionality of audience validation support with APIs and API products:

  • Test the response for a GET request without enabling audience validation.
  • Test blocking the response when an invalid audience is provided.
  • Test whether the response passes through when a valid audience is provided.

@CLAassistant
Copy link

CLAassistant commented Aug 22, 2024

CLA assistant check
All committers have signed the CLA.

@BiyonFernando BiyonFernando force-pushed the aud-auth-test branch 2 times, most recently from f28004b to 8ae8395 Compare August 22, 2024 12:29
Copy link

codecov bot commented Aug 22, 2024

Codecov Report

Attention: Patch coverage is 60.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 17.83%. Comparing base (2d070f7) to head (b36568c).
Report is 25 commits behind head on master.

Files Patch % Lines
...lients/publisher/api/v1/dto/APIProductInfoDTO.java 60.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13520      +/-   ##
============================================
+ Coverage     17.52%   17.83%   +0.30%     
- Complexity      879      897      +18     
============================================
  Files           255      255              
  Lines         11526    11551      +25     
  Branches       1202     1204       +2     
============================================
+ Hits           2020     2060      +40     
+ Misses         9490     9474      -16     
- Partials         16       17       +1     
Flag Coverage Δ
integration_tests 17.83% <60.00%> (+0.30%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@RakhithaRR RakhithaRR merged commit efd6ee7 into wso2:master Aug 24, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants