brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From duncangrant <...@git.apache.org>
Subject [GitHub] incubator-brooklyn pull request: Cobertura to use maven plugin rat...
Date Fri, 27 Feb 2015 15:22:01 GMT
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.
---

Mime
View raw message