padrino-core 0.16.0.pre3 → 0.16.0

data/padrino-core.gemspec

@@ -2,26 +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_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

data/test/fixtures/apps/complex.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 module LibDemo
   def self.give_me_a_random
@@ -8,22 +8,22 @@ end
 
 class Complex1Demo < Padrino::Application
   set :reload, true
-  get("/old"){ "Old Sinatra Way" }
+  get('/old') { 'Old Sinatra Way' }
 end
 
 class Complex2Demo < Padrino::Application
   set :reload, true
-  get("/old"){ "Old Sinatra Way" }
+  get('/old') { 'Old Sinatra Way' }
 
   controllers :var do
-    get(:destroy){ params.inspect }
+    get(:destroy) { params.to_json }
   end
 
-  get("/"){ "The magick number is: 12!" } # Change only the number!!!
+  get('/') { 'The magick number is: 12!' } # Change only the number!!!
 end
 
 Complex1Demo.controllers do
-  get("/"){ "Given random #{LibDemo.give_me_a_random}" }
+  get('/') { "Given random #{LibDemo.give_me_a_random}" }
 end
 
 Complex2Demo.controllers do

data/test/fixtures/apps/concerned/app.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class SystemConcernedClassDemo < Padrino::Application
   set :reload, true

data/test/fixtures/apps/custom_dependencies/custom_dependencies.rb

@@ -1,11 +1,11 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class CustomDependencies < Padrino::Application
   set :reload, true
 end
 
 CustomDependencies.controllers do
-  get "/" do
-    "foo"
+  get '/' do
+    'foo'
   end
 end

data/test/fixtures/apps/demo_app.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 module Demo
   class App < Padrino::Application

data/test/fixtures/apps/demo_demo.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 module Demo
   class Demo < Padrino::Application

data/test/fixtures/apps/demo_project/api/app.rb

@@ -1,7 +1,7 @@
 module DemoProject
   class API
     def self.call(_)
-      [200, {}, ["api app"]]
+      [200, {}, ['api app']]
     end
   end
 end

data/test/fixtures/apps/demo_project/app.rb

@@ -1,7 +1,7 @@
 module DemoProject
   class App < Padrino::Application
     get :index do
-      "padrino app"
+      'padrino app'
     end
   end
 end

data/test/fixtures/apps/helpers/class_methods_helpers.rb

@@ -1,4 +1,5 @@
 begin
   require '/tmp/padrino_class_demo'
 rescue LoadError
+  # do nothing
 end

data/test/fixtures/apps/helpers/instance_methods_helpers.rb

@@ -1,4 +1,5 @@
 begin
   require '/tmp/padrino_instance_demo'
 rescue LoadError
+  # do nothing
 end

data/test/fixtures/apps/helpers/system_helpers.rb

@@ -5,4 +5,3 @@ SystemDemo.helpers do
     Resolv.name
   end
 end
-

data/test/fixtures/apps/lib/myklass/mysubklass.rb

@@ -1,4 +1,4 @@
 class MyKlass
-    class MySubKlass
-    end
+  class MySubKlass
+  end
 end

data/test/fixtures/apps/mountable_apps/rack_apps.rb

@@ -1,10 +1,9 @@
-
 class RackApp
   def self.call(env)
     if env['PATH_INFO'] == '/404'
-      [404, {}, ["not found ;("]]
+      [404, {}, ['not found ;(']]
     else
-      [200, {}, ["hello rack app"]]
+      [200, {}, ['hello rack app']]
     end
   end
 
@@ -13,11 +12,12 @@ class RackApp
   end
 end
 
-RackApp2 = ->(_){ [200, {}, ["hello rack app2"]] }
+RackApp2 = ->(_) { [200, {}, ['hello rack app2']] }
 
 class SinatraApp < Sinatra::Base
-  set :public_folder, File.dirname(__FILE__)
-  get "/" do
-    "hello sinatra app"
+  set :public_folder, __dir__
+
+  get '/' do
+    'hello sinatra app'
   end
 end

data/test/fixtures/apps/precompiled_app.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 Padrino.configure_apps do
   set :precompile_routes, true
@@ -6,14 +6,15 @@ end
 
 module PrecompiledApp
   class App < Padrino::Application
-    10.times{|n| get("/#{n}"){} }
+    10.times { |n| get("/#{n}") {} }
   end
+
   class SubApp < Padrino::Application
-    10.times{|n| get("/#{n}"){} }
+    10.times { |n| get("/#{n}") {} }
   end
 end
 
-Padrino.mount("PrecompiledApp::SubApp").to("/subapp")
-Padrino.mount("PrecompiledApp::App").to("/")
+Padrino.mount('PrecompiledApp::SubApp').to('/subapp')
+Padrino.mount('PrecompiledApp::App').to('/')
 
 Padrino.load!

data/test/fixtures/apps/simple.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 # Remove this comment if you want do some like this: ruby RACK_ENV=test app.rb
 #
 # require 'padrino-core'
@@ -8,16 +8,16 @@ class SimpleDemo < Padrino::Application
   set :reload, true
   before { true }
   after  { true }
-  error(404) { "404" }
+  error(404) { '404' }
 end
 
 SimpleDemo.controllers do
-  get "/" do
+  get '/' do
     'The magick number is: 2767356926488785838763860464013972991031534522105386787489885890443740254365!' # Change only the number!!!
   end
 
-  get "/rand" do
-    rand(2 ** 256).to_s
+  get '/rand' do
+    rand(2**256).to_s
   end
 end
 

data/test/fixtures/apps/static.rb

@@ -1,9 +1,9 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class StaticDemo < Padrino::Application
   disable :reload
   def self.reload!
-    fail 'reload! called'
+    raise 'reload! called'
   end
 end
 

data/test/fixtures/apps/stealthy/app.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class SystemStealthyClassDemo < Padrino::Application
   set :reload, true

data/test/fixtures/apps/stealthy/helpers/stealthy_class_helpers.rb

@@ -1,6 +1,6 @@
 class Boo
   def self.class_method
-    "BooClass"
+    'BooClass'
   end
 end
 

data/test/fixtures/apps/system.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class SystemDemo < Padrino::Application
   set :reload, true

data/test/fixtures/apps/system_class_methods_demo.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class SystemClassMethodsDemo < Padrino::Application
   set :reload, true

data/test/fixtures/apps/system_instance_methods_demo.rb

@@ -1,4 +1,4 @@
-PADRINO_ROOT = File.dirname(__FILE__) unless defined? PADRINO_ROOT
+PADRINO_ROOT = __dir__ unless defined? PADRINO_ROOT
 
 class SystemInstanceMethodsDemo < Padrino::Application
   set :reload, true

data/test/fixtures/dependencies/a.rb

@@ -6,4 +6,4 @@
 # But here we need some of b.rb
 A_result = [B, C]
 
-A = "A"
+A = 'A'

data/test/fixtures/dependencies/b.rb

@@ -1,4 +1,4 @@
 # But here we need some of c.rb and a.rb
 B_result = C
 
-B = "B"
+B = 'B'

data/test/fixtures/dependencies/c.rb

@@ -1 +1 @@
-C = "C"
+C = 'C'

data/test/fixtures/dependencies/circular/e.rb

@@ -4,8 +4,9 @@ class E
   end
 
   def self.inherited(subclass)
+    super
     subclass.fields.replace fields.dup
   end
 
-  fields << "name"
+  fields << 'name'
 end

data/test/fixtures/dependencies/d.rb

@@ -1,4 +1,4 @@
 D  = 0 unless defined?(D)
 D += 1
 
-raise "SomeThing"
+raise 'SomeThing'

data/test/fixtures/dependencies/linear/i.rb

@@ -1 +1 @@
-fail
+raise

data/test/fixtures/dependencies/nested/l.rb

@@ -1,4 +1,4 @@
 Padrino.require_dependencies(
-  Padrino.root("fixtures/dependencies/nested/m.rb"),
-  Padrino.root("fixtures/dependencies/nested/n.rb")
+  Padrino.root('fixtures/dependencies/nested/m.rb'),
+  Padrino.root('fixtures/dependencies/nested/n.rb')
 )

data/test/fixtures/dependencies/nested/m.rb

@@ -1,5 +1,5 @@
 class M
   def self.hello
-    "hello"
+    'hello'
   end
 end

data/test/fixtures/dependencies/nested/qqq.rb

@@ -1,4 +1,4 @@
 Padrino.require_dependencies(
-  Padrino.root("fixtures/dependencies/nested/rrr.rb"),
-  Padrino.root("fixtures/dependencies/nested/sss.rb")
+  Padrino.root('fixtures/dependencies/nested/rrr.rb'),
+  Padrino.root('fixtures/dependencies/nested/sss.rb')
 )

data/test/fixtures/dependencies/nested/rrr.rb

@@ -1,5 +1,5 @@
 class RRR
   def self.hello
-    "hello"
+    'hello'
   end
 end

data/test/fixtures/dependencies/nested/sss.rb

@@ -1,6 +1,6 @@
 class RollbackTarget
   def self.hello
-    "hello"
+    'hello'
   end
 end
 

data/test/fixtures/reloadable_apps/external/app/app.rb

@@ -1,4 +1,3 @@
-
 module ReloadableApp
   class External < Padrino::Application
     set :reload, true

data/test/fixtures/reloadable_apps/external/app/controllers/base.rb

@@ -1,6 +1,5 @@
-
 ReloadableApp::External.controller :base do
   get :index do
-    "Hello External App"
+    'Hello External App'
   end
 end

data/test/fixtures/reloadable_apps/main/app.rb

@@ -1,10 +1,10 @@
-require File.expand_path(File.dirname(__FILE__) + '/../external/app/app')
+require_relative '../external/app/app'
 
 module ReloadableApp
   class Main < Padrino::Application
     set :reload, true
     get :index do
-      "hey"
+      'hey'
     end
   end
 end

data/test/helper.rb

@@ -1,5 +1,5 @@
 ENV['RACK_ENV'] = 'test'
-PADRINO_ROOT = File.dirname(__FILE__) unless defined?(PADRINO_ROOT)
+PADRINO_ROOT = __dir__ unless defined?(PADRINO_ROOT)
 
 require 'minitest/autorun'
 require 'minitest/pride'
@@ -19,7 +19,7 @@ class Minitest::Spec
   # Sets up a Sinatra::Base subclass defined with the block
   # given. Used in setup or individual spec methods to establish
   # the application.
-  def mock_app(base=Padrino::Application, &block)
+  def mock_app(base = Padrino::Application, &block)
     @app = Sinatra.new(base, &block)
   end