summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAna Krivokapic <akrivoka@redhat.com>2013-04-08 10:55:47 -0400
committerMartin Kosek <mkosek@redhat.com>2013-04-11 15:58:14 +0200
commitc0cdba78b01317a9ea5c423eda548a69ee046e26 (patch)
tree68ce4f82aab80b8bdf2c0623d13c0e2b10ad554e
parentff52c25ae299abba8bed653fe324951979a41293 (diff)
downloadfreeipa.git-c0cdba78b01317a9ea5c423eda548a69ee046e26.tar.gz
freeipa.git-c0cdba78b01317a9ea5c423eda548a69ee046e26.tar.xz
freeipa.git-c0cdba78b01317a9ea5c423eda548a69ee046e26.zip
Add missing summary message to dnszone_del
https://fedorahosted.org/freeipa/ticket/3503
-rw-r--r--ipalib/plugins/dns.py2
-rw-r--r--tests/test_xmlrpc/test_dns_plugin.py8
2 files changed, 6 insertions, 4 deletions
diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py
index f40360f8..becec142 100644
--- a/ipalib/plugins/dns.py
+++ b/ipalib/plugins/dns.py
@@ -1849,6 +1849,8 @@ api.register(dnszone_add)
class dnszone_del(LDAPDelete):
__doc__ = _('Delete DNS zone (SOA record).')
+ msg_summary = _('Deleted DNS zone "%(value)s"')
+
def post_callback(self, ldap, dn, *keys, **options):
try:
api.Command['permission_del'](self.obj.permission_name(keys[-1]),
diff --git a/tests/test_xmlrpc/test_dns_plugin.py b/tests/test_xmlrpc/test_dns_plugin.py
index fb548bb9..7b14fc67 100644
--- a/tests/test_xmlrpc/test_dns_plugin.py
+++ b/tests/test_xmlrpc/test_dns_plugin.py
@@ -394,7 +394,7 @@ class test_dns(Declarative):
command=('dnszone_del', [revdnszone1], {}),
expected={
'value': revdnszone1,
- 'summary': None,
+ 'summary': u'Deleted DNS zone "%s"' % revdnszone1,
'result': {'failed': u''},
},
),
@@ -905,7 +905,7 @@ class test_dns(Declarative):
command=('dnszone_del', [dnszone2], {}),
expected={
'value': dnszone2,
- 'summary': None,
+ 'summary': u'Deleted DNS zone "%s"' % dnszone2,
'result': {'failed': u''},
},
),
@@ -1397,7 +1397,7 @@ class test_dns(Declarative):
command=('dnszone_del', [dnszone1], {}),
expected={
'value': dnszone1,
- 'summary': None,
+ 'summary': u'Deleted DNS zone "%s"' % dnszone1,
'result': {'failed': u''},
},
),
@@ -1460,7 +1460,7 @@ class test_dns(Declarative):
command=('dnszone_del', [dnszone1], {}),
expected={
'value': dnszone1,
- 'summary': None,
+ 'summary': u'Deleted DNS zone "%s"' % dnszone1,
'result': {'failed': u''},
},
),