Download as
Browse Code »

Commit e2299a22870cbb8a5510e806893653e882e2523e

Authored by Heitor
1 parent e5467768
Exists in colab and in 4 other branches master, spb-master, spb-stable, stable

Unit test for deleting a kalibro range with a compound metric configuration

Showing 1 changed file with 18 additions and 1 deletions   Show diff stats
spec/controllers/kalibro_ranges_controller_spec.rb
  Diff comments   View file @e2299a2
... ... @@ -100,13 +100,30 @@ describe KalibroRangesController, :type => :controller do
100 100 subject.expects(:find_resource).with(KalibroRange, kalibro_range.id).returns(kalibro_range)
101 101 MetricConfiguration.expects(:find).with(kalibro_range.metric_configuration_id).returns(metric_configuration)
102 102  
103   - delete :destroy, id: kalibro_range.id.to_s, metric_configuration_id: metric_configuration.id.to_s, kalibro_configuration_id: metric_configuration.kalibro_configuration_id.to_s
  103 + delete :destroy, id: kalibro_range.id, metric_configuration_id: metric_configuration.id, kalibro_configuration_id: metric_configuration.kalibro_configuration_id
104 104 end
105 105  
106 106 it { is_expected.to redirect_to(kalibro_configuration_metric_configuration_path(metric_configuration.kalibro_configuration_id, metric_configuration.id)) }
107 107 it { is_expected.to respond_with(:redirect) }
108 108 end
109 109  
  110 + context 'when the user owns the compound metric configuration' do
  111 + let(:compound_metric_configuration) { FactoryGirl.build(:compound_metric_configuration_with_id) }
  112 + let(:new_kalibro_range) { FactoryGirl.build(:kalibro_range_with_id, metric_configuration_id: compound_metric_configuration.id) }
  113 +
  114 + before :each do
  115 + subject.expects(:metric_configuration_owner?).returns true
  116 + new_kalibro_range.expects(:destroy)
  117 + subject.expects(:find_resource).with(KalibroRange, new_kalibro_range.id).returns(new_kalibro_range)
  118 + MetricConfiguration.expects(:find).with(new_kalibro_range.metric_configuration_id).returns(compound_metric_configuration)
  119 +
  120 + delete :destroy, id: new_kalibro_range.id, metric_configuration_id: compound_metric_configuration.id, kalibro_configuration_id: compound_metric_configuration.kalibro_configuration_id
  121 + end
  122 +
  123 + it { is_expected.to redirect_to(kalibro_configuration_compound_metric_configuration_path(compound_metric_configuration.kalibro_configuration_id, compound_metric_configuration.id)) }
  124 + it { is_expected.to respond_with(:redirect) }
  125 + end
  126 +
110 127 context "when the user doesn't own the metric configuration" do
111 128 before :each do
112 129 delete :destroy, id: kalibro_range.id.to_s, metric_configuration_id: metric_configuration.id.to_s, kalibro_configuration_id: metric_configuration.kalibro_configuration_id.to_s
... ...