From 121d3d99fe1f1cc2ffb55955b94270dae67eb020 Mon Sep 17 00:00:00 2001
From: Brenda Wallace <brenda@wallace.net.nz>
Date: Sun, 21 May 2017 11:36:07 +1200
Subject: [PATCH] Tests for retrieving data of deleted members

---
 spec/models/harvest_spec.rb  | 7 +++++++
 spec/models/photo_spec.rb    | 9 ++++++++-
 spec/models/planting_spec.rb | 6 ++++++
 spec/models/post_spec.rb     | 8 +++++++-
 4 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/spec/models/harvest_spec.rb b/spec/models/harvest_spec.rb
index d2f840280..83f89c7c2 100644
--- a/spec/models/harvest_spec.rb
+++ b/spec/models/harvest_spec.rb
@@ -294,4 +294,11 @@ describe Harvest do
       end
     end
   end
+
+  it 'excludes deleted members' do
+    member = FactoryGirl.create :member
+    harvest = FactoryGirl.create :harvest, owner: member
+    member.destroy
+    expect(Harvest.all).not_to include(harvest)
+  end
 end
diff --git a/spec/models/photo_spec.rb b/spec/models/photo_spec.rb
index b1f4f2759..30a95f341 100644
--- a/spec/models/photo_spec.rb
+++ b/spec/models/photo_spec.rb
@@ -1,8 +1,9 @@
 require 'rails_helper'
 
 describe Photo do
+  let(:photo) { FactoryGirl.create(:photo, owner: member) }
+  let(:member) { FactoryGirl.create(:member) }
   describe 'add/delete functionality' do
-    let(:photo) { FactoryGirl.create(:photo) }
     let(:planting) { FactoryGirl.create(:planting) }
     let(:harvest) { FactoryGirl.create(:harvest) }
     let(:garden) { FactoryGirl.create(:garden) }
@@ -118,4 +119,10 @@ describe Photo do
       photo.should.respond_to? :flickr_metadata
     end
   end
+
+  it 'excludes deleted members' do
+    expect(Photo.all).to include(photo)
+    member.destroy
+    expect(Photo.all).not_to include(photo)
+  end
 end
diff --git a/spec/models/planting_spec.rb b/spec/models/planting_spec.rb
index aee1a3463..1e3529f4d 100644
--- a/spec/models/planting_spec.rb
+++ b/spec/models/planting_spec.rb
@@ -347,4 +347,10 @@ describe Planting do
       end
     end
   end
+
+  it 'excludes deleted members' do
+    expect(Planting.all).to include(planting)
+    planting.owner.destroy
+    expect(Planting.all).not_to include(planting)
+  end
 end
diff --git a/spec/models/post_spec.rb b/spec/models/post_spec.rb
index c8ff686fe..db115852d 100644
--- a/spec/models/post_spec.rb
+++ b/spec/models/post_spec.rb
@@ -2,7 +2,6 @@ require 'rails_helper'
 
 describe Post do
   let(:member) { FactoryGirl.create(:member) }
-
   it_behaves_like "it is likeable"
 
   it "should be sorted in reverse order" do
@@ -182,4 +181,11 @@ describe Post do
       end
     end
   end
+
+  it 'excludes deleted members' do
+    post = FactoryGirl.create :post, author: member
+    expect(Post.all).to include(post)
+    member.destroy
+    expect(Post.all).not_to include(post)
+  end
 end