Fix tests

Victor Costa
1 parent 7b7d1cf0
Showing 3 changed files with 20 additions and 15 deletions Show diff stats
test/unit/api_test.rb
test/unit/proposal_task_test.rb
test/unit/proposal_test.rb
@@ -5,6 +5,8 @@ class APITest &lt;  ActiveSupport::TestCase
   def setup
     login_api
+    environment = Environment.default
+    environment.enable_plugin(ProposalsDiscussionPlugin)
   end
   should 'return proposal ranking' do
@@ -0,0 +1,18 @@
+require_relative '../test_helper'
+
+class ProposalTaskTest < ActiveSupport::TestCase
+
+  attr_reader :profile, :proposal, :person, :discussion
+
+  def setup
+    @person = fast_create(Person)
+    @discussion = ProposalsDiscussionPlugin::Discussion.create!(:name => 'discussion', :profile => person, :allow_topics => false)
+  end
+
+  should 'check the source of a proposal in a task' do
+    topic = fast_create(ProposalsDiscussionPlugin::Topic, :parent_id => discussion.id)
+    task = ProposalsDiscussionPlugin::ProposalTask.new(:article_parent_id => topic.id)
+    assert_equal topic.name, task.proposal_source
+  end
+
+end
@@ -124,19 +124,4 @@ class ProposalTest &lt; ActiveSupport::TestCase
     assert_equal [location], proposal.locations
   end
-  should 'check the source of a proposal in a task' do
-
-    task_data = {
-      article: {name: "test proposal", abstract: "teste adadd"},
-      requestor: person,
-      target:  profile,
-      spam: false
-    }
-
-    task = ProposalsDiscussionPlugin::ProposalTask.new task_data
-
-    assert_equal task_data[:article][:name], task.proposal_source
-
-  end
-
 end
	@@ -5,6 +5,8 @@ class APITest < ActiveSupport::TestCase		@@ -5,6 +5,8 @@ class APITest < ActiveSupport::TestCase
5		5
6	def setup	6	def setup
7	login_api	7	login_api
		8	+ environment = Environment.default
		9	+ environment.enable_plugin(ProposalsDiscussionPlugin)
8	end	10	end
9		11
10	should 'return proposal ranking' do	12	should 'return proposal ranking' do
@@ -0,0 +1,18 @@		@@ -0,0 +1,18 @@
	1	+require_relative '../test_helper'
	2	+
	3	+class ProposalTaskTest < ActiveSupport::TestCase
	4	+
	5	+ attr_reader :profile, :proposal, :person, :discussion
	6	+
	7	+ def setup
	8	+ @person = fast_create(Person)
	9	+ @discussion = ProposalsDiscussionPlugin::Discussion.create!(:name => 'discussion', :profile => person, :allow_topics => false)
	10	+ end
	11	+
	12	+ should 'check the source of a proposal in a task' do
	13	+ topic = fast_create(ProposalsDiscussionPlugin::Topic, :parent_id => discussion.id)
	14	+ task = ProposalsDiscussionPlugin::ProposalTask.new(:article_parent_id => topic.id)
	15	+ assert_equal topic.name, task.proposal_source
	16	+ end
	17	+
	18	+end
	@@ -124,19 +124,4 @@ class ProposalTest < ActiveSupport::TestCase		@@ -124,19 +124,4 @@ class ProposalTest < ActiveSupport::TestCase
124	assert_equal [location], proposal.locations	124	assert_equal [location], proposal.locations
125	end	125	end
126		126
127	- should 'check the source of a proposal in a task' do
128	-
129	- task_data = {
130	- article: {name: "test proposal", abstract: "teste adadd"},
131	- requestor: person,
132	- target: profile,
133	- spam: false
134	- }
135	-
136	- task = ProposalsDiscussionPlugin::ProposalTask.new task_data
137	-
138	- assert_equal task_data[:article][:name], task.proposal_source
139	-
140	- end
141	-
142	end	127	end