Remove the test options suppressing ignoring differences in whitespace
outputted in the XML to allow capturing of issues where the code
incorrectly adds or removes significant whitespace.
Add a helper function to deal with the edge case of parsing XML directly
that contains non significant whitespace to avoid accidentally adding
excess whitespace when minidom outputs the resulting document.
Change-Id: I9936042cd82c204ba2b3c19f575703e33564f7fd
In Jenkins 1.575 the option to fingerprint build artefacts has been
moved from Fingerprinter to ArtifactArchiver.
(https://github.com/jenkinsci/jenkins/commit/f43a450)
This patch reflects the change by adding an optional fingerprint
parameter to the archive publisher. The parameter recordBuildArtifacts
in the fingerprint publisher remains unchanged for backward
compatibility.
Change-Id: I231fdc38eb8343d646f24e7a31cafd29534c9fc8
The archive publisher did not allow setting the allowEmptyArchive setting in the YAML.
This change adds configuring the setting if it is defined in the YAML file. If the value
is 'true' then it is set, else it is 'false'. The default Jenkins behavior is 'false' disallowing
builds to pass without storing the archived data.
Updated the doc string to add that there is a default value (false) which is also the default Jenkins behavior.
Cleand up the value assignment, and typed it to a str().
fixed pep8 violation
Addresses patchset 3 request to update the doc string to use the test directly.
Would like to have more discussion around the second point in patchset 3 around having
the new code add a line of XML.
Change-Id: I5e0c9ed7e8ea669a8d0e8f267ebb02d5f0b6ae73
Piotr Falkowski