Skip to content

Conversation

@Farix1337
Copy link

@Farix1337 Farix1337 commented Sep 21, 2021

As described in #330 currently the Plugin can't be build at jenkins.io due to the enforcement of a newer jenkins version for plugins.
This Pull Request

  • updates the Jenkins version
  • migrates library versions to the usage of jenkins tools bill of materials (bom) to avoid library version incompatibilities
  • fixes deprecations with for library updates
  • makes BootSourceTest#doCheckImageIdWhenNoValueSet assertion default locale independ

This Pull Request replaces #329

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue


public class BootSourceTest {
private static final FormValidation VALIDATION_REQUIRED = FormValidation.error(hudson.util.Messages.FormValidation_ValidateRequired());
private static final FormValidation VALIDATION_REQUIRED = FormValidation.error(hudson.util.Messages._FormValidation_ValidateRequired().toString(US));
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If running the Test jenkins.plugins.openstack.compute.slaveopts.BootSourceTest#doCheckImageIdWhenNoValueSet at a machine without US or GB localisation the assertion will fail, because the created VALIDATION_REQUIRED which is used for the expect value changes based on the jvm default java.util.Locale (See the Implementation org.jvnet.localizer.ResourceBundleHolder#format)
This change forces the US Localisation for the creation of the expected value of VALIDATION_REQUIRED.

- update org.jenkins-ci:acceptance-test-harness to 1.101
- update openstack4j to 3.10
- fix jackson version to avoid dependency enforce conflict of openstack4j and configuration as code
- specify test-harness version via jenkins-test-harness.version from the parent pom
- move jackson dependencies to compile dependencies
@olivergondza
Copy link
Member

Merging, Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants