fmrest-spyke 0.22.0 → 0.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9f606d5d9365cdd3e3dfdbd9eca86f03f95c950ad793d695b692c3177c2e695e
-  data.tar.gz: 17b268ee1f2ed93de74f624e90044dd60407c56c47b05b91cba1e5f634aaf835
+  metadata.gz: f9956beb060ba224625cbc91c6d15169fcc24002d8868c8736fa8ee474d64afc
+  data.tar.gz: 9991f5c849f766a9a0660c3072f2ac6cb677d311888a9d536ecbe0c0c18dfeb5
 SHA512:
-  metadata.gz: e74e883a25912e28813e60a18ec74f0ea51e2ec0e160706dec7a8c056b5a3e98d6003b01b9cafe86495e19a4d81a9d5ea45ff842a58858ab150eec286dfbb4af
-  data.tar.gz: babc9b457c7260fb4332aeeee2ba8b442161ac26a42d840c1410ba4ce0676e595ab7ca2de82c51eb8df6c5276b3c828cf364356b2f9b1aeed2a3d4320986635a
+  metadata.gz: b11f7d8f9e582185534aec7ba8497d2f74a371a8bc5322e4f48849ec8fe84f841bca705cf0cfb884b8f96298c677883ccaaf98e5816a7923b414a4dd6f815bc3
+  data.tar.gz: 41063c5e3993b525131ab84c27eda309eadfec7fe5ba4c0458605221a707e1e60ce87e3ba7f6737476e7cdb2b735fe686aeba68064b66e130b905a5d6550e3d3

data/.yardopts

@@ -1,5 +1,6 @@
 --markup markdown
 --plugin activesupport-concern
-lib/**/*.rb
+--exclude lib/generators/*
 -
+lib/**/*.rb
 docs/*

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 ## Changelog
 
+### 0.23.0
+
+* Add `find_one!` (aliased as `first!`) exception-raising method
+* Add mapping of API exceptions to HTTP responses in Rails
+
+### 0.22.0 
+
+* Add `fmrest-rails` gem with Rails integration (initializer, generators)
+
 ### 0.21.0
 
 * Support for Spyke 7 and Faraday 2

data/lib/fmrest/spyke/model/orm.rb

@@ -18,10 +18,6 @@ module FmRest
           class_attribute :default_limit, instance_accessor: false, instance_predicate: false
 
           class_attribute :default_sort, instance_accessor: false, instance_predicate: false
-
-          # Whether to raise an FmRest::APIError::NoMatchingRecordsError when a
-          # _find request has no results
-          class_attribute :raise_on_no_matching_records, instance_accessor: false, instance_predicate: false
         end
 
         class_methods do
@@ -42,10 +38,14 @@ module FmRest
           # options, as well as using the appropriate HTTP method/URL depending
           # on whether there's a query present in the current scope.
           #
+          # @option options [Boolean] :raise_on_no_matching_records whether to
+          #   raise `APIError::NoMatchingRecordsError` when no records match (FM
+          #   error 401). If not given it returns an empty resultset.
+          #
           # @example
           #   Person.query(first_name: "Stefan").fetch # -> POST .../_find
           #
-          def fetch
+          def fetch(options = {})
             if current_scope.has_query?
               scope = extend_scope_with_fm_params(current_scope, prefixed: false)
               scope = scope.where(query: scope.query_params)
@@ -60,9 +60,9 @@ module FmRest
             # nothing matches a _find request, so we need to catch it in order
             # to provide sane behavior (i.e. return an empty resultset)
             begin
-              current_scope.has_query? ? scoped_request(:post) : super
+              current_scope.has_query? ? scoped_request(:post) : super()
             rescue FmRest::APIError::NoMatchingRecordsError => e
-              raise e if raise_on_no_matching_records
+              raise e if options[:raise_on_no_matching_records]
               ::Spyke::Result.new({})
             end
           ensure

data/lib/fmrest/spyke/relation.rb

@@ -276,13 +276,15 @@ module FmRest
       # Finds a single instance of the model by forcing limit = 1, or simply
       # fetching the record by id if the primary key was set
       #
+      # @option (see FmRest::Spyke::Model::ORM.fetch)
+      #
       # @return [FmRest::Spyke::Base]
-      def find_one
+      def find_one(options = {})
         @find_one ||=
           if primary_key_set?
-            without_collection_params { super }
+            without_collection_params { super() }
           else
-            klass.new_collection_from_result(limit(1).fetch).first
+            klass.new_collection_from_result(limit(1).fetch(options)).first
           end
       rescue ::Spyke::ConnectionError => error
         fallback_or_reraise(error, default: nil)
@@ -290,6 +292,18 @@ module FmRest
       alias_method :first, :find_one
       alias_method :any, :find_one
 
+      # Same as `#find_one`, but raises `APIError::NoMatchingRecordsError` when
+      # no records match.
+      # Equivalent to calling `find_one(raise_on_no_matching_records: true)`.
+      #
+      # @option (see FmRest::Spyke::Model::ORM.fetch)
+      #
+      # @return [FmRest::Spyke::Base]
+      def find_one!(options = {})
+        find_one(options.merge(raise_on_no_matching_records: true))
+      end
+      alias_method :first!, :find_one!
+
       # Yields each batch of records that was found by the find options.
       #
       # NOTE: By its nature, batch processing is subject to race conditions if

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fmrest-spyke
 version: !ruby/object:Gem::Version
-  version: 0.22.0
+  version: 0.23.0
 platform: ruby
 authors:
 - Pedro Carbajal
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
 - !ruby/object:Gem::Dependency
   name: spyke
   requirement: !ruby/object:Gem::Requirement