fixes #110 - bug provisioner conditionals only/except conditionals ar…

…e missing

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ### BUG FIX:
 # builder/GoogleCompute: source_image_family / source_image exactly one should be required [GH-111]
+# provisioner: only/except conditionals are missing[GH-110]
+
 
 ## 0.8.0 (May 30, 2018)
 

src/packerlicious/provisioner.py

@@ -31,6 +31,8 @@ class PackerProvisioner(BasePackerObject):
     """
     provisioner_props = {
         'pause_before': (str, False),
+        'only': ([str], False),
+        'except': ([str], False),
     }
 
     def __init__(self, title=None, **kwargs):

tests/packerlicious/test_provisioner.py

@@ -0,0 +1,86 @@
+
+from packerlicious import Template
+import packerlicious.provisioner as provisioner
+import json
+
+
+class TestProvisionerAttributes(object):
+
+    def test_support_only(self):
+        expected_json = """
+        {
+          "provisioners": [
+            {
+              "type": "shell-local",
+              "inline": [ "ls" ],
+              "only": [ "azure-arm" ]
+            }
+          ]
+        }
+        """
+
+        p = provisioner.ShellLocal(
+            inline=["ls"],
+            only=["azure-arm"]
+        )
+
+        t = Template()
+        t.add_provisioner(p)
+
+        to_json = t.to_json()
+        assert to_json == json.dumps(json.loads(expected_json), sort_keys=True, indent=2,
+                                     separators=(',', ': '))
+
+    def test_support_only(self):
+        expected_json = """
+        {
+          "provisioners": [
+            {
+              "type": "shell",
+              "inline": [ "ls" ],
+              "except": [ "azure-arm" ]
+            }
+          ]
+        }
+        """
+
+        p = provisioner.Shell(
+            inline=["ls"])
+
+        p.__setattr__('except', ["azure-arm"])
+
+        t = Template()
+        t.add_provisioner(p)
+
+        to_json = t.to_json()
+        assert to_json == json.dumps(json.loads(expected_json), sort_keys=True, indent=2,
+                                     separators=(',', ': '))
+
+
+
+    def test_support_pause_before(self):
+        expected_json = """
+        {
+          "provisioners": [
+            {
+              "type": "shell",
+              "inline": [ "ls" ],
+              "pause_before": "10s"
+            }
+          ]
+        }
+        """
+
+        p = provisioner.Shell(
+            inline=["ls"],
+            pause_before="10s"
+        )
+
+        t = Template()
+        t.add_provisioner(p)
+
+        to_json = t.to_json()
+        assert to_json == json.dumps(json.loads(expected_json), sort_keys=True, indent=2,
+                                     separators=(',', ': '))
+
+