Skip to content

Commit c317621

Browse files
committed
Restore 100% branch coverage
There's an [issue in coverlet][1] that doesn't report proper coverage for pattern-matching. [1]: coverlet-coverage/coverlet#1313
1 parent 52a1975 commit c317621

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/Log4NetTextFormatter.cs

+2-1
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,8 @@ internal static string DefaultMessageFormatter(LogEvent logEvent, IFormatProvide
151151
// https://github.com/serilog/serilog-extensions-logging/blob/v9.0.1/src/Serilog.Extensions.Logging/Extensions/Logging/EventIdPropertyCache.cs#L58-L73
152152
var isMicrosoftExtensionsLoggingEvent = logEvent.Properties.TryGetValue(EventIdPropertyName, out var property)
153153
&& property is StructureValue structure
154-
&& structure.Properties.Any(e => e.Name is "Id" or "Name");
154+
// ReSharper disable once MergeIntoLogicalPattern -- Wrong coverage report, see https://github.com/coverlet-coverage/coverlet/issues/1313
155+
&& structure.Properties.Any(e => e.Name == "Id" || e.Name == "Name");
155156

156157
if (isMicrosoftExtensionsLoggingEvent)
157158
{

0 commit comments

Comments
 (0)