Title: [947] trunk/jbehave-maven-plugin/src/main/java/org/jbehave/mojo/AbstractScenarioMojo.java: Refactored mojo to fail on the individual scenario instantiation to provide more accurate information to user.
Revision
947
Author
mauro
Date
2008-09-26 01:53:19 -0500 (Fri, 26 Sep 2008)

Log Message

Refactored mojo to fail on the individual scenario instantiation to provide more accurate information to user.

Modified Paths

Diff

Modified: trunk/jbehave-maven-plugin/src/main/java/org/jbehave/mojo/AbstractScenarioMojo.java (946 => 947)

--- trunk/jbehave-maven-plugin/src/main/java/org/jbehave/mojo/AbstractScenarioMojo.java	2008-09-24 17:50:17 UTC (rev 946)
+++ trunk/jbehave-maven-plugin/src/main/java/org/jbehave/mojo/AbstractScenarioMojo.java	2008-09-26 06:53:19 UTC (rev 947)
@@ -141,15 +141,20 @@
         if (names.isEmpty()) {
             getLog().info("No scenarios to run.");
         }
+        ScenarioClassLoader classLoader = null;
         try {
-            ScenarioClassLoader classLoader = createScenarioClassLoader();
-            List<RunnableScenario> scenarios = new ArrayList<RunnableScenario>();
-            for (String name : names) {
+            classLoader = createScenarioClassLoader();
+        } catch (Exception e) {
+            throw new MojoExecutionException("Failed to create scenario class loader", e);
+        }
+        List<RunnableScenario> scenarios = new ArrayList<RunnableScenario>();
+        for (String name : names) {
+            try {
                 scenarios.add(classLoader.newScenario(name));
+            } catch (Exception e) {
+                throw new MojoExecutionException("Failed to instantiate scenario '" + name + "'", e);
             }
-            return scenarios;
-        } catch (Exception e) {
-            throw new MojoExecutionException("Failed to instantiate scenarios " + names, e);
         }
+        return scenarios;
     }
 }


To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email

Reply via email to