Missing many files in the GH Artifacts of CI ex-post.

[Tibor17]

According to the build result we have found out that the test failed (ForkedBooterTest.testThreadDump:190 NullPointer), the build crashed but we have no Artifact for the analysis.
We are missing a lots of files in the archive. Currently we can see only some XML reports of the ITs but that's not enough for the complete analysis. We are missing logs, dump files, etc. We are missing unit tests, and another ITs from the Failsafe module.

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

• [ x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[slawekjaranowski]

Paths are related to project workspace ....
so there is no directory maven-surefire

[slawekjaranowski]

we can try to add something like:

        **/target/*/log.txt
        **/target/**/surefire-reports/*
        **/target/**/failsafe-reports/*

so we will have included logs, reports from all modules

[Tibor17]

I was googling a bit and I found this structure could be generated in the runner .../maven-surefire/maven-surefire/... Let me please just break some test to see what would happen in the archive because I am also not sure. I have expected the ForkedBooterTest to fail. So let's break it now, see the archive content, and then I would take it back.

[slawekjaranowski]

According to the build result we have found out that the test failed (ForkedBooterTest.testThreadDump:190 NullPointer), the build crashed but we have no Artifact for the analysis.

Currently we save result only from ITs test ... from surefire-its module ... so when some module failed before IT we have empty archive

[Tibor17]

According to the build result we have found out that the test failed (ForkedBooterTest.testThreadDump:190 NullPointer), the build crashed but we have no Artifact for the analysis.

Currently we save result only from ITs test ... from surefire-its module ... so when some module failed before IT we have empty archive

Exactly. That's my point. Empty archive, or no existence of the archive was the reason why I opened this PR.
In Jenkins we observed everything and there was no problem to make the analysis. Currently, it is not possible because we don't have a relevant information from other modules. We don't have any results from unit tests. It does not mean that other modules except surefire-its have unit tests. The Failsafe has ITs as well.

[Tibor17]

@slawekjaranowski I have squashed the commits in one and dropped the fake test which forcibly failed.

[slawekjaranowski]

timeout can be done a simple separate PR

[slawekjaranowski]

I still not sure if we need everything eg, I see many files from source in archive ... do we need all

[slawekjaranowski]

I'm still not sure if we need all those files ... in archive

[Tibor17]

@slawekjaranowski so, i can proceed with this if there are no objections. Thx

[github-actions]

@Tibor17 Please assign appropriate label to PR according to the type of change.

[olamy]

@slawekjaranowski so, i can proceed with this if there are no objections. Thx

there was a question here https://github.com/apache/maven-surefire/pull/3219/files#r2534891172

not sure why *.java or even *.class are needed

[slawekjaranowski]

@slawekjaranowski so, i can proceed with this if there are no objections. Thx

there was a question here https://github.com/apache/maven-surefire/pull/3219/files#r2534891172

not sure why *.java or even *.class are needed

*.java , *.class are excluded ... but there are many other not needed files, like I show in my questions in thread

[olamy]

@slawekjaranowski so, i can proceed with this if there are no objections. Thx

there was a question here https://github.com/apache/maven-surefire/pull/3219/files#r2534891172
not sure why *.java or even *.class are needed

*.java , *.class are excluded ... but there are many other not needed files, like I show in my questions in thread

oh right sorry I didn't read the diff correctly.
Would be better to configure what we need/want rather than upload everything but those.