Merge pull request #5286 from jdi-testing/fix_npe_cookies

Add debug info for errors

Author: pnatashap

jdi-bdd-tests/pom.xml

@@ -26,7 +26,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.text=ALL-UNNAMED  --add-opens=java.desktop/java.awt.font=ALL-UNNAMED</aspectj.args>

jdi-light-angular-tests-with-mocks/pom.xml

@@ -25,7 +25,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-angular-tests/pom.xml

@@ -146,7 +146,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-applitools-tests/pom.xml

@@ -30,7 +30,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-bootstrap-tests/pom.xml

@@ -26,7 +26,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-core/src/main/java/com/epam/jdi/light/actions/ActionHelper.java

@@ -395,7 +395,7 @@ public static RuntimeException actionFailed(ActionObject jInfo, Throwable ex) {
         }
         addFailedMethod(jInfo.jp());
         if (jInfo.topLevel()) {
-            logFailure(jInfo);
+            logFailure(jInfo, ex);
             reverse(failedMethods);
             List<String> chainActions = new ArrayList<>(failedMethods);
             try {
@@ -415,8 +415,7 @@ public static RuntimeException actionFailed(ActionObject jInfo, Throwable ex) {
 
     public static JFunc2<ActionObject, Throwable, RuntimeException> ACTION_FAILED = ActionHelper::actionFailed;
 
-    public static void logFailure(ActionObject jInfo) {
-        logger.error("!>>> " + jInfo.object().toString());
+    public static void logFailure(ActionObject jInfo, Throwable ex) {
         if (ObjectUtils.isNotEmpty(ELEMENT.highlight) && !ELEMENT.highlight.contains(HighlightStrategy.OFF)) {
             if (ELEMENT.highlight.contains(HighlightStrategy.FAIL)) {
                 try {
@@ -427,6 +426,7 @@ public static void logFailure(ActionObject jInfo) {
         showElement(jInfo);
         AllureLogData logData = logDataToAllure(FAIL,
             "Failed" + capitalize(jInfo.methodName()), jInfo.isAssert());
+        logger.error("!>>> " + jInfo.object().toString() + " Failed " + capitalize(jInfo.methodName()), ex);
         failStep(jInfo.stepUId, logData);
     }
 

jdi-light-html-tests/pom.xml

@@ -135,7 +135,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-material-ui-tests/pom.xml

@@ -118,7 +118,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-mobile-selenium3-tests/pom.xml

@@ -33,7 +33,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>

jdi-light-mobile-tests/pom.xml

@@ -34,7 +34,7 @@
         <profile>
             <id>jdk16</id>
             <activation>
-                <jdk>[16, 20]</jdk>
+                <jdk>[16,)</jdk>
             </activation>
             <properties>
                 <aspectj.args>--add-opens java.base/java.lang=ALL-UNNAMED</aspectj.args>