Merge pull request softlayer#688 from sudorandom/issue-660

sudorandom · sudorandom · commit 02e546bd7dce · 2016-02-29T14:23:21.000-06:00
Returns the new record when calling DNSManager.create_record
diff --git a/SoftLayer/managers/dns.py b/SoftLayer/managers/dns.py
@@ -92,7 +92,7 @@ def create_record(self, zone_id, record, record_type, data, ttl=60):
         :param integer ttl: the TTL or time-to-live value (default: 60)
 
         """
-        self.record.createObject({
+        return self.record.createObject({
             'domainId': zone_id,
             'ttl': ttl,
             'host': record,
diff --git a/tests/managers/dns_tests.py b/tests/managers/dns_tests.py
@@ -77,7 +77,8 @@ def test_edit_zone(self):
                                 args=('example.com',))
 
     def test_create_record(self):
-        self.dns_client.create_record(1, 'test', 'TXT', 'testing', ttl=1200)
+        res = self.dns_client.create_record(1, 'test', 'TXT', 'testing',
+                                            ttl=1200)
 
         self.assert_called_with('SoftLayer_Dns_Domain_ResourceRecord',
                                 'createObject',
@@ -88,6 +89,7 @@ def test_create_record(self):
                                     'type': 'TXT',
                                     'data': 'testing'
                                 },))
+        self.assertEqual(res, {'name': 'example.com'})
 
     def test_delete_record(self):
         self.dns_client.delete_record(1)
