diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-03-06 04:37:09 +0900 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2018-03-06 16:53:24 +0900 |
commit | 7421604bd0855f7855f5562769acc9bc871fb631 (patch) | |
tree | 44ff0752fd74224f57fa625546fe90eb09141403 /spec/services | |
parent | 031794f57dd1b300e8442d57aa2822b70021ff3a (diff) | |
download | gitlab-ce-7421604bd0855f7855f5562769acc9bc871fb631.tar.gz |
Introduce FailedToExtractError. Fix spec. Add DNS test mock.
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/projects/update_pages_service_spec.rb | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/spec/services/projects/update_pages_service_spec.rb b/spec/services/projects/update_pages_service_spec.rb index bfb86284d86..505aff1acf4 100644 --- a/spec/services/projects/update_pages_service_spec.rb +++ b/spec/services/projects/update_pages_service_spec.rb @@ -34,6 +34,7 @@ describe Projects::UpdatePagesService do context 'with expiry date' do before do build.artifacts_expire_in = "2 days" + build.save! end it "doesn't delete artifacts" do @@ -105,6 +106,7 @@ describe Projects::UpdatePagesService do context 'with expiry date' do before do build.artifacts_expire_in = "2 days" + build.save! end it "doesn't delete artifacts" do @@ -159,6 +161,17 @@ describe Projects::UpdatePagesService do expect(execute).not_to eq(:success) end + + context 'when timeout happens by DNS error' do + before do + allow_any_instance_of(Projects::UpdatePagesService) + .to receive(:extract_zip_archive!).and_raise(SocketError) + end + + it 'raises an error' do + expect { execute }.to raise_error(SocketError) + end + end end end |