arturobernalg commented on PR #27: URL: https://github.com/apache/commons-exec/pull/27#issuecomment-1336519441
> > > @arturobernalg > > > Do you even run local builds before pushing out a PR? > > > Some of these tests will obviously fail, for example, see Exec65Test where the OS checks were not updated for JUnit 5 and cause the tests to fail. > > > > > > @garydgregory Honestly don't fallow you. AFAIU the test is in Junit5. > > No error in my local > > [WARNING] 24 warnings [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 01:43 min [INFO] Finished at: 2022-12-04T21:07:17+01:00 [INFO] ------------------------------------------------------------------------ > > Self explanatory @arturobernalg : > > ``` > Error: Errors: > Error: Exec65Test.testExec65WithSleepUsingShellScript:76 Execute The test 'testExec65WithSleepUsingShellScript()' is not possible for OS : Linux (Exit value: 0) > [INFO] > Error: Tests run: 104, Failures: 0, Errors: 1, Skipped: 9 > ``` I guest that is why there is a comment in the java docs ```This test currently only works for Mac OS X``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
