Skip to content

sdk/log: add TestRecordMethodsInputConcurrentSafe - #7378

Merged
pellared merged 3 commits into
open-telemetry:mainfrom
pellared:test-7364
Sep 19, 2025
Merged

sdk/log: add TestRecordMethodsInputConcurrentSafe#7378
pellared merged 3 commits into
open-telemetry:mainfrom
pellared:test-7364

Conversation

@pellared

@pellared pellared commented Sep 18, 2025

Copy link
Copy Markdown
Member

Why

Towards #7364

I want to split it into a few PRs as otherwise it would be a big PR which would be hard to review.

I plan to add benchmarks after this one.

What

Add a unit test reproducing all cases from #7364 (comment) when

if race() {
	t.Skip("TODO: Fix bug https://github.com/open-telemetry/opentelemetry-go/issues/7364.")
}

is removed.

@codecov

codecov Bot commented Sep 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.5%. Comparing base (6180f83) to head (d44cd3b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #7378   +/-   ##
=====================================
  Coverage   85.5%   85.5%           
=====================================
  Files        274     274           
  Lines      24614   24614           
=====================================
  Hits       21049   21049           
  Misses      3186    3186           
  Partials     379     379           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@pellared pellared added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 18, 2025
@pellared
pellared merged commit a189c6b into open-telemetry:main Sep 19, 2025
31 checks passed
@pellared
pellared deleted the test-7364 branch September 19, 2025 11:18
@MrAlias MrAlias added this to the v1.39.0 milestone Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

5 participants