fixes community track plugin

Signed-off-by: Evandro Magalhaes Leite Junior <evandro.leite@serpro.gov.br> Signed-off-by: Gustavo Jaruga <darkshades@gmail.com> Signed-off-by: Leandro Nunes dos Santos <leandro.santos@serpro.gov.br> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com> Signed-off-by: Victor Costa <vfcosta@gmail.com>

fixes community track plugin
Signed-off-by: Evandro Magalhaes Leite Junior <evandro.leite@serpro.gov.br> Signed-off-by: Gustavo Jaruga <darkshades@gmail.com> Signed-off-by: Leandro Nunes dos Santos <leandro.santos@serpro.gov.br> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com> Signed-off-by: Victor Costa <vfcosta@gmail.com>
Marcos Pereira
1 parent 251e6e43
Showing 16 changed files with 151 additions and 15 deletions Show diff stats
app/controllers/my_profile/profile_design_controller.rb
plugins/community_track/controllers/public/community_track_plugin_public_controller.rb
plugins/community_track/lib/community_track_plugin.rb
plugins/community_track/lib/community_track_plugin/step.rb
plugins/community_track/lib/community_track_plugin/track.rb
plugins/community_track/lib/community_track_plugin/track_list_block.rb
plugins/community_track/test/functional/community_track_plugin_cms_controller_test.rb
plugins/community_track/test/functional/community_track_plugin_public_controller_test.rb
plugins/community_track/test/unit/community_track_plugin/step_test.rb
plugins/community_track/test/unit/community_track_plugin/track_list_block_test.rb
plugins/community_track/test/unit/community_track_plugin/track_test.rb
plugins/community_track/test/unit/community_track_plugin_test.rb
plugins/community_track/views/blocks/_track_list_more.html.erb
plugins/community_track/views/box_organizer/community_track_plugin/_track_list_block.html.erb
plugins/community_track/views/cms/community_track_plugin/_step.html.erb
test/functional/profile_design_controller_test.rb
@@ -67,4 +67,14 @@ class ProfileDesignController &lt; BoxOrganizerController
     blocks
   end
  
+  def update_categories
+    @object = params[:id] ? @profile.blocks.find(params[:id]) : Block.new
+    @categories = @toplevel_categories = environment.top_level_categories
+    if params[:category_id]
+      @current_category = Category.find(params[:category_id])
+      @categories = @current_category.children
+    end
+    render :template => 'shared/update_categories', :locals => { :category => @current_category, :object_name => 'block' }
+  end
+
 end
@@ -6,14 +6,15 @@ class CommunityTrackPluginPublicController &lt; PublicController
  
   def view_tracks
     block = Block.find(params[:id])
+    instance_eval(&block.set_seed)
     p = params[:page].to_i
     per_page = params[:per_page]
     per_page ||= block.limit
     per_page = per_page.to_i
-    tracks = block.tracks(p, per_page)
+    @tracks = block.tracks(p, per_page)
  
     render :update do |page|
-      page.insert_html :bottom, "track_list_#{block.id}", :partial => "blocks/#{block.track_partial}", :collection => tracks, :locals => {:block => block}
+      page.insert_html :bottom, "track_list_#{block.id}", :partial => "blocks/#{block.track_partial}", :collection => @tracks, :locals => {:block => block}
  
       if block.has_page?(p+1, per_page)
         page.replace_html "track_list_more_#{block.id}", :partial => 'blocks/track_list_more', :locals => {:block => block, :page => p+1, :force_same_page => params[:force_same_page], :per_page => per_page}
@@ -26,6 +27,7 @@ class CommunityTrackPluginPublicController &lt; PublicController
   def all_tracks
     @per_page = 8 #FIXME
     @block = Block.find(params[:id])
+    instance_eval(&@block.set_seed)
     @tracks = @block.tracks(1, @per_page)
     @show_more = @block.has_page?(2, @per_page)
   end
@@ -13,7 +13,7 @@ class CommunityTrackPlugin &lt; Noosfero::Plugin
   end
  
   def content_types
-    if context.respond_to?(:params) && context.params
+    if context.kind_of?(CmsController) && context.respond_to?(:params) && context.params
       types = []
       parent_id = context.params[:parent_id]
       types << CommunityTrackPlugin::Track if context.profile.community? && !parent_id
@@ -60,7 +60,7 @@ class CommunityTrackPlugin::Step &lt; Folder
     accept_comments
   end
  
-  def self.enabled_tools
+  def enabled_tools
     [TinyMceArticle, Forum]
   end
  
@@ -7,6 +7,11 @@ class CommunityTrackPlugin::Track &lt; Folder
  
   attr_accessible :goals, :expected_results
  
+  def comments_count
+    @comments_count = sum_children_comments self unless @comments_count
+    @comments_count
+  end
+
   def validate_categories
     errors.add(:categories, _('should not be blank.')) if categories.empty? && pending_categorizations.blank?
   end
@@ -49,8 +54,13 @@ class CommunityTrackPlugin::Track &lt; Folder
     false
   end
  
-  def comments_count
-    steps_unsorted.joins(:children).sum('children_articles.comments_count')
+  def sum_children_comments node
+    result = 0
+    node.children.each do |c|
+      result += c.comments_count
+      result += sum_children_comments c
+    end
+    result
   end
  
   def css_class_name
@@ -5,6 +5,9 @@ class CommunityTrackPlugin::TrackListBlock &lt; Block
   settings_items :limit, :type => :integer, :default => 3
   settings_items :more_another_page, :type => :boolean, :default => false
   settings_items :category_ids, :type => Array, :default => []
+  settings_items :order, :type => :string, :default => 'hits'
+
+  attr_accessible :more_another_page, :category_ids, :order
  
   def self.description
     _('Track List')
@@ -23,7 +26,16 @@ class CommunityTrackPlugin::TrackListBlock &lt; Block
   end
  
   def tracks(page=1, per_page=limit)
-    all_tracks.order('hits DESC').paginate(:per_page => per_page, :page => page)
+    tracks = all_tracks
+    tracks = case order
+             when 'newer'
+               tracks.order('created_at DESC')
+             when 'random'
+               tracks.order('random()')
+             else
+               tracks.order('hits DESC')
+             end
+    tracks.paginate(:per_page => per_page, :page => page)
   end
  
   def count_tracks
@@ -53,6 +65,7 @@ class CommunityTrackPlugin::TrackListBlock &lt; Block
   def content(args={})
     block = self
     proc do
+      instance_eval(&block.set_seed(true))
       render :file => 'blocks/track_list', :locals => {:block => block}
     end
   end
@@ -73,4 +86,26 @@ class CommunityTrackPlugin::TrackListBlock &lt; Block
     { :profile => [:article, :category], :environment => [:article, :category] }
   end
  
+  def timeout
+    1.hour
+  end
+
+  def set_seed(new_seed=false)
+    block = self
+    proc do
+      if block.order == 'random'
+        if new_seed || cookies[:_noosfero_community_tracks_rand_seed].blank?
+          cookies[:_noosfero_community_tracks_rand_seed] = {value: rand, expires: Time.now + 600}
+        end
+        #XXX postgresql specific
+        seed_val = Environment.connection.quote(cookies[:_noosfero_community_tracks_rand_seed])
+        Environment.connection.execute("select setseed(#{seed_val})")
+      end
+    end
+  end
+
+  def self.order_options
+    {_('Hits') => 'hits', _('Random') => 'random', _('Most Recent') => 'newer'}
+  end
+
 end
@@ -3,6 +3,9 @@ require_relative &#39;../test_helper&#39;
 class CmsControllerTest < ActionController::TestCase
  
   def setup
+    @environment = Environment.default
+    @environment.enabled_plugins = ['CommunityTrackPlugin']
+    @environment.save!
     @profile = fast_create(Community)
     @track = create_track('track', @profile)
     @step = CommunityTrackPlugin::Step.create!(:name => 'step1', :body => 'body', :profile => @profile, :parent => @track, :published => false, :end_date => Date.today, :start_date => Date.today)
@@ -36,4 +39,15 @@ class CmsControllerTest &lt; ActionController::TestCase
     assert_equal 'changed', @step.name
   end
  
+  should 'have parent_id present in form' do
+    get :new, :parent_id => @track.id, :profile => @profile.identifier, :type => CommunityTrackPlugin::Step
+    assert_tag :tag => 'input', :attributes => { :name => 'parent_id' }
+  end
+
+  should 'be able to create an step with a parent' do
+    amount_of_steps = CommunityTrackPlugin::Step.count
+    post :new, :parent_id => @track.id, :profile => @profile.identifier, :type => CommunityTrackPlugin::Step, :article => {:name => 'some', :body => 'some'}
+    assert_equal amount_of_steps + 1, CommunityTrackPlugin::Step.count
+  end
+
 end
@@ -97,4 +97,19 @@ class CommunityTrackPluginPublicControllerTest &lt; ActionController::TestCase
     assert_tag :tag => 'div', :attributes => {:id => 'errorExplanation'}
   end
  
+  should 'do not repeat tracks when paging a block with random order' do
+    @track.destroy
+    @block.order = 'random'
+    @block.save!
+
+    per_page = 4
+    (per_page*3).times {|i| create_track("track_#{i}", @community) }
+
+    tracks = 3.times.map do |i|
+      xhr :get, :view_tracks, :id => @block.id, :page => i+1, :per_page => per_page
+      assigns[:tracks].all
+    end.flatten
+    assert_equal tracks.count, tracks.uniq.count
+  end
+
 end
@@ -235,8 +235,8 @@ class StepTest &lt; ActiveSupport::TestCase
   end
  
   should 'return enabled tools for a step' do
-    assert_includes CommunityTrackPlugin::Step.enabled_tools, TinyMceArticle
-    assert_includes CommunityTrackPlugin::Step.enabled_tools, Forum
+    assert_includes @step.enabled_tools, TinyMceArticle
+    assert_includes @step.enabled_tools, Forum
   end
  
   should 'return class for selected tool' do
@@ -9,7 +9,7 @@ class TrackListBlockTest &lt; ActiveSupport::TestCase
     @block = create(CommunityTrackPlugin::TrackListBlock, :box => box)
   end
  
-  attr_reader :profile
+  attr_reader :profile, :track
  
   should 'describe yourself' do
     assert CommunityTrackPlugin::TrackListBlock.description
@@ -111,4 +111,20 @@ class TrackListBlockTest &lt; ActiveSupport::TestCase
     assert_equivalent [], @block.categories
   end
  
+  should 'list tracks in hits order by default' do
+    track2 = create_track("track2", profile)
+    track.update_attribute(:hits, 2)
+    track2.update_attribute(:hits, 1)
+    assert_equal [track, track2], @block.tracks
+  end
+
+  should 'list tracks in newer order' do
+    @block.order = 'newer'
+    @block.save!
+    track2 = create_track("track2", profile)
+    track2.update_attribute(:created_at, Date.today)
+    track.update_attribute(:created_at, Date.today - 1.day)
+    assert_equal [track2, track], @block.tracks
+  end
+
 end
@@ -3,10 +3,12 @@ require_relative &#39;../../test_helper&#39;
 class TrackTest < ActiveSupport::TestCase
  
   def setup
-    @profile = fast_create(Community)
+    @profile = create(Community)
     @track = create_track('track', @profile)
-    @step = CommunityTrackPlugin::Step.create!(:parent => @track, :start_date => Date.today, :end_date => Date.today, :name => 'step', :profile => @profile)
+    @step = CommunityTrackPlugin::Step.create!(:parent => @track, :start_date => DateTime.now, :end_date => DateTime.now, :name => 'step', :profile => @profile)
+    @track.children << @step
     @tool = fast_create(Article, :parent_id => @step.id, :profile_id => @profile.id)
+    @step.children << @tool
   end
  
   should 'describe yourself' do
@@ -25,8 +27,18 @@ class TrackTest &lt; ActiveSupport::TestCase
     assert_equal 0, @track.comments_count
     owner = create_user('testuser').person
     article = create(Article, :name => 'article', :parent_id => @step.id, :profile_id => owner.id)
+    @track.children << @step
+    @step.children << article
     comment = create(Comment, :source => article, :author_id => owner.id)
-    assert_equal 1, @track.comments_count
+    @step2 = CommunityTrackPlugin::Step.create!(:parent => @track, :start_date => DateTime.now, :end_date => DateTime.now, :name => 'step2', :profile => @profile)
+    @step2.tool_type = 'Forum'
+    forum = fast_create(Forum, :parent_id => @step2.id, :profile_id => owner.id)
+    article_forum = create(Article, :name => 'article_forum', :parent_id => forum.id, :profile_id => owner.id)
+    forum.children << article_forum
+    forum_comment = create(Comment, :source => article_forum, :author_id => owner.id)
+    @track.children = [@step, @step2]
+    @track = Article.find(@track.id)
+    assert_equal 2, @track.comments_count
   end
  
   should 'return children steps' do
@@ -35,6 +47,7 @@ class TrackTest &lt; ActiveSupport::TestCase
  
   should 'do not return other articles type at steps' do
     article = fast_create(Article, :parent_id => @track.id, :profile_id => @track.profile.id)
+    @track = Article.find(@track.id)
     assert_includes @track.children, article
     assert_equal [@step], @track.steps_unsorted
   end
@@ -7,6 +7,7 @@ class CommunityTrackPluginTest &lt; ActiveSupport::TestCase
     @profile = fast_create(Community)
     @params = {}
     @context = mock
+    @context.stubs(:kind_of?).returns(CmsController)
     @context.stubs(:profile).returns(@profile)
     @context.stubs(:params).returns(@params)
     @plugin.stubs(:context).returns(@context)
@@ -6,7 +6,7 @@
     </div>
   <% else %>
     <div class="more">
-      <%= link_to_remote(c_('More'), :url => {:id => block.id, :controller => 'community_track_plugin_public', :action => 'view_tracks', :page => page, :per_page => per_page, :force_same_page => force_same_page}, :loaded => visual_effect(:highlight, "track_card_list_#{block.id}")) %>
+      <%= link_to_remote(c_('More'), :url => {:id => block.id, :controller => 'community_track_plugin_public', :action => 'view_tracks', :page => page, :per_page => per_page, :force_same_page => force_same_page}, :method => :get) %>
     </div>
   <% end %>
 </div>
 <div id='edit-track-list-block'>
   <%= labelled_form_field c_('Limit of items'), text_field(:block, :limit, :size => 3) %>
+
+  <%= labelled_form_field c_('Order'),
+    select(:block, :order,
+           options_for_select(CommunityTrackPlugin::TrackListBlock.order_options, :selected => @block.order)) %>
+
   <%= labelled_form_field check_box(:block, :more_another_page) + _('Show more at another page'), '' %>
   <%= select_categories(:block, _('Select Categories')) %>
 <br/>
@@ -11,8 +11,10 @@
                      { :size => 14 })
   )) %>
  
-  <%= labelled_form_field(_('Tool type'), select(:article, :tool_type, CommunityTrackPlugin::Step.enabled_tools.map {|t| [t.short_description, t.name]} )) %>
+  <%= labelled_form_field(_('Tool type'), select(:article, :tool_type, @article.enabled_tools.map {|t| [t.short_description, t.name]} )) %>
   <%= hidden_field_tag('success_back_to', url_for(@article.parent.view_url)) %>
+  <%= hidden_field_tag('parent_id', @article.parent_id) %>
+
 </div>
  
 <%= labelled_form_field check_box(:article, :hidden) + _('Hidden Step'), '' %>
@@ -711,4 +711,17 @@ class ProfileDesignControllerTest &lt; ActionController::TestCase
     end
   end
  
+  should 'update selected categories in blocks' do
+    env = Environment.default
+    c1 = env.categories.build(:name => "Test category 1"); c1.save!
+
+    block = profile.blocks.last
+
+    Block.any_instance.expects(:accept_category?).at_least_once.returns true
+
+    xhr :get, :update_categories, :profile => profile.identifier, :id => block.id, :category_id => c1.id
+
+    assert_equal assigns(:current_category), c1
+  end
+
 end
...	...	@@ -67,4 +67,14 @@ class ProfileDesignController < BoxOrganizerController
67	67	blocks
68	68	end
69	69
	70	+ def update_categories
	71	+ @object = params[:id] ? @profile.blocks.find(params[:id]) : Block.new
	72	+ @categories = @toplevel_categories = environment.top_level_categories
	73	+ if params[:category_id]
	74	+ @current_category = Category.find(params[:category_id])
	75	+ @categories = @current_category.children
	76	+ end
	77	+ render :template => 'shared/update_categories', :locals => { :category => @current_category, :object_name => 'block' }
	78	+ end
	79	+
70	80	end
...	...
...	...	@@ -6,14 +6,15 @@ class CommunityTrackPluginPublicController < PublicController
6	6
7	7	def view_tracks
8	8	block = Block.find(params[:id])
	9	+ instance_eval(&block.set_seed)
9	10	p = params[:page].to_i
10	11	per_page = params[:per_page]
11	12	per_page \|\|= block.limit
12	13	per_page = per_page.to_i
13		- tracks = block.tracks(p, per_page)
	14	+ @tracks = block.tracks(p, per_page)
14	15
15	16	render :update do \|page\|
16		- page.insert_html :bottom, "track_list_#{block.id}", :partial => "blocks/#{block.track_partial}", :collection => tracks, :locals => {:block => block}
	17	+ page.insert_html :bottom, "track_list_#{block.id}", :partial => "blocks/#{block.track_partial}", :collection => @tracks, :locals => {:block => block}
17	18
18	19	if block.has_page?(p+1, per_page)
19	20	page.replace_html "track_list_more_#{block.id}", :partial => 'blocks/track_list_more', :locals => {:block => block, :page => p+1, :force_same_page => params[:force_same_page], :per_page => per_page}
...	...	@@ -26,6 +27,7 @@ class CommunityTrackPluginPublicController < PublicController
26	27	def all_tracks
27	28	@per_page = 8 #FIXME
28	29	@block = Block.find(params[:id])
	30	+ instance_eval(&@block.set_seed)
29	31	@tracks = @block.tracks(1, @per_page)
30	32	@show_more = @block.has_page?(2, @per_page)
31	33	end
...	...
...	...	@@ -13,7 +13,7 @@ class CommunityTrackPlugin < Noosfero::Plugin
13	13	end
14	14
15	15	def content_types
16		- if context.respond_to?(:params) && context.params
	16	+ if context.kind_of?(CmsController) && context.respond_to?(:params) && context.params
17	17	types = []
18	18	parent_id = context.params[:parent_id]
19	19	types << CommunityTrackPlugin::Track if context.profile.community? && !parent_id
...	...
...	...	@@ -60,7 +60,7 @@ class CommunityTrackPlugin::Step < Folder
60	60	accept_comments
61	61	end
62	62
63		- def self.enabled_tools
	63	+ def enabled_tools
64	64	[TinyMceArticle, Forum]
65	65	end
66	66
...	...
...	...	@@ -7,6 +7,11 @@ class CommunityTrackPlugin::Track < Folder
7	7
8	8	attr_accessible :goals, :expected_results
9	9
	10	+ def comments_count
	11	+ @comments_count = sum_children_comments self unless @comments_count
	12	+ @comments_count
	13	+ end
	14	+
10	15	def validate_categories
11	16	errors.add(:categories, _('should not be blank.')) if categories.empty? && pending_categorizations.blank?
12	17	end
...	...	@@ -49,8 +54,13 @@ class CommunityTrackPlugin::Track < Folder
49	54	false
50	55	end
51	56
52		- def comments_count
53		- steps_unsorted.joins(:children).sum('children_articles.comments_count')
	57	+ def sum_children_comments node
	58	+ result = 0
	59	+ node.children.each do \|c\|
	60	+ result += c.comments_count
	61	+ result += sum_children_comments c
	62	+ end
	63	+ result
54	64	end
55	65
56	66	def css_class_name
...	...
...	...	@@ -5,6 +5,9 @@ class CommunityTrackPlugin::TrackListBlock < Block
5	5	settings_items :limit, :type => :integer, :default => 3
6	6	settings_items :more_another_page, :type => :boolean, :default => false
7	7	settings_items :category_ids, :type => Array, :default => []
	8	+ settings_items :order, :type => :string, :default => 'hits'
	9	+
	10	+ attr_accessible :more_another_page, :category_ids, :order
8	11
9	12	def self.description
10	13	_('Track List')
...	...	@@ -23,7 +26,16 @@ class CommunityTrackPlugin::TrackListBlock < Block
23	26	end
24	27
25	28	def tracks(page=1, per_page=limit)
26		- all_tracks.order('hits DESC').paginate(:per_page => per_page, :page => page)
	29	+ tracks = all_tracks
	30	+ tracks = case order
	31	+ when 'newer'
	32	+ tracks.order('created_at DESC')
	33	+ when 'random'
	34	+ tracks.order('random()')
	35	+ else
	36	+ tracks.order('hits DESC')
	37	+ end
	38	+ tracks.paginate(:per_page => per_page, :page => page)
27	39	end
28	40
29	41	def count_tracks
...	...	@@ -53,6 +65,7 @@ class CommunityTrackPlugin::TrackListBlock < Block
53	65	def content(args={})
54	66	block = self
55	67	proc do
	68	+ instance_eval(&block.set_seed(true))
56	69	render :file => 'blocks/track_list', :locals => {:block => block}
57	70	end
58	71	end
...	...	@@ -73,4 +86,26 @@ class CommunityTrackPlugin::TrackListBlock < Block
73	86	{ :profile => [:article, :category], :environment => [:article, :category] }
74	87	end
75	88
	89	+ def timeout
	90	+ 1.hour
	91	+ end
	92	+
	93	+ def set_seed(new_seed=false)
	94	+ block = self
	95	+ proc do
	96	+ if block.order == 'random'
	97	+ if new_seed \|\| cookies[:_noosfero_community_tracks_rand_seed].blank?
	98	+ cookies[:_noosfero_community_tracks_rand_seed] = {value: rand, expires: Time.now + 600}
	99	+ end
	100	+ #XXX postgresql specific
	101	+ seed_val = Environment.connection.quote(cookies[:_noosfero_community_tracks_rand_seed])
	102	+ Environment.connection.execute("select setseed(#{seed_val})")
	103	+ end
	104	+ end
	105	+ end
	106	+
	107	+ def self.order_options
	108	+ {_('Hits') => 'hits', _('Random') => 'random', _('Most Recent') => 'newer'}
	109	+ end
	110	+
76	111	end
...	...
...	...	@@ -3,6 +3,9 @@ require_relative '../test_helper'
3	3	class CmsControllerTest < ActionController::TestCase
4	4
5	5	def setup
	6	+ @environment = Environment.default
	7	+ @environment.enabled_plugins = ['CommunityTrackPlugin']
	8	+ @environment.save!
6	9	@profile = fast_create(Community)
7	10	@track = create_track('track', @profile)
8	11	@step = CommunityTrackPlugin::Step.create!(:name => 'step1', :body => 'body', :profile => @profile, :parent => @track, :published => false, :end_date => Date.today, :start_date => Date.today)
...	...	@@ -36,4 +39,15 @@ class CmsControllerTest < ActionController::TestCase
36	39	assert_equal 'changed', @step.name
37	40	end
38	41
	42	+ should 'have parent_id present in form' do
	43	+ get :new, :parent_id => @track.id, :profile => @profile.identifier, :type => CommunityTrackPlugin::Step
	44	+ assert_tag :tag => 'input', :attributes => { :name => 'parent_id' }
	45	+ end
	46	+
	47	+ should 'be able to create an step with a parent' do
	48	+ amount_of_steps = CommunityTrackPlugin::Step.count
	49	+ post :new, :parent_id => @track.id, :profile => @profile.identifier, :type => CommunityTrackPlugin::Step, :article => {:name => 'some', :body => 'some'}
	50	+ assert_equal amount_of_steps + 1, CommunityTrackPlugin::Step.count
	51	+ end
	52	+
39	53	end
...	...
...	...	@@ -97,4 +97,19 @@ class CommunityTrackPluginPublicControllerTest < ActionController::TestCase
97	97	assert_tag :tag => 'div', :attributes => {:id => 'errorExplanation'}
98	98	end
99	99
	100	+ should 'do not repeat tracks when paging a block with random order' do
	101	+ @track.destroy
	102	+ @block.order = 'random'
	103	+ @block.save!
	104	+
	105	+ per_page = 4
	106	+ (per_page*3).times {\|i\| create_track("track_#{i}", @community) }
	107	+
	108	+ tracks = 3.times.map do \|i\|
	109	+ xhr :get, :view_tracks, :id => @block.id, :page => i+1, :per_page => per_page
	110	+ assigns[:tracks].all
	111	+ end.flatten
	112	+ assert_equal tracks.count, tracks.uniq.count
	113	+ end
	114	+
100	115	end
...	...
...	...	@@ -235,8 +235,8 @@ class StepTest < ActiveSupport::TestCase
235	235	end
236	236
237	237	should 'return enabled tools for a step' do
238		- assert_includes CommunityTrackPlugin::Step.enabled_tools, TinyMceArticle
239		- assert_includes CommunityTrackPlugin::Step.enabled_tools, Forum
	238	+ assert_includes @step.enabled_tools, TinyMceArticle
	239	+ assert_includes @step.enabled_tools, Forum
240	240	end
241	241
242	242	should 'return class for selected tool' do
...	...
...	...	@@ -9,7 +9,7 @@ class TrackListBlockTest < ActiveSupport::TestCase
9	9	@block = create(CommunityTrackPlugin::TrackListBlock, :box => box)
10	10	end
11	11
12		- attr_reader :profile
	12	+ attr_reader :profile, :track
13	13
14	14	should 'describe yourself' do
15	15	assert CommunityTrackPlugin::TrackListBlock.description
...	...	@@ -111,4 +111,20 @@ class TrackListBlockTest < ActiveSupport::TestCase
111	111	assert_equivalent [], @block.categories
112	112	end
113	113
	114	+ should 'list tracks in hits order by default' do
	115	+ track2 = create_track("track2", profile)
	116	+ track.update_attribute(:hits, 2)
	117	+ track2.update_attribute(:hits, 1)
	118	+ assert_equal [track, track2], @block.tracks
	119	+ end
	120	+
	121	+ should 'list tracks in newer order' do
	122	+ @block.order = 'newer'
	123	+ @block.save!
	124	+ track2 = create_track("track2", profile)
	125	+ track2.update_attribute(:created_at, Date.today)
	126	+ track.update_attribute(:created_at, Date.today - 1.day)
	127	+ assert_equal [track2, track], @block.tracks
	128	+ end
	129	+
114	130	end
...	...