shark-on-lambda 1.0.0.rc2 → 2.0.0.rc1

data/lib/shark_on_lambda/middleware/lambda_logger.rb

@@ -10,37 +10,34 @@ module SharkOnLambda
         @logger = logger
       end
 
-      private
-
-      def _call(env)
+      def call!(env)
         start_time = Time.now
         response = app.call(env)
-        end_time = Time.now
+        duration = duration_in_ms(start_time, Time.now)
 
         if logger.info?
-          log_request(env: env,
-                      response: response,
-                      start_time: start_time,
-                      end_time: end_time)
+          log_request(env: env, response: response, duration: duration)
         end
 
         response
       end
 
+      private
+
       def body_size(body)
         size = 0
         body.each { |chunk| size += chunk.bytesize }
         size
       end
 
-      def log_request(env:, response:, start_time:, end_time:)
+      def log_request(env:, response:, duration:)
         log_object = {
           url: env['PATH_INFO'],
           method: env['REQUEST_METHOD'],
-          params: params(env),
+          params: env.fetch('action_dispatch.request.parameters', {}),
           status: response[0],
           length: body_size(response[2]),
-          duration: "#{duration_in_ms(start_time, end_time)} ms"
+          duration: "#{duration} ms"
         }
         logger.info log_object.to_json
       end
@@ -49,11 +46,6 @@ module SharkOnLambda
         duration = (end_time - start_time) * 1000
         duration.abs.floor(3)
       end
-
-      def params(env)
-        query_params = Rack::Utils.parse_nested_query(env['QUERY_STRING'])
-        query_params.merge(env['shark.path_parameters'] || {})
-      end
     end
   end
 end

data/lib/shark_on_lambda/rake_tasks.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require 'action_dispatch/routing/inspector'
+require 'rake'
+require 'shark_on_lambda'
+
+namespace :'shark-on-lambda' do
+  desc 'Print out all defined routes in match order, with names'
+  task :routes do
+    routes = SharkOnLambda.application.routes.routes
+    inspector = ActionDispatch::Routing::RoutesInspector.new(routes)
+    formatter = ActionDispatch::Routing::ConsoleFormatter::Sheet.new
+
+    puts inspector.format(formatter)
+  end
+end

data/lib/shark_on_lambda/request.rb

@@ -2,8 +2,5 @@
 
 module SharkOnLambda
   class Request < ActionDispatch::Request
-    def path_parameters
-      super.merge(env['shark.path_parameters'] || {})
-    end
   end
 end

data/lib/shark_on_lambda/rspec/env_builder.rb

@@ -3,22 +3,14 @@
 module SharkOnLambda
   module RSpec
     class EnvBuilder
-      attr_reader :action, :controller, :headers, :method, :params
-
-      def initialize(options = {})
-        @method = options.fetch(:method).to_s.upcase
-        @controller = options.fetch(:controller)
-        @action = options.fetch(:action)
-
-        @headers = (options[:headers] || {}).deep_stringify_keys
-        @params = options[:params]
-
-        initialize_env
-        add_headers
-        add_request_body
+      def initialize(**options)
+        @options = options
       end
 
       def build
+        initialize_env
+        add_headers
+        add_request_body_as_json if body? && jsonable_params? && json_request?
         env.deep_stringify_keys
       end
 
@@ -26,39 +18,89 @@ module SharkOnLambda
 
       attr_reader :env
 
-      def add_header(name, value)
-        name = name.upcase.tr('-', '_')
-        key = case name
-              when 'CONTENT_LENGTH', 'CONTENT_TYPE' then name
-              else "HTTP_#{name}"
-              end
-        @env[key] = value.to_s
+      def action
+        @options.fetch(:action)
       end
 
       def add_headers
-        headers.each_pair { |name, value| add_header(name, value) }
+        headers.each_pair do |name, value|
+          name = name.upcase.tr('-', '_')
+          key = case name
+                when 'CONTENT_LENGTH', 'CONTENT_TYPE' then name
+                else "HTTP_#{name}"
+                end
+          env[key] = value.to_s
+        end
       end
 
-      def add_request_body
-        return if %w[GET HEAD OPTIONS].include?(env['REQUEST_METHOD'])
-        return unless params.is_a?(Hash)
-
+      def add_request_body_as_json
         body = params.to_json
 
         env['rack.input'] = StringIO.new(body).set_encoding(Encoding::BINARY)
-        env['CONTENT_TYPE'] = 'application/json'
-        env['CONTENT_LENGTH'] = body.bytesize.to_s
+        env['CONTENT_TYPE'] = headers['content-type']
+        env['CONTENT_LENGTH'] = env['rack.input'].length.to_s
+      end
+
+      def as
+        @options.fetch(:as, :json)
+      end
+
+      def body?
+        !%w[GET HEAD OPTIONS].include?(env['REQUEST_METHOD'])
       end
 
       def initialize_env
         @env = Rack::MockRequest.env_for(
-          'https://localhost:9292',
+          request_uri.to_s,
           method: method,
-          params: params,
-          'shark.controller' => controller,
-          'shark.action' => action
+          params: params
         )
       end
+
+      def controller
+        @options.fetch(:controller, nil)
+      end
+
+      def headers
+        return @headers if defined?(@headers)
+
+        @headers = @options.fetch(:headers, {}).deep_stringify_keys
+        @headers.transform_keys!(&:downcase)
+        @headers
+      end
+
+      def json_request?
+        as == :json
+      end
+
+      def jsonable_params?
+        params.is_a?(Hash)
+      end
+
+      def method
+        @options.fetch(:method).to_s.upcase
+      end
+
+      def params
+        @options.fetch(:params, {}).deep_stringify_keys
+      end
+
+      def path_from_routes
+        path_params = {
+          controller: controller.name.underscore.sub(/_controller$/, ''),
+          action: action,
+          only_path: true
+        }
+        url = SharkOnLambda.application.routes.url_for(path_params, nil)
+        URI.parse(url).path
+      end
+
+      def request_uri
+        return @request_uri if defined?(@request_uri)
+
+        path = action.is_a?(String) ? action : path_from_routes
+        @request_uri = URI.join('https://localhost:9292', path)
+      end
     end
   end
 end

data/lib/shark_on_lambda/rspec/helpers.rb

@@ -3,81 +3,12 @@
 module SharkOnLambda
   module RSpec
     module Helpers
-      def delete(controller_method, options = {})
-        make_request('DELETE', controller_method, options)
-      end
-
-      def get(controller_method, options = {})
-        make_request('GET', controller_method, options)
-      end
-
-      def patch(controller_method, options = {})
-        make_request('PATCH', controller_method, options)
-      end
-
-      def post(controller_method, options = {})
-        make_request('POST', controller_method, options)
-      end
-
-      def put(controller_method, options = {})
-        make_request('PUT', controller_method, options)
-      end
-
-      def response
-        if @response.nil?
-          raise 'You must make a request before you can request a response.'
-        end
-
-        @response
-      end
-
-      private
-
-      def build_env(method, action, options = {})
-        env_builder = EnvBuilder.new(
-          method: method,
-          controller: controller_name,
-          action: action,
-          headers: options[:headers],
-          params: options[:params]
-        )
-        env_builder.build
-      end
-
-      def controller?
-        controller_name.present?
-      end
-
-      def controller_name
-        self.class.ancestors.find do |klass|
-          klass.name.end_with?('Controller')
-        end&.description
-      end
-
-      def dispatch_request(env, skip_middleware: false)
-        return SharkOnLambda.application.call(env) unless skip_middleware
-
-        controller_class = env['shark.controller'].constantize
-        action = env['shark.action']
-
-        request = Request.new(env)
-        response = Response.new
-        controller_class.dispatch(action, request, response)
-        response.prepare!
-      end
-
-      def make_request(method, action, options = {})
-        raise ArgumentError, 'Cannot find controller name.' unless controller?
-
-        options = options.with_indifferent_access
-        env = build_env(method, action, options)
+      extend ActiveSupport::Concern
+      include RequestHelpers
+      include ResponseHelpers
 
-        status, headers, body = dispatch_request(
-          env,
-          skip_middleware: options[:skip_middleware]
-        )
-        errors = env['rack.errors']
-        @response = Rack::MockResponse.new(status, headers, body, errors)
+      included do
+        include SharkOnLambda.application.routes.url_helpers
       end
     end
   end

data/lib/shark_on_lambda/rspec/request_helpers.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+module SharkOnLambda
+  module RSpec
+    module RequestHelpers
+      attr_writer :app
+
+      SUPPORTED_HTTP_METHODS = %w[
+        DELETE GET HEAD OPTIONS PATCH POST PUT
+      ].freeze
+
+      SUPPORTED_HTTP_METHODS.each do |http_method|
+        define_method(http_method.underscore) do |action, **options|
+          make_request(http_method, action, **options)
+        end
+      end
+
+      def app
+        @app ||= SharkOnLambda.application
+      end
+
+      def response
+        if @response.nil?
+          raise 'You must make a request before you can request a response.'
+        end
+
+        @response
+      end
+
+      private
+
+      def build_env(method, action, **options)
+        headers = options.fetch(:headers, {})
+        env_builder = EnvBuilder.new(
+          method: method,
+          controller: described_class,
+          action: action,
+          headers: normalized_headers(headers),
+          params: options.fetch(:params, {})
+        )
+        env_builder.build
+      end
+
+      def default_content_type
+        'application/vnd.api+json'
+      end
+
+      def make_request(method, action, **options)
+        env = build_env(method, action, **options)
+
+        status, headers, body = app.call(env)
+        errors = env['rack.errors']
+        @response = Rack::MockResponse.new(status, headers, body, errors)
+      end
+
+      def normalized_headers(headers)
+        headers.transform_keys! { |key| key.to_s.downcase }
+        headers['content-type'] ||= default_content_type
+        headers
+      end
+    end
+  end
+end

data/lib/shark_on_lambda/rspec/{jsonapi_helpers.rb → response_helpers.rb}

@@ -2,19 +2,17 @@
 
 module SharkOnLambda
   module RSpec
-    module JsonapiHelpers
-      include Helpers
-
+    module ResponseHelpers
       def jsonapi_attributes
-        jsonapi_data[:attributes] || {}
+        jsonapi_data.fetch(:attributes, {})
       end
 
       def jsonapi_data
-        parsed_body[:data] || {}
+        parsed_body.fetch(:data, {})
       end
 
       def jsonapi_errors
-        parsed_body[:errors] || []
+        parsed_body.fetch(:errors, [])
       end
 
       private

data/lib/shark_on_lambda/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SharkOnLambda
-  VERSION = '1.0.0.rc2'
+  VERSION = '2.0.0.rc1'
 end

data/shark-on-lambda.gemspec

@@ -30,18 +30,20 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'actionpack'
-  spec.add_dependency 'activesupport'
-  spec.add_dependency 'jsonapi-rb'
+  spec.required_ruby_version = '>= 2.5'
+
+  spec.add_dependency 'actionpack', '~> 6.0.0'
+  spec.add_dependency 'activesupport', '~> 6.0.0'
+  spec.add_dependency 'jsonapi-rb', '~> 0.5.0'
   spec.add_dependency 'rack', '>= 2.0.8', '< 3'
-  spec.add_dependency 'zeitwerk', '~> 2.2'
+  spec.add_dependency 'rack-on-lambda', '~> 1.0'
+  spec.add_dependency 'zeitwerk', '~> 2.3'
 
   # TODO: Do we really need `activemodel`?
   #       Or can we get away with mocking out ActiveModel::Errors?
   spec.add_development_dependency 'activemodel'
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'factory_bot'
-  spec.add_development_dependency 'fasterer'
   spec.add_development_dependency 'pry'
   spec.add_development_dependency 'pry-byebug'
   spec.add_development_dependency 'rake'

metadata

@@ -1,57 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: shark-on-lambda
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc2
+  version: 2.0.0.rc1
 platform: ruby
 authors:
 - Huy Dinh
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2020-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
 - !ruby/object:Gem::Dependency
   name: jsonapi-rb
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.5.0
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
@@ -73,35 +73,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
-  name: zeitwerk
+  name: rack-on-lambda
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: activemodel
+  name: zeitwerk
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '2.3'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.3'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -115,7 +115,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: factory_bot
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -129,7 +129,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: fasterer
+  name: factory_bot
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -260,36 +260,30 @@ files:
 - bin/console
 - bin/setup
 - changelog.md
-- gems.locked
+- doc/upgrade-from-0.6.x-to-1.x.md
 - gems.rb
 - lib/shark-on-lambda.rb
 - lib/shark_on_lambda.rb
 - lib/shark_on_lambda/api_gateway_handler.rb
 - lib/shark_on_lambda/application.rb
 - lib/shark_on_lambda/base_controller.rb
-- lib/shark_on_lambda/concerns/resettable_singleton.rb
-- lib/shark_on_lambda/concerns/yaml_config_loader.rb
 - lib/shark_on_lambda/configuration.rb
-- lib/shark_on_lambda/dispatcher.rb
 - lib/shark_on_lambda/errors/base.rb
 - lib/shark_on_lambda/errors/base_serializer.rb
-- lib/shark_on_lambda/inferrers/name_inferrer.rb
 - lib/shark_on_lambda/inferrers/serializer_inferrer.rb
-- lib/shark_on_lambda/jsonapi_controller.rb
 - lib/shark_on_lambda/jsonapi_parameters.rb
 - lib/shark_on_lambda/jsonapi_renderer.rb
 - lib/shark_on_lambda/middleware/base.rb
+- lib/shark_on_lambda/middleware/honeybadger.rb
 - lib/shark_on_lambda/middleware/jsonapi_rescuer.rb
 - lib/shark_on_lambda/middleware/lambda_logger.rb
-- lib/shark_on_lambda/middleware/rescuer.rb
-- lib/shark_on_lambda/query.rb
-- lib/shark_on_lambda/rack_adapters/api_gateway.rb
+- lib/shark_on_lambda/rake_tasks.rb
 - lib/shark_on_lambda/request.rb
 - lib/shark_on_lambda/response.rb
 - lib/shark_on_lambda/rspec/env_builder.rb
 - lib/shark_on_lambda/rspec/helpers.rb
-- lib/shark_on_lambda/rspec/jsonapi_helpers.rb
-- lib/shark_on_lambda/secrets.rb
+- lib/shark_on_lambda/rspec/request_helpers.rb
+- lib/shark_on_lambda/rspec/response_helpers.rb
 - lib/shark_on_lambda/version.rb
 - shark-on-lambda.gemspec
 homepage: https://github.com/Skudo/shark-on-lambda
@@ -304,7 +298,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"