Merge pull request #681 from HelenCampbell/emptystringing

Ensure release allows empty strings
puppetlabs · May 30, 2017 · 61f4e6a · 61f4e6a
2 parents bb82b88 + cd15d61
commit 61f4e6a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/manifests/source.pp b/manifests/source.pp
@@ -19,7 +19,7 @@
 
   $_before = Apt::Setting["list-${title}"]
 
-  if ! $release {
+  if !$release {
     if $facts['lsbdistcodename'] {
       $_release = $facts['lsbdistcodename']
     } else {

diff --git a/spec/defines/source_spec.rb b/spec/defines/source_spec.rb
@@ -373,6 +373,20 @@
       end
     end
 
+    context 'release is empty string' do
+      let :facts do
+        {
+          :os => { :family => 'Debian', :name => 'Debian', :release => { :major => '7', :full => '7.0' }},
+          :lsbdistid       => 'Debian',
+          :osfamily        => 'Debian',
+          :puppetversion   => Puppet.version,
+        }
+      end
+      let(:params) { { :location => 'hello.there', :release => '' } }
+
+      it { is_expected.to contain_apt__setting('list-my_source').with_content(/hello\.there  main/) }
+    end
+
     context 'invalid pin' do
       let :facts do
         {