Merge pull request #242 from bastelfreak/tests2

openssl::certificate::pkcs12: Add test for x509 without password

Author: bastelfreak

examples/export_pkcs12_from_key.pp

@@ -18,3 +18,22 @@
   in_pass  => 'mahje1Qu',
   out_pass => 'mahje1Qu',
 }
+
+# same as above, just no password for the X509
+openssl::certificate::x509 { 'foo2.example.com':
+  ensure       => present,
+  country      => 'CH',
+  organization => 'Example.com',
+  commonname   => 'foo2.example.com',
+  base_dir     => '/tmp',
+  owner        => 'nobody',
+  # This is just to speed up CI - use 2048 or more in production
+  key_size     => 1024,
+}
+-> openssl::export::pkcs12 { 'export2.pkcs12':
+  ensure   => 'present',
+  basedir  => '/tmp',
+  pkey     => '/tmp/foo2.example.com.key',
+  cert     => '/tmp/foo2.example.com.crt',
+  out_pass => 'mahje1Qu',
+}

spec/acceptance/pkcs12_spec.rb

@@ -7,8 +7,17 @@
     it { expect(file('/tmp/foo.example.com.crt')).to be_file.and(have_attributes(owner: 'nobody', group: 'root')) }
     it { expect(file('/tmp/foo.example.com.key')).to be_file.and(have_attributes(owner: 'nobody', group: 'root')) }
     it { expect(file('/tmp/export.pkcs12.p12')).to be_file.and(have_attributes(owner: 'root', group: 'root')) }
+    it { expect(file('/tmp/foo2.example.com.crt')).to be_file.and(have_attributes(owner: 'nobody', group: 'root')) }
+    it { expect(file('/tmp/foo2.example.com.key')).to be_file.and(have_attributes(owner: 'nobody', group: 'root')) }
+    it { expect(file('/tmp/export2.pkcs12.p12')).to be_file.and(have_attributes(owner: 'root', group: 'root')) }
   end
+  # rubocop:disable RSpec/RepeatedExampleGroupBody
   describe file('/tmp/export.pkcs12.p12') do
     its(:size) { is_expected.to be > 0 }
   end
+
+  describe file('/tmp/export2.pkcs12.p12') do
+    its(:size) { is_expected.to be > 0 }
+  end
+  # rubocop:enable RSpec/RepeatedExampleGroupBody
 end