safrano 0.4.2 → 0.5.0

data/lib/odata/error.rb

@@ -1,21 +1,31 @@
+# frozen_string_literal: true
+
 require 'json'
 require 'rexml/document'
-require 'safrano.rb'
+require_relative '../safrano/contract'
 
 # Error handling
-module OData
+module Safrano
   # for errors occurring in API (publishing) usage --> Exceptions
   module API
     # when published class is not a Sequel Model
     class ModelNameError < NameError
       def initialize(name)
-        super("class #{name} is not a Sequel Model", name)
+        super("class #{name} is not a Sequel Model")
       end
     end
+
+    # when published complex type is not a Complex Type
+    class ComplexTypeNameError < NameError
+      def initialize(name)
+        super("class #{name} is not a ComplexType")
+      end
+    end
+
     # when published class as media does not have the mandatory media fields
     class MediaModelError < NameError
       def initialize(name)
-        super("Model #{name} does not have the mandatory media attributes content_type/media_src", name)
+        super("Model #{name} does not have the mandatory media attributes content_type/media_src")
       end
     end
 
@@ -27,47 +37,120 @@ module OData
         super(msg, symbname)
       end
     end
+
+    # duplicate attribute name
+    class ModelDuplicateAttributeError < NameError
+      def initialize(klass, symb)
+        symbname = symb.to_s
+        msg = "There is already an attribute :#{symbname} defined in class #{klass}"
+        super(msg, symbname)
+      end
+    end
   end
 
   # base module for HTTP errors, when used as a Error Class
   module ErrorClass
+    include ::Safrano::Contract::Invalid
+    def http_code
+      const_get(:HTTP_CODE)
+    end
+    EMPTYH = {}.freeze
     def odata_get(req)
+      message = (m = @msg.to_s).empty? ? to_s : m
       if req.accept?(APPJSON)
-        [const_get(:HTTP_CODE), CT_JSON,
-         { 'odata.error' => { 'code' => '', 'message' => @msg } }.to_json]
+        # json is default content type so we dont need to specify it here again
+        [self.http_code, EMPTY_HASH,
+         { 'odata.error' => { 'code' => "#{http_code}",
+                              'type' => to_s,
+                              'message' => message } }.to_json]
       else
-        [const_get(:HTTP_CODE), CT_TEXT, @msg]
+        [self.http_code, CT_TEXT, message]
       end
     end
   end
 
   # base module for HTTP errors, when used as an Error instance
   module ErrorInstance
+    include ::Safrano::Contract::Invalid
+    # can(should) be overriden in subclasses
+    def msg
+      @msg
+    end
+
     def odata_get(req)
+      message = (m = msg.to_s).empty? ? self.class.to_s : m
       if req.accept?(APPJSON)
-        [self.class.const_get(:HTTP_CODE), CT_JSON,
-         { 'odata.error' => { 'code' => '', 'message' => @msg } }.to_json]
+        # json is default content type so we dont need to specify it here again
+        [self.class.http_code, EMPTY_HASH,
+         { 'odata.error' => { 'code' => "#{self.class.http_code}",
+                              'type' => "#{self.class}",
+                              'message' => message } }.to_json]
       else
-        [self.class.const_get(:HTTP_CODE), CT_TEXT, @msg]
+        [self.class.http_code, CT_TEXT, message]
       end
     end
   end
 
+  # generic http 500 server err
+  class ServerError
+    extend  ErrorClass
+    HTTP_CODE = 500
+    @msg = 'Server error'
+  end
+
+  # for outputing Sequel exceptions that we could not prevent
+  class SequelExceptionError < ServerError
+    include ErrorInstance
+    def initialize(seqle7n)
+      @msg = seqle7n.message
+    end
+  end
+
+  # for outputing Ruby StandardError exceptions that we could not prevent
+  class RubyStandardErrorException < ServerError
+    include ErrorInstance
+    def initialize(rubye7n)
+      @msg = rubye7n.message
+    end
+  end
+
+  # http Unprocessable Entity for example when trying to
+  # upload duplicated media ressource
+  class UnprocessableEntityError
+    extend  ErrorClass
+    include ErrorInstance
+    HTTP_CODE = 422
+    @msg = 'Unprocessable Entity'
+    def initialize(reason)
+      @msg = reason
+    end
+  end
+
   # http Bad Req.
   class BadRequestError
     extend  ErrorClass
+    include ErrorInstance
     HTTP_CODE = 400
-    @msg = 'Bad Request Error'
+    @msg = 'Bad Request'
+    def initialize(reason)
+      @msg = "Bad Request : #{reason}"
+    end
+  end
+
+  # for upload empty media
+  class BadRequestEmptyMediaUpload < BadRequestError
+    include ErrorInstance
+    def initialize(path)
+      @msg = "Bad Request: empty media file #{path}"
+    end
   end
   # Generic failed changeset
   class BadRequestFailedChangeSet < BadRequestError
-    HTTP_CODE = 400
     @msg = 'Bad Request: Failed changeset '
   end
 
   # $value request for a non-media entity
   class BadRequestNonMediaValue < BadRequestError
-    HTTP_CODE = 400
     @msg = 'Bad Request: $value request for a non-media entity'
   end
   class BadRequestSequelAdapterError < BadRequestError
@@ -79,59 +162,127 @@ module OData
 
   # for Syntax error in Filtering
   class BadRequestFilterParseError < BadRequestError
-    HTTP_CODE = 400
     @msg = 'Bad Request: Syntax error in $filter'
   end
 
-  # for Syntax error in $expand param
-  class BadRequestExpandParseError < BadRequestError
-    HTTP_CODE = 400
-    @msg = 'Bad Request: Syntax error in $expand'
+  # for invalid attribute in path
+  class BadRequestInvalidAttribPath < BadRequestError
+    include ErrorInstance
+    def initialize(model, attr)
+      @msg = "Bad Request: the attribute #{attr} is invalid for entityset #{model.entity_set_name}"
+    end
+  end
+
+  # for invalid attribute in $expand param
+  class BadRequestExpandInvalidPath < BadRequestError
+    include ErrorInstance
+    def initialize(model, path)
+      @msg = "Bad Request: the $expand path #{path} is invalid for entityset #{model.entity_set_name}"
+    end
+  end
+  # for invalid properti(es) in $select param
+  class BadRequestSelectInvalidProps < BadRequestError
+    include ErrorInstance
+    def initialize(model, iprops)
+      @msg = ((iprops.size > 1) ? "Bad Request: the $select properties #{iprops.to_a.join(', ')} are invalid for entityset #{model.entity_set_name}" : "Bad Request: the $select property #{iprops.first} is invalid for entityset #{model.entity_set_name}")
+    end
   end
 
   # for Syntax error in $orderby param
   class BadRequestOrderParseError < BadRequestError
-    HTTP_CODE = 400
     @msg = 'Bad Request: Syntax error in $orderby'
   end
 
   # for $inlinecount error
   class BadRequestInlineCountParamError < BadRequestError
-    HTTP_CODE = 400
     @msg = 'Bad Request: wrong $inlinecount parameter'
   end
+
   # http not found
   class ErrorNotFound
     extend  ErrorClass
     HTTP_CODE = 404
     @msg = 'The requested ressource was not found'
   end
+
+  # http not found segment
+  class ErrorNotFoundSegment < ErrorNotFound
+    include ErrorInstance
+
+    def initialize(segment)
+      @msg = "The requested ressource segment #{segment} was not found"
+    end
+  end
+
   # Transition error (Safrano specific)
-  class ServerTransitionError
-    extend  ErrorClass
-    HTTP_CODE = 500
+  class ServerTransitionError < ServerError
     @msg = 'Server error: Segment could not be parsed'
   end
-  # generic http 500 server err
-  class ServerError
-    extend  ErrorClass
-    HTTP_CODE = 500
-    @msg = 'Server error'
-  end
+
   # not implemented (Safrano specific)
   class NotImplementedError
     extend  ErrorClass
     HTTP_CODE = 501
   end
-  # batch not implemented (Safrano specific)
-  class BatchNotImplementedError
+
+  # version not implemented (Safrano specific)
+  class VersionNotImplementedError
     extend  ErrorClass
     HTTP_CODE = 501
+    @msg = 'The requested OData version is not yet supported'
+  end
+  # batch not implemented (Safrano specific)
+  class BatchNotImplementedError < NotImplementedError
     @msg = 'Not implemented: OData batch'
   end
+
   # error in filter parsing (Safrano specific)
   class FilterParseError < BadRequestError
     extend  ErrorClass
-    HTTP_CODE = 400
+  end
+
+  class FilterFunctionNotImplementedError < BadRequestError
+    extend  ErrorClass
+    include ErrorInstance
+    @msg = 'the requested $filter function is Not implemented'
+    def initialize(xmsg)
+      @msg = xmsg
+    end
+  end
+  class FilterUnknownFunctionError < BadRequestError
+    include ErrorInstance
+    def initialize(badfuncname)
+      @msg = "Bad request: unknown function #{badfuncname} in $filter"
+    end
+  end
+  class FilterParseErrorWrongColumnName < BadRequestError
+    extend ErrorClass
+    @msg = 'Bad request: invalid property name in $filter'
+  end
+  class FilterParseWrappedError < BadRequestError
+    include ErrorInstance
+    def initialize(exception)
+      @msg = exception.to_s
+    end
+  end
+  class ServiceOperationParameterMissing < BadRequestError
+    include ErrorInstance
+    def initialize(missing:, sopname:)
+      @msg = "Bad request: Parameter(s) missing for for service operation #{sopname} : #{missing.join(', ')}"
+    end
+  end
+
+  class ServiceOperationParameterError < BadRequestError
+    include ErrorInstance
+    def initialize(type:, value:, param:, sopname:)
+      @type = type
+      @value = value
+      @param = param
+      @sopname = sopname
+    end
+
+    def msg
+      "Bad request: Parameter #{@param} with value '#{@value}' cannot be converted to type #{@type} for service operation #{@sopname}"
+    end
   end
 end

data/lib/odata/expand.rb

@@ -1,22 +1,24 @@
+# frozen_string_literal: true
+
 require 'odata/error.rb'
 
 # all dataset expanding related classes in our OData module
 # ie  do eager loading
-module OData
+module Safrano
   # base class for expanding
   class ExpandBase
     EmptyExpand = new # re-useable empty expanding (idempotent)
     EMPTYH = {}.freeze
 
-    def self.factory(expandstr)
-      expandstr.nil? ? EmptyExpand : MultiExpand.new(expandstr)
+    def self.factory(expandstr, model)
+      expandstr.nil? ? EmptyExpand : MultiExpand.new(expandstr, model)
     end
 
     # output template
     attr_reader :template
 
     def apply_to_dataset(dtcx)
-      dtcx
+      Contract.valid(dtcx)
     end
 
     def empty?
@@ -24,7 +26,7 @@ module OData
     end
 
     def parse_error?
-      false
+      Contract::OK
     end
 
     def template
@@ -60,7 +62,7 @@ module OData
     end
 
     def apply_to_dataset(dtcx)
-      dtcx
+      Contract.valid(dtcx)
     end
 
     def build_arg
@@ -70,11 +72,6 @@ module OData
       @template = DEEPH_1.call(@nodes)
     end
 
-    def parse_error?
-      # todo
-      false
-    end
-
     def empty?
       false
     end
@@ -85,19 +82,22 @@ module OData
     COMASPLIT = /\s*,\s*/.freeze
     attr_reader :template
 
-    def initialize(expandstr)
+    # Note: if you change this method, please also update arity_full_monkey?
+    #       see below
+    def initialize(expandstr, model)
       expandstr.strip!
+      @model = model
       @expandp = expandstr
-      @exlist = []
 
-      @exlist = expandstr.split(COMASPLIT).map { |exstr| Expand.new(exstr) }
+      @exstrlist = expandstr.split(COMASPLIT)
+      @exlist = @exstrlist.map { |exstr| Expand.new(exstr) }
       build_template
     end
 
     def apply_to_dataset(dtcx)
       # use eager loading for each used association
       @exlist.each { |exp| dtcx = dtcx.eager(exp.arg) }
-      dtcx
+      Contract.valid(dtcx)
     end
 
     def build_template
@@ -112,8 +112,11 @@ module OData
     end
 
     def parse_error?
-      # todo
-      false
+      @exstrlist.each do |expstr|
+        return BadRequestExpandInvalidPath.new(@model, expstr) unless @model.expand_path_valid? expstr
+      end
+
+      Contract::OK
     end
 
     def empty?

data/lib/odata/filter/base.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Safrano
+  module Filter
+    # Base class for Leaves, Trees, RootTrees etc
+    class Node
+      def success(res)
+        Contract.valid(res)
+      end
+    end
+
+    # Leaves are Nodes with a parent but no children
+    class Leave < Node
+    end
+
+    # RootTrees have childrens but no parent
+    class RootTree < Node
+    end
+
+    # Tree's have Parent and children
+    class Tree < RootTree
+    end
+
+    # For functions... should have a single child---> the argument list
+    class FuncTree < Tree
+    end
+
+    # Indentity Func to use as "parent" func of parenthesis expressions
+    # --> allow to handle generically parenthesis always as argument of
+    # some function
+    class IdentityFuncTree < FuncTree
+    end
+
+    # unary op eg. NOT
+    class UnopTree < Tree
+    end
+
+    # Bin ops
+    class BinopTree < Tree
+    end
+
+    class BinopBool < BinopTree
+    end
+
+    class BinopArithm < BinopTree
+    end
+
+    # Arguments or lists
+    class ArgTree < Tree
+    end
+
+    # Numbers (floating point, ints, dec)
+    class FPNumber < Leave
+    end
+
+    # Literals are unquoted words without /
+    class Literal < Leave
+    end
+
+    # Null Literal is unquoted null word
+    class NullLiteral < Literal
+      LEUQES = nil
+    end
+
+    # Qualit (qualified lits) are words separated by /
+    # path/path/path/attrib
+    class Qualit < Literal
+    end
+
+    # Quoted Strings
+    class QString < Leave
+    end
+  end
+end