Skip to content

Commit 808cf80

Browse files
committed
Remove unnecessary throwing in test methods
Hoping that it will fix those Codacy issues: > Remove this redundant jump.
1 parent d50f34b commit 808cf80

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/Log4NetTextFormatterOptionsBuilderTest.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public void UseNoIndentation()
8787
public void UsePropertyFilter()
8888
{
8989
// Arrange
90-
PropertyFilter filterProperty = (logEvent, propertyName) => throw new NotImplementedException();
90+
PropertyFilter filterProperty = (logEvent, propertyName) => true;
9191

9292
// Act
9393
var builder = new Log4NetTextFormatterOptionsBuilder().UsePropertyFilter(filterProperty);
@@ -111,7 +111,7 @@ public void SettingPropertyFilterToNullThrowsArgumentNullException()
111111
public void UseExceptionFormatter()
112112
{
113113
// Arrange
114-
ExceptionFormatter formatException = exception => throw new NotImplementedException();
114+
ExceptionFormatter formatException = exception => "";
115115

116116
// Act
117117
var builder = new Log4NetTextFormatterOptionsBuilder().UseExceptionFormatter(formatException);

0 commit comments

Comments
 (0)