diff options
| author | Marc Alexander <admin@m-a-styles.de> | 2019-09-07 13:24:20 +0200 |
|---|---|---|
| committer | Marc Alexander <admin@m-a-styles.de> | 2019-09-07 21:59:01 +0200 |
| commit | cf898133a4fa0c412911a9b65a533c84524edc34 (patch) | |
| tree | 8acf3bff9dca1d50e8bd3d75d2c47f2d91c4fc69 /tests/functions | |
| parent | 56e2f1a3f66602efa2977e5c2abe31e884e56bf6 (diff) | |
| download | forums-cf898133a4fa0c412911a9b65a533c84524edc34.tar forums-cf898133a4fa0c412911a9b65a533c84524edc34.tar.gz forums-cf898133a4fa0c412911a9b65a533c84524edc34.tar.bz2 forums-cf898133a4fa0c412911a9b65a533c84524edc34.tar.xz forums-cf898133a4fa0c412911a9b65a533c84524edc34.zip | |
[ticket/16123] Add tests to cover different ban reasons
PHPBB3-16123
Diffstat (limited to 'tests/functions')
| -rw-r--r-- | tests/functions/fixtures/validate_email.xml | 21 | ||||
| -rw-r--r-- | tests/functions/validate_user_email_test.php | 10 |
2 files changed, 28 insertions, 3 deletions
diff --git a/tests/functions/fixtures/validate_email.xml b/tests/functions/fixtures/validate_email.xml index eb4fd90217..5a21e51d13 100644 --- a/tests/functions/fixtures/validate_email.xml +++ b/tests/functions/fixtures/validate_email.xml @@ -1,5 +1,26 @@ <?xml version="1.0" encoding="UTF-8" ?> <dataset> + <table name="phpbb_banlist"> + <column>ban_userid</column> + <column>ban_exclude</column> + <column>ban_end</column> + <column>ban_email</column> + <column>ban_give_reason</column> + <row> + <value>0</value> + <value>0</value> + <value>0</value> + <value>banned@example.com</value> + <value></value> + </row> + <row> + <value>0</value> + <value>0</value> + <value>0</value> + <value>banned2@example.com</value> + <value>just because</value> + </row> + </table> <table name="phpbb_users"> <column>user_id</column> <column>username</column> diff --git a/tests/functions/validate_user_email_test.php b/tests/functions/validate_user_email_test.php index 8dcec88103..f64d01517c 100644 --- a/tests/functions/validate_user_email_test.php +++ b/tests/functions/validate_user_email_test.php @@ -28,10 +28,14 @@ class phpbb_functions_validate_user_email_test extends phpbb_database_test_case protected function setUp() { + global $phpbb_dispatcher, $phpbb_root_path, $phpEx; + parent::setUp(); $this->db = $this->new_dbal(); - $this->user = new phpbb_mock_user; + $phpbb_dispatcher = new phpbb_mock_event_dispatcher(); + $language = new phpbb\language\language(new phpbb\language\language_file_loader($phpbb_root_path, $phpEx)); + $this->user = new phpbb\user($language, '\phpbb\datetime'); $this->helper = new phpbb_functions_validate_data_helper($this); } @@ -47,7 +51,6 @@ class phpbb_functions_validate_user_email_test extends phpbb_database_test_case $config['email_check_mx'] = $check_mx; $db = $this->db; $user = $this->user; - $user->optionset('banned_users', array('banned@example.com')); } public static function validate_user_email_data() @@ -58,7 +61,8 @@ class phpbb_functions_validate_user_email_test extends phpbb_database_test_case array('valid_complex', array(), "'%$~test@example.com"), array('invalid', array('EMAIL_INVALID'), 'fööbar@example.com'), array('taken', array('EMAIL_TAKEN'), 'admin@example.com'), - array('banned', array('EMAIL_BANNED'), 'banned@example.com'), + array('banned', ['just because'], 'banned2@example.com'), + array('banned', ['EMAIL_BANNED'], 'banned@example.com') ); } |