invenia / Memento.jl

A flexible logging library for Julia
https://invenia.github.io/Memento.jl/latest
Other
87 stars 14 forks source link

Updates `AsyncHandler` test case to accommodate fix added in v1.8.0-beta1 #189

Closed BaileyBox closed 1 year ago

BaileyBox commented 1 year ago

Previously, there was a test case for where serializing io with an AsyncHandler caused an exception. Memento was changed to accommodate this fix here.

With the changes made to v1.8.0-beta1 (the addition of support AbstractLock and GenericCondition), this case no longer needs to be covered.

Fixes: #184

codecov[bot] commented 1 year ago

Codecov Report

Merging #189 (5e2feb0) into master (d73d2da) will increase coverage by 0.28%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
+ Coverage   98.26%   98.55%   +0.28%     
==========================================
  Files          14       14              
  Lines         404      414      +10     
==========================================
+ Hits          397      408      +11     
+ Misses          7        6       -1     
Impacted Files Coverage Δ
src/io.jl 100.00% <0.00%> (ø)
src/loggers.jl 99.30% <0.00%> (+0.03%) :arrow_up:
src/formatters.jl 100.00% <0.00%> (+1.96%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.