diff --git a/CHANGELOG.md b/CHANGELOG.md index ca627b6..e7aba3a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,13 @@ +## [4.1.4](https://github.com/extent-framework/extentreports-java/compare/v4.1.3...v4.1.4) +#### Improvements +* [#114] ExtentSparkReporter to display hierarchical name in tags view +* [#119] ExtentSparkReporter to display thumbnail screenshots +* [#125] ExtentSparkReporter displays clip icon for tests with leaf containing screenshots +* [#132] ExtentSparkReporter to accept file-name instead of Index.html default + +#### Fixes +* [#135] ExtentSparkReporter issue with not parsing the `scripts` tag in config.xml + ## [4.1.3](https://github.com/extent-framework/extentreports-java/compare/v4.1.2...v4.1.3) #### Improvements * [#119] `ExtentSparkReporter` now displays thumbnails as default diff --git a/config/logger-config.xml b/config/logger-config.xml index f3b5742..a0094a6 100644 --- a/config/logger-config.xml +++ b/config/logger-config.xml @@ -1,44 +1,44 @@ - - - - standard - - - - UTF-8 - - - true - - - - https - - - Extent Framework - - - Build 1 + + + + standard + + + + UTF-8 + + + true + + + + https + + + Extent Framework + + + Build 1 MMM dd, yyyy HH:mm:ss - - - + + + - - - - + + + + - - + + diff --git a/config/spark-config.xml b/config/spark-config.xml index 5da2252..0bb21fe 100644 --- a/config/spark-config.xml +++ b/config/spark-config.xml @@ -1,37 +1,37 @@ - + standard - + UTF-8 - + https - + - false - + false + Extent Framework - + Build 1 - + MMM dd, yyyy HH:mm:ss - + true false true true - + - + - + \ No newline at end of file diff --git a/config/v3html-config.xml b/config/v3html-config.xml index 5da2252..0bb21fe 100644 --- a/config/v3html-config.xml +++ b/config/v3html-config.xml @@ -1,37 +1,37 @@ - + standard - + UTF-8 - + https - + - false - + false + Extent Framework - + Build 1 - + MMM dd, yyyy HH:mm:ss - + true false true true - + - + - + \ No newline at end of file diff --git a/pom-nexus.xml b/pom-nexus.xml index 084aa94..7aa8e5c 100644 --- a/pom-nexus.xml +++ b/pom-nexus.xml @@ -5,7 +5,7 @@ com.aventstack extentreports - 4.1.3 + 4.1.4 extentreports www.extentreports.com diff --git a/pom.xml b/pom.xml index 084aa94..f1d2aeb 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ com.aventstack extentreports - 4.1.3 + 4.1.4 extentreports www.extentreports.com @@ -114,20 +114,6 @@ maven-resources-plugin 2.7 - - org.apache.maven.plugins - maven-gpg-plugin - 1.6 - - - sign-artifacts - verify - - sign - - - - org.jacoco jacoco-maven-plugin @@ -150,56 +136,6 @@ - - - release-sign-artifacts - - - performRelease - true - - - - 41414BBD - - anshooarora - - - - - - - - org.apache.maven.plugins - maven-source-plugin - 2.3 - - - attach-sources - - jar - - - - - - org.apache.maven.plugins - maven-javadoc-plugin - 2.3 - - - attach-javadocs - - jar - - - - - - - - - ossrh diff --git a/src/main/java/com/aventstack/extentreports/ExtentTest.java b/src/main/java/com/aventstack/extentreports/ExtentTest.java index d346436..0eff914 100644 --- a/src/main/java/com/aventstack/extentreports/ExtentTest.java +++ b/src/main/java/com/aventstack/extentreports/ExtentTest.java @@ -136,7 +136,7 @@ public class ExtentTest implements IAddsMedia, RunResult, Serializab * * @return {@link ExtentTest} object */ - public ExtentTest createNode(Class type, String name, String description) { + public synchronized ExtentTest createNode(Class type, String name, String description) { if (name == null || name.isEmpty()) throw new IllegalArgumentException("nodeName cannot be null or empty"); @@ -160,7 +160,7 @@ public ExtentTest createNode(Class type, Strin * * @return {@link ExtentTest} object */ - public ExtentTest createNode(String name, String description) { + public synchronized ExtentTest createNode(String name, String description) { if (name == null || name.isEmpty()) throw new IllegalArgumentException("nodeName cannot be null or empty"); @@ -275,7 +275,7 @@ public ExtentTest createNode(String name) { return createNode(name, null); } - private void applyCommonNodeSettings(ExtentTest extentTest) { + private synchronized void applyCommonNodeSettings(ExtentTest extentTest) { extentTest.getModel().setLevel(test.getLevel() + 1); extentTest.getModel().setParent(getModel()); test.getNodeContext().add(extentTest.getModel()); diff --git a/src/main/java/com/aventstack/extentreports/model/service/TestService.java b/src/main/java/com/aventstack/extentreports/model/service/TestService.java index 1de5b69..d967747 100644 --- a/src/main/java/com/aventstack/extentreports/model/service/TestService.java +++ b/src/main/java/com/aventstack/extentreports/model/service/TestService.java @@ -4,6 +4,7 @@ import java.lang.reflect.Method; import com.aventstack.extentreports.gherkin.model.IGherkinFormatterModel; +import com.aventstack.extentreports.gherkin.model.ScenarioOutline; import com.aventstack.extentreports.model.Test; public class TestService { @@ -39,6 +40,18 @@ public static Boolean testHasLog(Test test) { public static Boolean testHasScreenCapture(Test test) { return !test.getScreenCaptureContext().isEmpty(); } + + public static Boolean testHasScreenCapture(Test test, Boolean deep) { + if (deep) { + Boolean hasScreenCapture = !test.getScreenCaptureContext().isEmpty() + || test.getLogContext().getAll().stream().anyMatch(LogService::logHasScreenCapture); + if (!hasScreenCapture) { + hasScreenCapture = test.getNodeContext().getAll().stream().anyMatch(x -> testHasScreenCapture(x, deep)); + } + return hasScreenCapture; + } + return testHasScreenCapture(test); + } public static Boolean isTestBehaviorDriven(Test test) { return test.getBddType() != null; @@ -76,7 +89,8 @@ public static String getHierarchicalName(Test test) { StringBuilder sb = new StringBuilder(test.getName()); while (test.getParent() != null) { test = test.getParent(); - sb.insert(0, test.getName() + "."); + if (!test.isBehaviorDrivenType() || test.getBddType() != ScenarioOutline.class) + sb.insert(0, test.getName() + "."); } return sb.toString(); } diff --git a/src/main/java/com/aventstack/extentreports/reporter/ExtentSparkReporter.java b/src/main/java/com/aventstack/extentreports/reporter/ExtentSparkReporter.java index 5ff5f7f..0ea22ea 100644 --- a/src/main/java/com/aventstack/extentreports/reporter/ExtentSparkReporter.java +++ b/src/main/java/com/aventstack/extentreports/reporter/ExtentSparkReporter.java @@ -73,8 +73,9 @@ public synchronized void flush(ReportAggregates reportAggregates) { try { if (viewStyle == ViewStyle.SPA) { + String fileName = getFileFile().isDirectory() ? getDestinationPath() + "Index.html" : getFilePath(); Template template = getFreemarkerConfig().getTemplate(SPA_TEMPLATE_NAME); - processTemplate(template, new File(getDestinationPath() + "index.html")); + processTemplate(template, new File(fileName)); return; } Template template = getFreemarkerConfig().getTemplate(TEST_TEMPLATE_NAME); diff --git a/src/main/java/com/aventstack/extentreports/reporter/configuration/ExtentHtmlReporterConfiguration.java b/src/main/java/com/aventstack/extentreports/reporter/configuration/ExtentHtmlReporterConfiguration.java index b44b73b..578b62b 100644 --- a/src/main/java/com/aventstack/extentreports/reporter/configuration/ExtentHtmlReporterConfiguration.java +++ b/src/main/java/com/aventstack/extentreports/reporter/configuration/ExtentHtmlReporterConfiguration.java @@ -6,7 +6,6 @@ import com.aventstack.extentreports.ExtentReports; import com.aventstack.extentreports.offline.OfflineResxDelegate; import com.aventstack.extentreports.reporter.BasicFileReporter; -import com.aventstack.extentreports.reporter.ExtentHtmlReporter; import com.aventstack.extentreports.utils.FileUtil; /** @@ -14,7 +13,8 @@ */ public class ExtentHtmlReporterConfiguration extends RichViewReporterConfiguration { - public ExtentHtmlReporterConfiguration(ExtentHtmlReporter reporter) { + @SuppressWarnings("deprecation") + public ExtentHtmlReporterConfiguration(com.aventstack.extentreports.reporter.ExtentHtmlReporter reporter) { super(reporter); } diff --git a/src/main/resources/com/aventstack/extentreports/offline/spark/css/spark-style.css b/src/main/resources/com/aventstack/extentreports/offline/spark/css/spark-style.css index 5ab7ae0..aaf555d 100644 --- a/src/main/resources/com/aventstack/extentreports/offline/spark/css/spark-style.css +++ b/src/main/resources/com/aventstack/extentreports/offline/spark/css/spark-style.css @@ -312,7 +312,8 @@ textarea { width: 110px; } img.r-img { - width: 5%; + width: 120px; + height: 75px; } /* nodes */ diff --git a/src/main/resources/com/aventstack/extentreports/view/commons/commons-inject-js.ftl b/src/main/resources/com/aventstack/extentreports/view/commons/commons-inject-js.ftl index 0a02f3f..dcf035d 100644 --- a/src/main/resources/com/aventstack/extentreports/view/commons/commons-inject-js.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/commons/commons-inject-js.ftl @@ -1,3 +1,4 @@ \ No newline at end of file diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/macros/recurse_nodes.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/macros/recurse_nodes.ftl index 4e5f2be..edf43a8 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/macros/recurse_nodes.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/macros/recurse_nodes.ftl @@ -7,10 +7,15 @@
diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/partials/bdd-content.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/partials/bdd-content.ftl index 4c74b0e..9747f4e 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/partials/bdd-content.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/partials/bdd-content.ftl @@ -23,6 +23,11 @@
+ <#if TestService.testHasScreenCapture(node, true)> +
+ +
+
<#if TestService.testHasChildren(node)> @@ -31,12 +36,17 @@ <#list node.nodeContext.all as child>
<#list child.nodeContext.all as step> diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/partials/head.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/partials/head.ftl index 99ea49a..0a8de70 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/partials/head.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/partials/head.ftl @@ -1,4 +1,4 @@ -<#assign resourceCDN=config.getConfig('resourceCDN') cdnURI="cdn.jsdelivr.net/gh/extent-framework/extent-github-cdn@" csscommit="b00a2d0486596e73dd7326beacf352c639623a0e" jscommit="b00a2d0486596e73dd7326beacf352c639623a0e" iconcommit="b00a2d0486596e73dd7326beacf352c639623a0e"> +<#assign resourceCDN=config.getConfig('resourceCDN') cdnURI="cdn.jsdelivr.net/gh/extent-framework/extent-github-cdn@" csscommit="74cc0d02f0116d1b5665bd07f6aea22c71033efe" jscommit="b00a2d0486596e73dd7326beacf352c639623a0e" iconcommit="b00a2d0486596e73dd7326beacf352c639623a0e"> <#if resourceCDN=="extentreports"> <#assign cdnURI="extentreports.com/resx" csscommit="" jscommit="" iconcommit=""> diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/partials/scripts.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/partials/scripts.ftl index 755f96a..8703fa7 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/partials/scripts.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/partials/scripts.ftl @@ -1,8 +1,6 @@ <#if offline=="true"> - - <#else> - - -<#if config.containsConfig("js") && config.getConfig("js")?has_content> -<#include "../../commons/commons-inject-js.ftl"> + +<#else> + +<#include "../../commons/commons-inject-js.ftl"> diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/partials/standard-content.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/partials/standard-content.ftl index c03fe81..e50b374 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/partials/standard-content.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/partials/standard-content.ftl @@ -7,11 +7,7 @@
diff --git a/src/main/resources/com/aventstack/extentreports/view/spark/partials/tag.ftl b/src/main/resources/com/aventstack/extentreports/view/spark/partials/tag.ftl index 3afb465..4bfbf91 100644 --- a/src/main/resources/com/aventstack/extentreports/view/spark/partials/tag.ftl +++ b/src/main/resources/com/aventstack/extentreports/view/spark/partials/tag.ftl @@ -61,7 +61,7 @@ ${test.name} <#if test.parent??>
- ${test.parent.name} + ${TestService.getHierarchicalName(test)}