URL: https://github.com/freeipa/freeipa/pull/1354
Author: slaykovsky
 Title: #1354: test_backup_and_restore.py AssertionError fix
Action: opened

PR body:
"""
prefix in the backup function expects output to have
'ipa.ipaserver.install.ipa_backup.Backup:' and it's wrong. The right
one is 'ipaserver.install.ipa_backup:'.
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1354/head:pr1354
git checkout pr1354
From edf110810c8059ca8fdd2d4a71aa3e00a019d467 Mon Sep 17 00:00:00 2001
From: Aleksei Slaikovskii <aslai...@redhat.com>
Date: Mon, 4 Dec 2017 11:56:50 +0100
Subject: [PATCH] test_backup_and_restore.py AssertionError fix

prefix in the backup function expects output to have
'ipa.ipaserver.install.ipa_backup.Backup:' and it's wrong. The right
one is 'ipaserver.install.ipa_backup:'.
---
 ipatests/test_integration/test_backup_and_restore.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ipatests/test_integration/test_backup_and_restore.py b/ipatests/test_integration/test_backup_and_restore.py
index 4912bc2476..cc1a85ffe6 100644
--- a/ipatests/test_integration/test_backup_and_restore.py
+++ b/ipatests/test_integration/test_backup_and_restore.py
@@ -126,7 +126,7 @@ def backup(host):
 
     # Get the backup location from the command's output
     for line in result.stderr_text.splitlines():
-        prefix = ('ipa.ipaserver.install.ipa_backup.Backup: '
+        prefix = ('ipaserver.install.ipa_backup: '
                   'INFO: Backed up to ')
         if line.startswith(prefix):
             backup_path = line[len(prefix):].strip()
@@ -136,7 +136,6 @@ def backup(host):
         raise AssertionError('Backup directory not found in output')
 
 
-
 class TestBackupAndRestore(IntegrationTest):
     topology = 'star'
 
@@ -409,6 +408,7 @@ def test_full_backup_restore_with_vault(self):
         """backup, uninstall, restore"""
         self._full_backup_restore_with_vault(reinstall=False)
 
+
 class TestBackupReinstallRestoreWithKRA(BaseBackupAndRestoreWithKRA):
     def test_full_backup_reinstall_restore_with_vault(self):
         """backup, uninstall, reinstall, restore"""
_______________________________________________
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