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

ASF GitHub Bot commented on SCM-981:
------------------------------------

nielsbasjes commented on code in PR #140:
URL: https://github.com/apache/maven-scm/pull/140#discussion_r879140561


##########
maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java:
##########
@@ -112,14 +114,19 @@ protected List<String> getScmFileNames()
     public abstract void initRepo()
         throws Exception;
 
+    private static final Logger LOG = LoggerFactory.getLogger( 
ScmTckTestCase.class );
+
     @Before
     public void checkScmPresence()
     {
         String scmProviderCommand = getScmProviderCommand();
         if ( scmProviderCommand != null )
         {
+            LOG.warn( "Checking for required SCM command \"{}\" --> {}",
+                    scmProviderCommand, ScmTestCase.isSystemCmd( 
scmProviderCommand ) );
             assumeTrue( "Skipping tests because the required command " + 
scmProviderCommand + " is not available.",
                 ScmTestCase.isSystemCmd( scmProviderCommand ) );
+            LOG.warn( "Required SCM command \"{}\" exists.", 
scmProviderCommand );

Review Comment:
   It works now, on MacOS on Github I see:
   ```
   2022-05-23T08:00:42.9665730Z [INFO] 
   2022-05-23T08:00:42.9767310Z [INFO] 
-------------------------------------------------------
   2022-05-23T08:00:42.9868750Z [INFO]  T E S T S
   2022-05-23T08:00:42.9969860Z [INFO] 
-------------------------------------------------------
   2022-05-23T08:00:43.2415810Z [INFO] Running 
org.apache.maven.scm.provider.hg.repository.HgScmProviderRepositoryTest
   2022-05-23T08:00:43.3204390Z [INFO] Tests run: 6, Failures: 0, Errors: 0, 
Skipped: 0, Time elapsed: 0.077 s - in 
org.apache.maven.scm.provider.hg.repository.HgScmProviderRepositoryTest
   2022-05-23T08:00:43.3223180Z [INFO] Running 
org.apache.maven.scm.provider.hg.HgUtilsTest
   2022-05-23T08:00:43.3792390Z [INFO] Tests run: 2, Failures: 0, Errors: 0, 
Skipped: 0, Time elapsed: 0.056 s - in 
org.apache.maven.scm.provider.hg.HgUtilsTest
   2022-05-23T08:00:43.3820030Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.update.HgUpdateCommandTckTest
   2022-05-23T08:00:43.4054560Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.024 s - in 
org.apache.maven.scm.provider.hg.command.update.HgUpdateCommandTckTest
   2022-05-23T08:00:43.4057710Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.changelog.HgChangeLogCommandTckTest
   2022-05-23T08:00:43.4130830Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.006 s - in 
org.apache.maven.scm.provider.hg.command.changelog.HgChangeLogCommandTckTest
   2022-05-23T08:00:43.4138580Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.diff.HgDiffCommandTckTest
   2022-05-23T08:00:43.4208220Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.007 s - in 
org.apache.maven.scm.provider.hg.command.diff.HgDiffCommandTckTest
   2022-05-23T08:00:43.4224000Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.status.HgStatusCommandTckTest
   2022-05-23T08:00:43.4280470Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.006 s - in 
org.apache.maven.scm.provider.hg.command.status.HgStatusCommandTckTest
   2022-05-23T08:00:43.4287380Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.checkout.HgCheckOutCommandTckTest
   2022-05-23T08:00:43.4345300Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.005 s - in 
org.apache.maven.scm.provider.hg.command.checkout.HgCheckOutCommandTckTest
   2022-05-23T08:00:43.4363420Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.blame.HgBlameCommandTckTest
   2022-05-23T08:00:43.4419300Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.006 s - in 
org.apache.maven.scm.provider.hg.command.blame.HgBlameCommandTckTest
   2022-05-23T08:00:43.4468160Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.checkin.HgCheckInCommandTckTest
   2022-05-23T08:00:43.4567260Z [WARNING] Tests run: 2, Failures: 0, Errors: 0, 
Skipped: 2, Time elapsed: 0.013 s - in 
org.apache.maven.scm.provider.hg.command.checkin.HgCheckInCommandTckTest
   2022-05-23T08:00:43.4574170Z [INFO] Running 
org.apache.maven.scm.provider.hg.command.tag.HgTagCommandTckTest
   2022-05-23T08:00:43.4628770Z [WARNING] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 1, Time elapsed: 0.005 s - in 
org.apache.maven.scm.provider.hg.command.tag.HgTagCommandTckTest
   2022-05-23T08:00:44.6834330Z [INFO] 
   2022-05-23T08:00:44.6834780Z [INFO] Results:
   2022-05-23T08:00:44.6837080Z [INFO] 
   2022-05-23T08:00:44.6837810Z [WARNING] Tests run: 17, Failures: 0, Errors: 
0, Skipped: 9
   ```





> Several integration tests are never run and fail if you do.
> -----------------------------------------------------------
>
>                 Key: SCM-981
>                 URL: https://issues.apache.org/jira/browse/SCM-981
>             Project: Maven SCM
>          Issue Type: Bug
>            Reporter: Niels Basjes
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 2.0.0-M2
>
>
> The README.md states
> {quote}Run all the tests with `mvn -Prun-its verify` to assure nothing else 
> was accidentally broken.
> {quote}
> The Github actions reuses for the CI build
> {code:java}
> apache/maven-gh-actions-shared/.github/workflows/maven-verify.yml@v2{code}
> which does the same
> {code:java}
> ./mvnw --errors --batch-mode --show-version -D"invoker.streamLogsOnFailures" 
> -P run-its verify{code}
> [https://github.com/apache/maven-gh-actions-shared/blob/v2/.github/workflows/maven-verify.yml#L167]
> [https://github.com/apache/maven-gh-actions-shared/blob/v2/.github/workflows/maven-verify.yml#L93]
> The snag is that this profile does not exist in this project.
> I asked this and was told to use the `tck-git` profile
> [https://github.com/apache/maven-scm/pull/134#issuecomment-1130214768]
> I found that there is also a `tck-svn` profile and a `tck` profile.
> When I run `mvn clean verify -Ptck` there are several tests that fail and are 
> apparently never run by anyone.
>  
> Proposal: 
>  * Create a profile run-its that simply activates the appropriate tck 
> profiles.
>  * Fix the failing tests.
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to