Fix tests

reportportal · Apr 4, 2024 · b144db7 · b144db7
1 parent 786fe1e
commit b144db7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/main/java/com/epam/reportportal/testng/TestNGService.java b/src/main/java/com/epam/reportportal/testng/TestNGService.java
@@ -376,7 +376,7 @@ protected FinishTestItemRQ buildFinishTestMethodRq(@Nonnull ItemStatus status, @
 		FinishTestItemRQ rq = new FinishTestItemRQ();
 		rq.setEndTime(new Date(testResult.getEndMillis()));
 		rq.setStatus(status.name());
-		if (!testResult.isSuccess()) {
+		if (!testResult.isSuccess() && ItemStatus.SKIPPED != status) {
 			rq.setDescription(getLogMessage(testResult));
 		}
 		return rq;

diff --git a/src/test/java/com/epam/reportportal/testng/TestNGServiceTest.java b/src/test/java/com/epam/reportportal/testng/TestNGServiceTest.java
@@ -177,6 +177,7 @@ public void startTestMethod() {
 	public void finishTestMethod() {
 		when(launch.getParameters()).thenReturn(new ListenerParameters());
 		when(testResult.getAttribute(RP_ID)).thenReturn(id);
+		when(testResult.isSuccess()).thenReturn(true);
 
 		testNGService.finishTestMethod(ItemStatus.PASSED, testResult);
 		verify(launch, times(1)).finishTestItem(eq(id), any(FinishTestItemRQ.class));