From: Neil Smith <neil.git@njae.me.uk>
Date: Tue, 7 Jan 2014 23:40:14 +0000 (+0000)
Subject: Tidied some specs
X-Git-Url: https://git.njae.me.uk/?p=feedcatcher.git;a=commitdiff_plain;h=9e1c293eb864b0ab812b1620e4f91c494b792147

Tidied some specs
---

diff --git a/spec/models/feed_item_spec.rb b/spec/models/feed_item_spec.rb
index f932e43..d2aabc6 100644
--- a/spec/models/feed_item_spec.rb
+++ b/spec/models/feed_item_spec.rb
@@ -1,55 +1,49 @@
 require 'spec_helper'
 
 describe FeedItem do
-  before(:each) do
-    @feed_item = FeedItem.new(
-      :feed_name => "feed",
-      :title => "foo",
-      :description => "bar"
-    )
-  end
-
+ 
   it "is valid with valid attributes" do
-    expect(@feed_item).to be_valid
+    feed_item = FactoryGirl.build(:feed_item)
+    expect(feed_item).to be_valid
   end
 
   it "is not valid without a feed name" do
-    @feed_item.feed_name = nil
-    expect(@feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: nil)
+    expect(feed_item).not_to be_valid
   end
 
   it "is not valid without a title" do
-    @feed_item.title = nil
-    expect(@feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, title: nil)
+    expect(feed_item).not_to be_valid
   end
 
   it "is not valid without a description" do
-    @feed_item.description = nil
-    expect(@feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, description: nil)
+    expect(feed_item).not_to be_valid
   end
 
   it "is not valid with an improper feed name" do
-    @feed_item.feed_name = 'feed name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed%20name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed&name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed%26name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed<name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed%3Cname'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'feed%name'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'index'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'show'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'update'
-    expect(@feed_item).not_to be_valid
-    @feed_item.feed_name = 'action'
-    expect(@feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed%20name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed&name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed%26name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed<name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed%3Cname')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'feed%name')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'index')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'show')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'update')
+    expect(feed_item).not_to be_valid
+    feed_item = FactoryGirl.build(:feed_item, feed_name: 'action')
+    expect(feed_item).not_to be_valid
   end
 end
diff --git a/spec/requests/format_spec.rb b/spec/requests/format_spec.rb
index b55c6f5..216cc39 100644
--- a/spec/requests/format_spec.rb
+++ b/spec/requests/format_spec.rb
@@ -1,12 +1,6 @@
 require 'spec_helper'
 
 describe "format" do
-  let!(:feed_item1) { FactoryGirl.create(:feed_item, 
-      title: "item 1") }
-  let!(:feed_item2) { FactoryGirl.create(:feed_item, 
-      title: "item 2") }
-  let!(:other_feed_item) { FactoryGirl.create(:feed_item, 
-      feed_name: "other_test_feed") }
 
   it "serves index as html by default" do
     get_via_redirect '/index'