From de940802bb6631fbbc97afd11869d87cba18f47f Mon Sep 17 00:00:00 2001 From: Florence Blanc-Renaud Date: Wed, 17 Jul 2024 18:32:37 +0200 Subject: [PATCH] ipatests: remove xfail for test_ipa_migrate_version_option The test test_ipa_ipa_migration.py::TestIPAMigrateScenario1:: test_ipa_migrate_version_option is now passing, issue has been fixed. The -V option has been removed. Related: https://pagure.io/freeipa/issue/9620 Signed-off-by: Florence Blanc-Renaud Reviewed-By: Sudhir Menon --- ipatests/test_integration/test_ipa_ipa_migration.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/ipatests/test_integration/test_ipa_ipa_migration.py b/ipatests/test_integration/test_ipa_ipa_migration.py index 7e2d4a342..9aa8a9f32 100644 --- a/ipatests/test_integration/test_ipa_ipa_migration.py +++ b/ipatests/test_integration/test_ipa_ipa_migration.py @@ -846,20 +846,18 @@ class TestIPAMigrateScenario1(IntegrationTest): assert DNS_LOG2 in install_msg assert DNS_LOG3 in install_msg - @pytest.mark.xfail(reason="https://issues.redhat.com/browse/RHEL-46003", - strict=True) def test_ipa_migrate_version_option(self): """ - This testcase checks the version of - the ipa-migrate tool using -v option + The -V option has been removed. """ CONSOLE_LOG = ( "ipa-migrate: error: the following arguments are " "required: mode, hostname" ) - result = self.master.run_command(["ipa-migrate", "-V"]) - assert result.returncode == 0 - assert CONSOLE_LOG not in result.stderr_text + result = self.master.run_command(["ipa-migrate", "-V"], + raiseonerr=False) + assert result.returncode == 2 + assert CONSOLE_LOG in result.stderr_text def test_ipa_migrate_with_log_file_option(self): """