From: Bryan Jen <bryan.jen@gmail.com>
Date: Tue, 7 Jun 2016 16:20:12 +0000 (-0700)
Subject: Merge pull request #610 from puppetlabs/unused-lsb
X-Git-Tag: 2.3.0~11
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=5476d9033bfee6807800b1d14c8bbb9da4b4c01c;hp=47b9b6dee7c11576c37f7f9366da6a23531e9892;p=puppet-modules%2Fpuppetlabs-apt.git

Merge pull request #610 from puppetlabs/unused-lsb

apt/params: Remove unused LSB facts
---

diff --git a/manifests/key.pp b/manifests/key.pp
index 8fbb47a..914ec88 100644
--- a/manifests/key.pp
+++ b/manifests/key.pp
@@ -49,7 +49,7 @@ define apt::key (
   }
 
   validate_re($_id, ['\A(0x)?[0-9a-fA-F]{8}\Z', '\A(0x)?[0-9a-fA-F]{16}\Z', '\A(0x)?[0-9a-fA-F]{40}\Z'])
-  validate_re($ensure, ['\Aabsent|present\Z',])
+  validate_re($ensure, ['\A(absent|present)\Z',])
 
   if $_content {
     validate_string($_content)
diff --git a/spec/defines/key_spec.rb b/spec/defines/key_spec.rb
index d4a85a4..14a3efb 100644
--- a/spec/defines/key_spec.rb
+++ b/spec/defines/key_spec.rb
@@ -268,13 +268,15 @@ describe 'apt::key' do
     end
 
     context 'invalid ensure' do
-      let :params do
-        {
-          :ensure => 'foo',
-        }
-      end
-      it 'fails' do
-        expect { subject.call }.to raise_error(/does not match/)
+      %w(foo aabsent absenta apresent presenta).each do |param|
+        let :params do
+          {
+            :ensure => param,
+          }
+          end
+        it 'fails' do
+          expect { subject.call }.to raise_error(/does not match/)
+        end
       end
     end