praxis 2.0.pre.9 → 2.0.pre.10

data/lib/praxis/request.rb

@@ -82,7 +82,7 @@ module Praxis
       PATH_VERSION_PREFIX
     end
 
-    # DEPRECATED: remove with ResourceDefinition.version using: :path
+    # DEPRECATED: remove with EndpointDefinition.version using: :path
     PATH_VERSION_MATCHER = %r{^#{self.path_version_prefix}(?<version>[^\/]+)\/}.freeze
 
     def path_version_matcher

data/lib/praxis/resource_definition.rb

@@ -1,312 +1,3 @@
-require 'active_support/concern'
-require 'active_support/inflector'
 
-module Praxis
-  module ResourceDefinition
-    extend ActiveSupport::Concern
-
-    DEFAULT_RESOURCE_HREF_ACTION = :show
-
-    included do
-      @version = 'n/a'.freeze
-      @actions = Hash.new
-      @responses = Hash.new
-
-      @action_defaults = Trait.new &ResourceDefinition.generate_defaults_block
-
-      @version_options = {}
-      @metadata = {}
-      @traits = []
-
-      if self.name
-        @prefix = '/' + self.name.split("::").last.underscore
-      else
-        @prefix = '/'
-      end
-
-      @version_prefix = ''
-
-      @parent = nil
-      @parent_prefix = ''
-
-      @routing_prefix = nil
-
-      @on_finalize = Array.new
-
-      Application.instance.resource_definitions << self
-    end
-
-    def self.generate_defaults_block( version: nil )
-
-      # Ensure we inherit any base params defined in the API definition for the passed in version
-      base_attributes = if (base_params = ApiDefinition.instance.info(version).base_params)
-        base_params.attributes
-      else
-        {}
-      end
-
-      Proc.new do
-        unless base_attributes.empty?
-          params do
-            base_attributes.each do |base_name, base_attribute|
-              attribute base_name, base_attribute.type, **base_attribute.options
-            end
-          end
-        end
-      end
-    end
-
-    def self.finalize!
-      Application.instance.resource_definitions.each do |resource_definition|
-        while (block = resource_definition.on_finalize.shift)
-          block.call
-        end
-      end
-    end
-
-
-
-    module ClassMethods
-      attr_reader :actions
-      attr_reader :responses
-      attr_reader :version_options
-      attr_reader :traits
-      attr_reader :version_prefix
-      attr_reader :parent_prefix
-
-      # opaque hash of user-defined medata, used to decorate the definition,
-      # and also available in the generated JSON documents
-      attr_reader :metadata
-
-      attr_accessor :controller
-
-      def display_name( string=nil )
-        unless string
-          return  @display_name ||= self.name.split("::").last  # Best guess at a display name?
-        end
-        @display_name = string
-      end
-
-      def on_finalize(&block)
-        if block_given?
-          @on_finalize << proc(&block)
-        end
-
-        @on_finalize
-      end
-
-      def prefix(prefix=nil)
-        return @prefix if prefix.nil?
-        @routing_prefix = nil # reset routing_prefix
-        @prefix = prefix
-      end
-
-      def media_type(media_type=nil)
-        return @media_type if media_type.nil?
-
-        if media_type.kind_of?(String)
-          media_type = SimpleMediaType.new(media_type)
-        end
-        @media_type = media_type
-      end
-
-
-      def parent(parent=nil, **mapping)
-        return @parent if parent.nil?
-
-        @routing_prefix = nil # reset routing_prefix
-
-        parent_action = parent.canonical_path
-        parent_route = parent_action.route.path
-
-        # if a mapping is passed, it *must* resolve any param name conflicts
-        unless mapping.any?
-          # assume last capture is the relevant one to replace
-          # if not... then I quit.
-          parent_param_name = parent_route.names.last
-
-          # more assumptions about names
-          parent_name = parent.name.demodulize.underscore.singularize
-
-          # put it together to find what we should call this new param
-          param = "#{parent_name}_#{parent_param_name}".to_sym
-          mapping[parent_param_name.to_sym] = param
-        end
-
-        # complete the mapping and massage the route
-        parent_route.names.collect(&:to_sym).each do |name|
-          if mapping.key?(name)
-            param = mapping[name]
-            # FIXME: this won't handle URI Template type paths, ie '/{parent_id}'
-            prefixed_path = parent_action.route.prefixed_path
-            @parent_prefix = prefixed_path.gsub(/(:)(#{name})(\W+|$)/, "\\1#{param.to_s}\\3")
-          else
-            mapping[name] = name
-          end
-        end
-
-        self.on_finalize do
-          self.inherit_params_from_parent(parent_action, **mapping)
-        end
-
-        @parent = parent
-      end
-
-      def inherit_params_from_parent(parent_action, **mapping)
-        actions.each do |name, action|
-          action.params do
-            mapping.each do |parent_name, name|
-              next if action.params && action.params.attributes.key?(name)
-
-              parent_attribute = parent_action.params.attributes[parent_name]
-
-              attribute name, parent_attribute.type, **parent_attribute.options
-            end
-          end
-        end
-
-      end
-
-      attr_writer :routing_prefix
-
-      def routing_prefix
-        return @routing_prefix if @routing_prefix
-
-        @routing_prefix = parent_prefix + prefix
-      end
-
-
-      def version(version=nil, options=nil)
-        return @version unless version
-
-        @version = version
-
-        unless options.nil?
-          warn 'DEPRECATED: ResourceDefinition.version with options is no longer supported. Define in api global info instead.'
-
-          @version_options = options
-          version_using = Array(@version_options[:using])
-          if version_using.include?(:path)
-            @version_prefix = "#{Praxis::Request::path_version_prefix}#{self.version}"
-          end
-        end
-
-        @action_defaults.instance_eval &ResourceDefinition.generate_defaults_block( version: version )
-      end
-
-
-      def canonical_path(action_name=nil)
-        if action_name
-          raise "Canonical path for #{self.name} is already defined as: '#{@canonical_action_name}'. 'canonical_path' can only be defined once." if @canonical_action_name
-          @canonical_action_name = action_name
-        else
-          # Resolution of the actual action definition needs to be done lazily, since we can use the `canonical_path` stanza
-          # at the top of the resource, well before the actual action is defined.
-          unless @canonical_action
-            href_action = @canonical_action_name || DEFAULT_RESOURCE_HREF_ACTION
-            @canonical_action = actions.fetch(href_action) do
-              raise "Error: trying to set canonical_href of #{self.name}. Action '#{href_action}' does not exist"
-            end
-          end
-          return @canonical_action
-        end
-      end
-
-      def to_href( params )
-        canonical_path.route.path.expand(params)
-      end
-
-      def parse_href(path)
-        if path.kind_of?(::URI::Generic)
-          path = path.path
-        end
-        param_values = canonical_path.route.path.params(path)
-        attrs = canonical_path.params.attributes
-        param_values.each_with_object({}) do |(key,value),hash|
-          hash[key.to_sym] = attrs[key.to_sym].load(value,[key])
-        end
-      rescue => e
-        raise Praxis::Exception.new("Error parsing or coercing parameters from href: #{path}\n"+e.message)
-      end
-
-      def trait(trait_name)
-        unless ApiDefinition.instance.traits.has_key? trait_name
-          raise Exceptions::InvalidTrait.new("Trait #{trait_name} not found in the system")
-        end
-        trait = ApiDefinition.instance.traits.fetch(trait_name)
-        @traits << trait_name
-      end
-      alias_method :use, :trait
-
-      def action_defaults(&block)
-        if block_given?
-          @action_defaults.instance_eval(&block)
-        end
-
-        @action_defaults
-      end
-
-      def params(type=Attributor::Struct, **opts, &block)
-        warn 'DEPRECATED: ResourceDefinition.params is deprecated. Use it in action_defaults instead.'
-        action_defaults do
-          params type, **opts, &block
-        end
-      end
-
-      def payload(type=Attributor::Struct, **opts, &block)
-        warn 'DEPRECATED: ResourceDefinition.payload is deprecated. Use action_defaults instead.'
-        action_defaults do
-          payload type, **opts, &block
-        end
-      end
-
-      def headers(**opts, &block)
-        warn 'DEPRECATED: ResourceDefinition.headers is deprecated. Use action_defaults instead.'
-        action_defaults do
-          headers **opts, &block
-        end
-      end
-
-      def response(name, **args)
-        warn 'DEPRECATED: ResourceDefinition.response is deprecated. Use action_defaults instead.'
-        action_defaults do
-          response name, **args
-        end
-      end
-
-      def action(name, &block)
-        raise ArgumentError, "can not create ActionDefinition without block" unless block_given?
-        raise ArgumentError, "Action names must be defined using symbols (Got: #{name} (of type #{name.class}))" unless name.is_a? Symbol
-        @actions[name] = ActionDefinition.new(name, self, &block)
-      end
-
-      def description(text=nil)
-        @description = text if text
-        @description
-      end
-
-      def id
-        self.name.gsub('::'.freeze,'-'.freeze)
-      end
-
-      def describe(context: nil)
-        {}.tap do |hash|
-          hash[:description] = description
-          hash[:media_type] = media_type.describe(true) if media_type
-          hash[:actions] = actions.values.collect{|action| action.describe(context: context)}
-          hash[:name] = self.name
-          hash[:parent] = self.parent.id if self.parent
-          hash[:display_name] = self.display_name
-          hash[:metadata] = metadata
-          hash[:traits] = self.traits
-        end
-      end
-
-      def nodoc!
-        metadata[:doc_visibility] = :none
-      end
-
-    end
-
-  end
-end
+raise "ResourceDefinition class has changed name in this version of Praxis.\n" \
+      'Please blanket-replace all uses of ResourceDefinition with EndpointDefinition instead.'

data/lib/praxis/response_definition.rb

@@ -311,16 +311,8 @@ module Praxis
     def validate_parts!(response)
       return unless parts
 
-      case response.body
-      when Praxis::Types::MultipartArray
-        response.body.each do |part|
-          parts.validate(part)
-        end
-      else
-        # TODO: remove with other Multipart deprecations.
-        response.parts.each do |name, part|
-          parts.validate(part)
-        end
+      response.body.each do |part|
+        parts.validate(part)
       end
     end
 

data/lib/praxis/response_template.rb

@@ -9,15 +9,15 @@ module Praxis
     end
 
     def compile(action=nil, **args)
-      # Default media_type to the resource_definition one, if the block has it in
+      # Default media_type to the endpoint_definition one, if the block has it in
       # its required args but no value is passed (funky, but can help in the common case)
       if block.parameters.any? { |(type, name)| name == :media_type && type == :keyreq } && action
         unless args.has_key? :media_type
-          media_type = action.resource_definition.media_type
+          media_type = action.endpoint_definition.media_type
           unless media_type
             raise Exceptions::InvalidConfiguration.new(
               "Could not default :media_type argument for response template #{@name}." +
-               " Resource #{action.resource_definition} does not have an associated mediatype and none was passed"
+               " Endpoint #{action.endpoint_definition} does not have an associated mediatype and none was passed"
             )
           end
           args[:media_type] = media_type

data/lib/praxis/router.rb

@@ -52,8 +52,8 @@ module Praxis
     def add_route(target, route)
       path_versioning = (Application.instance.versioning_scheme == :path)
 
-      # DEPRECATED: remove with ResourceDefinition.version using: :path
-      path_versioning ||= (target.action.resource_definition.version_options[:using] == :path)
+      # DEPRECATED: remove with EndpointDefinition.version using: :path
+      path_versioning ||= (target.action.endpoint_definition.version_options[:using] == :path)
 
       unless path_versioning
         target = VersionMatcher.new(target, version: route.version)

data/lib/praxis/routing_config.rb

@@ -53,7 +53,7 @@ module Praxis
       end
       prefixed_path = path.gsub('//','/')
       path = (base + path).gsub('//','/')
-      pattern = Mustermann.new(path, {ignore_unknown_options: true}.merge( options ))
+      pattern = Mustermann.new(path, **{ignore_unknown_options: true}.merge( options ))
       @route = Route.new(verb, pattern, version, prefixed_path: prefixed_path, **options)
     end
 

data/lib/praxis/tasks/api_docs.rb

@@ -2,68 +2,8 @@ namespace :praxis do
 
   namespace :docs do
 
-    def base_path
-      require 'uri'
-      documentation_url = Praxis::ApiDefinition.instance.global_info.documentation_url
-      URI(documentation_url).path.gsub(/\/[^\/]*$/, '/') if documentation_url
-    end
-
-    path = File.expand_path(File.join(File.dirname(__FILE__), '../../api_browser'))
-
-    desc "Install dependencies"
-    task :install do
-      unless system("npm install --production", chdir: path)
-        raise Exception.new("NPM Install Failed")
-      end
-
-      docs_dir = File.join(Dir.pwd, 'docs')
-      FileUtils.mkdir_p docs_dir unless File.directory? docs_dir
-
-      # The doc browser will need to have a minimal app.js and styles.css file at the root
-      # Let's add them if the app has not overriden them
-      js_file = File.join(Dir.pwd, 'docs', 'app.js')
-      scss_file = File.join(Dir.pwd, 'docs', 'styles.scss')
-      template_directory = File.expand_path(File.join(File.dirname(__FILE__), '../../../tasks/thor/templates/generator/empty_app/docs'))
-
-      unless File.exists? js_file
-        FileUtils.cp File.join(template_directory, 'app.js'), js_file
-      end
-      unless File.exists? scss_file
-        FileUtils.cp File.join(template_directory, 'styles.scss'), scss_file
-      end
-    end
-
-    desc "Run API Documentation Browser"
-    task :preview, [:port] => [:install, :generate]  do |t, args|
-      doc_port = args[:port] || '9090'
-      exec({
-        'USER_DOCS_PATH' => File.join(Dir.pwd, 'docs'),
-        'DOC_PORT' => doc_port,
-        'PLUGIN_PATHS' => Praxis::Application.instance.doc_browser_plugin_paths.join(':'),
-        'BASE_PATH' => '/'
-      }, "#{path}/node_modules/.bin/grunt serve --gruntfile '#{path}/Gruntfile.js'")
-    end
-
-    desc "Build docs that can be shipped"
-    task :build => [:install, :generate] do
-      exec({
-        'USER_DOCS_PATH' => File.join(Dir.pwd, 'docs'),
-        'PLUGIN_PATHS' => Praxis::Application.instance.doc_browser_plugin_paths.join(':'),
-        'BASE_PATH' => base_path
-      }, "#{path}/node_modules/.bin/grunt build --gruntfile '#{path}/Gruntfile.js'")
-    end
-
-    desc "Generate API docs (JSON definitions) for a Praxis App"
-    task :generate => [:environment] do |t, args|
-      require 'fileutils'
-
-      Praxis::Blueprint.caching_enabled = false
-      generator = Praxis::Docs::Generator.new(Dir.pwd)
-      generator.save!
-    end
-
     desc "Generate OpenAPI 3 docs for a Praxis App"
-    task :openapi => [:environment] do |t, args|
+    task :generate => [:environment] do |t, args|
       require 'fileutils'
 
       Praxis::Blueprint.caching_enabled = false
@@ -72,7 +12,7 @@ namespace :praxis do
     end
 
     desc "Preview (and Generate) OpenAPI 3 docs for a Praxis App"
-    task :openapipreview => [:openapi] do |t, args|
+    task :preview => [:generate] do |t, args|
       require 'webrick' 
       docs_port = 9090
       root = Dir.pwd + '/docs/openapi/'
@@ -84,6 +24,16 @@ namespace :praxis do
       `open http://localhost:#{docs_port}/`
       wb.join
     end
-
+    desc "Generate and package all OpenApi Docs into a zip, ready for a Web server (like S3...) to present it"
+    task :package => [:generate] do |t, args|
+      docs_root = Dir.pwd + '/docs/openapi/'
+      zip_file = Dir.pwd + '/docs/openapi.zip'
+      `rm -f #{zip_file}`
+      # NOTE: This assumes the "zip" utility is installed, supporting the recursive flag.
+      `zip -r #{zip_file} #{docs_root}`
+      puts
+      puts "Left packaged API docs in #{zip_file}"
+      puts " --> To view the docs, unzip the file under a web server (or S3...) and access the index.hml files from a browser"
+    end
   end
 end