more cleanup, spec tests pass

3 files changed, 268 insertions, 201 deletions

diff --git a/spec/defines/openvpn_client_spec.rb b/spec/defines/openvpn_client_spec.rb
index da71d63..143b76f 100644
--- a/spec/defines/openvpn_client_spec.rb
+++ b/spec/defines/openvpn_client_spec.rb
@@ -23,94 +23,153 @@ describe 'openvpn::client', :type => :define do
     'command' => '/bin/rm test_client.tar.gz; tar --exclude=\*.conf.d -chzvf test_client.tar.gz test_client'
   ) }
 
-  it { should contain_openvpn__option('ca test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'ca', 
-    'value'   => 'keys/ca.crt'
-  )}
-  it { should contain_openvpn__option('cert test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'cert', 
-    'value'   => 'keys/test_client.crt'
-  )}
-  it { should contain_openvpn__option('key test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'key', 
-    'value'   => 'keys/test_client.key'
-  )}
-  it { should contain_openvpn__option('client test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'client'
-  )}
-  it { should contain_openvpn__option('dev test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'dev', 
-    'value'   => 'tun'
-  )}
-  it { should contain_openvpn__option('proto test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'proto', 
-    'value'   => 'tcp'
-  )}
-  it { should contain_openvpn__option('remote test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'remote', 
-    'value'   => 'somehost 1194'
-  )}
-  it { should contain_openvpn__option('resolv-retry test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'resolv-retry', 
-    'value'   => 'infinite'
-  )}
-  it { should contain_openvpn__option('nobind test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'nobind'
-  )}
-  it { should contain_openvpn__option('persist-key test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'persist-key'
-  )}
-  it { should contain_openvpn__option('persist-tun test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'persist-tun'
-  )}
-  it { should contain_openvpn__option('mute-replay-warnings test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'mute-replay-warnings'
-  )}
-  it { should contain_openvpn__option('ns-cert-type test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'ns-cert-type', 
-    'value'   => 'server'
-  )}
-  it { should contain_openvpn__option('comp-lzo test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'comp-lzo'
-  )}
-  it { should contain_openvpn__option('verb test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'verb', 
-    'value'   => '3'
-  )}
-  it { should contain_openvpn__option('mute test_server with test_client').with(
-    'server'  => 'test_server', 
-    'client'  => 'test_client',
-    'key'     => 'mute', 
-    'value'   => '20'
-  )}
+  context "setting the minimum parameters" do
+    let(:params) { { 'server' => 'test_server' } }
+    let(:facts) { { :fqdn => 'somehost', :concat_basedir => '/var/lib/puppet/concat' } }
+
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^client$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^ca\s+keys\/ca\.crt$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^cert\s+keys\/test_client.crt$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^key\s+keys\/test_client\.key$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^dev\s+tun$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^proto\s+tcp$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^remote\s+somehost\s+1194$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^comp-lzo$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^resolv-retry\s+infinite$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^nobind$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^persist-key$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^persist-tun$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^mute-replay-warnings$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^ns\-cert\-type\s+server$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^verb\s+3$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^mute\s+20$/)}
+  end
+
+  context "setting all of the parameters" do
+    let(:params) { { 
+      'server'                => 'test_server',
+      'compression'           => 'comp-something',
+      'dev'                   => 'tap',
+      'mute'                  => 10,
+      'mute_replay_warnings'  => false,
+      'nobind'                => false,
+      'ns_cert_type'          => 'client',
+      'persist_key'           => false,
+      'persist_tun'           => false,
+      'port'                  => '123',
+      'proto'                 => 'udp',
+      'remote_host'           => 'somewhere',
+      'resolv_retry'          => '2m',
+      'verb'                  => '1'
+    } }
+    let(:facts) { { :fqdn => 'somehost', :concat_basedir => '/var/lib/puppet/concat' } }
+
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^client$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^ca\s+keys\/ca\.crt$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^cert\s+keys\/test_client.crt$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^key\s+keys\/test_client\.key$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^dev\s+tap$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^proto\s+udp$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^remote\s+somewhere\s+123$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^comp-something$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^resolv-retry\s+2m$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^ns\-cert\-type\s+client$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^verb\s+1$/)}
+    it { should contain_file('/etc/openvpn/test_server/download-configs/test_client/test_client.conf').with_content(/^mute\s+10$/)}
+  end
+
+
+
+
+
+#  it { should contain_openvpn__option('ca test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'ca', 
+#    'value'   => 'keys/ca.crt'
+#  )}
+#  it { should contain_openvpn__option('cert test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'cert', 
+#    'value'   => 'keys/test_client.crt'
+#  )}
+#  it { should contain_openvpn__option('key test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'key', 
+#    'value'   => 'keys/test_client.key'
+#  )}
+#  it { should contain_openvpn__option('client test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'client'
+#  )}
+#  it { should contain_openvpn__option('dev test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'dev', 
+#    'value'   => 'tun'
+#  )}
+#  it { should contain_openvpn__option('proto test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'proto', 
+#    'value'   => 'tcp'
+#  )}
+#  it { should contain_openvpn__option('remote test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'remote', 
+#    'value'   => 'somehost 1194'
+#  )}
+#  it { should contain_openvpn__option('resolv-retry test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'resolv-retry', 
+#    'value'   => 'infinite'
+#  )}
+#  it { should contain_openvpn__option('nobind test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'nobind'
+#  )}
+#  it { should contain_openvpn__option('persist-key test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'persist-key'
+#  )}
+#  it { should contain_openvpn__option('persist-tun test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'persist-tun'
+#  )}
+#  it { should contain_openvpn__option('mute-replay-warnings test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'mute-replay-warnings'
+#  )}
+#  it { should contain_openvpn__option('ns-cert-type test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'ns-cert-type', 
+#    'value'   => 'server'
+#  )}
+#  it { should contain_openvpn__option('comp-lzo test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'comp-lzo'
+#  )}
+#  it { should contain_openvpn__option('verb test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'verb', 
+#    'value'   => '3'
+#  )}
+#  it { should contain_openvpn__option('mute test_server with test_client').with(
+#    'server'  => 'test_server', 
+#    'client'  => 'test_client',
+#    'key'     => 'mute', 
+#    'value'   => '20'
+#  )}
 end
diff --git a/spec/defines/openvpn_option_spec.rb b/spec/defines/openvpn_option_spec.rb
deleted file mode 100644
index a2d1661..0000000
--- a/spec/defines/openvpn_option_spec.rb
+++ /dev/null
@@ -1,42 +0,0 @@
-require 'spec_helper'
- 
-describe 'openvpn::option', :type => :define do
-
-  let(:title) { 'test_param' }
-  
-  context "when key => 'test_key', server => 'test_server'" do
-    let(:params) { { 'key' => 'test_key', 'server' => 'test_server' } }
-
-    it { should contain_concat__fragment('openvpn.test_server..test_param').with(
-      'target'  => '/etc/openvpn/test_server.conf',
-      'content' => "test_key\n"
-    ) }
-  end
-
-  context "when key => 'test_key', value => 'test_value', server => 'test_server'" do
-    let(:params) { { 'key' => 'test_key', 'value' => 'test_value', 'server' => 'test_server' } }
-
-    it { should contain_concat__fragment('openvpn.test_server..test_param').with(
-      'target'  => '/etc/openvpn/test_server.conf',
-      'content' => "test_key test_value\n"
-    ) }
-  end
-  
-  context "when key => 'test_key', server => 'test_server', client => 'test_client'" do
-    let(:params) { { 'key' => 'test_key', 'server' => 'test_server', 'client' => 'test_client' } }
-
-    it { should contain_concat__fragment('openvpn.test_server.test_client.test_param').with(
-      'target'  => '/etc/openvpn/test_server/download-configs/test_client/test_client.conf',
-      'content' => "test_key\n"
-    ) }
-  end
-
-  context "when key => 'test_key', server => 'test_server', client => 'test_client', csc => true" do
-    let(:params) { { 'key' => 'test_key', 'server' => 'test_server', 'client' => 'test_client', 'csc' => 'true' } }
-
-    it { should contain_concat__fragment('openvpn.test_server.test_client.test_param').with(
-      'target'  => '/etc/openvpn/test_server/client-configs/test_client',
-      'content' => "test_key\n"
-    ) }
-  end
-end
diff --git a/spec/defines/openvpn_server_spec.rb b/spec/defines/openvpn_server_spec.rb
index 1032302..85dadf2 100644
--- a/spec/defines/openvpn_server_spec.rb
+++ b/spec/defines/openvpn_server_spec.rb
@@ -3,82 +3,124 @@ require 'spec_helper'
 describe 'openvpn::server', :type => :define do
   
   let(:title) { 'test_server' }
-  let(:params) { {
-    'country'       => 'CO',
-    'province'      => 'ST',
-    'city'          => 'Some City',
-    'organization'  => 'example.org',
-    'email'         => 'testemail@example.org'
-  } }
+  
+  context "creating a server with the minimum parameters" do
+    let(:params) { {
+      'country'       => 'CO',
+      'province'      => 'ST',
+      'city'          => 'Some City',
+      'organization'  => 'example.org',
+      'email'         => 'testemail@example.org'
+    } }
 
-  let (:facts) { { :concat_basedir => '/var/lib/puppet/concat' } }
+    let (:facts) { { 
+      :ipaddress_eth0 => '1.2.3.4', 
+      :network_eth0   => '1.2.3.0',
+      :netmask_eth0   => '255.255.255.0',
+      :concat_basedir => '/var/lib/puppet/concat' 
+    } }
   
-  # Files associated with a server config
-  it { should contain_file('/etc/openvpn/test_server').with('ensure' => 'directory')}
-  it { should contain_file('/etc/openvpn/test_server/client-configs').with('ensure' => 'directory')}
-  it { should contain_file('/etc/openvpn/test_server/download-configs').with('ensure' => 'directory')}
-  it { should contain_file('/etc/openvpn/test_server/easy-rsa/vars')}
-  it { should contain_file('/etc/openvpn/test_server/easy-rsa/openssl.cnf')}
-  it { should contain_file('/etc/openvpn/test_server/keys').with(
-    'ensure'  => 'link',
-    'target'  => '/etc/openvpn/test_server/easy-rsa/keys'
-  )}
+    # Files associated with a server config
+    it { should contain_file('/etc/openvpn/test_server').with('ensure' => 'directory')}
+    it { should contain_file('/etc/openvpn/test_server/client-configs').with('ensure' => 'directory')}
+    it { should contain_file('/etc/openvpn/test_server/download-configs').with('ensure' => 'directory')}
+    it { should contain_file('/etc/openvpn/test_server/easy-rsa/vars')}
+    it { should contain_file('/etc/openvpn/test_server/easy-rsa/openssl.cnf')}
+    it { should contain_file('/etc/openvpn/test_server/keys').with(
+      'ensure'  => 'link',
+      'target'  => '/etc/openvpn/test_server/easy-rsa/keys'
+    )}
   
-  it { should contain_concat__fragment('openvpn.default.autostart.test_server').with(
-    'content' => "AUTOSTART=\"$AUTOSTART test_server\"\n",
-    'target'  => '/etc/default/openvpn'
-  )}
+    # Execs to working with certificates
+    it { should contain_exec('copy easy-rsa to openvpn config folder test_server').with(
+      'command' => '/bin/cp -r /usr/share/doc/openvpn/examples/easy-rsa/2.0 /etc/openvpn/test_server/easy-rsa'
+    )}
+    it { should contain_exec('generate dh param test_server') }
+    it { should contain_exec('initca test_server') }
+    it { should contain_exec('generate server cert test_server') }
+
+    # Configure to start vpn session
+    it { should contain_concat__fragment('openvpn.default.autostart.test_server').with(
+      'content' => "AUTOSTART=\"$AUTOSTART test_server\"\n",
+      'target'  => '/etc/default/openvpn'
+    )}
   
-  # Execs to working with certificates
-  it { should contain_exec('copy easy-rsa to openvpn config folder test_server').with(
-    'command' => '/bin/cp -r /usr/share/doc/openvpn/examples/easy-rsa/2.0 /etc/openvpn/test_server/easy-rsa'
-  )}
-  it { should contain_exec('generate dh param test_server') }
-  it { should contain_exec('initca test_server') }
-  it { should contain_exec('generate server cert test_server') }
+    # VPN server config file itself
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^mode\s+server$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^client\-config\-dir\s+\/etc\/openvpn\/test_server\/client\-configs$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^ca\s+\/etc\/openvpn\/test_server\/keys\/ca.crt$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^cert\s+\/etc\/openvpn\/test_server\/keys\/server.crt$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^key\s+\/etc\/openvpn\/test_server\/keys\/server.key$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^dh\s+\/etc\/openvpn\/test_server\/keys\/dh1024.pem$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^proto\s+tcp$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^port\s+1194$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^comp-lzo$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^group\s+nobody$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^user\s+nobody$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^log\-append\s+test_server\/openvpn\.log$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^status\s+test_server\/openvpn\-status\.log$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^dev\s+tun0$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^local\s+1\.2\.3\.4$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^server\s+1\.2\.3\.0\s+255\.255\.255\.0$/) }
+  end
   
-  # Options that should be set
-  it { should contain_openvpn__option('client-config-dir test_server').with(
-    'server'  => 'test_server',
-    'key'     => 'client-config-dir',
-    'value'   => '/etc/openvpn/test_server/client-configs'
-  )}
-  it { should contain_openvpn__option('mode test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'mode', 
-    'value'   => 'server'
-  )}
-  it { should contain_openvpn__option('ca test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'ca', 
-    'value'   => '/etc/openvpn/test_server/keys/ca.crt'
-  )}
-  it { should contain_openvpn__option('cert test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'cert', 
-    'value'   => '/etc/openvpn/test_server/keys/server.crt'
-  )}
-  it { should contain_openvpn__option('key test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'key', 
-    'value'   => '/etc/openvpn/test_server/keys/server.key'
-  )}
-  it { should contain_openvpn__option('dh test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'dh', 
-    'value'   => '/etc/openvpn/test_server/keys/dh1024.pem'
-  )}
-  it { should contain_openvpn__option('proto test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'proto', 
-    'value'   => 'tcp'
-  )}
-  it { should contain_openvpn__option('comp-lzo test_server').with(
-    'server'  => 'test_server', 
-    'key'     => 'comp-lzo'
-  )}  
+  context "creating a server setting all parameters" do
+    let(:params) { {
+      'country'       => 'CO',
+      'province'      => 'ST',
+      'city'          => 'Some City',
+      'organization'  => 'example.org',
+      'email'         => 'testemail@example.org',
+      'compression'   => 'fake_compression',
+      'port'          => '123',
+      'proto'         => 'udp',
+      'group'         => 'someone',
+      'user'          => 'someone',
+      'logfile'       => '/var/log/openvpn/test_server.log',
+      'status_log'    => '/var/log/openvpn/test_server_status.log',
+      'dev'           => 'tun1',
+      'local'         => '2.3.4.5',
+      'ipp'           => true,
+      'server'        => '2.3.4.0 255.255.0.0',
+      'push'          => [ 'dhcp-option DNS 172.31.0.30', 'route 172.31.0.0 255.255.0.0' ]
+    } }
+
+    let (:facts) { { 
+      :ipaddress_eth0 => '1.2.3.4', 
+      :network_eth0   => '1.2.3.0',
+      :netmask_eth0   => '255.255.255.0',
+      :concat_basedir => '/var/lib/puppet/concat' 
+    } }
+    
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^mode\s+server$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^client\-config\-dir\s+\/etc\/openvpn\/test_server\/client\-configs$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^ca\s+\/etc\/openvpn\/test_server\/keys\/ca.crt$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^cert\s+\/etc\/openvpn\/test_server\/keys\/server.crt$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^key\s+\/etc\/openvpn\/test_server\/keys\/server.key$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^dh\s+\/etc\/openvpn\/test_server\/keys\/dh1024.pem$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^proto\s+udp$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^port\s+123$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^fake_compression$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^group\s+someone$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^user\s+someone$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^log\-append\s+\/var\/log\/openvpn\/test_server\.log$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^status\s+\/var\/log\/openvpn\/test_server_status\.log$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^dev\s+tun1$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^local\s+2\.3\.4\.5$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^server\s+2\.3\.4\.0\s+255\.255\.0\.0$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^push\s+dhcp-option\s+DNS\s+172\.31\.0\.30$/) }
+    it { should contain_file('/etc/openvpn/test_server.conf').with_content(/^push\s+route\s+172\.31\.0\.0\s+255\.255\.0\.0$/) }
+  end
 
   context "when RedHat based machine" do
+    let(:params) { {
+      'country'       => 'CO',
+      'province'      => 'ST',
+      'city'          => 'Some City',
+      'organization'  => 'example.org',
+      'email'         => 'testemail@example.org'
+    } }
+    
     let(:facts) { { :osfamily => 'RedHat', :concat_basedir => '/var/lib/puppet/concat' } }
     
     it { should contain_file('/etc/openvpn/test_server/easy-rsa/openssl.cnf').with(
@@ -93,6 +135,14 @@ describe 'openvpn::server', :type => :define do
   end
     
   context "when Debian based machine" do 
+    let(:params) { {
+      'country'       => 'CO',
+      'province'      => 'ST',
+      'city'          => 'Some City',
+      'organization'  => 'example.org',
+      'email'         => 'testemail@example.org'
+    } }
+    
     let(:facts) { { :osfamily => 'Debian', :concat_basedir => '/var/lib/puppet/concat' } }
 
     it { should contain_file('/etc/openvpn/test_server/easy-rsa/openssl.cnf').with(