search

eclipse / kapua

Eclipse Public License 2.0
222 stars 160 forks source link

FIX - Added tests for CORS filtering logic #4010

Closed Agnul97 closed 5 months ago

Agnul97 commented 5 months ago

Brief description of the PR. I created a batch of scenarios for the CORS filtering that uses my latest functions for the rest API client. here, I added them along with a proper job in the ci configuration. This PR fills a gap in the tests for CORS filtering

Description of the solution adopted a bunch of scenarios have been added for the CORS filtering testing, similar to what done in the "restAuth.feature" file. In order to test for the CORS origins list refresh, I needed to tweak the refresh time to a small amount so, just like done in the past, I added an env. variable to set this value and I set it to a very low value (to speed-up execution time of tests). This env. value could be used in the future if we want to tweak this setting for other reasons

codecov[bot] commented 5 months ago

Codecov Report

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

Project coverage is 16.87%. Comparing base (21d4941) to head (836c5a7). Report is 3 commits behind head on develop.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse/kapua/pull/4010/graphs/tree.svg?width=650&height=150&src=pr&token=1P4N4CApH8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)](https://app.codecov.io/gh/eclipse/kapua/pull/4010?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) ```diff @@ Coverage Diff @@ ## develop #4010 +/- ## ========================================== Coverage 16.87% 16.87% Complexity 10 10 ========================================== Files 1989 1989 Lines 51791 51791 Branches 4420 4420 ========================================== Hits 8739 8739 Misses 42648 42648 Partials 404 404 ```