[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2018-11-03 Thread Rustam Abdullaev (JIRA)


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

Rustam Abdullaev commented on SUREFIRE-1198:


Ah OK thanks! We don't inherit from spring-boot-starter-parent (because we must 
inherit from our corporate parent pom), so didn't have that configuration. It's 
working now indeed.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Bug
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>Priority: Major
> Fix For: 2.20
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2018-11-03 Thread Stephane Nicoll (JIRA)


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

Stephane Nicoll commented on SUREFIRE-1198:
---

I don't thinks that error is related to this issue. Spring Boot is using 
Failsafe 2.22.1 in {{spring-boot-starter-parent}} with this extra configuration

{{noformat}}

${project.build.outputDirectory}

{{noformat}}

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Bug
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>Priority: Major
> Fix For: 2.20
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2018-11-03 Thread Rustam Abdullaev (JIRA)


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

Rustam Abdullaev commented on SUREFIRE-1198:


Failsafe is NOT working with Spring Boot in repackage mode since versionĀ 2.19.

{code}

org.springframework.boot
spring-boot-maven-plugin
2.1.0.RELEASE



repackage




{code}

Causes

{code}
java.lang.ArrayStoreException: 
sun.reflect.annotation.TypeNotPresentExceptionProxy
at 
sun.reflect.annotation.AnnotationParser.parseClassArray(AnnotationParser.java:724)
at 
sun.reflect.annotation.AnnotationParser.parseArray(AnnotationParser.java:531)
at 
sun.reflect.annotation.AnnotationParser.parseMemberValue(AnnotationParser.java:355)
at 
sun.reflect.annotation.AnnotationParser.parseAnnotation2(AnnotationParser.java:286)
at 
sun.reflect.annotation.AnnotationParser.parseAnnotations2(AnnotationParser.java:120)
at 
sun.reflect.annotation.AnnotationParser.parseAnnotations(AnnotationParser.java:72)
at java.lang.Class.createAnnotationData(Class.java:3521)
at java.lang.Class.annotationData(Class.java:3510)
at java.lang.Class.getAnnotation(Class.java:3415)
at 
org.apache.maven.surefire.common.junit4.JUnit4TestChecker.isValidJUnit4Test(JUnit4TestChecker.java:66)
at 
org.apache.maven.surefire.common.junit4.JUnit4TestChecker.accept(JUnit4TestChecker.java:53)
at 
org.apache.maven.surefire.util.DefaultScanResult.applyFilter(DefaultScanResult.java:102)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.scanClassPath(JUnit4Provider.java:309)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.setTestsToRun(JUnit4Provider.java:189)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:132)
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:384)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:345)
at 
org.apache.maven.surefire.booter.ForkedBooter.execute(ForkedBooter.java:126)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:418)
{code}

So we're stuck with version *2.18.1* at the moment.

How to use Surefire with the repackage goal?

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Bug
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>Priority: Major
> Fix For: 2.20
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-24 Thread Hudson (JIRA)

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

Hudson commented on SUREFIRE-1198:
--

SUCCESS: Integrated in Jenkins build maven-surefire #1627 (See 
[https://builds.apache.org/job/maven-surefire/1627/])
[SUREFIRE-1198] Failsafe does not allow to configure the jar file to use 
(tibor17: rev 432231e7e9d01b9ef109acd39176045e0b18e5a5)
* (edit) 
maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java


> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-24 Thread Hudson (JIRA)

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

Hudson commented on SUREFIRE-1198:
--

SUCCESS: Integrated in Jenkins build maven-surefire #1626 (See 
[https://builds.apache.org/job/maven-surefire/1626/])
[SUREFIRE-1198] Failsafe does not allow to configure the jar file to use 
(tibor17: rev f1aea63320b66e635e0dc27e4f9ecd0e155099d9)
* (add) 
maven-failsafe-plugin/src/test/java/org/apache/maven/plugin/failsafe/IntegrationTestMojoTest.java
* (edit) 
maven-surefire-plugin/src/site/apt/examples/configuring-classpath.apt.vm
* (edit) maven-failsafe-plugin/pom.xml
* (edit) 
maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
* (edit) maven-surefire-plugin/src/site/fml/faq.fml
* (edit) pom.xml


> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-24 Thread Phillip Webb (JIRA)

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

Phillip Webb commented on SUREFIRE-1198:


Thanks [~tibor17]!! Exactly what I had in mind.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-24 Thread Phillip Webb (JIRA)

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

Phillip Webb commented on SUREFIRE-1198:


> I think here is the fundamental problem in Spring group using integration 
> tests

Just to be clear, the issue here was raised by Spring Boot *users*, we've not 
hit the problem in our own build.

> The ITs are inherently testing the interaction between jar files which is the 
> purpose. So I am wondering why if these jars can run in the application 
> server, the same file have problem to run in tests

Spring Boot creates "Fat Jar" files which a fully executable, there is no need 
for an application server. The layout is conceptually similar to a WAR file. 
Classes are under a {{BOOT-INF/classes}} folder and dependent libraries are 
under {{BOOT-INF/lib}}. We have a custom {{URLHandler}} and {{Classloader}} 
that actually launches the application.

> I think it is unit tests started by maven-surefire-plugin which you should 
> rather use instead of maven-failsafe-plugin. Surefire tests classes in 
> particular module which is smaller granularity of tests.


I think it's a little confusing to suggest that users should switch to surefire 
for integration tests, especially given that the plugin previously worked well. 
I'm especially concerned that there is so much documentation out there for 
using failsafe, and so many differences with configuration (e.g. the bound 
lifecycle, the use of *IT.java) that it will be quite hard to explain why it 
shouldn't be used.

I did look into how we might be able to plug in a different classloader, but 
it's quite hard because most of the classpath is actually built in 
{{AbstractSurefireMojo.generateTestClasspath}}. We really don't want to mess 
with that, we just want to return a different JAR file (as it appeared before 
we repackaged it) from {{getClassesDirectory()}} or failing that use the 
{{project.build.outputDirectory}}.

I'm still wondering why we can't use the {{classesDirectory}} parameter. It 
seems odd to me that the setting is completely ignored when dealing with 
{{.jar}} files. I haven't had time to actually try the change yet, but I was 
hoping to submit a PR that:

* Changed {{classesDirectory}} so that it no longer has a {{defaultValue}}.

* Updated {{getClassesDirectory()}} so that if {{classesDirectory}} is not 
{{null}} it's returned

* Update the final line to {{return useArtifactFile ? artifactFile : 
${project.build.outputDirectory};}}

Is that something that's worth pursuing? It seems a lot simpler than trying to 
write another classloader, and I think it would be generally useful for any 
project that does something different with the primary artifact JAR.






> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-24 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on SUREFIRE-1198:
--

{quote}In the short term ...{quote}
There are no temporary solutions. If we introduce a parameter, it'll stay there 
until at least the next major version. I have good hope that this can be fixed 
with the custom classloader option, so let's fix it immediately correct.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-23 Thread Tibor Digana (JIRA)

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

Tibor Digana commented on SUREFIRE-1198:


[~pwebb]
I think here is the fundamental problem in Spring group using integration 
tests. The ITs are inherently testing the interaction between jar files which 
is the purpose. So I am wondering why if these jars can run in the application 
server, the same file have problem to run in tests. I think it is unit tests 
started by {{maven-surefire-plugin}} which you should rather use instead of 
{{maven-failsafe-plugin}}. Surefire tests classes in particular module which is 
smaller granularity of tests.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-19 Thread Phillip Webb (JIRA)

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

Phillip Webb commented on SUREFIRE-1198:


That would be an ideal solution. In the short term, if we could just tell the 
plugin to use the backed up jar rather than the repackaged one, then we should 
be good. Spring Boot effectively creates a backup of the original JAR before it 
repackages it, I imagine if we pointed failsafe at that backup JAR then 
everything would work as before.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>Assignee: Tibor Digana
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-19 Thread Tibor Digana (JIRA)

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

Tibor Digana commented on SUREFIRE-1198:


[~pwebb]
A custom classloader is a good idea. I would propose the issue in 3.0 or 3.1.
We want to let the users give a chance to "inject" the code to 
Surefire/Failsafe from outside without asking the committer.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-19 Thread Phillip Webb (JIRA)

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

Phillip Webb commented on SUREFIRE-1198:


The javadoc for {{classesDirectory}} appears to suggest that it could be used 
to specify an alternative JAR file, but this is not the case when the project 
artifact is set:

{code:java}
/**
 * The path representing project JAR file, if exists; Otherwise 
the directory containing generated
 * classes of the project being tested. This will be included after the 
test classes in the test classpath.
 */
@Parameter( defaultValue = "${project.build.outputDirectory}" )
private File classesDirectory;
{code}

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-09-19 Thread Phillip Webb (JIRA)

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

Phillip Webb commented on SUREFIRE-1198:


I spoke with [~rfscholte] a little about this issue today. One option we might 
have is to find some way of plugging in a custom classloader that can deal with 
Spring Boot's layouts. This does seem quite involved so I wondering if instead 
we shouldn't look at what can be done with the {{getClassesDirectory()}} 
method. Currently it looks like this:

{code:java}
public File getClassesDirectory()
{
Artifact artifact = getProject().getArtifact();
File artifactFile = artifact.getFile();

boolean useArtifactFile = artifactFile != null && artifactFile.isFile()
&& artifactFile.getName().toLowerCase().endsWith( ".jar" );

return useArtifactFile ? artifactFile : classesDirectory;
}
{code}

This is appears to be saying "use the main artifact, if it's a JAR, otherwise 
use the classesDirectory". So, if the main artifact is set (which it will be if 
failsafe is bound the expected lifecyle) there is no way to force the 
{{classesDirectory}} to be used. Perhaps we could change that logic so that a 
specific {{classesDirectory}} attribute has been configured, it's always used. 
That way Spring Boot could either set {{classesDirectory}} to 
{{${project.build.outputDirectory}} to force use of the folder or, more likely, 
set it to point to the original jar (before it was repackaged).



> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-07-01 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on SUREFIRE-1198:
--

Failsafe *prefers* to use the jar. If you bind it to an earlier phase when 
there's no jar, it'll use the folder with the test-classes. But then you should 
wonder why not using the maven-surefire-plugin instead?
So I had a look at the {{Spring-Boot-Maven-Failsafe-Plugin-Issue-master}} 
project. It seems like you've decided to introduce a {{BOOT-INFO/classes}}, the 
{{META-INF/MANIFEST.MF}} has an entry to this folder.
This made me aware that the plugin is actually still not good enough. In case 
of an executable jar ( i.e the Main-Class entry is specified) where the 
Class-Path is specified as well, it should use those entries. Spring Boot does 
something similar, but with own specified entries.
So the fix should be to be able to map or redirect classes.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-07-01 Thread Stephane Nicoll (JIRA)

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

Stephane Nicoll commented on SUREFIRE-1198:
---

I disagree. Builds have different requirements. This particular one makes it so 
that the jar that should be used for ITs is not the main artifact of the 
project. Since we're forced to use the jar, making it configurable doesn't 
sound insane to me.

> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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


[jira] [Commented] (SUREFIRE-1198) Failsafe does not allow to configure the jar file to use

2016-06-30 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on SUREFIRE-1198:
--

IIUC the Spring Boot jar is a jar-in-name-only: it doesn't follow the 
specifications of a jar, classes are not in the root but in a subdirectory. I 
assume you're using a custom classloader to execute this jar, which means that 
the maven-surefire-plugin should do the same.


> Failsafe does not allow to configure the jar file to use
> 
>
> Key: SUREFIRE-1198
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1198
> Project: Maven Surefire
>  Issue Type: Improvement
>Reporter: Stephane Nicoll
>
> See [this Spring Boot 
> issue|https://github.com/spring-projects/spring-boot/issues/4510#issuecomment-159448634]
> It seems that SUREFIRE-855 does not allow {{target/classes}} to be used 
> anymore. Is there a reason why this behaviour was completely removed in 
> favour of only the jar file?
> It would be nice if we had an option to chose between the two (defaulting to 
> the jar)



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