From 7bc5420503174672a1d5039f60f3b9b067b0af71 Mon Sep 17 00:00:00 2001 From: Aki Tuomi Date: Mon, 6 Jan 2020 20:43:15 +0200 Subject: [PATCH] regression-tests.api: Add future test for nothing under DNAME --- regression-tests.api/test_Zones.py | 39 +++++++++++++++++++++++++++++- 1 file changed, 38 insertions(+), 1 deletion(-) diff --git a/regression-tests.api/test_Zones.py b/regression-tests.api/test_Zones.py index 3b30aecbb5..47b5a719df 100644 --- a/regression-tests.api/test_Zones.py +++ b/regression-tests.api/test_Zones.py @@ -1443,7 +1443,7 @@ $ORIGIN %NAME% 'ttl': 3600, 'records': [ { - "content": "example.org.", + "content": "example.com.", "disabled": False } ] @@ -1454,6 +1454,43 @@ $ORIGIN %NAME% self.assertEquals(r.status_code, 422) self.assertIn('Cannot have both NS and DNAME except in zone apex', r.json()['error']) +## FIXME: Enable this when it's time for it +# def test_rrset_dname_nothing_under(self): +# name, payload, zone = self.create_zone() +# rrset = { +# 'changetype': 'replace', +# 'name': 'delegation.'+name, +# 'type': 'DNAME', +# 'ttl': 3600, +# 'records': [ +# { +# "content": "example.com.", +# "disabled": False +# } +# ] +# } +# payload = {'rrsets': [rrset]} +# r = self.session.patch(self.url("/api/v1/servers/localhost/zones/" + name), data=json.dumps(payload), +# headers={'content-type': 'application/json'}) +# self.assert_success(r) +# rrset = { +# 'changetype': 'replace', +# 'name': 'sub.delegation.'+name, +# 'type': 'A', +# 'ttl': 3600, +# 'records': [ +# { +# "content": "1.2.3.4", +# "disabled": False +# } +# ] +# } +# payload = {'rrsets': [rrset]} +# r = self.session.patch(self.url("/api/v1/servers/localhost/zones/" + name), data=json.dumps(payload), +# headers={'content-type': 'application/json'}) +# self.assertEquals(r.status_code, 422) +# self.assertIn('You cannot have record(s) under CNAME/DNAME', r.json()['error']) + def test_create_zone_with_leading_space(self): # Actual regression. name, payload, zone = self.create_zone() -- 2.47.2