Merge pull request #75 from aetrion/bugfix/type-vs-record-type

Record type should be :type not :record_type
dnsimple · Feb 26, 2016 · c9b231b · c9b231b
2 parents 9fe81b3 + 60e4853
commit c9b231b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/dnsimple/client/zones_records.rb b/lib/dnsimple/client/zones_records.rb
@@ -59,7 +59,7 @@ def all_records(account_id, zone_id, options = {})
       #
       # @raise  [Dnsimple::RequestError]
       def create_record(account_id, zone_id, attributes = {}, options = {})
-        Extra.validate_mandatory_attributes(attributes, [:record_type, :name, :content])
+        Extra.validate_mandatory_attributes(attributes, [:type, :name, :content])
         options  = options.merge(attributes)
         response = client.post(Client.versioned("/%s/zones/%s/records" % [account_id, zone_id]), options)
 

diff --git a/spec/dnsimple/client/zones_records_spec.rb b/spec/dnsimple/client/zones_records_spec.rb
@@ -87,7 +87,7 @@
           .to_return(read_http_fixture("createZoneRecord/created.http"))
     end
 
-    let(:attributes) { { record_type: "A", name: "www", content: "127.0.0.1" } }
+    let(:attributes) { { type: "A", name: "www", content: "127.0.0.1" } }
 
     it "builds the correct request" do
       subject.create_record(account_id, zone_id, attributes)