jclouds: Utilize convert_mapping_to_xml

Change-Id: I319aee86daf63f17778f020dea6b3756db04127d
Co-Authored-By: Anil Belur <askb23@gmail.com>
Signed-off-by: Tan Heng Yeow <E0032242@u.nus.edu>
Signed-off-by: Anil Belur <askb23@gmail.com>

jenkins_jobs/modules/publishers.py

@@ -186,28 +186,27 @@ def jclouds(registry, xml_parent, data):
     deployer = XML.SubElement(xml_parent,
                               'jenkins.plugins.jclouds.blobstore.'
                               'BlobStorePublisher')
-
-    if 'profile' not in data:
-        raise JenkinsJobsException('profile parameter is missing')
-    XML.SubElement(deployer, 'profileName').text = data.get('profile')
-
     entries = XML.SubElement(deployer, 'entries')
-
     deployer_entry = XML.SubElement(entries,
                                     'jenkins.plugins.jclouds.blobstore.'
                                     'BlobStoreEntry')
-
+    deployer_mapping = [
+        ('profile', 'profileName', None),
+    ]
+    helpers.convert_mapping_to_xml(
+        deployer, data, deployer_mapping, fail_required=True)
     try:
         XML.SubElement(deployer_entry, 'container').text = data['container']
-        XML.SubElement(deployer_entry, 'path').text = data.get('basedir', '')
         XML.SubElement(deployer_entry, 'sourceFile').text = data['files']
     except KeyError as e:
         raise JenkinsJobsException("blobstore requires '%s' to be set"
                                    % e.args[0])
-
+    deployer_entry_mapping = [
-    mapping = [('hierarchy', 'keepHierarchy', False)]
+        ('hierarchy', 'keepHierarchy', False),
+        ('basedir', 'path', ''),
+    ]
     helpers.convert_mapping_to_xml(
-        deployer_entry, data, mapping, fail_required=True)
+        deployer_entry, data, deployer_entry_mapping, fail_required=True)
 
 
 def javadoc(registry, xml_parent, data):

tests/publishers/fixtures/jclouds001.xml

@@ -2,15 +2,15 @@
 <project>
   <publishers>
     <jenkins.plugins.jclouds.blobstore.BlobStorePublisher>
-      <profileName>hp</profileName>
       <entries>
         <jenkins.plugins.jclouds.blobstore.BlobStoreEntry>
           <container>jenkins</container>
-          <path/>
           <sourceFile>*.tar.gz</sourceFile>
           <keepHierarchy>false</keepHierarchy>
+          <path>test base dir</path>
         </jenkins.plugins.jclouds.blobstore.BlobStoreEntry>
       </entries>
+      <profileName>hp</profileName>
     </jenkins.plugins.jclouds.blobstore.BlobStorePublisher>
   </publishers>
 </project>

tests/publishers/fixtures/jclouds001.yaml

@@ -3,4 +3,4 @@ publishers:
       profile: hp
       files: '*.tar.gz'
       container: jenkins
-      basedir:
+      basedir: test base dir