diff options
author | Bob Campbell <bobcampbell@catalyst.net.nz> | 2016-12-09 09:13:11 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2018-07-12 04:31:58 +0200 |
commit | 59657418107e40d5b95e85dcff164db5bb60e504 (patch) | |
tree | 9d0f4c43dca81f30d9cc0a869d532ba4bb74eec3 | |
parent | aaffc4d1aadc643d8587159e716b0f35ea3413cf (diff) | |
download | samba-59657418107e40d5b95e85dcff164db5bb60e504.tar.gz |
python/tests: check setting values on dnsRecord attributes
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12451
Signed-off-by: Bob Campbell <bobcampbell@catalyst.net.nz>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
-rw-r--r-- | python/samba/tests/dcerpc/dnsserver.py | 34 |
1 files changed, 32 insertions, 2 deletions
diff --git a/python/samba/tests/dcerpc/dnsserver.py b/python/samba/tests/dcerpc/dnsserver.py index 93d4478997c..53e1abde042 100644 --- a/python/samba/tests/dcerpc/dnsserver.py +++ b/python/samba/tests/dcerpc/dnsserver.py @@ -212,7 +212,7 @@ class DnsserverTests(RpcInterfaceTestCase): zone_dn = None for zone in zones: - if zone_name in str(zone.dn): + if "DC=%s," % zone_name in str(zone.dn): zone_dn = zone.dn break @@ -225,7 +225,8 @@ class DnsserverTests(RpcInterfaceTestCase): for old_packed_record in records: if record_name in str(old_packed_record.dn): - return (old_packed_record.dn, ndr_unpack(dnsp.DnssrvRpcRecord, old_packed_record["dnsRecord"][0])) + rec = ndr_unpack(dnsp.DnssrvRpcRecord, old_packed_record["dnsRecord"][0]) + return (old_packed_record.dn, rec) def test_duplicate_matching(self): """ @@ -323,6 +324,35 @@ class DnsserverTests(RpcInterfaceTestCase): self.assert_num_records(self.custom_zone, "testrecord", record_type_str) self.delete_record(self.custom_zone, "testrecord", record_type_str, record_str) + def check_params(self, wDataLength, rank, flags, dwTtlSeconds, dwReserved, data, + wType, dwTimeStamp=0, zone="zone", rec_name="testrecord"): + res = self.get_record_from_db(zone, rec_name) + self.assertIsNotNone(res, "Expected record %s but was not found over LDAP." % data) + (rec_dn, rec) = res + self.assertEqual(wDataLength, rec.wDataLength, "Unexpected data length for record %s. Got %s, expected %s." % (data, rec.wDataLength, wDataLength)) + self.assertEqual(rank, rec.rank, "Unexpected rank for record %s. Got %s, expected %s." % (data, rec.rank, rank)) + self.assertEqual(flags, rec.flags, "Unexpected flags for record %s. Got %s, expected %s." % (data, rec.flags, flags)) + self.assertEqual(dwTtlSeconds, rec.dwTtlSeconds, "Unexpected time to live for record %s. Got %s, expected %s." % (data, rec.dwTtlSeconds, dwTtlSeconds)) + self.assertEqual(dwReserved, rec.dwReserved, "Unexpected dwReserved for record %s. Got %s, expected %s." % (data, rec.dwReserved, dwReserved)) + self.assertEqual(data.lower(), rec.data.lower(), "Unexpected data for record %s. Got %s, expected %s." % (data, rec.data.lower(), data.lower())) + self.assertEqual(wType, rec.wType, "Unexpected wType for record %s. Got %s, expected %s." % (data, rec.wType, wType)) + self.assertEqual(dwTimeStamp, rec.dwTimeStamp, "Unexpected timestamp for record %s. Got %s, expected %s." % (data, rec.dwTimeStamp, dwTimeStamp)) + + def test_record_params(self): + """ + Make sure that, when we add records to the database, + they're added with reasonable parameters. + """ + self.add_record(self.custom_zone, "testrecord", "A", "192.168.50.50") + self.check_params(4, 240, 0, 900, 0, "192.168.50.50", 1) + self.delete_record(self.custom_zone, "testrecord", "A", "192.168.50.50") + self.add_record(self.custom_zone, "testrecord", "AAAA", "AAAA:AAAA::") + self.check_params(16, 240, 0, 900, 0, "AAAA:AAAA:0000:0000:0000:0000:0000:0000", 28) + self.delete_record(self.custom_zone, "testrecord", "AAAA", "AAAA:AAAA::") + self.add_record(self.custom_zone, "testrecord", "CNAME", "cnamedest") + self.check_params(13, 240, 0, 900, 0, "cnamedest", 5) + self.delete_record(self.custom_zone, "testrecord", "CNAME", "cnamedest") + def test_reject_invalid_commands(self): """ Make sure that we can't add a variety of invalid records, |