Merge pull request #121 from ryanhertz/MWEB-1179-defaultable-find-options

defaultable find options

Author: ryanhertz

CHANGELOG

@@ -1,3 +1,6 @@
+v1.4.5
+ - Fixed problem where options are dropped in Defaultable when `find` is called with 'default'
+
 v1.4.4
   - Fixed symbol hash key bug in Defaultable
 

VERSION

@@ -1 +1 @@
-1.4.4
+1.4.5

lib/spark_api/models/defaultable.rb

@@ -28,7 +28,8 @@ def default(options = {})
 
         def find(*arguments)
           if arguments.first == DEFAULT_ID
-            default
+            options = arguments.slice!(1) || {}
+            default(options)
           else
             original_find(*arguments)
           end

spec/unit/spark_api/models/defaultable_spec.rb

@@ -43,6 +43,11 @@ class TestClass < Base
       TestClass.find(TestClass::DEFAULT_ID)
     end
 
+    it "passes options to 'default'" do
+      expect(TestClass).to receive(:default).with(foo: true)
+      TestClass.find(TestClass::DEFAULT_ID, foo: true)
+    end
+
     it "calls Finders.find when given a normal id" do
       connection = double
       expect(connection).to receive(:get).with("/testclass/5", {foo: true}).and_return([{}])