Skip to content

airframe-log: Fixes #2676 flaky test of AsyncHandler#2800

Merged
xerial merged 3 commits intomasterfrom
fix-flaky-log-test
Mar 21, 2023
Merged

airframe-log: Fixes #2676 flaky test of AsyncHandler#2800
xerial merged 3 commits intomasterfrom
fix-flaky-log-test

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Mar 21, 2023

No description provided.

@github-actions github-actions Bot added the bug label Mar 21, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 21, 2023

Codecov Report

Merging #2800 (ff8aa61) into master (e470956) will increase coverage by 0.03%.
The diff coverage is 95.14%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2800      +/-   ##
==========================================
+ Coverage   82.26%   82.30%   +0.03%     
==========================================
  Files         337      337              
  Lines       14179    14190      +11     
  Branches     2313     2321       +8     
==========================================
+ Hits        11665    11679      +14     
+ Misses       2514     2511       -3     
Impacted Files Coverage Δ
...g/.jvm/src/main/scala/wvlet/log/AsyncHandler.scala 97.05% <88.88%> (-2.95%) ⬇️
...wvlet/airframe/http/router/HttpRequestMapper.scala 94.84% <95.65%> (+0.28%) ⬆️
...irframe-log/src/main/scala/wvlet/log/Handler.scala 78.57% <100.00%> (+1.64%) ⬆️

... and 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 593bc08...ff8aa61. Read the comment docs.

@xerial xerial added internal Internal changes (usually non-user facing) and removed bug labels Mar 21, 2023
@xerial xerial merged commit 3903556 into master Mar 21, 2023
@xerial xerial deleted the fix-flaky-log-test branch March 21, 2023 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes (usually non-user facing)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant