pytest/dcerpc/dnsserver.py: use dnsserver.flag_from_string
authorDouglas Bagnall <douglas.bagnall@catalyst.net.nz>
Fri, 26 Mar 2021 08:04:20 +0000 (21:04 +1300)
committerDouglas Bagnall <dbagnall@samba.org>
Thu, 8 Apr 2021 21:54:35 +0000 (21:54 +0000)
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andreas Schneider <asn@samba.org>
python/samba/tests/dcerpc/dnsserver.py

index a91ab67df730c4ddf7b82620bc63cc25eb7445ff..5f7aa378575189e99f5e6257a906dd10a84b6b45 100644 (file)
@@ -26,7 +26,7 @@ from samba.samdb import SamDB
 from samba.ndr import ndr_unpack, ndr_pack
 from samba.dcerpc import dnsp, dnsserver, security
 from samba.tests import RpcInterfaceTestCase, env_get_var_value
 from samba.ndr import ndr_unpack, ndr_pack
 from samba.dcerpc import dnsp, dnsserver, security
 from samba.tests import RpcInterfaceTestCase, env_get_var_value
-from samba.dnsserver import record_from_string, ARecord
+from samba.dnsserver import record_from_string, flag_from_string, ARecord
 from samba import sd_utils, descriptor
 from samba import WERRORError, werror
 
 from samba import sd_utils, descriptor
 from samba import WERRORError, werror
 
@@ -178,7 +178,7 @@ class DnsserverTests(RpcInterfaceTestCase):
                                                  self.custom_zone,
                                                  "@",
                                                  None,
                                                  self.custom_zone,
                                                  "@",
                                                  None,
-                                                 self.record_type_int(record_type_str),
+                                                 flag_from_string(record_type_str),
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
@@ -219,7 +219,7 @@ class DnsserverTests(RpcInterfaceTestCase):
                                                  self.custom_zone,
                                                  "@",
                                                  None,
                                                  self.custom_zone,
                                                  "@",
                                                  None,
-                                                 self.record_type_int(record_type_str),
+                                                 flag_from_string(record_type_str),
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
@@ -261,7 +261,7 @@ class DnsserverTests(RpcInterfaceTestCase):
                                                  self.custom_zone,
                                                  "a.b",
                                                  None,
                                                  self.custom_zone,
                                                  "a.b",
                                                  None,
-                                                 self.record_type_int(record_type_str),
+                                                 flag_from_string(record_type_str),
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
@@ -291,7 +291,7 @@ class DnsserverTests(RpcInterfaceTestCase):
                                                  self.custom_zone,
                                                  "a.b",
                                                  None,
                                                  self.custom_zone,
                                                  "a.b",
                                                  None,
-                                                 self.record_type_int(record_type_str),
+                                                 flag_from_string(record_type_str),
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
                                                  dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA,
                                                  None,
                                                  None)
@@ -734,14 +734,11 @@ class DnsserverTests(RpcInterfaceTestCase):
                                             zone,
                                             name,
                                             None,
                                             zone,
                                             name,
                                             None,
-                                            self.record_type_int(record_type_str),
+                                            flag_from_string(record_type_str),
                                             dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA | dnsserver.DNS_RPC_VIEW_NO_CHILDREN,
                                             None,
                                             None)
 
                                             dnsserver.DNS_RPC_VIEW_AUTHORITY_DATA | dnsserver.DNS_RPC_VIEW_NO_CHILDREN,
                                             None,
                                             None)
 
-    def record_type_int(self, s):
-        return getattr(dnsp, 'DNS_TYPE_' + s)
-
     def add_record(self, zone, name, record_type_str, record_str,
                    assertion=True, client_version=dnsserver.DNS_CLIENT_VERSION_LONGHORN):
         """
     def add_record(self, zone, name, record_type_str, record_str,
                    assertion=True, client_version=dnsserver.DNS_CLIENT_VERSION_LONGHORN):
         """