diff options
Diffstat (limited to 'lib/gitlab/metrics/dashboard/validator.rb')
| -rw-r--r-- | lib/gitlab/metrics/dashboard/validator.rb | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/lib/gitlab/metrics/dashboard/validator.rb b/lib/gitlab/metrics/dashboard/validator.rb index a2450c59886..8edd9c397f9 100644 --- a/lib/gitlab/metrics/dashboard/validator.rb +++ b/lib/gitlab/metrics/dashboard/validator.rb @@ -8,18 +8,20 @@ module Gitlab class << self def validate(content, schema_path = DASHBOARD_SCHEMA_PATH, dashboard_path: nil, project: nil) - errors(content, schema_path, dashboard_path: dashboard_path, project: project).empty? + errors = _validate(content, schema_path, dashboard_path: dashboard_path, project: project) + errors.empty? end def validate!(content, schema_path = DASHBOARD_SCHEMA_PATH, dashboard_path: nil, project: nil) - errors = errors(content, schema_path, dashboard_path: dashboard_path, project: project) + errors = _validate(content, schema_path, dashboard_path: dashboard_path, project: project) errors.empty? || raise(errors.first) end - def errors(content, schema_path = DASHBOARD_SCHEMA_PATH, dashboard_path: nil, project: nil) - Validator::Client - .new(content, schema_path, dashboard_path: dashboard_path, project: project) - .execute + private + + def _validate(content, schema_path, dashboard_path: nil, project: nil) + client = Validator::Client.new(content, schema_path, dashboard_path: dashboard_path, project: project) + client.execute end end end |