Mailing-List: contact dev-help@brooklyn.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@brooklyn.incubator.apache.org
From: duncangrant <git@git.apache.org>
To: dev@brooklyn.incubator.apache.org
Reply-To: dev@brooklyn.incubator.apache.org
References: <git-pr-455-incubator-brooklyn@git.apache.org>
In-Reply-To: <git-pr-455-incubator-brooklyn@git.apache.org>
Subject: [GitHub] incubator-brooklyn pull request: Cobertura to use maven
 plugin rat...
Content-Type: text/plain
Message-Id: <20150227152201.C3F1BE052F@git1-us-west.apache.org>
Date: Fri, 27 Feb 2015 15:22:01 +0000 (UTC)

Github user duncangrant commented on the pull request:

    https://github.com/apache/incubator-brooklyn/pull/455#issuecomment-76410811
  
    The current ant-run based integration does produce output.xml.  This is useful for integration with jenkins and if we want to produce reports from the output.  For this reason I am going to close rather than merge until I can investigate the issue.
    
    I hadn't notice it running the tests twice before.  According to @grkvlt this is because cobertura runs the tests once to confirm they pass and then instruments them and runs them again.
    
    I haven't ever managed to get the integration tests to run on my machine so I still don't know what the coverage will look like for those.  Presumably it is possible to get both unit tests and integration. 
    

---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---