diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-09-03 09:30:48 +0100 |
---|---|---|
committer | Gareth Rees <gareth@mysociety.org> | 2014-09-12 17:17:17 +0100 |
commit | 94364952fb08abe82a0f47a1389bbe6586fc634c (patch) | |
tree | b8047584c4c5551a8b23a12556f882fd02f1941f | |
parent | cee0aec2cdb3f07c6d8eeaed9ee3c7111935ac68 (diff) |
Fix regressions in CensorRule specs
Added extra validations to match Postgres constraints
-rw-r--r-- | spec/models/censor_rule_spec.rb | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/spec/models/censor_rule_spec.rb b/spec/models/censor_rule_spec.rb index 089f0648c..4ecd2d3e1 100644 --- a/spec/models/censor_rule_spec.rb +++ b/spec/models/censor_rule_spec.rb @@ -112,7 +112,10 @@ describe 'when validating rules' do before do @censor_rule = CensorRule.new(:regexp => true, - :text => '*') + :text => '*', + :replacement => '---', + :last_edit_comment => 'test', + :last_edit_editor => 'rspec') end it 'should try to create a regexp from the text' do @@ -145,7 +148,10 @@ describe 'when validating rules' do describe 'when the allow_global flag has been set' do before do - @censor_rule = CensorRule.new(:text => 'some text') + @censor_rule = CensorRule.new(:text => 'some text', + :replacement => '---', + :last_edit_comment => 'test', + :last_edit_editor => 'rspec') @censor_rule.allow_global = true end @@ -158,7 +164,10 @@ describe 'when validating rules' do describe 'when the allow_global flag has not been set' do before do - @censor_rule = CensorRule.new(:text => '/./') + @censor_rule = CensorRule.new(:text => '/./', + :replacement => '---', + :last_edit_comment => 'test', + :last_edit_editor => 'rspec') end it 'should not allow a global text censor rule (without user_id, request_id or public_body_id)' do |