Skip to content

Commit

Permalink
[TouchRunner] Rename TestElement.Update to TestElement.TestFinished t…
Browse files Browse the repository at this point in the history
…o make it clearer when it's used. (#73)
  • Loading branch information
rolfbjarne authored Jul 14, 2020
1 parent f8d9a21 commit e5a46fd
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion NUnitLite/TouchRunner/TestCaseElement.cs
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public void Run ()
Runner.Run (TestCase);
}

public override void Update ()
public override void TestFinished ()
{
if (Result.IsIgnored ()) {
Value = Result.GetMessage ();
Expand Down
6 changes: 3 additions & 3 deletions NUnitLite/TouchRunner/TestElement.cs
Original file line number Diff line number Diff line change
Expand Up @@ -65,13 +65,13 @@ public TestResult Result {

protected ITest Test { get; private set; }

public void Update (TestResult result)
public void TestFinished (TestResult result)
{
Result = result;

Update ();
TestFinished ();
}

abstract public void Update ();
abstract public void TestFinished ();
}
}
2 changes: 1 addition & 1 deletion NUnitLite/TouchRunner/TestSuiteElement.cs
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public void Run ()
Runner.Run (Suite);
}

public override void Update ()
public override void TestFinished ()
{
int positive = Result.PassCount + Result.InconclusiveCount;
int failure = Result.FailCount;
Expand Down
4 changes: 2 additions & 2 deletions NUnitLite/TouchRunner/TouchRunner.cs
Original file line number Diff line number Diff line change
Expand Up @@ -829,11 +829,11 @@ public override void TestFinished (ITestResult r)
if (ts != null) {
TestSuiteElement tse;
if (suite_elements.TryGetValue (ts, out tse))
tse.Update (result);
tse.TestFinished (result);
} else {
TestMethod tc = result.Test as TestMethod;
if (tc != null)
case_elements [tc].Update (result);
case_elements [tc].TestFinished (result);
}
});
}
Expand Down

0 comments on commit e5a46fd

Please sign in to comment.