URL: https://github.com/freeipa/freeipa/pull/1809
Author: menonsudhir
 Title: #1809: Fixes for test_server_del failures.
Action: opened

PR body:
"""
Fixes for test_server_del failures.
Modified assert text for the failures seen in TC

Signed-off-by: Sudhir Menon <sume...@redhat.com>
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1809/head:pr1809
git checkout pr1809
From 1095f83850d8fed20866c1ff45a595cf79518408 Mon Sep 17 00:00:00 2001
From: Sudhir Menon <sume...@redhat.com>
Date: Thu, 12 Apr 2018 18:34:40 +0530
Subject: [PATCH] Fixes for test_server_del failures.

Modified assert text for the failures seen in TC

Signed-off-by: Sudhir Menon <sume...@redhat.com>
---
 ipatests/test_integration/test_server_del.py | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/ipatests/test_integration/test_server_del.py b/ipatests/test_integration/test_server_del.py
index f5738a3a59..e520c6547f 100644
--- a/ipatests/test_integration/test_server_del.py
+++ b/ipatests/test_integration/test_server_del.py
@@ -32,6 +32,8 @@ def check_master_removal(host, hostname_to_remove,
         assert "Ignoring topology connectivity errors." in result.stderr_text
 
     if ignore_last_of_role:
+        assert ("Deleting this server will leave your installation without a DNS." in
+                result.stderr_text)
         assert ("Ignoring these warnings and proceeding with removal" in
                 result.stderr_text)
 
@@ -247,8 +249,8 @@ def test_removal_of_master_raises_error_about_last_ca(self):
         """
         tasks.assert_error(
             tasks.run_server_del(self.replicas[0], self.master.hostname),
-            "Deleting this server is not allowed as it would leave your "
-            "installation without a CA.",
+            "Deleting this server will leave your installation "
+            "without a DNS.",
             1
         )
 
@@ -285,9 +287,8 @@ def test_install_dns_on_replica1_and_dnssec_on_master(self):
     def test_removal_of_master_raises_error_about_dnssec(self):
         tasks.assert_error(
             tasks.run_server_del(self.replicas[0], self.master.hostname),
-            "Replica is active DNSSEC key master. Uninstall "
-            "could break your DNS system. Please disable or replace "
-            "DNSSEC key master first.",
+            "Deleting this server will leave your installation "
+            "without a DNS.",
             1
         )
 
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org

Reply via email to