#21

thiagolvlsantos · Jan 25, 2016 · 08b22ff · 08b22ff
1 parent f1ad8bc
commit 08b22ff
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/README.txt b/README.txt
@@ -1,5 +1,6 @@
 -----------------------------------------------------------
 1.4.57
+OK - Make 'execute' protected in scenario listener. #21 https://github.com/thiagolvlsantos/specrunner/issues/21
 OK - Remove title atribute setttings (refactoring of getPath(...)) for scenarios. #17 https://github.com/thiagolvlsantos/specrunner/issues/17
 OK - Define a tag interface to exceptions where stack traces are irrelevant to error undestanding #20 https://github.com/thiagolvlsantos/specrunner/issues/20
 	- UnstackedException interface created.

diff --git a/specrunner-core/src/main/java/org/specrunner/junit/JUnitUtils.java b/specrunner-core/src/main/java/org/specrunner/junit/JUnitUtils.java
@@ -302,8 +302,11 @@ public void afterScenario(String title, Node node, IContext context, IResultSet
                         } else if (r == null || r.countErrors() == 0) {
                             runner.getNotifier().fireTestFinished(description);
                         } else {
-                            String msg = "OUTPUT: " + runner.getStatement().getOutput().getAbsoluteFile() + "\n" + r.asString();
-                            runner.getNotifier().fireTestFailure(new Failure(description, new Exception(msg)));
+                            StringBuilder msg = new StringBuilder("OUTPUT: ");
+                            msg.append(runner.getStatement().getOutput().getAbsoluteFile());
+                            msg.append('\n');
+                            msg.append(r.asString());
+                            runner.getNotifier().fireTestFailure(new Failure(description, new Exception(msg.toString())));
                         }
                         time = System.currentTimeMillis() - time;
                         IOutput out = SRServices.get(IOutputFactory.class).currentOutput();

diff --git a/specrunner-core/src/main/java/org/specrunner/listeners/core/ScenarioFrameListener.java b/specrunner-core/src/main/java/org/specrunner/listeners/core/ScenarioFrameListener.java
@@ -93,7 +93,7 @@ public abstract class ScenarioFrameListener implements INodeListener {
     /**
      * Check if execute should be prioritized.
      */
-    private Boolean execute;
+    protected Boolean execute;
     /**
      * Scenario listeners.
      */