Re: [papyrus-rt-dev] [Papyrus RT] Local execution : **/*AllTests

[MAGGI Benoit <Benoit.MAGGI@xxxxxx>]

Hi,

 

Thanks Christian for the explanatation

Thanks Céline for your proposition, Are you sure it works with [1] ?

 

I will take any solution that allow to build on both unix/windows.

 

Regards,

Benoit

 

1 : tests\junit\umlrt\tooling\org.eclipse.papyrusrt.umlrt.tooling.types.tests\pom.xml

<properties>

       <!-- Exclude the EMF-generated tests, because they are integrated explicitly by a suite. -->         <test.excludes>${test.excludes.default},**/papyrusrt/umlrt/tooling/types/umlrttypes/tests/*.java</test.excludes>

  </properties>

 

De : papyrus-rt-dev-bounces@xxxxxxxxxxx [mailto:papyrus-rt-dev-bounces@xxxxxxxxxxx] De la part de Céline JANSSENS
Envoyé : mardi 21 juin 2016 15:42
À : papyrus-rt-dev@xxxxxxxxxxx
Objet : Re: [papyrus-rt-dev] [Papyrus RT] Local execution : **/*AllTests

 

Hi Benoit, Christian,

This can be fixed by encapsulating the Tests filter (i.e."**/*AllTests.java" ) into <exclude> or <include>  tag into the <excludes> and <includes> sections.

It has been include in the following gerrit:

https://git.eclipse.org/r/75605

Best regards

Céline

Le 21/06/2016 à 14:10, Christian Damus a écrit :

 

Hi, Benoit,

 

Yes, I originally specified the exclusions as a single comma-separated string because, on Linux, maven failed to parse the list-element syntax that seems to be required on Windows.

 

It would be difficult to remove this pattern because it is there to ensure that we don’t have redundant execution of tests for EMF models.  EMF generates test suites named AllTests and XyzAllTests.  I suppose we could use custom codegen templates to change the names of these suite classes, if it’s important.

 

Christian

 

On 21 June, 2016 at 05:23:04, MAGGI Benoit (benoit.maggi@xxxxxx) wrote:

Hi,

 

I have a problem when running the build on a windows machine (see [1])

I think it’s mandatory to have a build that is cross-platform especially for a java based product.

 

The problem comes from the exclusion pattern **/*AllTests

(There is maybe something wrong on  maven/Tycho side see [3] but it won’t be solved soon.)

 

From what I understand, it’s used to remove the Suite ‘AllTests’ from the build execution.

One proposition would be to rename the suites from “AllTests” to “Suite” (or just delete these suites if it’s not in the build execution)

It should work since we will be back following the convention [2]

 

Any suggestion ?

 

Regards,

Benoit

 

1 : https://git.eclipse.org/r/#/c/73962/

mvn clean install -Preleng

and got

[INFO] org.eclipse.papyrusrt.tests ........................ SUCCESS [ 0.010 s]

[INFO] org.eclipse.papyrusrt.junit ........................ SUCCESS [ 1.582 s]

[INFO] org.eclipse.papyrusrt.umlrt.profile.tests .......... FAILURE [ 1.086 s]

[INFO] org.eclipse.papyrusrt.umlrt.common.ui.tests ........ SKIPPED

 

2 : http://maven.apache.org/surefire/maven-surefire-plugin/examples/inclusion-exclusion.html

3 : https://bugs.eclipse.org/bugs/show_bug.cgi?id=496411

 

 

 

 

 

_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

_______________________________________________

papyrus-rt-dev mailing list

papyrus-rt-dev@xxxxxxxxxxx

To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

 

--

 

	Céline JANSSENS Software Engineer +33 (0)2 44 47 23 23
	Mail : cej@xxxxxxxxxxx
6 rue Léonard De Vinci - BP 0119 - 53001 LAVAL Cedex - FRANCE www.all4tec.net

 

Garanti sans virus. www.avast.com