You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by Carsten Ziegeler <cz...@apache.org> on 2009/06/23 10:09:28 UTC

Code Coverage and Metrics

Hi,

Sling has been added to the public Sonar instance:

http://nemo.sonarsource.org/project/index/org.apache.sling:sling-builder

So we get some nice graphics about code coverage, some metrics etc.

Atm we have 9% coverage :) Anyone interested in making us look better?

Regards
Carsten
-- 
Carsten Ziegeler
cziegeler@apache.org

Re: Code Coverage and Metrics

Posted by Alexander Klimetschek <ak...@day.com>.
2009/6/23 Dominik Süß <do...@gmail.com>:
> Cool stats :)
> Most metrics seam to be ok... but..
> Cyclomatic Complexity[1] of Apache Sling Scripting JSP Support with a value
> of 165 ... wow...

That is the Jasper JSP compiler, which is included as-is in Sling. And
compiler parsers are normally huge (because mostly code-generated)
classes. No need to worry about that ;-)

Regards,
Alex

-- 
Alexander Klimetschek
alexander.klimetschek@day.com

Re: Code Coverage and Metrics

Posted by Dominik Süß <do...@gmail.com>.
Cool stats :)
Most metrics seam to be ok... but..
Cyclomatic Complexity[1] of Apache Sling Scripting JSP Support with a value
of 165 ... wow...

"11+ refactor now !" (quote from the Sonar Hovertext)...

These metrics might really indicate where optimization could be required ;)

Best regards,
Dominik

[1]
http://nemo.sonarsource.org/drilldown/rules/107290?categ_id=11&metric=mandatory_violations&rule_id=13

On Tue, Jun 23, 2009 at 10:35 AM, Bertrand Delacretaz <
bdelacretaz@apache.org> wrote:

> On Tue, Jun 23, 2009 at 10:09 AM, Carsten Ziegeler<cz...@apache.org>
> wrote:
> > Sling has been added to the public Sonar instance:
> > http://nemo.sonarsource.org/project/index/org.apache.sling:sling-builder
> > So we get some nice graphics about code coverage, some metrics etc.
>
> Cool!
>
> > Atm we have 9% coverage :) Anyone interested in making us look better?
>
> That probably doesn't include our integration tests, or does it?
>
> I'd assume the only way to include those tests in the test coverage
> computation is to run the tests with an instrumented Sling server
> instance, can Sonar do that?
>
> -Bertrand
>

Re: Code Coverage and Metrics

Posted by Bertrand Delacretaz <bd...@apache.org>.
On Tue, Jun 23, 2009 at 10:09 AM, Carsten Ziegeler<cz...@apache.org> wrote:
> Sling has been added to the public Sonar instance:
> http://nemo.sonarsource.org/project/index/org.apache.sling:sling-builder
> So we get some nice graphics about code coverage, some metrics etc.

Cool!

> Atm we have 9% coverage :) Anyone interested in making us look better?

That probably doesn't include our integration tests, or does it?

I'd assume the only way to include those tests in the test coverage
computation is to run the tests with an instrumented Sling server
instance, can Sonar do that?

-Bertrand