Convert tar_extract to a class

manifests/init.pp

@@ -109,7 +109,8 @@
   $katello_default_ca_cert = "${pki_dir}/certs/${default_ca_name}.crt"
 
   if $tar_file {
-    certs::tar_extract { $tar_file:
+    class { 'certs::tar_extract':
+      path   => $tar_file,
       before => Class['certs::install'],
     }
   }

manifests/tar_extract.pp

@@ -1,15 +1,10 @@
-# Definition: certs::tar_extract
-#
-# This class extracts a tarball
-#
+# @summary This class extracts a tarball
+# @api private
 # Parameters:
 # - The $path of the tarball to extract
 #
-# Actions:
-# - Extracts a tarball
-#
-define certs::tar_extract (
-  Stdlib::Absolutepath $path = $title,
+class certs::tar_extract (
+  Stdlib::Absolutepath $path,
 ) {
   validate_file_exists($path)
 

spec/classes/certs_tar_extract_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe 'certs::tar_extract' do
+  on_supported_os.each do |os, os_facts|
+    context "on #{os}" do
+      let :facts do
+        os_facts
+      end
+
+      let(:params) { { path: '/some/other/path/with/certs.tar' } }
+
+      describe 'with default parameters' do
+        it { should compile.with_all_deps }
+      end
+
+      it do
+        allow(File).to receive(:exist?).and_call_original
+        allow(File).to receive(:exist?).with('/some/other/path/with/certs.tar').and_return(true)
+
+        is_expected.to compile.with_all_deps
+        is_expected.to contain_exec('extract /some/other/path/with/certs.tar')
+          .with_cwd('/root')
+          .with_path(['/usr/bin', '/bin'])
+          .with_command('tar -xaf /some/other/path/with/certs.tar')
+
+        expect(File).not_to have_received(:exist?).with('/path/to/certs.tar')
+        expect(File).to have_received(:exist?).with('/some/other/path/with/certs.tar')
+      end
+    end
+  end
+end