EventEspresso\Codeception\helpers\CountrySettingsAdmin

Complexity

Total Complexity

5

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	amOnCountrySettingsAdminPage()	0	4	1
A	setCurrencyDecimalPlacesTo()	0	4	1
A	setCurrencyDecimalMarkTo()	0	4	1
A	setCurrencyThousandsSeparatorTo()	0	4	1
A	saveCountrySettings()	0	6	1

<?php
namespace EventEspresso\Codeception\helpers;

use Page\CoreAdmin;
use Page\CountrySettingsAdmin as CountrySettings;

trait CountrySettingsAdmin
{
    /**
     * Instructs the actor to browse to the country settings page.
     * Assumes the actor is already logged in.
     * @param string $additional_params
     */
    public function amOnCountrySettingsAdminPage($additional_params = '')
    {
        $this->actor()->amOnAdminPage(CountrySettings::url($additional_params));

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }


    /**
     * Instructs the actor to select the given decimal places radio option.
     * Assumes the actor is already on the country settings page.
     * @param string $decimal_places
     * @param string $country_code
     */
    public function setCurrencyDecimalPlacesTo($decimal_places = '2', $country_code = 'US')
    {
        $this->actor()->click(CountrySettings::currencyDecimalPlacesRadioField($decimal_places, $country_code));

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }


    /**
     * Instructs the actor to select the given decimal mark radio option.
     * Assumes the actor is already on the country settings page.
     * @param string $decimal_mark
     */
    public function setCurrencyDecimalMarkTo($decimal_mark = '.')
    {
        $this->actor()->click(CountrySettings::currencyDecimalMarkRadioField($decimal_mark));

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }


    /**
     * Instructs the actor to select the given thousands separator radio option.
     * Assumes the actor is already on the country settings page.
     * @param string $thousands_seperator
     */
    public function setCurrencyThousandsSeparatorTo($thousands_seperator = ',')
    {
        $this->actor()->click(CountrySettings::currencyThousandsSeparatorField($thousands_seperator));

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }


    /**
     * Clicks the country settings submit button.
     * Assumes the actor is on the country settings admin page.
     */
    public function saveCountrySettings()
    {
        $this->actor()->click(CountrySettings::COUNTRY_SETTINGS_SAVE_BUTTON);

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        //no indicator on the page when stuff has been updated so just give a bit of time for it to finish.
        $this->actor()->wait(5);

It seems like actor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}