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

Remove named-pkcs11 workarounds from DNSSEC tests. #333

Closed

Conversation

pspacek
Copy link
Contributor

@pspacek pspacek commented Dec 13, 2016

As far as I can tell the tests are passing for some time in Jenkins so
maybe a bug in some underlying component was fixed. Let's remove
workarounds to make tests actually test real setups.

https://fedorahosted.org/freeipa/ticket/5348

@MartinBasti
Copy link
Contributor

Please fix the issue reported by pylint

************* Module ipatests.test_integration.test_dnssec
ipatests/test_integration/test_dnssec.py:9: [W0611(unused-import), ] Unused import pytest)

@MartinBasti MartinBasti self-assigned this Dec 14, 2016
As far as I can tell the tests are passing for some time in Jenkins so
maybe a bug in some underlying component was fixed. Let's remove
workarounds to make tests actually test real setups.

https://fedorahosted.org/freeipa/ticket/5348
@pspacek pspacek force-pushed the dnssec-remove-test-workarounds branch from 3c1cf30 to c433291 Compare December 14, 2016 10:51
@MartinBasti MartinBasti added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Dec 16, 2016
@MartinBasti
Copy link
Contributor

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