Commit fab7dacc authored by Tiago Botelho's avatar Tiago Botelho

Updates CE backport

parent f7abde93
...@@ -3,7 +3,7 @@ module Gitlab ...@@ -3,7 +3,7 @@ module Gitlab
class Metric class Metric
include ActiveModel::Model include ActiveModel::Model
attr_accessor :title, :required_metrics, :weight, :y_label, :queries attr_accessor :id, :title, :required_metrics, :weight, :y_label, :queries
validates :title, :required_metrics, :weight, :y_label, :queries, presence: true validates :title, :required_metrics, :weight, :y_label, :queries, presence: true
......
...@@ -14,12 +14,16 @@ module Gitlab ...@@ -14,12 +14,16 @@ module Gitlab
lambda do |group| lambda do |group|
metrics = group.metrics.map do |metric| metrics = group.metrics.map do |metric|
{ metric_hsh = {
title: metric.title, title: metric.title,
weight: metric.weight, weight: metric.weight,
y_label: metric.y_label, y_label: metric.y_label,
queries: metric.queries.map(&query_processor).select(&method(:query_with_result)) queries: metric.queries.map(&query_processor).select(&method(:query_with_result))
} }
metric_hsh[:id] = metric.id if metric.id
metric_hsh
end end
{ {
......
...@@ -41,37 +41,37 @@ describe Clusters::Applications::Prometheus do ...@@ -41,37 +41,37 @@ describe Clusters::Applications::Prometheus do
it 'returns true when installed' do it 'returns true when installed' do
application = build(:clusters_applications_prometheus, :installed, cluster: cluster) application = build(:clusters_applications_prometheus, :installed, cluster: cluster)
expect(application.ready?).to be true expect(application).to be_ready
end end
it 'returns false when not_installable' do it 'returns false when not_installable' do
application = build(:clusters_applications_prometheus, :not_installable, cluster: cluster) application = build(:clusters_applications_prometheus, :not_installable, cluster: cluster)
expect(application.ready?).to be false expect(application).not_to be_ready
end end
it 'returns false when installable' do it 'returns false when installable' do
application = build(:clusters_applications_prometheus, :installable, cluster: cluster) application = build(:clusters_applications_prometheus, :installable, cluster: cluster)
expect(application.ready?).to be false expect(application).not_to be_ready
end end
it 'returns false when scheduled' do it 'returns false when scheduled' do
application = build(:clusters_applications_prometheus, :scheduled, cluster: cluster) application = build(:clusters_applications_prometheus, :scheduled, cluster: cluster)
expect(application.ready?).to be false expect(application).not_to be_ready
end end
it 'returns false when installing' do it 'returns false when installing' do
application = build(:clusters_applications_prometheus, :installing, cluster: cluster) application = build(:clusters_applications_prometheus, :installing, cluster: cluster)
expect(application.ready?).to be false expect(application).not_to be_ready
end end
it 'returns false when errored' do it 'returns false when errored' do
application = build(:clusters_applications_prometheus, :errored, cluster: cluster) application = build(:clusters_applications_prometheus, :errored, cluster: cluster)
expect(application.ready?).to be false expect(application).not_to be_ready
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment