Fix TagsBlock unit test

Leandro Santos
1 parent 84df5586
Showing 4 changed files with 18 additions and 12 deletions Show diff stats
app/models/article.rb
test/unit/tags_block_test.rb
vendor/plugins/acts_as_taggable_on_steroids/lib/tag.rb
vendor/plugins/acts_as_taggable_on_steroids/lib/tag_list.rb
@@ -2,7 +2,7 @@ require &#39;hpricot&#39;
 class Article < ActiveRecord::Base
-  attr_accessible :name, :body, :abstract, :profile
+  attr_accessible :name, :body, :abstract, :profile, :tag_list
   SEARCHABLE_FIELDS = {
     :name => 10,
@@ -8,8 +8,12 @@ class TagsBlockTest &lt; ActiveSupport::TestCase
     @user.articles.build(:name => 'article 2', :tag_list => 'first-tag, second-tag').save!
     @user.articles.build(:name => 'article 3', :tag_list => 'first-tag, second-tag, third-tag').save!
-    box = Box.create!(:owner => @user)
-    @block = TagsBlock.create!(:box => box)
+    box = Box.new
+    box.owner = @user
+    box.save!
+    @block = TagsBlock.new
+    @block.box = box
+    @block.save
   end
   attr_reader :block
 class Tag < ActiveRecord::Base  
   has_many :taggings
+
+  attr_accessible :name
   validates_presence_of :name
   validates_uniqueness_of :name
@@ -40,19 +40,19 @@ class TagList
   end
   class << self
-    def from(tags)
-    	case tags
-    		when String 
-    			new(parse(tags))
-    		when Array 
-    			new(tags.map(&:to_s))
-    		else
-    			new([])
+    def from(tags)
+    	case tags
+    		when String 
+    			new(parse(tags))
+    		when Array 
+    			new(tags.map(&:to_s))
+    		else
+    			new([])
     	end
     end
     def parse(string)
-      returning [] do |names|
+      [].tap do |names|
         string = string.to_s.dup
         # Parse the quoted tags
	@@ -2,7 +2,7 @@ require 'hpricot'		@@ -2,7 +2,7 @@ require 'hpricot'
2		2
3	class Article < ActiveRecord::Base	3	class Article < ActiveRecord::Base
4		4
5	- attr_accessible :name, :body, :abstract, :profile	5	+ attr_accessible :name, :body, :abstract, :profile, :tag_list
6		6
7	SEARCHABLE_FIELDS = {	7	SEARCHABLE_FIELDS = {
8	:name => 10,	8	:name => 10,
1	class Tag < ActiveRecord::Base	1	class Tag < ActiveRecord::Base
2	has_many :taggings	2	has_many :taggings
		3	+
		4	+ attr_accessible :name
3		5
4	validates_presence_of :name	6	validates_presence_of :name
5	validates_uniqueness_of :name	7	validates_uniqueness_of :name