Macro now can define defaults for parameters

Change-Id: Idc6688a76b80b904d24ff537a3df514b6d24b700

jenkins_jobs/job.py

@@ -35,18 +35,18 @@ class JobBase(RootBase):
         folder = d.pop_loc_string("folder", None)
         contents, params = split_contents_params(d, job_contents_keys)
         return cls(
-            roots.defaults,
-            Expander(config),
-            keep_descriptions,
-            id,
-            name,
-            pos,
-            description,
-            defaults,
-            params,
-            contents,
-            project_type,
-            folder,
+            _defaults=roots.defaults,
+            _expander=Expander(config),
+            _keep_descriptions=keep_descriptions,
+            _id=id,
+            name=name,
+            pos=pos,
+            description=description,
+            defaults_name=defaults,
+            params=params,
+            _contents=contents,
+            project_type=project_type,
+            folder=folder,
         )
 
     @property

jenkins_jobs/macro.py

@@ -60,23 +60,19 @@ class Macro(ElementBase):
         name = d.pop_required_loc_string("name")
         defaults = d.pop_loc_string("defaults", "global")
         elements = d.pop_required_element(elements_name)
+        params = d
         expander = Expander(config)
         str_expander = StringsOnlyExpander(config)
-        if d:
-            example_key = next(iter(d.keys()))
-            raise JenkinsJobsException(
-                f"In {type_name} macro {name!r}: unexpected elements: {','.join(d.keys())}",
-                pos=data.key_pos.get(example_key),
-            )
         macro = cls(
-            roots.defaults,
-            expander,
-            str_expander,
-            type_name,
-            name,
-            defaults,
-            pos,
-            elements or [],
+            _defaults=roots.defaults,
+            _expander=expander,
+            _str_expander=str_expander,
+            _type_name=type_name,
+            name=name,
+            defaults_name=defaults,
+            pos=pos,
+            params=params,
+            elements=elements or [],
         )
         roots.assign(roots.macros[type_name], name, macro, "macro")
 
@@ -87,6 +83,7 @@ class Macro(ElementBase):
         defaults = self._pick_defaults(self.defaults_name)
         full_params = LocDict.merge(
             defaults.params,
+            self.params,
             params,
         )
         element_list = self.elements

jenkins_jobs/root_base.py

@@ -44,6 +44,7 @@ class ElementBase:
     """Base class for YAML elements - job, view, template, or macro"""
 
     _defaults: dict
+    params: dict
 
     @property
     def title(self):
@@ -75,7 +76,6 @@ class RootBase(ElementBase):
     pos: Pos
     description: str
     defaults_name: str
-    params: dict
     _contents: dict
 
     @property

jenkins_jobs/view.py

@@ -34,17 +34,17 @@ class ViewBase(RootBase):
         view_type = d.pop_loc_string("view-type", "list")
         contents, params = split_contents_params(d, view_contents_keys)
         return cls(
-            roots.defaults,
-            Expander(config),
-            keep_descriptions,
-            id,
-            name,
-            pos,
-            description,
-            defaults,
-            params,
-            contents,
-            view_type,
+            _defaults=roots.defaults,
+            _expander=Expander(config),
+            _keep_descriptions=keep_descriptions,
+            _id=id,
+            name=name,
+            pos=pos,
+            description=description,
+            defaults_name=defaults,
+            params=params,
+            _contents=contents,
+            view_type=view_type,
         )
 
     @property

tests/yamlparser/error_fixtures/unexpected_macro_elements.error

@@ -1,3 +0,0 @@
-unexpected_macro_elements.yaml:3:5: In builder macro 'sample-builder': unexpected elements: something_unexpected
-        something_unexpected: sample-value
-        ^

tests/yamlparser/error_fixtures/unexpected_macro_elements.yaml

@@ -1,4 +0,0 @@
-- builder:
-    name: sample-builder
-    something_unexpected: sample-value
-    builders: []

tests/yamlparser/job_fixtures/macro-parameter-precenence.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8"?>
+<project>
+  <actions/>
+  <description>&lt;!-- Managed by Jenkins Job Builder --&gt;</description>
+  <keepDependencies>false</keepDependencies>
+  <blockBuildWhenDownstreamBuilding>false</blockBuildWhenDownstreamBuilding>
+  <blockBuildWhenUpstreamBuilding>false</blockBuildWhenUpstreamBuilding>
+  <concurrentBuild>false</concurrentBuild>
+  <canRoam>true</canRoam>
+  <properties/>
+  <scm class="hudson.scm.NullSCM"/>
+  <builders>
+    <hudson.tasks.Shell>
+      <command># 1-call
+echo param_1=1-call
+# 2-macro
+echo param_2=2-macro
+# 3-defaults
+echo param_3=3-defaults
+# 4-defaults
+echo param_4=4-defaults
+</command>
+    </hudson.tasks.Shell>
+  </builders>
+  <publishers/>
+  <buildWrappers/>
+</project>

tests/yamlparser/job_fixtures/macro-parameter-precenence.yaml

@@ -0,0 +1,35 @@
+- defaults:
+    name: global
+    param_1: '1-defaults'
+    param_2: '2-defaults'
+    param_3: '3-defaults'
+    param_4: '4-defaults'
+
+- builder:
+    name: sample-builder
+    param_1: 1-macro
+    param_2: 2-macro
+    builders:
+    - shell: |
+        # 1-call
+        echo param_1={param_1}
+        # 2-macro
+        echo param_2={param_2}
+        # 3-defaults
+        echo param_3={param_3}
+        # 4-defaults
+        echo param_4={param_4}
+
+- job-template:
+    name: sample-job
+    param_1: '1-template'
+    param_2: '2-template'
+    param_3: '3-template'
+    builders:
+    - sample-builder:
+        param_1: 1-call
+
+- project:
+    name: test-project
+    jobs:
+    - sample-job