Refactor validation of abstract max length

Victor Costa
1 parent 3d090c90
Showing 2 changed files with 16 additions and 10 deletions Show diff stats
lib/ext/proposal.rb
test/unit/proposal_test.rb
 require_relative '../../../proposals_discussion/lib/proposals_discussion_plugin/proposal.rb'
-class ProposalsDiscussionPlugin::Proposal < TinyMceArticle
-  validate :abstract_max_length
+class ProposalsDiscussionPlugin::Proposal
+
+  validates :abstract, length: {maximum: 200}, if: proc { |a| a.environment.plugin_enabled?(DialogaPlugin)}
-  def abstract_max_length
-    if abstract.size > 200 and environment.plugin_enabled? "Dialoga"
-      errors.add(:proposal, "Proposal must be no longer than 200 characters")
-    end
-  end
 end
@@ -8,17 +8,27 @@ class ProposalTest &lt; ActiveSupport::TestCase
     @discussion = ProposalsDiscussionPlugin::Discussion.create!(:name => 'discussion', :profile => person, :allow_topics => false)
     @proposal = ProposalsDiscussionPlugin::Proposal.new(:name => 'test', :abstract => 'abstract', :profile => @profile, :parent => @discussion)
     @proposal.created_by = @person
+    @environment = Environment.default
+    environment.enable_plugin(DialogaPlugin)
   end
-  attr_reader :profile, :proposal, :person, :discussion
+  attr_reader :profile, :proposal, :person, :discussion, :environment
-  should 'should not save a proposal with an abstract > 200 chars' do
+  should 'should not save a proposal with an abstract greater than 200 chars' do
     proposal.abstract = 201.times.map{'B'}.join
     refute proposal.valid?
     refute proposal.save
+    assert proposal.errors[:abstract].present?
   end
-  should 'should save a proposal with  abstract <= 200 chars' do
+  should 'should save a proposal with an abstract greater than 200 chars and dialoga plugin is not enabled' do
+    environment.disable_plugin(DialogaPlugin)
+    proposal.abstract = 201.times.map{'B'}.join
+    assert proposal.valid?
+    assert proposal.save
+  end
+
+  should 'should save a proposal with an abstract not greater than 200 chars' do
     proposal.abstract = 200.times.map{'B'}.join
     assert proposal.valid?
     assert proposal.save