jjb/jenkins-job-builder
Code Issues Proposed changes

Merge "maven-deploy: don't require id"

Browse Source
This commit is contained in:
Jenkins 2014-01-16 09:41:13 +00:00 committed by Gerrit Code Review
commit 777a29aa8a
3 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
jenkins_jobs/modules/publishers.py
View File

@ -2022,7 +2022,8 @@ def maven_deploy(parser, xml_parent, data):
""" """
p = XML.SubElement(xml_parent, 'hudson.maven.RedeployPublisher') p = XML.SubElement(xml_parent, 'hudson.maven.RedeployPublisher')
XML.SubElement(p, 'id').text = data['id'] if 'id' in data:
XML.SubElement(p, 'id').text = data['id']
XML.SubElement(p, 'url').text = data['url'] XML.SubElement(p, 'url').text = data['url']
XML.SubElement(p, 'uniqueVersion').text = str( XML.SubElement(p, 'uniqueVersion').text = str(
data.get('unique-version', True)).lower() data.get('unique-version', True)).lower()

10
tests/publishers/fixtures/mavendeploy001.xml Normal file
View File

@ -0,0 +1,10 @@
<?xml version="1.0" ?>
<project>
<publishers>
<hudson.maven.RedeployPublisher>
<url>file:///path/to/repo</url>
<uniqueVersion>true</uniqueVersion>
<evenIfUnstable>false</evenIfUnstable>
</hudson.maven.RedeployPublisher>
</publishers>
</project>

4
tests/publishers/fixtures/mavendeploy001.yaml Normal file
View File

@ -0,0 +1,4 @@
publishers:
- maven-deploy:
url: file:///path/to/repo