Do not run recipe without username/password/domain

recipes/default.rb

@@ -18,14 +18,9 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 
-if node.attribute? 'sendmail_ses'
-  log 'check username, password, and domain' do
-    level :fatal
-    message 'Username, password and domain must be defined'
-    not_if { node['sendmail_ses']['username'] }
-    not_if { node['sendmail_ses']['password'] }
-    not_if { node['sendmail_ses']['domain'] }
-  end
+if node['sendmail_ses'].attribute?('username') &&
+   node['sendmail_ses'].attribute?('password') &&
+   node['sendmail_ses'].attribute?('domain')
 
   %w(m4 sendmail-cf).each do |p|
     package p

spec/default_spec.rb

@@ -16,10 +16,16 @@
     stub_command('grep ses.cf /etc/mail/sendmail.mc').and_return(true)
     chef_run.node.set['sendmail_ses'] = {}
     chef_run.converge described_recipe
-    expect(chef_run).to write_log(
-      'Username, password and domain must be defined'
-    )
-    expect(chef_run).not_to restart_service('sendmail')
+
+    ses_cf_path = chef_run.node['sendmail']['ses_cf_path']
+
+    expect(chef_run).not_to create_template('/etc/mail/authinfo.ses')
+    expect(chef_run).not_to create_template('/etc/mail/access.ses')
+    expect(chef_run).not_to create_directory(ses_cf_path)
+    expect(chef_run).not_to create_template("#{ses_cf_path}/ses.cf")
+    template = chef_run.template("#{ses_cf_path}/ses.cf")
+    expect(template)
+      .not_to notify('execute[sendmail_test]').to(:run).immediately
   end
 
   context 'basic setup' do