diff options
author | Sean McGivern <sean@gitlab.com> | 2019-08-23 13:05:38 +0100 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2019-08-28 16:39:33 +0100 |
commit | 9bfb012a56918e3412ce767c19f5f9ca2329a78c (patch) | |
tree | 1939721d21ad320347d614fac8ff1c6a5ba9f24b /spec/lib/gitlab | |
parent | 23c1cfcc110b52c1060f35a4a4a7816fb38adc6c (diff) | |
download | gitlab-ce-9bfb012a56918e3412ce767c19f5f9ca2329a78c.tar.gz |
Add top-level warnings key to performance bar responseperformance-bar-warnings
This key is useful to reduce the amount of logic needed on the frontend:
if `has_warnings` is true, then the frontend knows that the request in
question has warnings for some metric.
Diffstat (limited to 'spec/lib/gitlab')
-rw-r--r-- | spec/lib/gitlab/performance_bar/with_top_level_warnings_spec.rb | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/spec/lib/gitlab/performance_bar/with_top_level_warnings_spec.rb b/spec/lib/gitlab/performance_bar/with_top_level_warnings_spec.rb new file mode 100644 index 00000000000..3b92261f0fe --- /dev/null +++ b/spec/lib/gitlab/performance_bar/with_top_level_warnings_spec.rb @@ -0,0 +1,29 @@ +# frozen_string_literal: true + +require 'fast_spec_helper' +require 'rspec-parameterized' + +describe Gitlab::PerformanceBar::WithTopLevelWarnings do + using RSpec::Parameterized::TableSyntax + + subject { Module.new } + + before do + subject.singleton_class.prepend(described_class) + end + + describe '#has_warnings?' do + where(:has_warnings, :results) do + false | { data: {} } + false | { data: { gitaly: { warnings: [] } } } + true | { data: { gitaly: { warnings: [1] } } } + true | { data: { gitaly: { warnings: [] }, redis: { warnings: [1] } } } + end + + with_them do + it do + expect(subject.has_warnings?(results)).to eq(has_warnings) + end + end + end +end |