Skip to content

Conversation

@Archidoit
Copy link
Member

Related issue

#13901

@Archidoit Archidoit self-assigned this Jan 6, 2026
@Archidoit Archidoit added the filigran team use to identify PR from the Filigran team label Jan 6, 2026
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 58.33333% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.83%. Comparing base (396f8da) to head (4b98c39).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
...pencti-platform/opencti-graphql/src/domain/user.js 58.33% 15 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #13902   +/-   ##
=======================================
  Coverage   30.82%   30.83%           
=======================================
  Files        2911     2911           
  Lines      192849   192847    -2     
  Branches    39368    39364    -4     
=======================================
+ Hits        59449    59464   +15     
+ Misses     133400   133383   -17     
Flag Coverage Δ
opencti 30.83% <58.33%> (+<0.01%) ⬆️
opencti-front 2.48% <ø> (+<0.01%) ⬆️
opencti-graphql 68.21% <58.33%> (+<0.01%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@labo-flg
Copy link
Member

labo-flg commented Jan 7, 2026

Thanks for the fix !
This is a quite sensitive area of the code base, can we had tests please.

@Archidoit
Copy link
Member Author

Backend tests have been added !

@Archidoit Archidoit merged commit bbd3bc6 into master Jan 8, 2026
39 checks passed
@Archidoit Archidoit deleted the issue/13901 branch January 8, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants