Hi Michael,

I think Robert Scholte's vision on the ordering of this list is to order
them on the date of implementation [1].
The rationale being that the latest implemented tests are potentially the
least stable.

I personally do not have a strong preference, but we should come to a
consensus.

Regards,
Martin

[1]
https://github.com/apache/maven-integration-testing/commit/077b9d3c9d9efcd9192082516d358b943ea3edcf#diff-02304634c9e39f8f1ec68f6cb88d51edR89

Op wo 5 aug. 2020 om 20:57 schreef <micha...@apache.org>:

> This is an automated email from the ASF dual-hosted git repository.
>
> michaelo pushed a commit to branch master
> in repository
> https://gitbox.apache.org/repos/asf/maven-integration-testing.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
>      new 59bcf77  Sort ITs in reverse numerical order
> 59bcf77 is described below
>
> commit 59bcf77d4f43115e3779933e4eeef2676af60300
> Author: Michael Osipov <micha...@apache.org>
> AuthorDate: Wed Aug 5 20:57:33 2020 +0200
>
>     Sort ITs in reverse numerical order
> ---
>  .../java/org/apache/maven/it/IntegrationTestSuite.java     | 14
> +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git
> a/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
> b/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
> index 1198133..ee41a83 100644
> ---
> a/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
> +++
> b/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
> @@ -108,15 +108,10 @@ public class IntegrationTestSuite
>          // suite.addTestSuite( MavenIT0108SnapshotUpdateTest.class ); --
> MNG-3137
>
>          suite.addTestSuite(
> MavenITmng6972AllowAccessToGraphPackageTest.class );
> -        suite.addTestSuite( MavenITmng5760ResumeFeatureTest.class );
> -        suite.addTestSuite( MavenITmng6656BuildConsumer.class );
> -        suite.addTestSuite( MavenITmng6562WarnDefaultBindings.class );
> -        suite.addTestSuite(
> MavenITmng5868NoDuplicateAttachedArtifacts.class );
> -        suite.addTestSuite( MavenITmng5937MavenWrapper.class );
> -        suite.addTestSuite( MavenITmng4660ResumeFromTest.class );
> -        suite.addTestSuite( MavenITmng4660OutdatedPackagedArtifact.class
> );
>          suite.addTestSuite(
> MavenITmng6759TransitiveDependencyRepositoriesTest.class );
>          suite.addTestSuite( MavenITmng6720FailFastTest.class );
> +        suite.addTestSuite( MavenITmng6656BuildConsumer.class );
> +        suite.addTestSuite( MavenITmng6562WarnDefaultBindings.class );
>          suite.addTestSuite(
> MavenITmng6558ToolchainsBuildingEventTest.class );
>          suite.addTestSuite( MavenITmng6506PackageAnnotationTest.class );
>          suite.addTestSuite( MavenITmng6391PrintVersionTest.class );
> @@ -139,10 +134,12 @@ public class IntegrationTestSuite
>          suite.addTestSuite( MavenITmng6057CheckReactorOrderTest.class );
>          suite.addTestSuite(
> MavenITmng5965ParallelBuildMultipliesWorkTest.class );
>          suite.addTestSuite(
> MavenITmng5958LifecyclePhaseBinaryCompat.class );
> +        suite.addTestSuite( MavenITmng5937MavenWrapper.class );
>          suite.addTestSuite(
> MavenITmng5935OptionalLostInTranstiveManagedDependenciesTest.class );
>          suite.addTestSuite(
> MavenITmng5898BuildMultimoduleWithEARFailsToResolveWARTest.class );
>          suite.addTestSuite(
> MavenITmng5895CIFriendlyUsageWithPropertyTest.class );
>          suite.addTestSuite( MavenITmng5889FindBasedir.class );
> +        suite.addTestSuite(
> MavenITmng5868NoDuplicateAttachedArtifacts.class );
>          suite.addTestSuite(
> MavenITmng5840RelativePathReactorMatching.class );
>          suite.addTestSuite( MavenITmng5840ParentVersionRanges.class );
>          suite.addTestSuite( MavenITmng5805PkgTypeMojoConfiguration2.class
> );
> @@ -151,6 +148,7 @@ public class IntegrationTestSuite
>          suite.addTestSuite(
> MavenITmng5774ConfigurationProcessorsTest.class );
>          suite.addTestSuite( MavenITmng5771CoreExtensionsTest.class );
>          suite.addTestSuite( MavenITmng5768CliExecutionIdTest.class );
> +        suite.addTestSuite( MavenITmng5760ResumeFeatureTest.class );
>          suite.addTestSuite(
> MavenITmng5753CustomMojoExecutionConfiguratorTest.class );
>          suite.addTestSuite(
> MavenITmng5742BuildExtensionClassloaderTest.class );
>          suite.addTestSuite( MavenITmng5716ToolchainsTypeTest.class );
> @@ -242,6 +240,8 @@ public class IntegrationTestSuite
>          suite.addTestSuite(
> MavenITmng4679SnapshotUpdateInPluginTest.class );
>          suite.addTestSuite(
> MavenITmng4677DisabledPluginConfigInheritanceTest.class );
>          suite.addTestSuite( MavenITmng4666CoreRealmImportTest.class );
> +        suite.addTestSuite( MavenITmng4660ResumeFromTest.class );
> +        suite.addTestSuite( MavenITmng4660OutdatedPackagedArtifact.class
> );
>          suite.addTestSuite(
> MavenITmng4654ArtifactHandlerForMainArtifactTest.class );
>          suite.addTestSuite(
> MavenITmng4644StrictPomParsingRejectsMisplacedTextTest.class );
>          // suite.addTestSuite(
> MavenITmng4633DualCompilerExecutionsWeaveModeTest.class );
>
>

Reply via email to