Skip to content

Commit 44d1093

Browse files
committed
TestCanceled fired from onTestSkipped should include the created formatter.
1 parent ea7aaeb commit 44d1093

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/main/scala/org/scalatestplus/testng/TestNGSuiteLike.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ trait TestNGSuiteLike extends Suite { thisSuite =>
338338
val testName = result.getName + params(result)
339339
val formatter = getIndentedTextForTest(testName, 1, true)
340340
val causedBy = result.getSkipCausedBy().asScala.map(_.getMethodName())
341-
report(TestCanceled(tracker.nextOrdinal(), "Skipped caused by ", thisSuite.suiteName, thisSuite.getClass.getName, Some(thisSuite.getClass.getName), testName, testName, Vector.empty))
341+
report(TestCanceled(tracker.nextOrdinal(), "Skipped caused by " + causedBy, thisSuite.suiteName, thisSuite.getClass.getName, Some(thisSuite.getClass.getName), testName, testName, Vector.empty, None, None, Some(formatter)))
342342
}
343343

344344
/**

src/test/scala/org/scalatestplus/testng/TestNGSuiteSuite.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ import org.scalatestplus.testng.SharedHelpers.EventRecordingReporter
8585
assert(reporter.testStartingEventsReceived.length == 2)
8686
assert(reporter.testFailedEventsReceived.length == 1)
8787
assert(reporter.testCanceledEventsReceived.length == 1)
88+
assert(reporter.testCanceledEventsReceived(0).formatter == Some(IndentedText("- depender", "depender", 1)))
8889
assert(reporter.suiteCompletedEventsReceived.isEmpty)
8990
}
9091

0 commit comments

Comments
 (0)