From 02b5d359b5ad9deeb0ac2d04487eb43a3089bc99 Mon Sep 17 00:00:00 2001
From: "http://jneen.net/" <jneen@jneen.net>
Date: Thu, 10 Aug 2017 13:13:52 -0700
Subject: [PATCH] restructure the #new_key notification spec

---
 spec/services/notification_service_spec.rb | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/spec/services/notification_service_spec.rb b/spec/services/notification_service_spec.rb
index d7ac0746df4..44d3f441382 100644
--- a/spec/services/notification_service_spec.rb
+++ b/spec/services/notification_service_spec.rb
@@ -80,21 +80,16 @@ describe NotificationService, :mailer do
 
   describe 'Keys' do
     describe '#new_key' do
-      let!(:key) { create(:personal_key) }
+      let(:key_options) { {} }
+      let!(:key) { create(:personal_key, key_options) }
 
       it { expect(notification.new_key(key)).to be_truthy }
+      it { should_email(key.user) }
 
-      it 'sends email to key owner' do
-        expect { notification.new_key(key) }.to change { ActionMailer::Base.deliveries.size }.by(1)
-      end
+      describe 'never emails the ghost user' do
+        let(:key_options) { { user: User.ghost } }
 
-      it 'never emails the ghost user' do
-        key.user = User.ghost
-
-        reset_delivered_emails!
-        notification.new_key(key)
-
-        should_not_email_anyone
+        it { should_not_email_anyone }
       end
     end
   end