diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-11-19 11:01:30 +0000 |
---|---|---|
committer | Gareth Rees <gareth@mysociety.org> | 2014-11-19 11:27:23 +0000 |
commit | 40579725f738a223155710fd440750c42b47f21b (patch) | |
tree | 22e27a38ace0a5f68797a54c0c8ac0089525d2b4 /spec/controllers | |
parent | 336142c324ecc993dcacc90b7304cd9d89a3bac7 (diff) |
Use instance variable for CensorRule in #destroy
Easier to check against the assigned variable, but the spec still fails
because an internal error is raised if the rule is not associated with
a User or InfoRequest
Diffstat (limited to 'spec/controllers')
-rw-r--r-- | spec/controllers/admin_censor_rule_controller_spec.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/spec/controllers/admin_censor_rule_controller_spec.rb b/spec/controllers/admin_censor_rule_controller_spec.rb index deaef1d7f..4df56a92b 100644 --- a/spec/controllers/admin_censor_rule_controller_spec.rb +++ b/spec/controllers/admin_censor_rule_controller_spec.rb @@ -412,10 +412,9 @@ describe AdminCensorRuleController do end it 'finds the correct censor rule to destroy' do - pending("Assign the CensorRule to an instance variable") + pending("This raises an internal error in most cases") # TODO: Replace :censor_rule_id with :id delete :destroy, :censor_rule_id => @censor_rule.id - # TODO: Assign the CensorRule to an instance variable expect(assigns[:censor_rule]).to eq(@censor_rule) end |