Skip to content

Fix CentOS 6 test failures#690

Merged
MarkEWaite merged 3 commits intojenkinsci:masterfrom
MarkEWaite:fix-centos-6-test-failures
Mar 12, 2019
Merged

Fix CentOS 6 test failures#690
MarkEWaite merged 3 commits intojenkinsci:masterfrom
MarkEWaite:fix-centos-6-test-failures

Conversation

@MarkEWaite
Copy link
Contributor

Skip some unit tests with CLI git 1.7.1

Since CLI git 1.7.1 is not officially supported, the tests are skipped for that older version of command line git. We prefer to run tests on CentOS 6 to reduce the risk of breaking use cases that work, even knowing that the default version of command line git on CentOS 6 is not supported by the git client plugin.

Git 1.7.1 functionality gaps are described in the unit tests of the git plugin and the git client plugin.

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that verify my changes
  • Unit tests pass locally with my changes
  • I have added documentation as necessary
  • No Javadoc warnings were introduced with my changes
  • No findbugs warnings were introduced with my changes

Types of changes

  • Non-breaking change

Git 2.21.0 has been released and is a good version to use.
@MarkEWaite MarkEWaite force-pushed the fix-centos-6-test-failures branch 4 times, most recently from d95b426 to f75092e Compare March 12, 2019 05:25
Preferred to keep the tests running and passing on multiple platforms
so that regressions are easier to detect.
Java 11 powermock fails due to a requirement for a Java XML module.
Test is not valuable enough to wrestle with the mocking framework.
Remove the test, replace it later or confirm that the case is already
covered elsewhere in JenkinsRule based tests.
@MarkEWaite MarkEWaite force-pushed the fix-centos-6-test-failures branch from f75092e to f925711 Compare March 12, 2019 10:05
@MarkEWaite MarkEWaite merged commit d2c3704 into jenkinsci:master Mar 12, 2019
@MarkEWaite MarkEWaite deleted the fix-centos-6-test-failures branch March 12, 2019 10:57
@MarkEWaite MarkEWaite added the skip-changelog Exclude from the changelog label Jul 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Exclude from the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant