diff options
| author | Tomas Babej <tbabej@redhat.com> | 2014-01-20 09:28:26 +0100 |
|---|---|---|
| committer | Petr Viktorin <pviktori@redhat.com> | 2014-01-20 15:35:43 +0100 |
| commit | cfaaeb9dadbb30cea0b1306f065c485d2dd82974 (patch) | |
| tree | de0c64904ac6a7f942535045c686529ab8f032ea /ipatests | |
| parent | dc1a1189e18c5d579570506bdf839f18aa94db6b (diff) | |
| download | freeipa-cfaaeb9dadbb30cea0b1306f065c485d2dd82974.tar.gz freeipa-cfaaeb9dadbb30cea0b1306f065c485d2dd82974.tar.xz freeipa-cfaaeb9dadbb30cea0b1306f065c485d2dd82974.zip | |
ipatests: Check for legacy_client attribute presence if unapplying fixes
When legacy client tests fail during IPA installation, the legacy
client test produces an additional misleading error
(the real cause is reported as well). This happens due the fact
that we try to cleanup host that was not yet defined. We need to
check for this attribute being defined before unapplying fixes there.
https://fedorahosted.org/freeipa/ticket/4124
Diffstat (limited to 'ipatests')
| -rw-r--r-- | ipatests/test_integration/test_legacy_clients.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/ipatests/test_integration/test_legacy_clients.py b/ipatests/test_integration/test_legacy_clients.py index 72b7ff492..6bbe54b32 100644 --- a/ipatests/test_integration/test_legacy_clients.py +++ b/ipatests/test_integration/test_legacy_clients.py @@ -233,7 +233,11 @@ class BaseTestLegacyClient(trust_tests.TestEnforcedPosixADTrust): def uninstall(cls): cls.master.run_command(['ipa', 'user-del', 'disabledipauser'], raiseonerr=False) - tasks.unapply_fixes(cls.legacy_client) + + # Also unapply fixes on the legacy client, if defined + if hasattr(cls, 'legacy_client'): + tasks.unapply_fixes(cls.legacy_client) + super(BaseTestLegacyClient, cls).uninstall() |