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

feature: add JWTTokenGenerator test #2784

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

hsiliev
Copy link
Collaborator

@hsiliev hsiliev commented Feb 26, 2025

Add JWTTokenGenerator unit test

Solves PZ-5694

Add JWTTokenGenerator unit test

Solves PZ-5694
@hsiliev hsiliev requested a review from a team as a code owner February 26, 2025 16:58
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has checked 2346 files.

Valid Invalid Ignored Fixed
2040 1 305 0
Click to see the invalid file list
  • src/test/kotlin/net/atos/client/util/JWTTokenGeneratorTest.kt
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

hsiliev and others added 2 commits February 26, 2025 19:30
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Feb 26, 2025

unit-test-results

 86 files  +1  124 suites  +1   1m 45s ⏱️ ±0s
475 tests +1  474 ✅ +1  1 💤 ±0  0 ❌ ±0 
579 runs  +2  577 ✅ +2  2 💤 ±0  0 ❌ ±0 

Results for commit 04657df. ± Comparison against base commit ab6e069.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Feb 26, 2025

integration-test-results

 31 files  ±0   31 suites  ±0   40s ⏱️ ±0s
114 tests ±0  114 ✅ ±0  0 💤 ±0  0 ❌ ±0 
127 runs  ±0  127 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 04657df. ± Comparison against base commit ab6e069.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Feb 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 42.16%. Comparing base (ab6e069) to head (04657df).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2784   +/-   ##
=========================================
  Coverage     42.16%   42.16%           
  Complexity     3476     3476           
=========================================
  Files          1336     1336           
  Lines         32271    32271           
  Branches       3045     3045           
=========================================
  Hits          13608    13608           
  Misses        17440    17440           
  Partials       1223     1223           
Flag Coverage Δ
integrationtests 51.38% <ø> (ø)
unittests 28.18% <ø> (+0.03%) ⬆️

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.

Copy link
Member

@edgarvonk edgarvonk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@hsiliev hsiliev enabled auto-merge February 27, 2025 08:28
@hsiliev hsiliev added this pull request to the merge queue Feb 27, 2025
Merged via the queue into main with commit d13833d Feb 27, 2025
21 checks passed
@hsiliev hsiliev deleted the feature/PZ-5693-add-jwt-generator-test branch February 27, 2025 08:55
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.

2 participants