[ 
https://issues.apache.org/jira/browse/SLING-4757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14602559#comment-14602559
 ] 

Satya Deep Maheshwari commented on SLING-4757:
----------------------------------------------

I need to remotely connect to the maven build while its executing the ITs to 
debug something I was trying out. By default, surefire forks a different thread 
for ITs. In order to be able to connect, I need it to run on the same thread as 
the maven build. Please see [1]

The mvn based url (i.e. mvn://...) seems to be related to 
{{<java.protocol.handler.pkgs>org.ops4j.pax.url</java.protocol.handler.pkgs>}}

[1] - 
http://maven.apache.org/surefire/maven-surefire-plugin/examples/debugging.html

> Improve test coverage and add integration tests to the contentdetection module
> ------------------------------------------------------------------------------
>
>                 Key: SLING-4757
>                 URL: https://issues.apache.org/jira/browse/SLING-4757
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>            Reporter: Bertrand Delacretaz
>            Assignee: Petr Shypila
>         Attachments: 
> FileNameExtractorImpl_Injection_failed__ClassNotFoundException.patch, 
> content-tampering-test.patch, week2-part1-unit-tests.patch, 
> week2-part2-integration-tests.patch
>
>
> The new SLING-4694 contentdetection module has reasonable test coverage but 
> there are some gaps, and it's also missing some integration tests to verify 
> that the services provided by the bundle are correctly made available in an 
> OSGI environment.
> I'll take this opportunity to ask our GSoC student [~petr.shypila] to expand 
> these tests, as that's a good example of what we're trying to do with his 
> tests improvement project.
> So [~petr.shypila] here's your "mission" ;-)
> Buliding bundles/commons/contentdetection with with -Pjacoco-report and 
> looking at target/site/jacoco/index.html shows that a few things are not 
> tested. The first step is to add the missing unit tests to improve that, 
> maybe provide a first patch with just that.
> Then, we'd like to add integration tests that start this bundle in an OSGi 
> environment and verify that the ContentAwareMimeTypeService and 
> FileNameExtractor services are available and work. No need to duplicate the 
> unit tests, just verify that the services are here and work in a simple case.
> The best way to do this is probably with Pax Exam, the 
> ResourceBundleProviderIT. is a good example of that and there are other 
> examples in our codebase if you search for *IT.java files that contain 
> "pax.exam".
> As usual, try to minimize changes to the pom (except for what's directly 
> related to testing) and to non-test code.
> Feel free to ask if you have any questions of course, either here for minor 
> ones or on the dev list for larger discussions.
> [1]  
> https://svn.apache.org/repos/asf/sling/trunk/bundles/extensions/i18n/src/test/java/org/apache/sling/i18n/it/ResourceBundleProviderIT.java



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to