diff options
author | Pawel Chojnacki <pawel@chojnacki.ws> | 2017-06-02 14:21:58 +0200 |
---|---|---|
committer | Pawel Chojnacki <pawel@chojnacki.ws> | 2017-06-02 19:46:29 +0200 |
commit | c86e1437eb415e816dcc29f0b1acafeed2dcc266 (patch) | |
tree | 5ec842bb0258e08cfd4a4816892ab0a8d3042cdc | |
parent | e21b1501ff16de7657a4a5eccb3b8124ba07709c (diff) | |
download | gitlab-ce-c86e1437eb415e816dcc29f0b1acafeed2dcc266.tar.gz |
Make fixture message more descriptive
+ use strip_heredoc to make the text in tests much more readable
-rw-r--r-- | db/fixtures/production/010_settings.rb | 2 | ||||
-rw-r--r-- | spec/lib/gitlab/health_checks/prometheus_text_format_spec.rb | 25 |
2 files changed, 14 insertions, 13 deletions
diff --git a/db/fixtures/production/010_settings.rb b/db/fixtures/production/010_settings.rb index 7978ceefa79..ed0718e7fa9 100644 --- a/db/fixtures/production/010_settings.rb +++ b/db/fixtures/production/010_settings.rb @@ -24,6 +24,6 @@ if envs.any? {|env_name| ENV[env_name].present? } if ENV['GITLAB_PROMETHEUS_METRICS_ENABLED'].present? value = Gitlab::Utils.to_boolean(ENV['GITLAB_PROMETHEUS_METRICS_ENABLED']) settings.prometheus_metrics_enabled = value - save(settings, 'GITLAB_PROMETHEUS_METRICS_ENABLED') + save(settings, 'Prometheus metrics enabled flag') end end diff --git a/spec/lib/gitlab/health_checks/prometheus_text_format_spec.rb b/spec/lib/gitlab/health_checks/prometheus_text_format_spec.rb index a9feab8ff78..b07f95443ee 100644 --- a/spec/lib/gitlab/health_checks/prometheus_text_format_spec.rb +++ b/spec/lib/gitlab/health_checks/prometheus_text_format_spec.rb @@ -11,12 +11,13 @@ describe Gitlab::HealthChecks::PrometheusTextFormat do end it 'marshal to text with non repeating type definition' do - expected = <<-EXPECTED -# TYPE metric1 gauge -metric1 1 -# TYPE metric2 gauge -metric2 2 -EXPECTED + expected = <<-EXPECTED.strip_heredoc + # TYPE metric1 gauge + metric1 1 + # TYPE metric2 gauge + metric2 2 + EXPECTED + expect(subject.marshal(sample_metrics)).to eq(expected.chomp) end @@ -30,12 +31,12 @@ EXPECTED end it 'marshal to text with non repeating type definition' do - expected = <<-EXPECTED -# TYPE metric1 gauge -metric1 1 -metric1 2 -# TYPE metric2 gauge -metric2 3 + expected = <<-EXPECTED.strip_heredoc + # TYPE metric1 gauge + metric1 1 + metric1 2 + # TYPE metric2 gauge + metric2 3 EXPECTED expect(subject.marshal(sample_metrics)).to eq(expected.chomp) end |