diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-09-02 18:04:12 +0100 |
---|---|---|
committer | Gareth Rees <gareth@mysociety.org> | 2014-09-12 17:17:17 +0100 |
commit | cee0aec2cdb3f07c6d8eeaed9ee3c7111935ac68 (patch) | |
tree | 15b934df6cbc6352541fb16321e36d437751d066 /spec/models/censor_rule_spec.rb | |
parent | fc1a27bfb306b189a4b4d936131d8cbc4d1f351e (diff) |
Add correct validations to CensorRule
Diffstat (limited to 'spec/models/censor_rule_spec.rb')
-rw-r--r-- | spec/models/censor_rule_spec.rb | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/spec/models/censor_rule_spec.rb b/spec/models/censor_rule_spec.rb index 5b41cc0d4..089f0648c 100644 --- a/spec/models/censor_rule_spec.rb +++ b/spec/models/censor_rule_spec.rb @@ -90,10 +90,22 @@ end describe 'when validating rules' do - it 'should be invalid without text' do + it 'must have the text to redact' do censor_rule = CensorRule.new - censor_rule.valid?.should == false - censor_rule.errors[:text].should == ["can't be blank"] + expect(censor_rule).to have(1).error_on(:text) + expect(censor_rule.errors[:text]).to eql(["can't be blank"]) + end + + it 'must have a replacement' do + expect(CensorRule.new).to have(1).error_on(:replacement) + end + + it 'must have a last_edit_editor' do + expect(CensorRule.new).to have(1).error_on(:last_edit_editor) + end + + it 'must have a last_edit_comment' do + expect(CensorRule.new).to have(1).error_on(:last_edit_comment) end describe 'when validating a regexp rule' do |