Skip to content

add additional validation tests#2621

Open
c1ly wants to merge 14 commits intoopen-telemetry:mainfrom
c1ly:additional-validation-framework-tests
Open

add additional validation tests#2621
c1ly wants to merge 14 commits intoopen-telemetry:mainfrom
c1ly:additional-validation-framework-tests

Conversation

@c1ly
Copy link
Copy Markdown
Contributor

@c1ly c1ly commented Apr 9, 2026

Add additional validation tests for testing the transform (kql and opt queries), log sampling, and temporal reaggregation processors

@github-actions github-actions Bot added the rust Pull requests that update Rust code label Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.86%. Comparing base (ebf2403) to head (ff94c1d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2621      +/-   ##
==========================================
- Coverage   87.87%   87.86%   -0.01%     
==========================================
  Files         638      638              
  Lines      244608   244608              
==========================================
- Hits       214942   214935       -7     
- Misses      29142    29149       +7     
  Partials      524      524              
Components Coverage Δ
otap-dataflow 89.90% <ø> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.74% <ø> (ø)
otel-arrow-go 51.92% <ø> (ø)
quiver 92.27% <ø> (ø)
🚀 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.

@c1ly c1ly marked this pull request as ready for review April 20, 2026 21:31
@c1ly c1ly requested a review from a team as a code owner April 20, 2026 21:31
@c1ly
Copy link
Copy Markdown
Contributor Author

c1ly commented Apr 20, 2026

@JakeDern @albertlockett I added some validation tests for the processors y'all worked, lmk if I missed anything

Copy link
Copy Markdown
Contributor

@JakeDern JakeDern left a comment

Choose a reason for hiding this comment

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

Thanks @c1ly, it's nice to have a second set of eyes testing these processors!

@github-actions github-actions Bot added the ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores label Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores rust Pull requests that update Rust code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants