657fc76 Fix test_ntp_options to use tasks' methods

Authored and Committed by tdudlak 5 years ago
    Fix test_ntp_options to use tasks' methods
    
    Use methods:
    - tasks.replica_install()
    - tasks.client_install()
    instead of custom methods.
    Move ntp_pool/server to class scope.
    Using teardown_method for cleanup.
    Edit tasks.client_install to return result of installation.
    Refactor install_replica task:
    Add promote parameter to install_replica task.
    Add ntp_args to install_client call and remove from
    replica installation from tasks.install_replica while promoting.
    Use case while not promoting has to have user allowed to enroll
    a replica and server to contact in case autodiscovery does not work.
    
    Related: https://pagure.io/freeipa/issue/7719
    Reviewed-By: Christian Heimes <cheimes@redhat.com>
    Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
    Reviewed-By: Sergey Orlov <sorlov@redhat.com>
    Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
    Reviewed-By: Armando Neto <abiagion@redhat.com>