Test against default to '0', it should not set

1 files changed, 22 insertions, 8 deletions

diff --git a/spec/requests/ci/api/builds_spec.rb b/spec/requests/ci/api/builds_spec.rb
index c7284be09b7..9948d1a9ea0 100644
--- a/spec/requests/ci/api/builds_spec.rb
+++ b/spec/requests/ci/api/builds_spec.rb
@@ -668,14 +668,28 @@ describe Ci::API::Builds do
               end
 
               context 'with application default' do
-                let(:default_artifacts_expire_in) { '5 days' }
-
-                it 'sets to application default' do
-                  build.reload
-                  expect(response).to have_http_status(201)
-                  expect(json_response['artifacts_expire_at']).not_to be_empty
-                  expect(build.artifacts_expire_at).
-                    to be_within(5.minutes).of(5.days.from_now)
+                context 'default to 5 days' do
+                  let(:default_artifacts_expire_in) { '5 days' }
+
+                  it 'sets to application default' do
+                    build.reload
+                    expect(response).to have_http_status(201)
+                    expect(json_response['artifacts_expire_at'])
+                      .not_to be_empty
+                    expect(build.artifacts_expire_at)
+                      .to be_within(5.minutes).of(5.days.from_now)
+                  end
+                end
+
+                context 'default to 0' do
+                  let(:default_artifacts_expire_in) { '0' }
+
+                  it 'does not set expire_in' do
+                    build.reload
+                    expect(response).to have_http_status(201)
+                    expect(json_response['artifacts_expire_at']).to be_nil
+                    expect(build.artifacts_expire_at).to be_nil
+                  end
                 end
               end
             end