Simplify the client class

The separate install class provided no benefit and only made it harder
to follow.

Author: ekohl

manifests/client.pp

@@ -2,17 +2,27 @@
 #
 # == Parameters
 #
-# [ensure] Desired ensure state of the package. Optional.
-#   Defaults to 'true'
+# $ensure:: Desired ensure state of the package. Optional. Defaults to 'true'
 #
-# [package_name] Name of the package to install the client from. Default
-#   is repository dependent.
+# $package_name:: Name of the package to install the client from. Default is
+#                 repository dependent.
 #
 class mongodb::client (
   Variant[Boolean, String] $ensure = $mongodb::params::package_ensure_client,
   Optional[String] $package_name   = $mongodb::params::client_package_name,
 ) inherits mongodb::params {
-  anchor { 'mongodb::client::start': }
-  -> class { 'mongodb::client::install': }
-  -> anchor { 'mongodb::client::end': }
+  $package_ensure = $ensure ? {
+    true     => 'present',
+    false    => 'purged',
+    'absent' => 'purged',
+    default  => $ensure,
+  }
+
+  if $package_name {
+    package { 'mongodb_client':
+      ensure => $package_ensure,
+      name   => $package_name,
+      tag    => 'mongodb',
+    }
+  }
 }

manifests/client/install.pp

@@ -8,6 +8,15 @@
       context 'with defaults' do
         it { is_expected.to compile.with_all_deps }
       end
+
+      context 'with manage_package' do
+        let(:pre_condition) do
+          "class { 'mongodb::globals': manage_package => true }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to create_package('mongodb_client').with_ensure('present') }
+      end
     end
   end
 end