[ticket/11538] Use abstract class for functional test cases for group colour

PHPBB3-11538

1 files changed, 5 insertions, 31 deletions

diff --git a/tests/functional/ucp_groups_test.php b/tests/functional/ucp_groups_test.php
index ae568e8182..8401cfdb09 100644
--- a/tests/functional/ucp_groups_test.php
+++ b/tests/functional/ucp_groups_test.php
@@ -7,41 +7,15 @@
 *
 */
 
+require_once dirname(__FILE__) . '/common_groups_test.php';
+
 /**
 * @group functional
 */
-class phpbb_functional_ucp_groups_test extends phpbb_functional_test_case
+class phpbb_functional_ucp_groups_test extends phpbb_functional_common_groups_test
 {
-	public function groups_manage_test_data()
+	protected function get_url()
 	{
-		return array(
-			array('#AA0000', 'WRONG_DATA_COLOUR'),
-			array('AA0000', 'GROUP_UPDATED'),
-			array('AA0000v', 'WRONG_DATA_COLOUR'),
-			array('vAA0000', 'WRONG_DATA_COLOUR'),
-			array('AAG000','WRONG_DATA_COLOUR'),
-			array('a00', 'GROUP_UPDATED'),
-			array('ag0', 'WRONG_DATA_COLOUR'),
-			array('#aa0', 'WRONG_DATA_COLOUR'),
-			array('AA0000 ', 'GROUP_UPDATED'),
-			array('AA0000 abf', 'WRONG_DATA_COLOUR'),
-			array('AA0000 AA0000', 'WRONG_DATA_COLOUR'),
-		);
-	}
-
-	/**
-	* @dataProvider groups_manage_test_data
-	*/
-	public function test_groups_manage($input, $expected)
-	{
-		$this->login();
-		$this->add_lang(array('ucp', 'acp/groups'));
-
-		$crawler = $this->request('GET', 'ucp.php?i=groups&mode=manage&action=edit&g=5&sid=' . $this->sid);
-		$this->assert_response_success();
-		$form = $crawler->selectButton($this->lang('SUBMIT'))->form();
-		$form['group_colour']->setValue($input);
-		$crawler = $this->client->submit($form);
-		$this->assertContains($this->lang($expected), $crawler->text());
+		return 'ucp.php?i=groups&mode=manage&action=edit&g=5';
 	}
 }