Hi! Thanks for the review, should be OK now. I won't change this email's subject, so adding it to the body only and will include it in subject of next patch.
[PATCH 0001] Regards, Peter Lacko ----- Original Message ----- From: "Martin Basti" <[email protected]> To: "Peter Lacko" <[email protected]>, [email protected] Sent: Thursday, April 28, 2016 4:09:17 PM Subject: Re: [Freeipa-devel] [TESTS][PATCH] Ping module tests in a non-declarative way On 08.04.2016 10:32, Peter Lacko wrote: > > Hello, I have a few comments: 1) Please set up your git name and email correctly (consistently for all patches) this is not right From: root<[email protected]> 2) -# Copyright (C) 2012 Red Hat +# Copyright (C) 2016 Red Hat leave there both years please +# Copyright (C) 2012, 2016 Red Hat 3) Please put the patch number to the email subject, it is easier to find correct patch for us Otherwise LGTM and works for me. Martin^2
From 79e29e40349b9ff77c89b8b7297ca1422ac2020b Mon Sep 17 00:00:00 2001 From: Peter Lacko <[email protected]> Date: Wed, 4 May 2016 13:17:57 +0200 Subject: [PATCH] Ping module tests. Test for ping module rewritten using non-declarative way. No new functionality has been added. --- ipatests/test_xmlrpc/test_ping_plugin.py | 49 ++++++++++++++++---------------- 1 file changed, 24 insertions(+), 25 deletions(-) diff --git a/ipatests/test_xmlrpc/test_ping_plugin.py b/ipatests/test_xmlrpc/test_ping_plugin.py index afd34fa1482735ac802b57a5d6cd387b18574f89..8f1ee714340bef0614757d8c7a318f7642563585 100644 --- a/ipatests/test_xmlrpc/test_ping_plugin.py +++ b/ipatests/test_xmlrpc/test_ping_plugin.py @@ -1,7 +1,8 @@ # Authors: # Petr Viktorin <[email protected]> +# Peter Lacko <[email protected]> # -# Copyright (C) 2012 Red Hat +# Copyright (C) 2012, 2016 Red Hat # see file 'COPYING' for use and warranty information # # This program is free software; you can redistribute it and/or modify @@ -21,34 +22,32 @@ Test the `ipalib/plugins/ping.py` module, and XML-RPC in general. """ -from ipalib import errors, _ -from ipatests.util import Fuzzy -from ipatests.test_xmlrpc.xmlrpc_test import Declarative import pytest +from ipalib import errors, _ +from ipatests.test_xmlrpc.tracker.base import Tracker +from ipatests.util import assert_equal, Fuzzy +from xmlrpc_test import XMLRPC_test, raises_exact + @pytest.mark.tier1 -class test_ping(Declarative): +class TestPing(XMLRPC_test): + """Test functionality of the `ipalib/plugins/ping.py` module.""" + tracker = Tracker() - tests = [ - dict( - desc='Ping the server', - command=('ping', [], {}), - expected=dict( - summary=Fuzzy('IPA server version .*. API version .*')), - ), + def test_ping(self): + """Ping the server.""" + result = self.tracker.run_command('ping') + exp = {'summary': Fuzzy('IPA server version .*. API version .*')} + assert_equal(result, exp) - dict( - desc='Try to ping with an argument', - command=('ping', ['bad_arg'], {}), - expected=errors.ZeroArgumentError(name='ping'), - ), + def test_ping_with_argument(self): + """Try to ping with an argument.""" + with raises_exact(errors.ZeroArgumentError(name='ping')): + self.tracker.run_command('ping', ['argument']) - dict( - desc='Try to ping with an option', - command=('ping', [], dict(bad_arg=True)), - expected=errors.OptionError(_('Unknown option: %(option)s'), - option='bad_arg'), - ), - - ] + def test_ping_with_option(self): + """Try to ping with an option.""" + with raises_exact(errors.OptionError( + _('Unknown option: %(option)s'), option='bad_arg')): + self.tracker.run_command('ping', bad_arg=True) -- 2.5.5
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
