Skip to content
This repository was archived by the owner on Dec 12, 2017. It is now read-only.

Commit c539f07

Browse files
committed
Nicer log messages
1 parent e73d6a3 commit c539f07

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

resharper/src/runner/Discoverer.cs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,14 +91,14 @@ private bool IsRequestedMethod(ITestCase testCase)
9191
if (IsTheory(displayName, typeName, methodName))
9292
{
9393
var hasTheoryTask = runContext.HasTheoryTask(displayName, typeName, methodName);
94-
Logger.LogVerbose(" Theory test case is {0}a requested theory: {1}",
94+
Logger.LogVerbose(" Test case is a theory, has {0}been previously seen: {1}",
9595
hasTheoryTask ? string.Empty : "NOT ",
9696
testCase.Format());
9797
return hasTheoryTask;
9898
}
9999

100100
var hasMethodTask = runContext.HasMethodTask(typeName, methodName);
101-
Logger.LogVerbose(" Test case is {0}a requested method: {1}", hasMethodTask ? string.Empty : "NOT ",
101+
Logger.LogVerbose(" Test case is a method, is {0}requested: {1}", hasMethodTask ? string.Empty : "NOT ",
102102
testCase.Format());
103103
return hasMethodTask;
104104
}
@@ -119,7 +119,7 @@ private bool IsDynamicMethod(ITestCase testCase)
119119
if (classTaskWrapper == null)
120120
{
121121
Logger.LogVerbose(
122-
" Test case class unknown. Cannot be a dynamic test of a requested class. {0} - {1}",
122+
" Test case does not belong to a known class. Cannot be a dynamic test of a requested class. {0} - {1}",
123123
testCase.TestMethod.TestClass.Class.Name, testCase.Format());
124124
return false;
125125
}
@@ -129,9 +129,8 @@ private bool IsDynamicMethod(ITestCase testCase)
129129
if (IsTheory(displayName, typeName, methodName))
130130
{
131131
var isDynamicTheory = !classTask.IsKnownMethod(displayName.Replace(typeName + ".", string.Empty));
132-
Logger.LogVerbose(" Theory test case is {0}a requested theory: {1}",
133-
isDynamicTheory ? string.Empty : "NOT ",
134-
testCase.Format());
132+
Logger.LogVerbose(" Test case is a previously unseen (dynamic) theory, {0} to a requested method: {1}",
133+
isDynamicTheory ? "belongs" : "does NOT belong", testCase.Format());
135134
return isDynamicTheory;
136135
}
137136

0 commit comments

Comments
 (0)