padrino-core 0.16.0.pre4 → 0.16.0

data/lib/padrino-core/path_router/route.rb

@@ -21,7 +21,7 @@ module Padrino
       #
       attr_accessor :action, :cache, :cache_key, :cache_expires,
                     :parent, :use_layout, :controller, :user_agent, :path_for_generation, :default_values
-  
+
       ##
       # Constructs an instance of PathRouter::Route.
       #
@@ -33,29 +33,29 @@ module Padrino
         @block   = block if block_given?
         merge_with_options!(options)
       end
-  
+
       ##
       # Calls the route block with arguments.
       #
       def call(app, *args)
         @block.call(app, *args)
       end
-  
+
       ##
       # Returns the route's verb as an array.
       #
       def request_methods
         [verb.to_s.upcase]
       end
-  
+
       ##
       # Returns the original path.
       #
       def original_path
         @path
       end
-  
-      SIGNIFICANT_VARIABLES_REGEX = /(^|[^\\])[:*]([a-zA-Z0-9_]+)/.freeze
+
+      SIGNIFICANT_VARIABLES_REGEX = /(^|[^\\])[:*]([a-zA-Z0-9_]+)/
 
       ##
       # Returns signficant variable names.
@@ -64,18 +64,18 @@ module Padrino
         @significant_variable_names ||=
           if @path.is_a?(String)
             @path.scan(SIGNIFICANT_VARIABLES_REGEX).map(&:last)
-          elsif @path.is_a?(Regexp) and @path.respond_to?(:named_captures)
+          elsif @path.is_a?(Regexp) && @path.respond_to?(:named_captures)
             @path.named_captures.keys
           else
             []
           end
       end
-  
+
       ##
       # Returns an instance of PathRouter::Matcher that is associated with the route.
       #
       def matcher
-        @matcher ||= Matcher.new(@path, :capture => @capture, :default_values => default_values)
+        @matcher ||= Matcher.new(@path, capture: @capture, default_values: default_values)
       end
 
       ##
@@ -84,7 +84,7 @@ module Padrino
       def match(pattern)
         matcher.match(pattern)
       end
-  
+
       ##
       # Associates a block with the route, and increments current order of the router.
       #
@@ -93,7 +93,7 @@ module Padrino
         @order = @router.current_order
         @router.increment_order
       end
-  
+
       ##
       # Expands the path by using parameters.
       # @see PathRouter::Matcher#expand
@@ -120,7 +120,7 @@ module Padrino
       def params_for(pattern, others = {})
         matcher.params_for(pattern, others)
       end
-  
+
       ##
       # Returns before_filters as an array.
       #
@@ -138,7 +138,7 @@ module Padrino
         @_after_filters << block if block_given?
         @_after_filters
       end
-  
+
       ##
       # Returns custom_conditions as an array.
       #
@@ -156,7 +156,7 @@ module Padrino
       end
 
       private
-  
+
       ##
       # Set value to accessor if option name has been defined as an accessora.
       #
@@ -166,7 +166,7 @@ module Padrino
           accessor?(key) ? __send__("#{key}=", value) : (@options[key] = value)
         end
       end
-  
+
       ##
       # Returns true if name has been defined as an accessor.
       #

data/lib/padrino-core/path_router.rb

@@ -57,11 +57,12 @@ module Padrino
       def path(name, *args)
         params = args.last.is_a?(Hash) ? args.pop : {}
         candidates = @routes.select { |route| route.name == name }
-        fail InvalidRouteException if candidates.empty?
+        raise InvalidRouteException if candidates.empty?
         i = 0
-        route = candidates.sort_by! { |candidate|
+        route = candidates.sort_by! do |candidate|
           # Tries to find the route that matches more, but with fewer names, in stable order
-          [(params.keys.map(&:to_s) - candidate.matcher.names).length, candidate.matcher.names.size, i += 1] }.shift
+          [(params.keys.map(&:to_s) - candidate.matcher.names).length, candidate.matcher.names.size, i += 1]
+        end.shift
         matcher = route.matcher
         params_for_expand = params.dup
         if !args.empty? && matcher.mustermann?

data/lib/padrino-core/reloader/rack.rb

@@ -7,7 +7,7 @@ module Padrino
     # during which no further action will be taken.
     #
     class Rack
-      def initialize(app, cooldown=1)
+      def initialize(app, cooldown = 1)
         @app = app
         @cooldown = cooldown
         @last = (Time.now - cooldown)

data/lib/padrino-core/reloader/storage.rb

@@ -13,8 +13,8 @@ module Padrino
 
       def remove(name)
         file = files[name] || return
-        file[:constants].each{ |constant| Reloader.remove_constant(constant) }
-        file[:features].each{ |feature| Reloader.remove_feature(feature) }
+        file[:constants].each { |constant| Reloader.remove_constant(constant) }
+        file[:features].each { |feature| Reloader.remove_feature(feature) }
         files.delete(name)
       end
 
@@ -22,18 +22,18 @@ module Padrino
         file = remove(name)
         @old_entries ||= {}
         @old_entries[name] = {
-          :constants => object_classes,
-          :features  => old_features = Set.new($LOADED_FEATURES.dup)
+          constants: object_classes,
+          features: old_features = Set.new($LOADED_FEATURES.dup)
         }
         features = file && file[:features] || []
-        features.each{ |feature| Reloader.safe_load(feature, :force => true) }
+        features.each { |feature| Reloader.safe_load(feature, force: true) }
         Reloader.remove_feature(name) if old_features.include?(name)
       end
 
       def commit(name)
         entry = {
-          :constants => new_classes(@old_entries[name][:constants]),
-          :features  => Set.new($LOADED_FEATURES) - @old_entries[name][:features] - [name]
+          constants: new_classes(@old_entries[name][:constants]),
+          features: Set.new($LOADED_FEATURES) - @old_entries[name][:features] - [name]
         }
         files[name] = entry
         @old_entries.delete(name)
@@ -42,9 +42,10 @@ module Padrino
       def rollback(name)
         new_classes(@old_entries[name][:constants]).each do |klass|
           loaded_in_name = files.each do |file, data|
-                             next if file == name
-                             break if data[:constants].include?(klass)
-                           end
+            next if file == name
+            break if data[:constants].include?(klass)
+          end
+
           Reloader.remove_constant(klass) if loaded_in_name
         end
         @old_entries.delete(name)
@@ -64,7 +65,7 @@ module Padrino
 
         ObjectSpace.each_object(::Class).each do |klass|
           if block_given?
-            if filtered_class = yield(klass)
+            if (filtered_class = yield(klass))
               klasses << filtered_class
             end
           else

data/lib/padrino-core/reloader.rb

@@ -24,7 +24,7 @@ module Padrino
     # Default excluded directories at Padrino.root are: test, spec, features, tmp, config, db and public
     #
     def exclude
-      @_exclude ||= Set.new(%w[test spec tmp features config public db].map{ |path| Padrino.root(path) })
+      @_exclude ||= Set.new(%w[test spec tmp features config public db].map { |path| Padrino.root(path) })
     end
 
     ##
@@ -74,8 +74,7 @@ module Padrino
     #
     def lock!
       klasses = Storage.send(:object_classes) do |klass|
-        original_klass_name = constant_name(klass)
-        original_klass_name.split('::').first if original_klass_name
+        constant_name(klass)&.split('::')&.first
       end
       klasses |= Padrino.mounted_apps.map(&:app_class)
       exclude_constants.merge(klasses)
@@ -84,7 +83,7 @@ module Padrino
     ##
     # A safe Kernel::require which issues the necessary hooks depending on results
     #
-    def safe_load(file, options={})
+    def safe_load(file, options = {})
       began_at = Time.now
       file     = figure_path(file)
       return unless options[:force] || file_changed?(file)
@@ -93,12 +92,12 @@ module Padrino
       Storage.prepare(file) # might call #safe_load recursively
       logger.devel(file_new?(file) ? :loading : :reload, began_at, file)
       begin
-        with_silence{ require(file) }
+        with_silence { require(file) }
         Storage.commit(file)
         update_modification_time(file)
-      rescue Exception => exception
+      rescue Exception => e
         unless options[:cyclic]
-          logger.exception exception, :short
+          logger.exception e, :short
           logger.error "Failed to load #{file}; removing partially defined constants"
         end
         Storage.rollback(file)
@@ -116,6 +115,7 @@ module Padrino
       base.send :remove_const, object
       logger.devel "Removed constant #{const} from #{base}"
     rescue NameError
+      # no const to remove
     end
 
     ##
@@ -157,7 +157,7 @@ module Padrino
     # Reloads the file if it's special. For now it's only I18n locale files.
     #
     def reload_special(file)
-      return unless special_files.any?{ |special_file| File.identical?(special_file, file) }
+      return unless special_files.any? { |special_file| File.identical?(special_file, file) }
       if defined?(I18n)
         began_at = Time.now
         I18n.reload!
@@ -237,14 +237,14 @@ module Padrino
     # Tells if a feature should be excluded from Reloader tracking.
     #
     def feature_excluded?(file)
-      !file.start_with?(Padrino.root) || exclude.any?{ |excluded_path| file.start_with?(excluded_path) }
+      !file.start_with?(Padrino.root) || exclude.any? { |excluded_path| file.start_with?(excluded_path) }
     end
 
     ##
     # Tells if a constant should be excluded from Reloader routines.
     #
     def constant_excluded?(const)
-      external_constant?(const) || (exclude_constants - include_constants).any?{ |excluded_constant| constant_name(const).start_with?(excluded_constant) }
+      external_constant?(const) || (exclude_constants - include_constants).any? { |excluded_constant| constant_name(const).start_with?(excluded_constant) }
     end
 
     ##
@@ -256,7 +256,7 @@ module Padrino
       sources = object_sources(const)
       # consider methodless constants not external
       return false if sources.empty?
-      !sources.any?{ |source| source.start_with?(Padrino.root) }
+      sources.none? { |source| source.start_with?(Padrino.root) }
     end
 
     ##
@@ -267,18 +267,17 @@ module Padrino
     def object_sources(target)
       sources = Set.new
       target.methods.each do |method_name|
-        next unless method_name.kind_of?(Symbol)
+        next unless method_name.is_a?(Symbol)
+
         method_object = target.method(method_name)
-        if method_object.owner == target.singleton_class
-          sources << method_object.source_location.first
-        end
+        sources << method_object.source_location.first if method_object.owner == target.singleton_class
       end
+
       target.instance_methods.each do |method_name|
-        next unless method_name.kind_of?(Symbol)
+        next unless method_name.is_a?(Symbol)
+
         method_object = target.instance_method(method_name)
-        if method_object.owner == target
-          sources << method_object.source_location.first
-        end
+        sources << method_object.source_location.first if method_object.owner == target
       end
       sources
     end

data/lib/padrino-core/router.rb

@@ -14,13 +14,13 @@ module Padrino
   # @example
   #
   #   routes = Padrino::Router.new do
-  #     map(:path => "/", :to => PadrinoWeb, :host => "padrino.local")
-  #     map(:path => "/", :to => Admin, :host => "admin.padrino.local")
+  #     map(path: '/', to: PadrinoWeb, host: 'padrino.local')
+  #     map(path: '/', to: Admin, host: 'admin.padrino.local')
   #   end
   #   run routes
   #
   #   routes = Padrino::Router.new do
-  #     map(:path => "/", :to => PadrinoWeb, :host => /*.padrino.local/)
+  #     map(path: '/', to: PadrinoWeb, host: /*.padrino.local/)
   #   end
   #   run routes
   #
@@ -45,17 +45,17 @@ module Padrino
     #  The host to map the specified application.
     #
     # @example
-    #  map(:path => "/", :to => PadrinoWeb, :host => "padrino.local")
+    #  map(path: '/', to: PadrinoWeb, host: 'padrino.local')
     #
     # @return [Array] The sorted route mappings.
     # @api semipublic
-    def map(options={})
-      path = options[:path] || "/"
+    def map(options = {})
+      path = options[:path] || '/'
       host = options[:host]
       app  = options[:to]
 
-      raise ArgumentError, "paths need to start with /" if path[0] != ?/
-      raise ArgumentError, "app is required" if app.nil?
+      raise ArgumentError, 'paths need to start with /' if path[0] != '/'
+      raise ArgumentError, 'app is required' if app.nil?
 
       path  = path.chomp('/')
       match = Regexp.new("^#{Regexp.quote(path).gsub('/', '/+')}(.*)", Regexp::NOENCODING)
@@ -67,17 +67,17 @@ module Padrino
     # The call handler setup to route a request given the mappings specified.
     def call(env)
       began_at = Time.now
-      path_info = env["PATH_INFO"].to_s
+      path_info = env['PATH_INFO'].to_s
       script_name = env['SCRIPT_NAME']
       http_host = env['HTTP_HOST']
       last_result = nil
 
       @mapping.each do |host, path, match, app|
         next unless host.nil? || http_host =~ host
-        next unless path_info =~ match && rest = ::Regexp.last_match(1)
-        next unless rest.empty? || rest[0] == ?/
+        next unless path_info =~ match && (rest = ::Regexp.last_match(1))
+        next unless rest.empty? || rest[0] == '/'
 
-        rest = "/" if rest.empty?
+        rest = '/' if rest.empty?
 
         env['SCRIPT_NAME'] = script_name + path
         env['PATH_INFO'] = rest
@@ -90,8 +90,8 @@ module Padrino
       last_result || begin
         env['SCRIPT_NAME'] = script_name
         env['PATH_INFO'] = path_info
-        Padrino::Logger::Rack.new(nil,'/').send(:log, env, 404, {}, began_at) if logger.debug?
-        [404, {"content-type" => "text/plain", "x-cascade" => "pass"}, ["Not Found: #{path_info}"]]
+        Padrino::Logger::Rack.new(nil, '/').send(:log, env, 404, {}, began_at) if logger.debug?
+        [404, { 'content-type' => 'text/plain', 'x-cascade' => 'pass' }, ["Not Found: #{path_info}"]]
       end
     end
   end

data/lib/padrino-core/server.rb

@@ -4,40 +4,37 @@ module Padrino
   # thin, mongrel, or WEBrick in that order.
   #
   # @example
-  #   Padrino.run! # with these defaults => host: "127.0.0.1", port: "3000", adapter: the first found
-  #   Padrino.run!("0.0.0.0", "4000", "mongrel") # use => host: "0.0.0.0", port: "4000", adapter: "mongrel"
+  #   Padrino.run! # with these defaults => host: '127.0.0.1', port: '3000', adapter: the first found
+  #   Padrino.run!('0.0.0.0', '4000', 'mongrel') # use => host: '0.0.0.0', port: '4000', adapter: 'mongrel'
   #
-  def self.run!(options={})
+  def self.run!(options = {})
     Padrino.load!
     Server.start(*detect_application(options))
   end
 
-
-  #
-  #
   def self.detect_application(options)
     default_config_file = 'config.ru'
-    if (config_file = options.delete(:config)) || File.file?(default_config_file)
-      config_file ||= default_config_file
-      fail "Rack config file `#{config_file}` must have `.ru` extension" unless config_file =~ /\.ru$/
-      rack_app, rack_options = Rack::Builder.parse_file(config_file)
-      [rack_app, (rack_options || {}).merge(options)]
-    else
-      [Padrino.application, options]
-    end
+
+    config_file = options.delete(:config)
+    config_file ||= default_config_file if File.file?(default_config_file)
+    return [Padrino.application, options] unless config_file
+
+    raise "Rack config file `#{config_file}` must have `.ru` extension" unless config_file =~ /\.ru$/
+    rack_app, rack_options = Rack::Builder.parse_file(config_file)
+    [rack_app, (rack_options || {}).merge(options)]
   end
 
   ##
   # This module builds a Padrino server to run the project based on available handlers.
   #
   class Server < Rackup::Server
-    DEFAULT_ADDRESS = { :Host => '127.0.0.1', :Port => 3000 }
+    DEFAULT_ADDRESS = { Host: '127.0.0.1', Port: 3000 }
 
     # Server Handlers
-    Handlers = [:thin, :puma, :'spider-gazelle', :mongrel, :trinidad, :webrick]
+    Handlers = %i[thin puma spider-gazelle mongrel trinidad webrick]
 
     # Starts the application on the available server with specified options.
-    def self.start(app, options={})
+    def self.start(app, options = {})
       options = Utils.symbolize_keys(options.to_hash)
       options.update(parse_server_options(options.delete(:options)))
       options.update(detect_address(options))
@@ -55,21 +52,20 @@ module Padrino
     # Starts the application on the available server with specified options.
     def start
       puts "=> Padrino/#{Padrino.version} has taken the stage #{Padrino.env} at http://#{options[:Host]}:#{options[:Port]}"
-      [:INT, :TERM].each { |sig| trap(sig) { exit } }
+      %i[INT TERM].each { |sig| trap(sig) { exit } }
       super do |server|
         server.threaded = true if server.respond_to?(:threaded=)
       end
     ensure
-      puts "<= Padrino leaves the gun, takes the cannoli" unless options[:daemonize]
+      puts '<= Padrino leaves the gun, takes the cannoli' unless options[:daemonize]
     end
 
     # The application the server will run.
     attr_reader :app
-    alias :wrapped_app :app
+    alias wrapped_app app
 
     attr_reader :options
 
-
     # Detects the supported handler to use.
     #
     # @example
@@ -81,7 +77,7 @@ module Padrino
       rescue LoadError, NameError
         # Ignored
       end
-      fail "Server handler (#{Handlers.join(', ')}) not found."
+      raise "Server handler (#{Handlers.join(', ')}) not found."
     end
 
     # Prepares a directory for pid file.
@@ -95,14 +91,14 @@ module Padrino
     # Parses an array of server options.
     #
     def self.parse_server_options(options)
-      parsed_server_options = Array(options).flat_map{ |option| option.split('=', 2) }
+      parsed_server_options = Array(options).flat_map { |option| option.split('=', 2) }
       Utils.symbolize_keys(Hash[*parsed_server_options])
     end
 
     # Detects Host and Port for Rack server.
     #
     def self.detect_address(options)
-      address = DEFAULT_ADDRESS.merge(options.select{ |key| [:Host, :Port].include?(key) })
+      address = DEFAULT_ADDRESS.merge(options.select { |key| %i[Host Port].include?(key) })
       address[:Host] = options[:host] if options[:host]
       address[:Port] = options[:port] if options[:port]
       address

data/lib/padrino-core/tasks.rb

@@ -10,7 +10,6 @@ module Padrino
   #   Padrino::Tasks.files.unshift("yourtask.rb")
   #
   module Tasks
-
     ##
     # Returns a list of files to handle with padrino rake
     #

data/lib/padrino-core/version.rb

@@ -6,7 +6,7 @@
 #
 module Padrino
   # The version constant for the current version of Padrino.
-  VERSION = '0.16.0.pre4' unless defined?(Padrino::VERSION)
+  VERSION = '0.16.0' unless defined?(Padrino::VERSION)
 
   #
   # The current Padrino version.

data/lib/padrino-core.rb

@@ -16,12 +16,12 @@ require 'padrino-core/server'
 require 'padrino-core/tasks'
 require 'padrino-core/module'
 
-if ENV["PADRINO_ENV"] || defined?(PADRINO_ENV)
+if ENV['PADRINO_ENV'] || defined?(PADRINO_ENV)
   warn 'Environment variable PADRINO_ENV is deprecated. Please, use RACK_ENV.'
-  ENV["RACK_ENV"] ||= ENV["PADRINO_ENV"] ||= PADRINO_ENV
+  ENV['RACK_ENV'] ||= ENV['PADRINO_ENV'] ||= PADRINO_ENV
 end
-RACK_ENV = ENV["RACK_ENV"] ||= "development"  unless defined?(RACK_ENV)
-PADRINO_ROOT = ENV["PADRINO_ROOT"] ||= File.dirname(Padrino.first_caller) unless defined?(PADRINO_ROOT)
+RACK_ENV = ENV['RACK_ENV'] ||= 'development' unless defined?(RACK_ENV)
+PADRINO_ROOT = ENV['PADRINO_ROOT'] ||= File.dirname(Padrino.first_caller) unless defined?(PADRINO_ROOT)
 
 module Padrino
   class ApplicationLoadError < RuntimeError # @private
@@ -89,8 +89,7 @@ module Padrino
     #   end
     #
     def configure_apps(&block)
-      return  unless block_given?
-      global_configurations << block
+      global_configurations << block if block_given?
     end
 
     ##
@@ -122,7 +121,7 @@ module Padrino
     #
     def add_middleware(router)
       builder = Rack::Builder.new
-      middleware.each{ |mw,args,block| builder.use(mw, *args, &block) }
+      middleware.each { |mw, args, block| builder.use(mw, *args, &block) }
       builder.run(router)
       builder.to_app
     end
@@ -160,8 +159,8 @@ module Padrino
     # @yield []
     #   The given block will be passed to the initialized middleware.
     #
-    def use(mw, *args, &block)
-      middleware << [mw, args, block]
+    def use(middleware_class, *args, &block)
+      middleware << [middleware_class, args, block]
     end
 
     ##
@@ -178,7 +177,7 @@ module Padrino
     #
     # @returns The root path of the loaded gem
     def gem(name, main_module)
-      _, spec = Gem.loaded_specs.find{|spec_pair| spec_pair[0] == name }
+      _, spec = Gem.loaded_specs.find { |spec_pair| spec_pair[0] == name }
       gems << spec
       modules << main_module
       spec.full_gem_path

data/padrino-core.gemspec

@@ -2,27 +2,27 @@
 require File.expand_path('lib/padrino-core/version.rb', __dir__)
 
 Gem::Specification.new do |s|
-  s.name = "padrino-core"
-  s.authors = ["Padrino Team", "Nathan Esquenazi", "Davide D'Agostino", "Arthur Chiu"]
-  s.email = "padrinorb@gmail.com"
-  s.summary = "The required Padrino core gem"
-  s.homepage = "http://www.padrinorb.com"
-  s.description = "The Padrino core gem required for use of this framework"
-  s.required_ruby_version = ">= 2.7.8"
-  s.required_rubygems_version = ">= 1.3.6"
+  s.name = 'padrino-core'
+  s.authors = ['Padrino Team', 'Nathan Esquenazi', "Davide D'Agostino", 'Arthur Chiu']
+  s.email = 'padrinorb@gmail.com'
+  s.summary = 'The required Padrino core gem'
+  s.homepage = 'http://www.padrinorb.com'
+  s.description = 'The Padrino core gem required for use of this framework'
+  s.required_ruby_version = '>= 2.7.8'
+  s.required_rubygems_version = '>= 1.3.6'
   s.version = Padrino.version
-  s.date = Time.now.strftime("%Y-%m-%d")
-  s.license = "MIT"
+  s.date = Time.now.strftime('%Y-%m-%d')
+  s.license = 'MIT'
 
-  s.extra_rdoc_files = Dir["*.rdoc"]
+  s.extra_rdoc_files = Dir['*.rdoc']
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
-  s.rdoc_options  = ["--charset=UTF-8"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
+  s.rdoc_options  = ['--charset=UTF-8']
 
-  s.add_dependency("padrino-support", Padrino.version)
-  s.add_dependency("rackup", "~> 2.1")
-  s.add_dependency("sinatra", "~> 4")
-  s.add_dependency("thor", "~> 1.0")
+  s.add_dependency('padrino-support', Padrino.version)
+  s.add_dependency('rackup', '~> 2.1')
+  s.add_dependency('sinatra', '~> 4')
+  s.add_dependency('thor', '~> 1.0')
 end

data/test/fixtures/app_gem/app_gem.gemspec

@@ -1,16 +1,17 @@
+require 'English'
 require File.expand_path('lib/app_gem/version', __dir__)
 
 Gem::Specification.new do |gem|
-  gem.authors       = ["Florian Gilcher"]
-  gem.email         = ["florian.gilcher@asquera.de"]
+  gem.authors       = ['Florian Gilcher']
+  gem.email         = ['florian.gilcher@asquera.de']
   gem.description   = 'TODO: Write a gem description'
   gem.summary       = 'TODO: Write a gem summary'
-  gem.homepage      = ""
+  gem.homepage      = ''
 
-  gem.files         = `git ls-files`.split($\)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  gem.files         = `git ls-files`.split($OUTPUT_RECORD_SEPARATOR)
+  gem.executables   = gem.files.grep(%r{^bin/}).map { |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.name          = "app_gem"
-  gem.require_paths = ["app", "lib"]
+  gem.name          = 'app_gem'
+  gem.require_paths = %w[app lib]
   gem.version       = AppGem::VERSION
 end

data/test/fixtures/app_gem/lib/app_gem/version.rb

@@ -1,3 +1,3 @@
 module AppGem
-  VERSION = "0.0.1"
+  VERSION = '0.0.1'
 end