Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/spec/lib/ci
diff options
context:
space:
mode:
authorStan Hu <stanhu@gmail.com>2016-05-23 09:35:18 +0300
committerStan Hu <stanhu@gmail.com>2016-05-23 09:52:50 +0300
commit9bb0d0b4079bc88134399f175ce9631005044060 (patch)
tree9c1af50c90db6756ecbd9b4a2ee7d57edf5f49e5 /spec/lib/ci
parent53ad33e4ba1b079e4f2aebceb42a7301c5ce2e4a (diff)
Fix Error 500 in CI charts by gracefully handling commits with no durations
Closes #17730
Diffstat (limited to 'spec/lib/ci')
-rw-r--r--spec/lib/ci/charts_spec.rb7
1 files changed, 7 insertions, 0 deletions
diff --git a/spec/lib/ci/charts_spec.rb b/spec/lib/ci/charts_spec.rb
index 50a77308cde..9d1215a5760 100644
--- a/spec/lib/ci/charts_spec.rb
+++ b/spec/lib/ci/charts_spec.rb
@@ -12,5 +12,12 @@ describe Ci::Charts, lib: true do
chart = Ci::Charts::BuildTime.new(@commit.project)
expect(chart.build_times).to eq([2])
end
+
+ it 'should handle nil build times' do
+ create(:ci_commit, duration: nil, project: @commit.project)
+
+ chart = Ci::Charts::BuildTime.new(@commit.project)
+ expect(chart.build_times).to eq([2, 0])
+ end
end
end