add warning() when an instance is defined but base class is set to ab…

…sent

manifests/instance.pp

@@ -78,24 +78,31 @@
     },
   }
 
-  # disable bucketting since the .inf file contains password information
-  file { $setup_inf_name:
-    ensure  => file,
-    path    => $setup_inf_path,
-    owner   => $::port389::user,
-    group   => $::port389::group,
-    mode    => '0600',
-    content => template("${module_name}/inf.erb"),
-    backup  => false,
-  } ->
-  # /usr/sbin/setup-ds-admin.pl needs:
-  #   /bin/{grep, cat, uname, sleep, ...}
-  #   /sbin/service
-  #   /usr/bin/env
-  exec { "setup-ds-admin.pl_${title}":
-    path      => [ '/bin', '/sbin', '/usr/bin', '/usr/sbin' ],
-    command   => "setup-ds-admin.pl --file=${setup_inf_path} --silent",
-    unless    => "/usr/bin/test -e /etc/dirsrv/slapd-${title}",
-    logoutput => true,
+  case $::port389::ensure {
+    'present', 'latest': {
+      # disable bucketting since the .inf file contains password information
+      file { $setup_inf_name:
+        ensure  => file,
+        path    => $setup_inf_path,
+        owner   => $::port389::user,
+        group   => $::port389::group,
+        mode    => '0600',
+        content => template("${module_name}/inf.erb"),
+        backup  => false,
+      } ->
+      # /usr/sbin/setup-ds-admin.pl needs:
+      #   /bin/{grep, cat, uname, sleep, ...}
+      #   /sbin/service
+      #   /usr/bin/env
+      exec { "setup-ds-admin.pl_${title}":
+        path      => [ '/bin', '/sbin', '/usr/bin', '/usr/sbin' ],
+        command   => "setup-ds-admin.pl --file=${setup_inf_path} --silent",
+        unless    => "/usr/bin/test -e /etc/dirsrv/slapd-${title}",
+        logoutput => true,
+      }
+    }
+    default: {
+      warning("it is meaningless to declare Port389::Instance[${name}] while Class[port389]{ ensure => 'absent|purged' }")
+    }
   }
 }

spec/defines/port389_instance_spec.rb

@@ -116,6 +116,34 @@
       end
 
     end # schema_file =>
-  end
+  end # setup.inf
+
+  context 'class { port389: ensure => ... }' do
+    let(:title) { 'ldap1' }
+
+    %w{ present latest }.each do |state|
+      context state do
+        let(:pre_condition) do <<-EOS
+          class { 'port389': ensure => #{state} }
+          EOS
+        end
+
+        it { should contain_file('setup_ldap1.inf') }
+        it { should contain_exec('setup-ds-admin.pl_ldap1') }
+      end
+    end
+
+    %w{ absent purged }.each do |state|
+      context state do
+        let(:pre_condition) do <<-EOS
+          class { 'port389': ensure => #{state} }
+          EOS
+        end
+
+        it { should_not contain_file('setup_ldap1.inf') }
+        it { should_not contain_exec('setup-ds-admin.pl_ldap1') }
+      end
+    end
+  end # class { port389: ensure => ... }
 
 end