Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ipatests: mark known failures as xfail #2437

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

Commit 5dbcc1a marked the base class method test_replica0_with_ca_kra_dns_install as known failure, but this does not work with inherited classes. The child class methods need to be marked
themselves as known failures with @pytest.mark.xfail.

Furthermore, TestInstallWithCA_KRA2 and TestInstallWithCA_KRA_DNS2 tests should succeed because the master is installed with KRA (issue 7651 is related to replica install with --setup-kra
when it is the first KRA instance).

Related to https://pagure.io/freeipa/issue/7651

Commit 5dbcc1a marked
the base class method test_replica0_with_ca_kra_dns_install
as known failure, but this does not work with inherited
classes. The child class methods need to be marked
themselves as known failures with @pytest.mark.xfail

Furthermore, TestInstallWithCA_KRA2 and TestInstallWithCA_KRA_DNS2
tests should succeed because the master is installed with KRA
(issue 7651 is related to replica install with --setup-kra
when it is the first KRA instance).

Related to https://pagure.io/freeipa/issue/7651
@rcritten rcritten added the ack Pull Request approved, can be merged label Oct 9, 2018
@flo-renaud
Copy link
Contributor Author

Removed the temp commit.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Oct 10, 2018
@flo-renaud
Copy link
Contributor Author

master:

  • f679304 ipatests: mark known failures as xfail

@flo-renaud flo-renaud closed this Oct 10, 2018
@flo-renaud flo-renaud deleted the t7651 branch October 10, 2018 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants