Skip to content

Conversation

@pnoltes
Copy link
Contributor

@pnoltes pnoltes commented Aug 14, 2025

This PR introduces some fuzz tests for celix utils.

Also:

  • Aligns enable_fuzzing and enable_benchmarks option
  • Updates conan package id so that test, benchmarks, etc options are not used for the package id.
  • Fixes an issues where the jar command was chosen to make bundles if jar and zip is available, even if the "use zip" option was enable
  • Update building document for testing, benchmarking and fuzzing

Also align enable benchmark option with other enable option and improve package id
Also improves benchmark setup
@pnoltes pnoltes self-assigned this Aug 14, 2025
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.49%. Comparing base (c6d0e2a) to head (9344b89).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #799      +/-   ##
==========================================
+ Coverage   91.44%   91.49%   +0.04%     
==========================================
  Files         234      234              
  Lines       28655    28655              
==========================================
+ Hits        26204    26218      +14     
+ Misses       2451     2437      -14     

☔ 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.

@PengZheng PengZheng self-requested a review August 15, 2025 02:42
@PengZheng
Copy link
Contributor

I'll finish reviewing it this week.

Copy link
Contributor

@PengZheng PengZheng left a comment

Choose a reason for hiding this comment

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

Great addition! This PR can be merged as is, though I left some minor remarks.

I read the libFuzzer documentation and learned a lot. Thanks.

@PengZheng PengZheng self-requested a review December 15, 2025 03:17
Copy link
Contributor

@PengZheng PengZheng left a comment

Choose a reason for hiding this comment

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

I have fixed the LD_PRELOAD issue.

@pnoltes
Copy link
Contributor Author

pnoltes commented Dec 18, 2025

I have fixed the LD_PRELOAD issue.

👍 , Thanks

@pnoltes pnoltes merged commit 7160174 into master Dec 18, 2025
36 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