On Tue, 2012-03-27 at 19:02 +0200, Petr Viktorin wrote:
> Updated with a regression test for 
> https://fedorahosted.org/freeipa/ticket/2581 (Unable to delete all sshfp 
> records interactively) which Martin fixed recently.

Thank's for adding a specific test for this I see 2 issues with this

1) DNS tests are not skipped when DNS is not configured:

$ ./make-test tests/test_cmdline/test_cli.py 
test_cli.TestCLIParsing.test_dnsrecord_add ... ok
test_cli.TestCLIParsing.test_dnsrecord_del_all ... ERROR
test_cli.TestCLIParsing.test_dnsrecord_del_one_by_one ... ERROR
test_cli.TestCLIParsing.test_group_add ... ok

We should rather skip these test as we do in test_dns_plugin.py instead
of failing with ERROR.

2) pprint is not needed, its output is not shown anyway as stdout is
+import pprint
+        pprint.pprint(kw_got)


Freeipa-devel mailing list

Reply via email to