Author: tchemit Date: 2008-10-18 02:09:55 +0000 (Sat, 18 Oct 2008) New Revision: 967 Modified: lutinjaxx/trunk/maven-jaxx-plugin/src/main/java/org/codelutin/jaxx/JaxxGeneratorMojo.java lutinjaxx/trunk/maven-jaxx-plugin/src/test/java/org/codelutin/jaxx/CompilerTest.java Log: minimize logging when testing errors and do not test n times the same files :) Modified: lutinjaxx/trunk/maven-jaxx-plugin/src/main/java/org/codelutin/jaxx/JaxxGeneratorMojo.java =================================================================== --- lutinjaxx/trunk/maven-jaxx-plugin/src/main/java/org/codelutin/jaxx/JaxxGeneratorMojo.java 2008-10-18 02:02:29 UTC (rev 966) +++ lutinjaxx/trunk/maven-jaxx-plugin/src/main/java/org/codelutin/jaxx/JaxxGeneratorMojo.java 2008-10-18 02:09:55 UTC (rev 967) @@ -250,6 +250,11 @@ return; } + doAction(); + + } + + public void doAction() throws MojoExecutionException { getLog().info("jaxx - will generate " + this.files.length + " jaxx file(s). "); try { @@ -271,7 +276,6 @@ throw new MojoExecutionException(e.getMessage(), e); } - } protected void fixCompileSourceRoots() { Modified: lutinjaxx/trunk/maven-jaxx-plugin/src/test/java/org/codelutin/jaxx/CompilerTest.java =================================================================== --- lutinjaxx/trunk/maven-jaxx-plugin/src/test/java/org/codelutin/jaxx/CompilerTest.java 2008-10-18 02:02:29 UTC (rev 966) +++ lutinjaxx/trunk/maven-jaxx-plugin/src/test/java/org/codelutin/jaxx/CompilerTest.java 2008-10-18 02:09:55 UTC (rev 967) @@ -67,6 +67,7 @@ String[] files = mojo.getFiles(); // execute mjo on each jaxx file to produce the error for (String file : files) { + getLog().info("test bad file "+file); mojo.setFiles(new String[]{file}); try { mojo.setLog(new SystemStreamLog() { @@ -86,15 +87,12 @@ //do nothing } }); - mojo.execute(); + mojo.doAction(); // should never pass fail(); } catch (MojoExecutionException e) { // ok jaxx compiler failed assertTrue(true); - } catch (MojoFailureException e) { - // this exception should not appear - fail(); } } }