diff options
Diffstat (limited to 'spec/graphql/mutations/container_expiration_policies/update_spec.rb')
-rw-r--r-- | spec/graphql/mutations/container_expiration_policies/update_spec.rb | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/spec/graphql/mutations/container_expiration_policies/update_spec.rb b/spec/graphql/mutations/container_expiration_policies/update_spec.rb index 6aedaab3b53..9c6016e0af4 100644 --- a/spec/graphql/mutations/container_expiration_policies/update_spec.rb +++ b/spec/graphql/mutations/container_expiration_policies/update_spec.rb @@ -35,7 +35,7 @@ RSpec.describe Mutations::ContainerExpirationPolicies::Update do it_behaves_like 'not creating the container expiration policy' - it "doesn't update the cadence" do + it 'doesn\'t update the cadence' do expect { subject } .not_to change { container_expiration_policy.reload.cadence } end @@ -47,6 +47,24 @@ RSpec.describe Mutations::ContainerExpirationPolicies::Update do ) end end + + context 'with blank regex' do + let_it_be(:params) { { project_path: project.full_path, name_regex: '', enabled: true } } + + it_behaves_like 'not creating the container expiration policy' + + it "doesn't update the cadence" do + expect { subject } + .not_to change { container_expiration_policy.reload.cadence } + end + + it 'returns an error' do + expect(subject).to eq( + container_expiration_policy: nil, + errors: ['Name regex can\'t be blank'] + ) + end + end end RSpec.shared_examples 'denying access to container expiration policy' do |