URL: https://github.com/freeipa/freeipa/pull/1378
Author: ndehadrai
Title: #1378: test_server_del: Added automation for bugzilla bz1506188.
Action: opened
PR body:
"""
Patch1:
1) Added automation for verification steps for bz1506188.
2) The code setusp up IPA-Master/ Replica and then checks for
deletion of Replica using 'ipa server-del' command.
3) It also checks for verification of 'ipa dnsserver-find' command
output once the replica server details are removed.
Patch2:
1) Removed unsed assert statement inside code.
2) Realigned code for doc-string information within
80 character limit length.
Signed-off-by: Nikhil Dehadrai <ndeha...@redhat.com>
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1378/head:pr1378
git checkout pr1378
From 435f983ad31ad572c2775f22615d2feed10de3b1 Mon Sep 17 00:00:00 2001
From: Nikhil Dehadrai <ndeha...@redhat.com>
Date: Fri, 8 Dec 2017 15:09:08 +0530
Subject: [PATCH] test_server_del: Added automation for bugzilla bz1506188.
Patch1:
1) Added automation for verification steps for bz1506188.
2) The code setusp up IPA-Master/ Replica and then checks for
deletion of Replica using 'ipa server-del' command.
3) It also checks for verification of 'ipa dnsserver-find' command
output once the replica server details are removed.
Patch2:
1) Removed unsed assert statement inside code.
2) Realigned code for doc-string information within
80 character limit length.
Signed-off-by: Nikhil Dehadrai <ndeha...@redhat.com>
---
ipatests/pytest_plugins/integration/tasks.py | 26 ++++++++++++++++++++++++++
ipatests/test_integration/test_server_del.py | 26 ++++++++++++++++++++++++++
2 files changed, 52 insertions(+)
diff --git a/ipatests/pytest_plugins/integration/tasks.py b/ipatests/pytest_plugins/integration/tasks.py
index b407145ace..e29a42c2fe 100644
--- a/ipatests/pytest_plugins/integration/tasks.py
+++ b/ipatests/pytest_plugins/integration/tasks.py
@@ -1381,3 +1381,29 @@ def add_dns_zone(master, zone, skip_overlap_check=False,
host.hostname + ".", '--a-rec', host.ip])
else:
logger.debug('Zone %s already added.', zone)
+
+
+def run_dnsserver_find(host, server_to_find, returncode=None):
+ """
+ Find DNS-server details
+ """
+ kinit_admin(host)
+ cmd = host.run_command(["ipa", "dnsserver-find", server_to_find],
+ raiseonerr=False)
+ if cmd.returncode == returncode:
+ print cmd.stdout_text
+ else:
+ print cmd.stderr_text
+
+
+def run_server_find(host, server_to_find, returncode=None):
+ """
+ Find server details
+ """
+ kinit_admin(host)
+ cmd = host.run_command(["ipa", "server-find", server_to_find],
+ raiseonerr=False)
+ if cmd.returncode == returncode:
+ print cmd.stdout_text
+ else:
+ print cmd.stderr_text
diff --git a/ipatests/test_integration/test_server_del.py b/ipatests/test_integration/test_server_del.py
index f5738a3a59..542c17caea 100644
--- a/ipatests/test_integration/test_server_del.py
+++ b/ipatests/test_integration/test_server_del.py
@@ -300,3 +300,29 @@ def test_forced_removal_of_master(self):
self.replicas[0], self.master.hostname,
ignore_last_of_role=True
)
+
+
+class TestServerDelReplicaRemoval(IntegrationTest):
+ """
+ Test the checks for Removal of Replica during server-del
+ as part for verification of bz1506188
+ """
+ num_replicas = 1
+ domain_level = DOMAIN_LEVEL_1
+ topology = 'line'
+
+ @classmethod
+ def install(cls, mh):
+ tasks.install_topo(
+ cls.topology, cls.master, cls.replicas, [],
+ domain_level=cls.domain_level, setup_replica_cas=False)
+
+ def test_removal_of_replica(self):
+ """
+ test that removal of replica bz1506188.
+ """
+ tasks.run_server_find(self.master, self.replicas[0].hostname, 0)
+ tasks.run_dnsserver_find(self.master, self.replicas[0].hostname, 0)
+ tasks.run_server_del(self.master, self.replicas[0].hostname, 0)
+ tasks.run_server_find(self.master, self.replicas[0].hostname, 1)
+ tasks.run_dnsserver_find(self.master, self.replicas[0].hostname, 1)
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org
