Fix for issue #233: accomodating for install_repo, with specs

sensu · Aug 27, 2014 · 9c9f481 · 9c9f481
1 parent 64aa8a8
commit 9c9f481
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/manifests/package.pp b/manifests/package.pp
@@ -12,12 +12,20 @@
 
     'Debian': {
       class { 'sensu::repo::apt': }
-      $repo_require = Apt::Source['sensu']
+      if str2bool($sensu::install_repo) {
+        $repo_require = Apt::Source['sensu']
+      } else {
+        $repo_require = undef
+      }
     }
 
     'RedHat': {
       class { 'sensu::repo::yum': }
-      $repo_require = Yumrepo['sensu']
+      if str2bool($sensu::install_repo) {
+        $repo_require = Yumrepo['sensu']
+      } else {
+        $repo_require = undef
+      }
     }
 
     default: { alert("${::osfamily} not supported yet") }

diff --git a/spec/classes/sensu_package_spec.rb b/spec/classes/sensu_package_spec.rb
@@ -88,6 +88,8 @@
               :key         => '7580C77F',
               :key_source  => 'http://repos.sensuapp.org/apt/pubkey.gpg'
             ) }
+
+            it { should contain_package('sensu').with( :require => nil ) }
           end
         end
 
@@ -121,6 +123,7 @@
         context 'install_repo => false' do
           let(:params) { { :install_repo => false } }
           it { should_not contain_yumrepo('sensu') }
+          it { should contain_package('sensu').with( :require => nil ) }
         end
       end
     end