Test and fix manage_repo => false problems on apt distributions

Author: tylerjl

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 #### Fixes
 * The Elasticsearch log directory is no longer recursively managed to avoid stomping on the user/mode settings that Elasticsearch prefers.
+* Package management on apt-based systems no longer encounters dependency errors when `manage_repo => false`.
 
 #### Features
 

manifests/package.pp

@@ -154,7 +154,7 @@
       }
 
     } else {
-      if ($facts['os']['family'] == 'Debian') {
+      if ($elasticsearch::manage_repo and $facts['os']['family'] == 'Debian') {
         Class['apt::update'] -> Package['elasticsearch']
       }
     }

spec/classes/000_elasticsearch_init_spec.rb

@@ -236,6 +236,16 @@
 
         it { should contain_class('elastic_stack::repo') }
       end
+
+      context 'When not managing the repository' do
+        let(:params) do
+          default_params.merge(
+            :manage_repo => false
+          )
+        end
+
+        it { should compile.with_all_deps }
+      end
     end
   end