Skip to content

Commit 62cdd0c

Browse files
authored
Merge pull request #5 from osandadeshan/refactor
Minor refactoring to the Extent Report service
2 parents f2c7b2f + 57cc7c8 commit 62cdd0c

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/main/java/com/appium/example/util/ExtentReportService.java

+12-8
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,18 @@ public void initializeExtentReporter(String timestamp) {
5252
public void updateExtentReport(ITestResult iTestResult, String timestamp) {
5353
ExtentTest test = appendTestInformation(iTestResult);
5454

55-
if (iTestResult.getStatus() == FAILURE) {
56-
appendFailureDetails(test, iTestResult, timestamp);
57-
} else if (iTestResult.getStatus() == SKIP) {
58-
appendSkipDetails(test, iTestResult, timestamp);
59-
} else if (iTestResult.getStatus() == SUCCESS) {
60-
appendSuccessDetails(test, iTestResult, timestamp);
61-
} else {
62-
logger.warn("Test result for the test method '{}' is unknown.", iTestResult.getMethod());
55+
switch (iTestResult.getStatus()) {
56+
case FAILURE:
57+
appendFailureDetails(test, iTestResult, timestamp);
58+
break;
59+
case SKIP:
60+
appendSkipDetails(test, iTestResult, timestamp);
61+
break;
62+
case SUCCESS:
63+
appendSuccessDetails(test, iTestResult, timestamp);
64+
break;
65+
default:
66+
logger.warn("Test result for the test method '{}' is unknown.", iTestResult.getMethod());
6367
}
6468

6569
addFeature(iTestResult, test);

0 commit comments

Comments
 (0)