Merge pull request #205 from bastelfreak/input

systemd::network: Validate if content/source are set for file resource

manifests/network.pp

@@ -19,6 +19,14 @@
     $notify = undef
   }
 
+  if $ensure == 'file' {
+    if $content =~ Undef and $source =~ Undef {
+      fail('Either content or source must be set')
+    }
+    if $content =~ NotUndef and $source =~ NotUndef {
+      fail('Either content or source must be set but not both')
+    }
+  }
   file { "${path}/${name}":
     ensure    => $ensure,
     content   => $content,

spec/defines/network_spec.rb

@@ -61,6 +61,23 @@
 
           it { is_expected.to create_file("/etc/systemd/network/#{title}").that_notifies('Service[systemd-networkd]') }
         end
+        context 'without content and without source' do
+          let :params do
+            {}
+          end
+
+          it { is_expected.to compile.and_raise_error(%r{Either content or source must be set}) }
+        end
+        context 'with content and source' do
+          let :params do
+            {
+              content: 'bla',
+              source: 'foo'
+            }
+          end
+
+          it { is_expected.to compile.and_raise_error(%r{Either content or source must be set but not both}) }
+        end
       end
     end
   end