diff --git a/spec/controllers/charts/gardens_controller_spec.rb b/spec/controllers/charts/gardens_controller_spec.rb index daa50a811..c42c043f0 100644 --- a/spec/controllers/charts/gardens_controller_spec.rb +++ b/spec/controllers/charts/gardens_controller_spec.rb @@ -5,18 +5,23 @@ describe Charts::GardensController do let(:valid_params) { { name: 'My second Garden' } } let(:garden) { FactoryBot.create :garden } + context "when not signed in" do describe 'GET timeline' do before { get :timeline, garden_id: garden.to_param } + it { expect(response).to be_success } end end + context "when signed in" do before(:each) { sign_in member } let!(:member) { FactoryBot.create(:member) } + describe 'GET timeline' do before { get :timeline, garden_id: garden.to_param } + it { expect(response).to be_success } end end diff --git a/spec/controllers/comments_controller_spec.rb b/spec/controllers/comments_controller_spec.rb index 2d265de5f..beee82b33 100644 --- a/spec/controllers/comments_controller_spec.rb +++ b/spec/controllers/comments_controller_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' describe CommentsController do subject { response } + let(:member) { FactoryBot.create(:member) } before(:each) do @@ -20,6 +21,7 @@ describe CommentsController do describe "returns an RSS feed" do before { get :index, format: "rss" } + it { is_expected.to be_success } it { is_expected.to render_template("comments/index") } it { expect(response.content_type).to eq("application/rss+xml") } @@ -81,11 +83,13 @@ describe CommentsController do expect(response).to redirect_to(comment.post) end end + describe "not my comment" do let(:comment) { FactoryBot.create :comment } it { expect(response).not_to be_success } end + describe "attempting to change post_id" do let(:post) { FactoryBot.create :post, subject: 'our post' } let(:other_post) { FactoryBot.create :post, subject: 'the other post' } diff --git a/spec/controllers/crops_controller_spec.rb b/spec/controllers/crops_controller_spec.rb index 5b682f6c2..afdea715c 100644 --- a/spec/controllers/crops_controller_spec.rb +++ b/spec/controllers/crops_controller_spec.rb @@ -16,6 +16,7 @@ describe CropsController do describe "GET crop wrangler homepage" do describe 'fetches the crop wrangler homepage' do before { get :wrangle } + it { is_expected.to be_success } it { is_expected.to render_template("crops/wrangle") } it { expect(assigns[:crop_wranglers]).to eq(Role.crop_wranglers) } @@ -25,6 +26,7 @@ describe CropsController do describe "GET crop hierarchy " do describe 'fetches the crop hierarchy page' do before { get :hierarchy } + it { is_expected.to be_success } it { is_expected.to render_template("crops/hierarchy") } end @@ -33,6 +35,7 @@ describe CropsController do describe "GET crop search" do describe 'fetches the crop search page' do before { get :search } + it { is_expected.to be_success } it { is_expected.to render_template("crops/search") } end @@ -41,6 +44,7 @@ describe CropsController do describe "GET RSS feed" do describe "returns an RSS feed" do before { get :index, format: "rss" } + it { is_expected.to be_success } it { is_expected.to render_template("crops/index") } it { expect(response.content_type).to eq("application/rss+xml") } diff --git a/spec/controllers/gardens_controller_spec.rb b/spec/controllers/gardens_controller_spec.rb index 2e31f2d78..a5620eb01 100644 --- a/spec/controllers/gardens_controller_spec.rb +++ b/spec/controllers/gardens_controller_spec.rb @@ -5,15 +5,20 @@ RSpec.describe GardensController, type: :controller do let(:valid_params) { { name: 'My second Garden' } } let(:garden) { FactoryBot.create :garden } + context "when not signed in" do describe 'GET new' do before { get :new, id: garden.to_param } + it { expect(response).to redirect_to(new_member_session_path) } end + describe 'PUT create' do before { put :create, garden: valid_params } + it { expect(response).to redirect_to(new_member_session_path) } end + describe 'changing existing records' do before do allow(Garden).to receive(:find).and_return(:garden) @@ -23,20 +28,27 @@ RSpec.describe GardensController, type: :controller do expect(garden).not_to receive(:update!) expect(garden).not_to receive(:destroy) end + describe 'GET edit' do before { get :edit, id: garden.to_param } + it { expect(response).to redirect_to(new_member_session_path) } end + describe 'POST update' do before { post :update, id: garden.to_param, garden: valid_params } + it { expect(response).to redirect_to(new_member_session_path) } end + describe 'DELETE' do before { delete :destroy, id: garden.to_param, params: { garden: valid_params } } + it { expect(response).to redirect_to(new_member_session_path) } end end end + context "when signed in" do before(:each) { sign_in member } @@ -56,14 +68,19 @@ RSpec.describe GardensController, type: :controller do describe 'GET edit' do before { get :edit, id: not_my_garden.to_param } + it { expect(response).to redirect_to(root_path) } end + describe 'POST update' do before { post :update, id: not_my_garden.to_param, garden: valid_params } + it { expect(response).to redirect_to(root_path) } end + describe 'DELETE' do before { delete :destroy, id: not_my_garden.to_param, params: { garden: valid_params } } + it { expect(response).to redirect_to(root_path) } end end diff --git a/spec/controllers/harvests_controller_spec.rb b/spec/controllers/harvests_controller_spec.rb index 705f954b5..08f5bb1e4 100644 --- a/spec/controllers/harvests_controller_spec.rb +++ b/spec/controllers/harvests_controller_spec.rb @@ -22,23 +22,27 @@ describe HarvestsController do describe "assigns all harvests as @harvests" do before { get :index, {} } + it { assigns(:harvests).should =~ [harvest1, harvest2] } end describe "picks up owner from params and shows owner's harvests only" do before { get :index, owner: member1.slug } + it { expect(assigns(:owner)).to eq member1 } it { expect(assigns(:harvests)).to eq [harvest1] } end describe "picks up crop from params and shows the harvests for the crop only" do before { get :index, crop: maize.name } + it { expect(assigns(:crop)).to eq maize } it { expect(assigns(:harvests)).to eq [harvest2] } end describe "generates a csv" do before { get :index, format: "csv" } + it { expect(response.status).to eq 200 } end end @@ -48,6 +52,7 @@ describe HarvestsController do describe "assigns the requested harvest as @harvest" do before { get :show, id: harvest.to_param } + it { expect(assigns(:harvest)).to eq(harvest) } end end @@ -69,6 +74,7 @@ describe HarvestsController do describe "assigns the requested harvest as @harvest" do before { get :edit, id: harvest.to_param } + it { expect(assigns(:harvest)).to eq(harvest) } end end @@ -96,6 +102,7 @@ describe HarvestsController do let(:planting) { FactoryBot.create(:planting, owner_id: member.id, garden: member.gardens.first) } before { post :create, harvest: valid_attributes.merge(planting_id: planting.id) } + it { expect(Harvest.last.planting.id).to eq(planting.id) } end end @@ -124,6 +131,7 @@ describe HarvestsController do allow(Harvest).to receive(:new).and_return(harvest) post :create, harvest: valid_attributes.merge(planting_id: not_my_planting.id) end + it { expect(harvest.planting_id).not_to eq(not_my_planting.id) } end end @@ -179,6 +187,7 @@ describe HarvestsController do put :update, id: harvest.to_param, harvest: valid_attributes.merge(planting_id: not_my_planting.id) end + it { expect(harvest.planting_id).to eq(nil) } end end diff --git a/spec/controllers/likes_controller_spec.rb b/spec/controllers/likes_controller_spec.rb index a80fc0d56..0e9b8ae6b 100644 --- a/spec/controllers/likes_controller_spec.rb +++ b/spec/controllers/likes_controller_spec.rb @@ -11,6 +11,7 @@ describe LikesController do describe "POST create" do it { expect(response.content_type).to eq "application/json" } before { post :create, post_id: blogpost.id, format: :json } + it { expect(Like.last.likeable_id).to eq(blogpost.id) } it { expect(Like.last.likeable_type).to eq('Post') } it { JSON.parse(response.body)["description"] == "1 like" } @@ -26,6 +27,7 @@ describe LikesController do describe "DELETE destroy" do before { delete :destroy, id: like.id, format: :json } + it { expect(response.content_type).to eq "application/json" } describe "un-liking something i liked before" do diff --git a/spec/controllers/photos_controller_spec.rb b/spec/controllers/photos_controller_spec.rb index 76869ebf3..4be3aec7f 100644 --- a/spec/controllers/photos_controller_spec.rb +++ b/spec/controllers/photos_controller_spec.rb @@ -39,6 +39,7 @@ describe PhotosController do describe "planting photos" do before(:each) { get :new, type: "planting", id: planting.id } + it { assigns(:flickr_auth).should be_an_instance_of(Authentication) } it { assigns(:item).should eq planting } it { expect(flash[:alert]).not_to be_present } @@ -47,12 +48,14 @@ describe PhotosController do describe "harvest photos" do before { get :new, type: "harvest", id: harvest.id } + it { assigns(:item).should eq harvest } it { expect(flash[:alert]).not_to be_present } end describe "garden photos" do before { get :new, type: "garden", id: garden.id } + it { assigns(:item).should eq garden } it { expect(flash[:alert]).not_to be_present } end @@ -76,6 +79,7 @@ describe PhotosController do describe "with valid params" do before { controller.stub(:current_member) { member } } + it "attaches the photo to a planting" do post :create, photo: { flickr_photo_id: photo.flickr_photo_id }, type: "planting", id: planting.id expect(flash[:alert]).not_to be_present @@ -87,6 +91,7 @@ describe PhotosController do post :create, photo: { flickr_photo_id: photo.flickr_photo_id }, type: "planting", id: planting.id post :create, photo: { flickr_photo_id: photo.flickr_photo_id }, type: "planting", id: planting.id end + it { expect(flash[:alert]).not_to be_present } it { expect(Photo.last.plantings.size).to eq 1 } end @@ -127,6 +132,7 @@ describe PhotosController do describe "with matching owners" do before { controller.stub(:current_member) { member } } + it "creates the planting/photo link" do planting = FactoryBot.create(:planting, garden: garden, owner: member) photo = FactoryBot.create(:photo, owner: member) diff --git a/spec/controllers/plantings_controller_spec.rb b/spec/controllers/plantings_controller_spec.rb index c4fe2396f..1d0fbe64f 100644 --- a/spec/controllers/plantings_controller_spec.rb +++ b/spec/controllers/plantings_controller_spec.rb @@ -20,17 +20,20 @@ describe PlantingsController do describe "assigns all plantings as @plantings" do before { get :index, {} } + it { expect(assigns(:plantings)).to match [planting1, planting2] } end describe "picks up owner from params and shows owner's plantings only" do before { get :index, owner: member1.slug } + it { expect(assigns(:owner)).to eq member1 } it { expect(assigns(:plantings)).to eq [planting1] } end describe "picks up crop from params and shows the plantings for the crop only" do before { get :index, crop: maize.name } + it { expect(assigns(:crop)).to eq maize } it { expect(assigns(:plantings)).to eq [planting2] } end @@ -41,11 +44,13 @@ describe PlantingsController do let(:crop) { FactoryBot.create(:crop) } before { get :new, crop_id: crop.id } + it { expect(assigns(:crop)).to eq(crop) } end describe "doesn't die if no crop specified" do before { get :new, {} } + it { expect(assigns(:crop)).to be_a_new(Crop) } end @@ -53,6 +58,7 @@ describe PlantingsController do let(:garden) { FactoryBot.create(:garden, owner: member) } before { get :new, garden_id: garden.id } + it { expect(assigns(:garden)).to eq(garden) } end @@ -61,6 +67,7 @@ describe PlantingsController do let(:garden) { FactoryBot.create(:garden, owner: another_member) } before { get :new, garden_id: garden.id } + it { expect(assigns(:garden)).not_to eq(garden) } end @@ -69,6 +76,7 @@ describe PlantingsController do let!(:garden) { FactoryBot.create(:garden, owner: member) } before { get :new, crop_id: crop.id } + it { expect(assigns(:crop)).not_to eq(crop) } end @@ -77,22 +85,27 @@ describe PlantingsController do let!(:garden) { FactoryBot.create(:garden, owner: member) } before { get :new, crop_id: crop.id } + it { expect(assigns(:crop)).not_to eq(crop) } end describe "doesn't die if no garden specified" do before { get :new, {} } + it { expect(assigns(:garden)).to be_a_new(Garden) } end describe "sets the date of the planting to today" do before { get :new, {} } + it { expect(assigns(:planting).planted_at).to eq Time.zone.today } end context 'with parent seed' do let(:seed) { FactoryBot.create :seed, owner: member } + before { get :new, seed_id: seed.to_param } + it { expect(assigns(:seed)).to eq(seed) } end end @@ -100,6 +113,7 @@ describe PlantingsController do describe 'POST :create' do describe "sets the owner automatically" do before { post :create, planting: valid_attributes } + it { expect(assigns(:planting).owner).to eq subject.current_member } end end diff --git a/spec/controllers/seeds_controller_spec.rb b/spec/controllers/seeds_controller_spec.rb index a803d8688..073c3ba99 100644 --- a/spec/controllers/seeds_controller_spec.rb +++ b/spec/controllers/seeds_controller_spec.rb @@ -5,6 +5,7 @@ describe SeedsController do describe "GET index" do before { get :index, owner: owner.slug } + it "picks up owner from params" do assigns(:owner).should eq(owner) end @@ -21,7 +22,9 @@ describe SeedsController do context 'with parent planting' do let(:planting) { FactoryBot.create :planting, owner: owner } + before { get :new, planting_id: planting.to_param } + it { expect(assigns(:planting)).to eq(planting) } end end diff --git a/spec/factories/notifications.rb b/spec/factories/notifications.rb index 7903035d2..186c7b905 100644 --- a/spec/factories/notifications.rb +++ b/spec/factories/notifications.rb @@ -5,6 +5,7 @@ FactoryBot.define do sender recipient subject { "MyString" } + body { "MyText" } read { false } post diff --git a/spec/factories/post.rb b/spec/factories/post.rb index 6a955b95a..f88a123be 100644 --- a/spec/factories/post.rb +++ b/spec/factories/post.rb @@ -1,6 +1,7 @@ FactoryBot.define do factory :post do subject { "A Post" } + body { "This is some text." } author created_at { Time.now } diff --git a/spec/features/crops/crop_detail_page_spec.rb b/spec/features/crops/crop_detail_page_spec.rb index b7dca2e8d..62f80d37f 100644 --- a/spec/features/crops/crop_detail_page_spec.rb +++ b/spec/features/crops/crop_detail_page_spec.rb @@ -10,6 +10,7 @@ feature "crop detail page", js: true do visit crop_path(crop) page end + let(:crop) { create :crop } context "varieties" do @@ -190,6 +191,7 @@ feature "crop detail page", js: true do planting: FactoryBot.create(:planting, planted_at: 200.days.ago, crop: crop)) planting.crop.update_medians end + it "predicts harvest" do is_expected.to have_text("First harvest expected 20 days after planting") end @@ -254,18 +256,21 @@ feature "crop detail page", js: true do context 'annual and perennial' do before { visit crop_path(crop) } + context 'crop is an annual' do let(:crop) { FactoryBot.create :annual_crop } it { expect(page).to have_text 'annual crop (living and reproducing in a single year or less)' } it { expect(page).not_to have_text 'perennial crop (living more than two years)' } end + context 'crop is perennial' do let(:crop) { FactoryBot.create :perennial_crop } it { expect(page).to have_text 'perennial crop (living more than two years)' } it { expect(page).not_to have_text 'annual crop (living and reproducing in a single year or less)' } end + context 'crop perennial value is null' do let(:crop) { FactoryBot.create :crop, perennial: nil } diff --git a/spec/features/crops/crop_photos_spec.rb b/spec/features/crops/crop_photos_spec.rb index ccc5288f0..03e2bc853 100644 --- a/spec/features/crops/crop_photos_spec.rb +++ b/spec/features/crops/crop_photos_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' feature "crop detail page", js: true do subject { page } + let(:member) { create :member } let(:crop) { create :crop, plantings: [planting], harvests: [harvest] } @@ -34,10 +35,12 @@ feature "crop detail page", js: true do it { is_expected.to have_xpath("//img[contains(@src,'#{photo1.thumbnail_url}')]") } it { is_expected.to have_xpath("//img[contains(@src,'#{photo2.thumbnail_url}')]") } end + describe "show harvest photos" do it { is_expected.to have_xpath("//img[contains(@src,'#{photo3.thumbnail_url}')]") } it { is_expected.to have_xpath("//img[contains(@src,'#{photo4.thumbnail_url}')]") } end + describe "link to more photos" do it { is_expected.to have_link "more photos" } end @@ -47,10 +50,12 @@ feature "crop detail page", js: true do background { login_as(create(:member)) } include_examples "shows photos" end + context "when signed in as photos owner" do background { login_as(member) } include_examples "shows photos" end + context "when not signed in " do include_examples "shows photos" end diff --git a/spec/features/gardens/actions_spec.rb b/spec/features/gardens/actions_spec.rb index 7342fe55e..a22ab6886 100644 --- a/spec/features/gardens/actions_spec.rb +++ b/spec/features/gardens/actions_spec.rb @@ -3,9 +3,11 @@ require 'custom_matchers' feature "Gardens" do context 'logged in' do + subject { page } + let(:member) { FactoryBot.create :member } background { login_as member } - subject { page } + let(:garden) { member.gardens.first } describe '#index' do @@ -21,6 +23,7 @@ feature "Gardens" do context 'my gardens' do before { visit gardens_path(owner: member) } + include_examples "has buttons bar at top" it "has actions on garden" do within '.garden-actions' do @@ -32,12 +35,16 @@ feature "Gardens" do end end end + context 'all gardens' do before { visit gardens_path } + include_examples "has buttons bar at top" end + context "other member's garden" do before { visit gardens_path(owner: FactoryBot.create(:member)) } + include_examples "has buttons bar at top" it 'does not show actions on other member garden' do is_expected.not_to have_link 'Plant something' diff --git a/spec/features/gardens/gardens_index_spec.rb b/spec/features/gardens/gardens_index_spec.rb index 0a973e305..03fc28447 100644 --- a/spec/features/gardens/gardens_index_spec.rb +++ b/spec/features/gardens/gardens_index_spec.rb @@ -4,6 +4,7 @@ require 'custom_matchers' feature "Gardens#index", :js do context "Logged in as member" do let(:member) { FactoryBot.create :member, login_name: 'shadow' } + background { login_as member } context "with 10 gardens" do @@ -58,6 +59,7 @@ feature "Gardens#index", :js do before do visit gardens_path(member: member) end + it "shows planting in garden" do expect(page).to have_link(planting.crop.name, href: planting_path(planting)) end @@ -71,6 +73,7 @@ feature "Gardens#index", :js do let(:garden) { member.gardens.first } let(:member) { FactoryBot.create :member, login_name: 'robbieburns' } let(:crop) { FactoryBot.create :crop } + before(:each) do # time to harvest = 50 day # time to finished = 90 days @@ -98,6 +101,7 @@ feature "Gardens#index", :js do garden: garden, planted_at: Time.zone.today end + it { expect(page).to have_link href: planting_path(planting) } it { expect(page).to have_link href: garden_path(planting.garden) } it { expect(page).to have_text '50 days until harvest' } @@ -112,6 +116,7 @@ feature "Gardens#index", :js do owner: member, garden: garden, planted_at: 51.days.ago end + it { expect(crop.median_days_to_first_harvest).to eq 50 } it { expect(crop.median_lifespan).to eq 90 } @@ -126,6 +131,7 @@ feature "Gardens#index", :js do crop: crop, owner: member, garden: garden, planted_at: 260.days.ago end + it { expect(page).to have_text 'super late' } it { expect(page).not_to have_text 'harvesting now' } it { expect(page).not_to have_text 'days until harvest' } diff --git a/spec/features/harvests/browse_harvests_spec.rb b/spec/features/harvests/browse_harvests_spec.rb index c51be9f41..010433be2 100644 --- a/spec/features/harvests/browse_harvests_spec.rb +++ b/spec/features/harvests/browse_harvests_spec.rb @@ -1,13 +1,16 @@ require 'rails_helper' feature "browse harvests" do + subject { page } + let!(:member) { create :member } let!(:harvest) { create :harvest, owner: member } background { login_as member } - subject { page } + feature 'blank optional fields' do let!(:harvest) { create :harvest, :no_description } + before { visit harvests_path } scenario 'read more' do diff --git a/spec/features/home/home_spec.rb b/spec/features/home/home_spec.rb index 4137b5e5a..c8759d48b 100644 --- a/spec/features/home/home_spec.rb +++ b/spec/features/home/home_spec.rb @@ -1,6 +1,8 @@ require 'rails_helper' feature "home page" do + subject { page } + let(:member) { FactoryBot.create :member } # let(:seed_photo) { FactoryBot.create :photo } let(:photo) { FactoryBot.create :photo } @@ -13,6 +15,7 @@ feature "home page" do let!(:tradable_seed) { FactoryBot.create :tradable_seed, finished: false } let!(:finished_seed) { FactoryBot.create :tradable_seed, finished: true } let!(:untradable_seed) { FactoryBot.create :untradable_seed } + background do # Add photos, so they can appear on home page planting.photos << photo @@ -20,7 +23,7 @@ feature "home page" do harvest.photos << photo end - subject { page } + before { visit root_path } shared_examples 'shows seeds' do @@ -55,12 +58,14 @@ feature "home page" do it { is_expected.to have_text 'Some of our crops' } it { is_expected.to have_link href: crop_path(crop) } end + describe 'shows recently added crops' do it { is_expected.to have_text 'Recently Added' } it 'link to newest crops' do is_expected.to have_link crop.name, href: crop_path(crop) end end + it 'includes a link to all crops' do is_expected.to have_link 'View all crops' end diff --git a/spec/features/seeds/seed_photos.rb b/spec/features/seeds/seed_photos.rb index 7e26ece28..ff8fd1db0 100644 --- a/spec/features/seeds/seed_photos.rb +++ b/spec/features/seeds/seed_photos.rb @@ -7,6 +7,7 @@ feature "Seeds", :js do visit seed_path(seed) page end + let(:member) { FactoryBot.create :member } let!(:seed) { FactoryBot.create :seed, owner: member } @@ -17,13 +18,16 @@ feature "Seeds", :js do # end context 'has one photo' do before { seed.photos = [photo] } + let!(:photo) { FactoryBot.create :photo, title: 'hello photo' } it { is_expected.to have_xpath("//img[contains(@src,'#{photo.thumbnail_url}')]") } it { is_expected.to have_xpath("//a[contains(@href,'#{photo_path(photo)}')]") } end + context 'has 50 photos' do before { seed.photos = photos } + let!(:photos) { FactoryBot.create_list :photo, 50 } it "shows newest photo" do diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb index d1eae5cf6..5c1d11d9e 100644 --- a/spec/helpers/application_helper_spec.rb +++ b/spec/helpers/application_helper_spec.rb @@ -20,6 +20,7 @@ describe ApplicationHelper do before :each do @member = FactoryBot.build(:member, email: 'example@example.com', preferred_avatar_uri: nil) end + it 'should render a gravatar uri' do expect(avatar_uri(@member)).to eq 'http://www.gravatar.com/avatar/23463b99b62a72f26ed677cc556c44e8?size=150&default=identicon' end @@ -33,6 +34,7 @@ describe ApplicationHelper do before :each do @member = FactoryBot.build(:member, email: 'example@example.com', preferred_avatar_uri: 'http://media.catmoji.com/post/ujg/cat-in-hat.jpg') end + it 'should render a the specified uri' do expect(avatar_uri(@member)).to eq 'http://media.catmoji.com/post/ujg/cat-in-hat.jpg' end @@ -87,6 +89,7 @@ describe ApplicationHelper do expect(build_alert_classes(:info)).not_to include ' alert-danger' end end + context 'warning' do it 'works when :warning' do expect(build_alert_classes(:warning)).to include 'alert-warning' @@ -101,6 +104,7 @@ describe ApplicationHelper do expect(build_alert_classes(:info)).not_to include ' alert-warning' end end + context 'success' do it 'works when :notice' do expect(build_alert_classes(:notice)).to include 'alert-success' @@ -115,6 +119,7 @@ describe ApplicationHelper do expect(build_alert_classes(:info)).not_to include ' alert-success' end end + context 'info' do it 'works when :info' do expect(build_alert_classes(:info)).to include 'alert-info' diff --git a/spec/helpers/photos_helper_spec.rb b/spec/helpers/photos_helper_spec.rb index 589cf8104..63a45a548 100644 --- a/spec/helpers/photos_helper_spec.rb +++ b/spec/helpers/photos_helper_spec.rb @@ -19,6 +19,7 @@ describe PhotosHelper do describe "with a planting" do before { planting.photos << planting_photo } + it "uses planting photos" do is_expected.to eq planting_photo.thumbnail_url end @@ -26,6 +27,7 @@ describe PhotosHelper do describe "with a harvest photos" do before { harvest.photos << harvest_photo } + it "uses harvest photos" do is_expected.to eq harvest_photo.thumbnail_url end @@ -33,6 +35,7 @@ describe PhotosHelper do describe "uses seed photo" do before { seed.photos << seed_photo } + it "uses seed photos" do is_expected.to eq seed_photo.thumbnail_url end @@ -41,38 +44,46 @@ describe PhotosHelper do describe "gardens" do subject { garden_image_path(garden) } + it { is_expected.to eq 'placeholder_150.png' } describe "uses garden's own photo" do before { garden.photos << garden_photo } + it { is_expected.to eq garden_photo.thumbnail_url } end end describe 'plantings' do subject { planting_image_path(planting) } + it { is_expected.to eq 'placeholder_150.png' } describe "uses planting's own photo" do before { planting.photos << planting_photo } + it { is_expected.to eq planting_photo.thumbnail_url } end end describe 'harvests' do subject { harvest_image_path(harvest) } + it { is_expected.to eq 'placeholder_150.png' } describe "uses harvest's own photo" do before { harvest.photos << harvest_photo } + it { is_expected.to eq harvest_photo.thumbnail_url } end end describe 'seeds' do subject { seed_image_path(seed) } + it { is_expected.to eq 'placeholder_150.png' } describe "uses seed's own photo" do before { seed.photos << seed_photo } + it { is_expected.to eq seed_photo.thumbnail_url } end end diff --git a/spec/models/crop_spec.rb b/spec/models/crop_spec.rb index 3f49a249d..ec4ae63c5 100644 --- a/spec/models/crop_spec.rb +++ b/spec/models/crop_spec.rb @@ -142,6 +142,7 @@ describe Crop do before :each do @crop = FactoryBot.create(:tomato) end + context 'with a planting photo' do before :each do @planting = FactoryBot.create(:planting, crop: @crop) @@ -277,6 +278,7 @@ describe Crop do context 'interesting' do subject { Crop.interesting } + let(:photo) { FactoryBot.create :photo } # first, a couple of candidate crops let(:crop1) { FactoryBot.create(:crop) } @@ -307,6 +309,7 @@ describe Crop do # ... and photos crop1_planting.photos = FactoryBot.create_list(:photo, 3) end + it { is_expected.to include crop1 } it { is_expected.not_to include crop2 } it { expect(subject.size).to eq 1 } @@ -321,6 +324,7 @@ describe Crop do # but only crop1 has photos crop1_planting.photos = FactoryBot.create_list(:photo, 3) end + it { is_expected.to include crop1 } it { is_expected.not_to include crop2 } it { expect(subject.size).to eq 1 } diff --git a/spec/models/member_spec.rb b/spec/models/member_spec.rb index eb4e105b1..5f278c32c 100644 --- a/spec/models/member_spec.rb +++ b/spec/models/member_spec.rb @@ -341,6 +341,7 @@ describe 'member' do context 'queries a scope' do before { member.destroy } + it { expect(Member.all).not_to include(member) } it { expect(Member.confirmed).not_to include(member) } it { expect(Member.located).not_to include(member) } @@ -350,6 +351,7 @@ describe 'member' do it { expect(Member.interesting).not_to include(member) } it { expect(Member.has_plantings).not_to include(member) } end + it "unsubscribes from mailing list" do expect(member).to receive(:newsletter_unsubscribe).and_return(true) member.destroy diff --git a/spec/models/planting_spec.rb b/spec/models/planting_spec.rb index 6b49f1c0a..7d4a94221 100644 --- a/spec/models/planting_spec.rb +++ b/spec/models/planting_spec.rb @@ -19,6 +19,7 @@ describe Planting do it { expect(planting.days_since_planted).to eq(30) } it { expect(planting.percentage_grown).to eq(nil) } end + describe 'planting not planted yet' do let(:planting) { FactoryBot.create :planting, planted_at: nil, finished_at: nil, finished: false } @@ -27,6 +28,7 @@ describe Planting do it { expect(planting.days_since_planted).to eq(nil) } it { expect(planting.percentage_grown).to eq(nil) } end + describe 'planting finished, no planted_at' do let(:planting) { FactoryBot.create :planting, planted_at: nil, finished_at: 1.day.ago, finished: true } @@ -35,6 +37,7 @@ describe Planting do it { expect(planting.days_since_planted).to eq(nil) } it { expect(planting.percentage_grown).to eq(100) } end + describe 'planting all finished' do let(:planting) { FactoryBot.create :planting, planted_at: 30.days.ago, finished_at: 1.day.ago, finished: true } @@ -90,6 +93,7 @@ describe Planting do describe 'planting finished 10 days, but was never planted' do let(:planting) { FactoryBot.create :planting, planted_at: nil, finished_at: 10.days.ago } + it { expect(planting.percentage_grown).to eq 100 } end @@ -112,9 +116,11 @@ describe Planting do it { expect(planting.days_to_last_harvest).to eq(nil) } it { expect(planting.expected_lifespan).to eq(nil) } end + context 'lots of data' do let(:crop) { FactoryBot.create :crop } # this is a method so it creates a new one each time + def one_hundred_day_old_planting FactoryBot.create(:planting, crop: crop, planted_at: 100.days.ago) end @@ -133,33 +139,42 @@ describe Planting do planting.crop.update_lifespan_medians planting.crop.update_harvest_medians end + it { expect(crop.median_days_to_first_harvest).to eq(20) } describe 'sets median time to harvest' do let(:planting) { FactoryBot.create :planting, crop: crop, planted_at: Time.zone.today } + it { expect(planting.first_harvest_predicted_at).to eq(Time.zone.today + 20.days) } end describe 'harvest still growing' do let(:planting) { FactoryBot.create :planting, crop: crop, planted_at: Time.zone.today } + it { expect(planting.before_harvest_time?).to eq true } it { expect(planting.harvest_time?).to eq false } end + describe 'harvesting ready now' do let(:planting) { FactoryBot.create :planting, crop: crop, planted_at: 21.days.ago } + it { expect(planting.first_harvest_predicted_at).to eq(1.day.ago.to_date) } it { expect(planting.before_harvest_time?).to eq false } it { expect(planting.harvest_time?).to eq true } end end + describe 'planting has no harvests' do let(:planting) { FactoryBot.create :planting } + before do planting.update_harvest_days! planting.crop.update_harvest_medians end + it { expect(planting.days_to_first_harvest).to eq(nil) } it { expect(planting.days_to_last_harvest).to eq(nil) } end + describe 'planting has first harvest' do let(:planting) { FactoryBot.create :planting, planted_at: 100.days.ago } @@ -171,11 +186,13 @@ describe Planting do planting.update_harvest_days! planting.crop.update_harvest_medians end + it { expect(planting.days_to_first_harvest).to eq(90) } it { expect(planting.days_to_last_harvest).to eq(nil) } it { expect(planting.crop.median_days_to_first_harvest).to eq(90) } it { expect(planting.crop.median_days_to_last_harvest).to eq(nil) } end + describe 'planting has last harvest' do let(:planting) { FactoryBot.create :planting, planted_at: 100.days.ago, finished_at: 1.day.ago, finished: true } @@ -185,6 +202,7 @@ describe Planting do planting.update_harvest_days! planting.crop.update_harvest_medians end + it { expect(planting.days_to_first_harvest).to eq(10) } it { expect(planting.days_to_last_harvest).to eq(90) } it { expect(planting.crop.median_days_to_first_harvest).to eq(10) } @@ -463,6 +481,7 @@ describe Planting do context 'ancestry' do let(:parent_seed) { FactoryBot.create :seed } let(:planting) { FactoryBot.create :planting, parent_seed: parent_seed } + it "planting has a parent seed" do expect(planting.parent_seed).to eq(parent_seed) end @@ -471,6 +490,7 @@ describe Planting do end describe 'grandchildren' do let(:grandchild_seed) { FactoryBot.create :seed, parent_planting: planting } + it { expect(grandchild_seed.parent_planting).to eq planting } it { expect(grandchild_seed.parent_planting.parent_seed).to eq parent_seed } end diff --git a/spec/models/role_spec.rb b/spec/models/role_spec.rb index 76e309aa0..0fc5f92b7 100644 --- a/spec/models/role_spec.rb +++ b/spec/models/role_spec.rb @@ -6,6 +6,7 @@ describe Role do role.members << member role end + let(:member) { FactoryBot.create(:member) } it 'has members' do diff --git a/spec/models/seed_spec.rb b/spec/models/seed_spec.rb index 269cb888f..f15a57398 100644 --- a/spec/models/seed_spec.rb +++ b/spec/models/seed_spec.rb @@ -153,6 +153,7 @@ describe Seed do let(:seed) { FactoryBot.create :seed } before { seed.photos << FactoryBot.create(:photo) } + it 'is found in has_photos scope' do Seed.has_photos.should include(seed) end @@ -161,6 +162,7 @@ describe Seed do context 'ancestry' do let(:parent_planting) { FactoryBot.create :planting } let(:seed) { FactoryBot.create :seed, parent_planting: parent_planting } + it "seed has a parent planting" do expect(seed.parent_planting).to eq(parent_planting) end @@ -172,6 +174,7 @@ describe Seed do context "finished" do describe 'has finished fields' do let(:seed) { FactoryBot.create(:finished_seed) } + it { expect(seed.finished).to eq true } it { expect(seed.finished_at).to be_an_instance_of Date } end diff --git a/spec/requests/api/v1/crop_request_spec.rb b/spec/requests/api/v1/crop_request_spec.rb index 6a392eb74..f6cc2d4a5 100644 --- a/spec/requests/api/v1/crop_request_spec.rb +++ b/spec/requests/api/v1/crop_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Plantings', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:crop) { FactoryBot.create :crop } let(:crop_encoded_as_json_api) do @@ -57,11 +58,13 @@ RSpec.describe 'Plantings', type: :request do describe '#index' do before { get '/api/v1/crops', {}, headers } + it { expect(subject['data']).to include(crop_encoded_as_json_api) } end describe '#show' do before { get "/api/v1/crops/#{crop.id}", {}, headers } + it { expect(subject['data']['attributes']).to eq(attributes) } it { expect(subject['data']['relationships']).to include("plantings" => plantings_as_json_api) } it { expect(subject['data']['relationships']).to include("harvests" => harvests_as_json_api) } @@ -72,16 +75,19 @@ RSpec.describe 'Plantings', type: :request do describe '#create' do before { post '/api/v1/crops', { 'crop' => { 'name' => 'can i make this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#update' do before { post "/api/v1/crops/#{crop.id}", { 'crop' => { 'name' => 'can i modify this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#delete' do before { delete "/api/v1/crops/#{crop.id}", {}, headers } + it { expect(response.code).to eq "404" } end end diff --git a/spec/requests/api/v1/gardens_request_spec.rb b/spec/requests/api/v1/gardens_request_spec.rb index d6005e286..c0c33c363 100644 --- a/spec/requests/api/v1/gardens_request_spec.rb +++ b/spec/requests/api/v1/gardens_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Gardens', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:garden) { FactoryBot.create :garden } let(:garden_encoded_as_json_api) do diff --git a/spec/requests/api/v1/harvest_request_spec.rb b/spec/requests/api/v1/harvest_request_spec.rb index af2d4201e..48094de1b 100644 --- a/spec/requests/api/v1/harvest_request_spec.rb +++ b/spec/requests/api/v1/harvest_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Harvests', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:harvest) { FactoryBot.create :harvest } let(:harvest_encoded_as_json_api) do @@ -58,11 +59,13 @@ RSpec.describe 'Harvests', type: :request do describe '#index' do before { get '/api/v1/harvests', {}, headers } + it { expect(subject['data']).to include(harvest_encoded_as_json_api) } end describe '#show' do before { get "/api/v1/harvests/#{harvest.id}", {}, headers } + it { expect(subject['data']['attributes']).to eq(attributes) } it { expect(subject['data']['relationships']).to include("planting" => planting_as_json_api) } it { expect(subject['data']['relationships']).to include("crop" => crop_as_json_api) } @@ -73,16 +76,19 @@ RSpec.describe 'Harvests', type: :request do describe '#create' do before { post '/api/v1/harvests', { 'harvest' => { 'description' => 'can i make this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#update' do before { post "/api/v1/harvests/#{harvest.id}", { 'harvest' => { 'description' => 'can i modify this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#delete' do before { delete "/api/v1/harvests/#{harvest.id}", {}, headers } + it { expect(response.code).to eq "404" } end end diff --git a/spec/requests/api/v1/member_request_spec.rb b/spec/requests/api/v1/member_request_spec.rb index 6067e59b6..3767f8a43 100644 --- a/spec/requests/api/v1/member_request_spec.rb +++ b/spec/requests/api/v1/member_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Members', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:member) { FactoryBot.create :member } let(:member_encoded_as_json_api) do @@ -58,11 +59,13 @@ RSpec.describe 'Members', type: :request do describe '#index' do before { get '/api/v1/members', {}, headers } + it { expect(subject['data']).to include(member_encoded_as_json_api) } end describe '#show' do before { get "/api/v1/members/#{member.id}", {}, headers } + it { expect(subject['data']['relationships']).to include("gardens" => gardens_as_json_api) } it { expect(subject['data']['relationships']).to include("plantings" => plantings_as_json_api) } it { expect(subject['data']['relationships']).to include("seeds" => seeds_as_json_api) } @@ -73,16 +76,19 @@ RSpec.describe 'Members', type: :request do describe '#create' do before { post '/api/v1/members', { 'member' => { 'login_name' => 'can i make this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#update' do before { post "/api/v1/members/#{member.id}", { 'member' => { 'login_name' => 'can i modify this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#delete' do before { delete "/api/v1/members/#{member.id}", {}, headers } + it { expect(response.code).to eq "404" } end end diff --git a/spec/requests/api/v1/photos_request_spec.rb b/spec/requests/api/v1/photos_request_spec.rb index e9e584a60..ed16505f6 100644 --- a/spec/requests/api/v1/photos_request_spec.rb +++ b/spec/requests/api/v1/photos_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Photos', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:photo) { FactoryBot.create :photo } let(:photo_encoded_as_json_api) do @@ -56,11 +57,13 @@ RSpec.describe 'Photos', type: :request do describe '#index' do before { get '/api/v1/photos', {}, headers } + it { expect(subject['data']).to include(photo_encoded_as_json_api) } end describe '#show' do before { get "/api/v1/photos/#{photo.id}", {}, headers } + it { expect(subject['data']['attributes']).to eq(attributes) } it { expect(subject['data']['relationships']).to include("plantings" => plantings_as_json_api) } it { expect(subject['data']['relationships']).to include("harvests" => harvests_as_json_api) } @@ -70,16 +73,19 @@ RSpec.describe 'Photos', type: :request do describe '#create' do before { post '/api/v1/photos', { 'photo' => { 'name' => 'can i make this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#update' do before { post "/api/v1/photos/#{photo.id}", { 'photo' => { 'name' => 'can i modify this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#delete' do before { delete "/api/v1/photos/#{photo.id}", {}, headers } + it { expect(response.code).to eq "404" } end end diff --git a/spec/requests/api/v1/plantings_request_spec.rb b/spec/requests/api/v1/plantings_request_spec.rb index 4e96cefb4..81bc06a14 100644 --- a/spec/requests/api/v1/plantings_request_spec.rb +++ b/spec/requests/api/v1/plantings_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Plantings', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:planting) { FactoryBot.create :planting } let(:planting_encoded_as_json_api) do diff --git a/spec/requests/api/v1/seeds_request_spec.rb b/spec/requests/api/v1/seeds_request_spec.rb index f518e91de..406412dc8 100644 --- a/spec/requests/api/v1/seeds_request_spec.rb +++ b/spec/requests/api/v1/seeds_request_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' RSpec.describe 'Photos', type: :request do subject { JSON.parse response.body } + let(:headers) { { 'Accept' => 'application/vnd.api+json' } } let!(:seed) { FactoryBot.create :seed } let(:seed_encoded_as_json_api) do @@ -45,11 +46,13 @@ RSpec.describe 'Photos', type: :request do describe '#index' do before { get '/api/v1/seeds', {}, headers } + it { expect(subject['data']).to include(seed_encoded_as_json_api) } end describe '#show' do before { get "/api/v1/seeds/#{seed.id}", {}, headers } + it { expect(subject['data']['attributes']).to eq(attributes) } it { expect(subject['data']['relationships']).to include("owner" => owner_as_json_api) } it { expect(subject['data']['relationships']).to include("crop" => crop_as_json_api) } @@ -58,16 +61,19 @@ RSpec.describe 'Photos', type: :request do describe '#create' do before { post '/api/v1/seeds', { 'seed' => { 'name' => 'can i make this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#update' do before { post "/api/v1/seeds/#{seed.id}", { 'seed' => { 'name' => 'can i modify this' } }, headers } + it { expect(response.code).to eq "404" } end describe '#delete' do before { delete "/api/v1/seeds/#{seed.id}", {}, headers } + it { expect(response.code).to eq "404" } end end diff --git a/spec/views/crops/_planting_advice.html.haml_spec.rb b/spec/views/crops/_planting_advice.html.haml_spec.rb index 12064a1b8..f5adc5b29 100644 --- a/spec/views/crops/_planting_advice.html.haml_spec.rb +++ b/spec/views/crops/_planting_advice.html.haml_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' describe "crops/_planting_advice" do subject { rendered } + let(:planting) { FactoryBot.create(:planting) } shared_examples "render planting_advice" do @@ -18,6 +19,7 @@ describe "crops/_planting_advice" do context "with sunniness frequencies" do before { FactoryBot.create(:sunny_planting, crop: planting.crop) } + include_examples "render planting_advice" it { is_expected.to have_content "Plant in:" } it { is_expected.to have_content "sun (1)" } @@ -28,6 +30,7 @@ describe "crops/_planting_advice" do FactoryBot.create_list(:sunny_planting, 2, crop: planting.crop) FactoryBot.create(:shady_planting, crop: planting.crop) end + include_examples "render planting_advice" it { is_expected.to have_content "Plant in:" } it { is_expected.to have_content "sun (2), shade (1)" } @@ -44,6 +47,7 @@ describe "crops/_planting_advice" do context "with planted_from frequencies" do before { FactoryBot.create(:seed_planting, crop: planting.crop) } + include_examples "render planting_advice" it { is_expected.to have_content "Plant from:" } it { is_expected.to have_content "seed (1)" } @@ -54,6 +58,7 @@ describe "crops/_planting_advice" do FactoryBot.create_list(:seed_planting, 2, crop: planting.crop) FactoryBot.create(:cutting_planting, crop: planting.crop) end + include_examples "render planting_advice" it { is_expected.to have_content "Plant from:" } it { is_expected.to have_content "seed (2), cutting (1)" } diff --git a/spec/views/devise/registrations/edit_spec.rb b/spec/views/devise/registrations/edit_spec.rb index 3d9dae906..64af90017 100644 --- a/spec/views/devise/registrations/edit_spec.rb +++ b/spec/views/devise/registrations/edit_spec.rb @@ -64,11 +64,13 @@ describe 'devise/registrations/edit.html.haml', type: "view" do assert_select "a", "Connect to Twitter" end end + context 'connected to twitter' do before(:each) do @twitter_auth = FactoryBot.create(:authentication, member: @member) render end + it 'has a link to twitter profile' do assert_select "a", href: "http://twitter.com/#{@twitter_auth.name}" end @@ -84,11 +86,13 @@ describe 'devise/registrations/edit.html.haml', type: "view" do assert_select "a", "Connect to Flickr" end end + context 'connected to flickr' do before(:each) do @flickr_auth = FactoryBot.create(:flickr_authentication, member: @member) render end + it 'has a link to flickr photostream' do assert_select "a", href: "http://flickr.com/photos/#{@flickr_auth.uid}" end diff --git a/spec/views/harvests/show.html.haml_spec.rb b/spec/views/harvests/show.html.haml_spec.rb index 4aec453ff..f3c17f2b5 100644 --- a/spec/views/harvests/show.html.haml_spec.rb +++ b/spec/views/harvests/show.html.haml_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' describe "harvests/show" do subject { render } + let!(:harvest) { FactoryBot.create(:harvest) } before do diff --git a/spec/views/members/show.rss.haml_spec.rb b/spec/views/members/show.rss.haml_spec.rb index 23366a9a8..46a986ae3 100644 --- a/spec/views/members/show.rss.haml_spec.rb +++ b/spec/views/members/show.rss.haml_spec.rb @@ -1,6 +1,8 @@ require 'rails_helper' describe 'members/show.rss.haml', type: "view" do + subject { rendered } + before(:each) do @member = assign(:member, FactoryBot.create(:member)) @post1 = FactoryBot.create(:post, id: 1, author: @member) @@ -9,7 +11,6 @@ describe 'members/show.rss.haml', type: "view" do render end - subject { rendered } it 'shows RSS feed title' do is_expected.to match(/member\d+'s recent posts/) diff --git a/spec/views/photos/show.html.haml_spec.rb b/spec/views/photos/show.html.haml_spec.rb index cecb0008b..d727be8a4 100644 --- a/spec/views/photos/show.html.haml_spec.rb +++ b/spec/views/photos/show.html.haml_spec.rb @@ -49,6 +49,7 @@ describe "photos/show" do controller.stub(:current_user) { member } render end + include_examples "photo data renders" it "has a delete button" do @@ -61,6 +62,7 @@ describe "photos/show" do controller.stub(:current_user) { FactoryBot.create :member } render end + include_examples "photo data renders" include_examples "No links to change data" end @@ -70,6 +72,7 @@ describe "photos/show" do controller.stub(:current_user) { nil } render end + include_examples "photo data renders" include_examples "No links to change data" end @@ -84,6 +87,7 @@ describe "photos/show" do @photo.gardens << garden render end + it "links to the CC license" do assert_select "a", href: @photo.license_url, text: @photo.license_name diff --git a/spec/views/posts/show.html.haml_spec.rb b/spec/views/posts/show.html.haml_spec.rb index ba6921e4e..ffdabfa36 100644 --- a/spec/views/posts/show.html.haml_spec.rb +++ b/spec/views/posts/show.html.haml_spec.rb @@ -2,6 +2,7 @@ require 'rails_helper' describe "posts/show" do subject { rendered } + let(:author) { FactoryBot.create(:member) } before(:each) do @@ -11,6 +12,7 @@ describe "posts/show" do describe 'render post' do before { render } + describe "basic post" do let(:post) { FactoryBot.create(:post, author: author) } @@ -35,11 +37,13 @@ describe "posts/show" do it { is_expected.to have_content('EVIL') } it { is_expected.not_to have_link("http://evil.com") } end + describe 'script tag in post body' do let(:post) { FactoryBot.create(:post, author: author, body: "") } it { is_expected.not_to have_selector('script') } end + describe 'script tag in post title' do let(:post) { FactoryBot.create(:post, author: author, subject: "") } @@ -99,6 +103,7 @@ describe "posts/show" do let(:post) { FactoryBot.create(:forum_post, author: author) } before { render } + it "shows forum name" do is_expected.to have_content "in #{post.forum.name}" end