Merge branch 'autostart_all' of https://github.com/jsok/puppet-openvpn …

…into jsok-autostart_all

spec/classes/openvpn_config_spec.rb

@@ -5,10 +5,28 @@
   it { should create_class('openvpn::config') }
 
   context "on Debian based machines" do
-    let (:facts) { { :osfamily => 'Debian', :concat_basedir => '/var/lib/puppet/concat' } }
+    let (:facts) { {
+      :osfamily => 'Debian',
+      :operatingsystem => 'Debian',
+      :concat_basedir => '/var/lib/puppet/concat'
+    } }
 
     it { should contain_concat('/etc/default/openvpn') }
     it { should contain_concat__fragment('openvpn.default.header') }
+
+    context "enabled autostart_all" do
+      let(:pre_condition) { 'class { "openvpn": autostart_all => true }' }
+      it { should contain_concat__fragment('openvpn.default.header').with(
+        'content' => /^AUTOSTART="all"/
+      )}
+    end
+
+    context "disabled autostart_all" do
+      let(:pre_condition) { 'class { "openvpn": autostart_all => false }' }
+      it { should contain_concat__fragment('openvpn.default.header').with(
+        'content' => /^AUTOSTART=""/
+      )}
+    end
   end
 
 end

templates/etc-default-openvpn.erb

@@ -17,4 +17,8 @@
 #STATUSREFRESH=0
 # Optional arguments to openvpn's command line
 OPTARGS=""
+<% if scope.lookupvar('::openvpn::autostart_all') -%>
+AUTOSTART="all"
+<% else -%>
 AUTOSTART=""
+<% end -%>