[Mezuro] Completed reading_group model.

João M. M. da Silva + Alessandro Palmeira · João M. M. da Silva
1 parent 59026c1e
Showing 5 changed files with 57 additions and 10 deletions Show diff stats
plugins/mezuro/lib/kalibro/configuration.rb
plugins/mezuro/lib/kalibro/model.rb
plugins/mezuro/lib/kalibro/reading_group.rb
plugins/mezuro/test/unit/kalibro/base_tool_test.rb
plugins/mezuro/test/unit/kalibro/reading_group_test.rb
@@ -16,9 +16,11 @@ class Kalibro::Configuration &lt; Kalibro::Model
   end
 =end
-  def self.exists?(id)
-    request("Configuration", :configuration_exists, {:configuration_id => id})[:exists]
-  end
+#  Should be on parent class
+#
+#  def self.exists?(id)
+#    request("Configuration", :configuration_exists, {:configuration_id => id})[:exists]
+#  end
   def self.find(id)
     if(exists?(id))
@@ -66,6 +66,10 @@ class Kalibro::Model
     end
   end
+  def self.exists?(id)
+    request(exists_endpoint, exists_action, exists_params(id))
+  end
+  
   protected
   def fields
@@ -124,11 +128,27 @@ class Kalibro::Model
   end
   def destroy_action
-    "remove_#{class_name.underscore}".to_sym
+    "delete_#{class_name.underscore}".to_sym
   end
   def destroy_params
-    {"#{class_name.underscore}_name".to_sym => self.name}
+    {"#{class_name.underscore}_id".to_sym => self.id}
+  end
+
+  def self.exists_class_name
+    self.name.gsub(/Kalibro::/,"")
+  end
+  
+  def self.exists_endpoint
+    self.exists_class_name
+  end
+  
+  def self.exists_action
+    "#{exists_class_name.underscore}_exists".to_sym
+  end
+  
+  def self.exists_params(id)
+    {"#{exists_class_name.underscore}_id".to_sym => id}
   end
   def add_error(exception)
@@ -0,0 +1,27 @@
+class Kalibro::ReadingGroup < Kalibro::Model
+
+  attr_accessor :id, :name, :description
+
+  def self.find(id)
+    new request("ReadingGroup", :get_reading_group, {:group_id => id})[:reading_group]
+  end
+
+  def self.all
+    request("ReadingGroup", :all_reading_groups)[:reading_group].to_a.map { |reading_group| new reading_group }
+  end
+  
+  def self.reading_group_of( metric_configuration_id )
+    new request("ReadingGroup", :reading_group_of, {:metric_configuration_id => metric_configuration_id} )[:reading_group]
+  end
+
+  private
+
+  def self.exists_params(id)
+    {:group_id => id}
+  end
+  
+  def destroy_params
+    {:group_id => self.id}
+  end
+
+end
 require "test_helper"
-
 require "#{RAILS_ROOT}/plugins/mezuro/test/fixtures/base_tool_fixtures"
 class BaseToolTest < ActiveSupport::TestCase
@@ -10,7 +9,7 @@ class BaseToolTest &lt; ActiveSupport::TestCase
   end
   should 'create base tool from hash' do
-    assert_equal @base_tool.name, Kalibro::BaseTool.new(@hash).name
+    assert_equal @hash[:name], Kalibro::BaseTool.new(@hash).name
   end
 #  Mezuro will not send a base_tool hash back to Kalibro
@@ -28,13 +28,13 @@ class ReadingGroupTest &lt; ActiveSupport::TestCase
   end
   should 'get all reading groups' do
-    Kalibro::ReadingGroup.expects(:request).with("ReadingGroup", :all_reading_group).returns({:reading_group => [@hash]})
+    Kalibro::ReadingGroup.expects(:request).with("ReadingGroup", :all_reading_groups).returns({:reading_group => [@hash]})
     assert_equal @hash[:name], Kalibro::ReadingGroup.all[0].name
   end
   should 'get reading group of a metric configuration' do
     id = 31
-    Kalibro::ReadingGroup.expects(:request).with("ReadingGroup", :reading_group_of, {:metricConfigurationId => id}).returns({:reading_group => @hash})
+    Kalibro::ReadingGroup.expects(:request).with("ReadingGroup", :reading_group_of, {:metric_configuration_id => id}).returns({:reading_group => @hash})
     assert_equal @hash[:name], Kalibro::ReadingGroup.reading_group_of(id).name
   end
@@ -48,7 +48,6 @@ class ReadingGroupTest &lt; ActiveSupport::TestCase
     assert !(@reading_group.save)
   end
-
   should 'destroy reading group by id' do
     Kalibro::ReadingGroup.expects(:request).with("ReadingGroup", :delete_reading_group, {:group_id => @reading_group.id})
     @reading_group.destroy
	@@ -66,6 +66,10 @@ class Kalibro::Model		@@ -66,6 +66,10 @@ class Kalibro::Model
66	end	66	end
67	end	67	end
68		68
		69	+ def self.exists?(id)
		70	+ request(exists_endpoint, exists_action, exists_params(id))
		71	+ end
		72	+
69	protected	73	protected
70		74
71	def fields	75	def fields
	@@ -124,11 +128,27 @@ class Kalibro::Model		@@ -124,11 +128,27 @@ class Kalibro::Model
124	end	128	end
125		129
126	def destroy_action	130	def destroy_action
127	- "remove_#{class_name.underscore}".to_sym	131	+ "delete_#{class_name.underscore}".to_sym
128	end	132	end
129		133
130	def destroy_params	134	def destroy_params
131	- {"#{class_name.underscore}_name".to_sym => self.name}	135	+ {"#{class_name.underscore}_id".to_sym => self.id}
		136	+ end
		137	+
		138	+ def self.exists_class_name
		139	+ self.name.gsub(/Kalibro::/,"")
		140	+ end
		141	+
		142	+ def self.exists_endpoint
		143	+ self.exists_class_name
		144	+ end
		145	+
		146	+ def self.exists_action
		147	+ "#{exists_class_name.underscore}_exists".to_sym
		148	+ end
		149	+
		150	+ def self.exists_params(id)
		151	+ {"#{exists_class_name.underscore}_id".to_sym => id}
132	end	152	end
133		153
134	def add_error(exception)	154	def add_error(exception)
@@ -0,0 +1,27 @@		@@ -0,0 +1,27 @@
	1	+class Kalibro::ReadingGroup < Kalibro::Model
	2	+
	3	+ attr_accessor :id, :name, :description
	4	+
	5	+ def self.find(id)
	6	+ new request("ReadingGroup", :get_reading_group, {:group_id => id})[:reading_group]
	7	+ end
	8	+
	9	+ def self.all
	10	+ request("ReadingGroup", :all_reading_groups)[:reading_group].to_a.map { \|reading_group\| new reading_group }
	11	+ end
	12	+
	13	+ def self.reading_group_of( metric_configuration_id )
	14	+ new request("ReadingGroup", :reading_group_of, {:metric_configuration_id => metric_configuration_id} )[:reading_group]
	15	+ end
	16	+
	17	+ private
	18	+
	19	+ def self.exists_params(id)
	20	+ {:group_id => id}
	21	+ end
	22	+
	23	+ def destroy_params
	24	+ {:group_id => self.id}
	25	+ end
	26	+
	27	+end