Prep Ruby 3: Add frozen string magic comments

cupakromer · cupakromer · commit 2c680398a206 · 2018-05-11T13:39:41.000-04:00
diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Declare your gem's dependencies in kracken.gemspec.
diff --git a/Rakefile b/Rakefile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'bundler/setup'
 rescue LoadError
diff --git a/app/controllers/kracken/sessions_controller.rb b/app/controllers/kracken/sessions_controller.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class SessionsController < ActionController::Base
 
diff --git a/app/helpers/kracken/application_helper.rb b/app/helpers/kracken/application_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module ApplicationHelper
   end
diff --git a/config/routes.rb b/config/routes.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Kracken::Engine.routes.draw do
   get '/:provider/callback', to: 'sessions#create'
   get '/failure', to: 'sessions#failure'
diff --git a/kracken.gemspec b/kracken.gemspec
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $:.push File.expand_path('../lib', __FILE__)
 
 # Maintain your gem's version:
diff --git a/lib/kracken.rb b/lib/kracken.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "kracken/env"
 require "kracken/error"
 require "kracken/engine"
diff --git a/lib/kracken/authenticator.rb b/lib/kracken/authenticator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class Authenticator
     attr_reader :auth_hash
diff --git a/lib/kracken/config.rb b/lib/kracken/config.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class Config
     attr_accessor :app_id, :app_secret
diff --git a/lib/kracken/controllers/authenticatable.rb b/lib/kracken/controllers/authenticatable.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module Controllers
     module Authenticatable
diff --git a/lib/kracken/controllers/json_api_compatible.rb b/lib/kracken/controllers/json_api_compatible.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module Controllers
     module JsonApiCompatible
diff --git a/lib/kracken/controllers/token_authenticatable.rb b/lib/kracken/controllers/token_authenticatable.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module Controllers
     module TokenAuthenticatable
diff --git a/lib/kracken/credential_authenticator.rb b/lib/kracken/credential_authenticator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class CredentialAuthenticator
     attr_reader :response
diff --git a/lib/kracken/engine.rb b/lib/kracken/engine.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'omniauth/strategies/radius'
 
 module Kracken
diff --git a/lib/kracken/env.rb b/lib/kracken/env.rb
@@ -1,4 +1,6 @@
 
+# frozen_string_literal: true
+
 # If this gem is loaded via bundler before the rails initializer we need to
 # manually pull in dotenv or the .env file will not be read. Ugly hack, but
 # since the omniauth strategy sets the options when the class is defined we
diff --git a/lib/kracken/error.rb b/lib/kracken/error.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   KrackenError        = Class.new(StandardError)
   RequestError        = Class.new(KrackenError)
diff --git a/lib/kracken/json_api.rb b/lib/kracken/json_api.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'json_api/exception_wrapper'
 require_relative 'json_api/path'
 require_relative 'json_api/public_exceptions'
diff --git a/lib/kracken/json_api/exception_wrapper.rb b/lib/kracken/json_api/exception_wrapper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     class ExceptionWrapper < ActionDispatch::ExceptionWrapper
diff --git a/lib/kracken/json_api/path.rb b/lib/kracken/json_api/path.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     class Path
diff --git a/lib/kracken/json_api/public_exceptions.rb b/lib/kracken/json_api/public_exceptions.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     # Error logging methods used by `ActionDispatch::DebugExceptions`
diff --git a/lib/kracken/json_api/routing_mapper.rb b/lib/kracken/json_api/routing_mapper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     module RoutingMapper
diff --git a/lib/kracken/json_api/validation.rb b/lib/kracken/json_api/validation.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     module Validation
diff --git a/lib/kracken/json_api/validation/conflictable.rb b/lib/kracken/json_api/validation/conflictable.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module JsonApi
     module Validation
diff --git a/lib/kracken/railtie.rb b/lib/kracken/railtie.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "kracken/json_api"
 
 module Kracken
diff --git a/lib/kracken/registration.rb b/lib/kracken/registration.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class Registration
     attr_reader :response, :status
diff --git a/lib/kracken/rspec.rb b/lib/kracken/rspec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   module SpecHelper
     @@current_user = nil
diff --git a/lib/kracken/token_authenticator.rb b/lib/kracken/token_authenticator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   class TokenAuthenticator
     attr_reader :response
diff --git a/lib/kracken/version.rb b/lib/kracken/version.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kracken
   VERSION = "0.3.0"
 end
diff --git a/lib/omniauth/strategies/radius.rb b/lib/omniauth/strategies/radius.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'omniauth-oauth2'
 
 begin
diff --git a/lib/tasks/kracken_tasks.rake b/lib/tasks/kracken_tasks.rake
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # desc "Explaining what the task does"
 # task :kracken do
 #   # Task goes here
diff --git a/spec/dummy/Rakefile b/spec/dummy/Rakefile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
diff --git a/spec/dummy/app/controllers/api_controller.rb b/spec/dummy/app/controllers/api_controller.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ApiController < ActionController::Base
   include Kracken::Controllers::TokenAuthenticatable
   include Kracken::Controllers::JsonApiCompatible
diff --git a/spec/dummy/app/controllers/application_controller.rb b/spec/dummy/app/controllers/application_controller.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ApplicationController < ActionController::Base
   # Prevent CSRF attacks by raising an exception.
   # For APIs, you may want to use :null_session instead.
diff --git a/spec/dummy/app/controllers/welcome_controller.rb b/spec/dummy/app/controllers/welcome_controller.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class WelcomeController < ApplicationController
   skip_before_action :authenticate_user!, only: [:index]
 
diff --git a/spec/dummy/app/helpers/application_helper.rb b/spec/dummy/app/helpers/application_helper.rb
@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 module ApplicationHelper
 end
diff --git a/spec/dummy/app/models/user.rb b/spec/dummy/app/models/user.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # In memory user model
 class User
   @@users = {}
diff --git a/spec/dummy/bin/bundle b/spec/dummy/bin/bundle
@@ -1,3 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
 load Gem.bin_path('bundler', 'bundle')
diff --git a/spec/dummy/bin/rails b/spec/dummy/bin/rails
@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 APP_PATH = File.expand_path('../../config/application',  __FILE__)
 require_relative '../config/boot'
 require 'rails/commands'
diff --git a/spec/dummy/bin/rake b/spec/dummy/bin/rake
@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 require_relative '../config/boot'
 require 'rake'
 Rake.application.run
diff --git a/spec/dummy/config.ru b/spec/dummy/config.ru
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file is used by Rack-based servers to start the application.
 
 require ::File.expand_path('../config/environment',  __FILE__)
diff --git a/spec/dummy/config/application.rb b/spec/dummy/config/application.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require File.expand_path('../boot', __FILE__)
 
 # Pick the frameworks you want:
diff --git a/spec/dummy/config/boot.rb b/spec/dummy/config/boot.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Set up gems listed in the Gemfile.
 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../../../Gemfile', __FILE__)
 
diff --git a/spec/dummy/config/environment.rb b/spec/dummy/config/environment.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Load the Rails application.
 require File.expand_path('../application', __FILE__)
 
diff --git a/spec/dummy/config/environments/development.rb b/spec/dummy/config/environments/development.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Dummy::Application.configure do
   # Settings specified here will take precedence over those in config/application.rb.
 
diff --git a/spec/dummy/config/environments/production.rb b/spec/dummy/config/environments/production.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Dummy::Application.configure do
   # Settings specified here will take precedence over those in config/application.rb.
 
diff --git a/spec/dummy/config/environments/test.rb b/spec/dummy/config/environments/test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Dummy::Application.configure do
   # Settings specified here will take precedence over those in config/application.rb.
   config.cache_store = :null_store
diff --git a/spec/dummy/config/initializers/backtrace_silencers.rb b/spec/dummy/config/initializers/backtrace_silencers.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # You can add backtrace silencers for libraries that you're using but don't wish to see in your backtraces.
diff --git a/spec/dummy/config/initializers/filter_parameter_logging.rb b/spec/dummy/config/initializers/filter_parameter_logging.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # Configure sensitive parameters which will be filtered from the log file.
diff --git a/spec/dummy/config/initializers/inflections.rb b/spec/dummy/config/initializers/inflections.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # Add new inflection rules using the following format. Inflections
diff --git a/spec/dummy/config/initializers/kracken.rb b/spec/dummy/config/initializers/kracken.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Kracken.setup do |config|
   config.app_id = 'DUMMY_OAUTH_APP_ID'
   config.app_secret = 'DUMMY_OAUTH_APP_SECRET'
diff --git a/spec/dummy/config/initializers/mime_types.rb b/spec/dummy/config/initializers/mime_types.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # Add new mime types for use in respond_to blocks:
diff --git a/spec/dummy/config/initializers/secret_token.rb b/spec/dummy/config/initializers/secret_token.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # Your secret key is used for verifying the integrity of signed cookies.
diff --git a/spec/dummy/config/initializers/session_store.rb b/spec/dummy/config/initializers/session_store.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 Dummy::Application.config.session_store :cookie_store, key: '_dummy_session'
diff --git a/spec/dummy/config/initializers/wrap_parameters.rb b/spec/dummy/config/initializers/wrap_parameters.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # This file contains settings for ActionController::ParamsWrapper which
diff --git a/spec/dummy/config/routes.rb b/spec/dummy/config/routes.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Rails.application.routes.draw do
   mount Kracken::Engine => "/auth"
   get "welcome/index"
diff --git a/spec/kracken/authenticator_spec.rb b/spec/kracken/authenticator_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/kracken/config_spec.rb b/spec/kracken/config_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/kracken/controllers/authenticatable_spec.rb b/spec/kracken/controllers/authenticatable_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 
diff --git a/spec/kracken/controllers/token_authenticatable_spec.rb b/spec/kracken/controllers/token_authenticatable_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rails_helper"
 require "support/base_controller_double"
 require "support/using_cache"
diff --git a/spec/kracken/credential_authenticator_spec.rb b/spec/kracken/credential_authenticator_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/kracken/registration_spec.rb b/spec/kracken/registration_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/kracken/token_authenticator_spec.rb b/spec/kracken/token_authenticator_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/kracken_spec.rb b/spec/kracken_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe Kracken do
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file is copied to spec/ when you run 'rails generate rspec:install'
 ENV['RAILS_ENV'] ||= 'test'
 require File.expand_path("../dummy/config/environment.rb",  __FILE__)
diff --git a/spec/requests/api_spec.rb b/spec/requests/api_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/requests/authentication_spec.rb b/spec/requests/authentication_spec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 module Kracken
diff --git a/spec/routing/oauth_spec.rb b/spec/routing/oauth_spec.rb
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
diff --git a/spec/support/base_controller_double.rb b/spec/support/base_controller_double.rb
diff --git a/spec/support/fixtures.rb b/spec/support/fixtures.rb
diff --git a/spec/support/omniauth.rb b/spec/support/omniauth.rb
diff --git a/spec/support/using_cache.rb b/spec/support/using_cache.rb

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`source "https://rubygems.org"`
`2`	`4`
`3`	`5`	`# Declare your gem's dependencies in kracken.gemspec.`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`begin`
`2`	`4`	`require 'bundler/setup'`
`3`	`5`	`rescue LoadError`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`module Kracken`
`2`	`4`	`class SessionsController < ActionController::Base`
`3`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`Kracken::Engine.routes.draw do`
`2`	`4`	`get '/:provider/callback', to: 'sessions#create'`
`3`	`5`	`get '/failure', to: 'sessions#failure'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`$:.push File.expand_path('../lib', __FILE__)`
`2`	`4`
`3`	`5`	`# Maintain your gem's version:`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require "kracken/env"`
`2`	`4`	`require "kracken/error"`
`3`	`5`	`require "kracken/engine"`