I think I won't make optional, but always return ZERO as you suggested on the patch. AFIK, cobertura does java only.
On Mon, Sep 12, 2011 at 2:19 PM, [email protected] < [email protected]> wrote: > I apologize for naming this "branch coverage" it was a typo - it is > really about cobertura complexity calculation. > > Mark, > It's getting 0 complexity for every class for me, so I have to assume > it is not working or at least is not working in every case. > > I submitted a pull request to make the complexity calculation > optional. I'm not sure if this is what you would like to see. Please > give me some feedback if it is not. > > https://github.com/Flexmojos/flexmojos/pull/34 > > On Sep 9, 10:30 pm, MarkV <[email protected]> wrote: > > Cobertura tries to parse the flex code as java, i know it doesn't make > > sense that it still works, but it does. > > > > This is particular problem you are having is because the Java and > > flash declare their packages differently. > > > > On Sep 9, 1:15 pm, "[email protected]" > > > > > > > > > > > > > > > > <[email protected]> wrote: > > > When using 4.0-RC1 branch coverage I get a ton of errors from > > > Cobertura when it's doing the complexity calculations (output below > > > hundreds of times). Is this because the complexity calculations do > > > not work on flex code? Can the calculations be disabled? This does > > > not prevent the coverage information from being created correctly, it > > > just results in a lot of output when building. > > > > > net.sourceforge.cobertura.javancss.parser.ParseException: Encountered > > > " "{" "{ "" at line 24, column 1. > > > Was expecting: > > > ";" ... > > > > > at > > > > net.sourceforge.cobertura.javancss.parser.JavaParser.generateParseException > (JavaParser.java: > > > 10296) > > > at > > > > net.sourceforge.cobertura.javancss.parser.JavaParser.jj_consume_token(JavaP > arser.java: > > > 10172) > > > at > > > > net.sourceforge.cobertura.javancss.parser.JavaParser.PackageDeclaration(Jav > aParser.java: > > > 558) > > > at > > > > net.sourceforge.cobertura.javancss.parser.JavaParser.CompilationUnit(JavaPa > rser.java: > > > 312) > > > at > > > > net.sourceforge.cobertura.javancss.parser.JavaParser.parse(JavaParser.java: > > > 137) > > > at > > > > net.sourceforge.cobertura.javancss.Javancss._measureSource(Javancss.java: > > > 256) > > > at > > > net.sourceforge.cobertura.javancss.Javancss._measureRoot(Javancss.java: > > > 339) > > > at > > > net.sourceforge.cobertura.javancss.Javancss.<init>(Javancss.java:419) > > > at > > > > net.sourceforge.cobertura.reporting.ComplexityCalculator.getAccumlatedCCNFo > rSource(ComplexityCalculator.java: > > > 102) > > > at > > > > net.sourceforge.cobertura.reporting.ComplexityCalculator.getAccumlatedCCNFo > rSingleFile(ComplexityCalculator.java: > > > 139) > > > at > > > > net.sourceforge.cobertura.reporting.ComplexityCalculator.getCCNForSourceFil > eNameInternal(ComplexityCalculator.java: > > > 224) > > > at > > > > net.sourceforge.cobertura.reporting.ComplexityCalculator.getCCNForPackageIn > ternal(ComplexityCalculator.java: > > > 194) > > > at > > > > net.sourceforge.cobertura.reporting.ComplexityCalculator.getCCNForProject(C > omplexityCalculator.java: > > > 164) > > > at > > > > net.sourceforge.cobertura.reporting.html.HTMLReport.generateTableRowForTota > l(HTMLReport.java: > > > 704) > > > at > > > > net.sourceforge.cobertura.reporting.html.HTMLReport.generateOverview(HTMLRe > port.java: > > > 336) > > > at > > > > net.sourceforge.cobertura.reporting.html.HTMLReport.generateOverviews(HTMLR > eport.java: > > > 271) > > > at > > > > net.sourceforge.cobertura.reporting.html.HTMLReport.<init>(HTMLReport.java: > > > 96) > > > at > > > > org.sonatype.flexmojos.coverage.cobertura.CoberturaCoverageReport.generateR > eport(CoberturaCoverageReport.java: > > > 94) > > > at > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.run(TestRunMojo.java: > > > 341) > > > at > > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.execute_aroundBody0(TestRunM > ojo.java: > > > 283) > > > at > > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.execute_aroundBody1$advice(T > estRunMojo.java: > > > 24) > > > at > > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.execute_aroundBody2(TestRunM > ojo.java: > > > 270) > > > at > > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.execute_aroundBody3$advice(T > estRunMojo.java: > > > 17) > > > at > > > > org.sonatype.flexmojos.plugin.test.TestRunMojo.execute(TestRunMojo.java: > > > 1) > > > at > > > > org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildP > luginManager.java: > > > 101) > > > at > > > > org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java: > > > 209) > > > at > > > > org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java: > > > 153) > > > at > > > > org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java: > > > 145) > > > at > > > > org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(Lif > ecycleModuleBuilder.java: > > > 84) > > > at > > > > org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(Lif > ecycleModuleBuilder.java: > > > 59) > > > at > > > > org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(Li > fecycleStarter.java: > > > 183) > > > at > > > > org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStart > er.java: > > > 161) > > > at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java: > > > 319) > > > at org.apache.maven.DefaultMaven.execute(DefaultMaven.java: > > > 156) > > > at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537) > > > at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196) > > > at org.apache.maven.cli.MavenCli.main(MavenCli.java:141) > > > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > > > at > > > > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java: > > > 39) > > > at > > > > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImp > l.java: > > > 25) > > > at java.lang.reflect.Method.invoke(Method.java:597) > > > at > > > > org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.j > ava: > > > 290) > > > at > > > org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java: > > > 230) > > > at > > > > org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher > .java: > > > 409) > > > at > > > org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java: > > > 352) > > > WARN getAccumlatedCCNForSource, JavaNCSS got an error while parsing > > > the java file {removed file names .as} > > > ParseException in STDIN > > > Last useful checkpoint: "" > > > Encountered " "{" "{ "" at line 24, column 1. > > > Was expecting: > > > ";" ... > > -- > You received this message because you are subscribed to the Google > Groups "Flex Mojos" group. > To post to this group, send email to [email protected] > To unsubscribe from this group, send email to > [email protected] > For more options, visit this group at > http://groups.google.com/group/flex-mojos > > http://flexmojos.sonatype.org/ > -- You received this message because you are subscribed to the Google Groups "Flex Mojos" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/flex-mojos http://flexmojos.sonatype.org/
