Add an endpoint to update all the values of a DNS record

Author: Kaka Ruto

README.md

@@ -49,6 +49,11 @@ Cloudflare.connect(key: key, email: email) do |connection|
 	# Add a DNS record. Here we add an A record for `batman.example.com`:
 	zone = zones.find_by_name("example.com")
 	zone.dns_records.create('A', 'batman', '1.2.3.4', proxied: false)
+
+	# Update a DNS record. Here we update the A record above to be a CNAME record to 'nairobi.kanairo.com'
+	record = zone.dns_records.find_by_name("example.com") }
+	record.update(type: "CNAME", name: "nairobi", content: "kanairo.com", proxied: true)
+
 
 	# Get firewall rules:
 	all_rules = zone.firewall_rules

lib/cloudflare/dns.rb

@@ -45,6 +45,17 @@ def update_content(content, **options)
 				@value = response.result
 			end
 
+			def update(type: nil, name: nil, content: nil, **options)
+				response = put(
+					type: type || @record[:type],
+					name: name || @record[:name],
+					content: content || @record[:content],
+					**options
+				)
+
+				@value = response.result
+			end
+
 			def type
 				value[:type]
 			end

spec/cloudflare/dns_spec.rb

@@ -12,7 +12,7 @@
 		end
 	end
 
-	context "new record" do
+	describe "#create" do
 		it "can create dns record" do
 			@record = zone.dns_records.create("A", subdomain, "1.2.3.4")
 			expect(@record.type).to be == "A"
@@ -29,8 +29,9 @@
 		end
 	end
 
-	context "with existing record" do
+	describe "#update_content" do
 		let(:record) {@record = zone.dns_records.create("A", subdomain, "1.2.3.4")}
+
 		it "can update dns content" do
 			record.update_content("4.3.2.1")
 			expect(record.content).to be == "4.3.2.1"
@@ -47,4 +48,26 @@
 			expect(fetched_record.proxied).to be_truthy
 		end
 	end
+
+	describe "#update" do
+		let(:subject) { record.update(**new_params)}
+
+		let(:record) { @record = zone.dns_records.create("A", "old", "1.2.3.4", proxied: false) }
+
+		let(:new_params) do
+			{
+				type: "CNAME",
+				name: "new",
+				content: "example.com",
+				proxied: true
+			}
+		end
+
+		it "can update dns record" do
+			expect { subject }.to change { record.name }.to("#{new_params[:name]}.#{zone.name}")
+				.and change { record.type }.to(new_params[:type])
+				.and change { record.content }.to(new_params[:content])
+				.and change { record.proxied }.to(new_params[:proxied])
+		end
+	end
 end