diff options
| author | gfyoung <gfyoung17@gmail.com> | 2018-09-29 15:34:47 -0700 |
|---|---|---|
| committer | gfyoung <gfyoung17@gmail.com> | 2018-09-29 21:04:50 -0700 |
| commit | 3836d69119fee2a625edae2a564132eff9520bc8 (patch) | |
| tree | 826596df417b6620e626d3638c553a4f2402d3f2 /lib/api/helpers.rb | |
| parent | 227cc997fb107672e3293c56e0dcb1df72ad82d5 (diff) | |
| download | gitlab-ce-3836d69119fee2a625edae2a564132eff9520bc8.tar.gz | |
Enable frozen string in lib/api and lib/backup
Partially addresses #47424.
Had to make changes to spec files because
stubbing methods on frozen objects is a mess
in RSpec and leads to failures:
https://github.com/rspec/rspec-mocks/issues/1190
Diffstat (limited to 'lib/api/helpers.rb')
| -rw-r--r-- | lib/api/helpers.rb | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/api/helpers.rb b/lib/api/helpers.rb index 85e3e06e4fd..a7ba8066233 100644 --- a/lib/api/helpers.rb +++ b/lib/api/helpers.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module API module Helpers include Gitlab::Utils @@ -381,9 +383,10 @@ module API # lifted from https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/middleware/debug_exceptions.rb#L60 trace = exception.backtrace - message = "\n#{exception.class} (#{exception.message}):\n" + message = ["\n#{exception.class} (#{exception.message}):\n"] message << exception.annoted_source_code.to_s if exception.respond_to?(:annoted_source_code) message << " " << trace.join("\n ") + message = message.join API.logger.add Logger::FATAL, message |