diff --git a/tests/php/Forms/EmailFieldTest.php b/tests/php/Forms/EmailFieldTest.php index 9eeaa4915..85733d8a6 100644 --- a/tests/php/Forms/EmailFieldTest.php +++ b/tests/php/Forms/EmailFieldTest.php @@ -67,23 +67,16 @@ class EmailFieldTest extends FunctionalTest * * @see SimpleTagBuilder::_createInputTag() */ - function testEmailFieldPopulation() + public function testEmailFieldPopulation() { - $this->get('EmailFieldTest_Controller'); - $this->submitForm( + + $response = $this->submitForm( 'Form_Form', null, - array( - 'Email' => 'test@test.com' - ) + ['Email' => 'test@test.com'] ); - $this->assertPartialMatchBySelector( - 'p.good', - array( - 'Test save was successful' - ) - ); + $this->assertContains('Test save was successful', $response->getBody()); } } diff --git a/tests/php/Forms/FormTest.php b/tests/php/Forms/FormTest.php index ab7a91367..ae7e1d7bd 100644 --- a/tests/php/Forms/FormTest.php +++ b/tests/php/Forms/FormTest.php @@ -2,33 +2,31 @@ namespace SilverStripe\Forms\Tests; +use SilverStripe\Control\Controller; +use SilverStripe\Control\HTTPRequest; use SilverStripe\Control\Session; -use SilverStripe\Core\Config\Config; +use SilverStripe\Dev\CSSContentParser; +use SilverStripe\Dev\FunctionalTest; +use SilverStripe\Forms\DateField; +use SilverStripe\Forms\FieldList; +use SilverStripe\Forms\FileField; +use SilverStripe\Forms\Form; +use SilverStripe\Forms\FormAction; +use SilverStripe\Forms\HeaderField; +use SilverStripe\Forms\LookupField; +use SilverStripe\Forms\NumericField; use SilverStripe\Forms\PasswordField; -use SilverStripe\Forms\Tests\FormTest\TestController; use SilverStripe\Forms\Tests\FormTest\ControllerWithSecurityToken; use SilverStripe\Forms\Tests\FormTest\ControllerWithStrictPostCheck; use SilverStripe\Forms\Tests\FormTest\Player; use SilverStripe\Forms\Tests\FormTest\Team; +use SilverStripe\Forms\Tests\FormTest\TestController; +use SilverStripe\Forms\TextareaField; +use SilverStripe\Forms\TextField; use SilverStripe\ORM\ValidationResult; use SilverStripe\Security\NullSecurityToken; -use SilverStripe\Security\Security; -use SilverStripe\Security\SecurityToken; use SilverStripe\Security\RandomGenerator; -use SilverStripe\Dev\CSSContentParser; -use SilverStripe\Dev\FunctionalTest; -use SilverStripe\Control\Controller; -use SilverStripe\Control\HTTPRequest; -use SilverStripe\Forms\TextField; -use SilverStripe\Forms\FieldList; -use SilverStripe\Forms\Form; -use SilverStripe\Forms\HeaderField; -use SilverStripe\Forms\TextareaField; -use SilverStripe\Forms\DateField; -use SilverStripe\Forms\NumericField; -use SilverStripe\Forms\LookupField; -use SilverStripe\Forms\FileField; -use SilverStripe\Forms\FormAction; +use SilverStripe\Security\SecurityToken; use SilverStripe\View\SSViewer; /** @@ -50,6 +48,8 @@ class FormTest extends FunctionalTest ControllerWithStrictPostCheck::class, ]; + protected static $disable_themes = true; + protected function setUp() { parent::setUp(); diff --git a/tests/php/Forms/GridField/GridFieldDetailFormTest.php b/tests/php/Forms/GridField/GridFieldDetailFormTest.php index 81d425867..9bc0226d8 100644 --- a/tests/php/Forms/GridField/GridFieldDetailFormTest.php +++ b/tests/php/Forms/GridField/GridFieldDetailFormTest.php @@ -2,20 +2,19 @@ namespace SilverStripe\Forms\Tests\GridField; -use SilverStripe\Dev\CSSContentParser; -use SilverStripe\Dev\Debug; -use SilverStripe\Dev\FunctionalTest; use SilverStripe\Control\Controller; -use SilverStripe\Forms\HiddenField; -use SilverStripe\Forms\GridField\GridFieldDetailForm; +use SilverStripe\Dev\CSSContentParser; +use SilverStripe\Dev\FunctionalTest; use SilverStripe\Forms\GridField\GridField; +use SilverStripe\Forms\GridField\GridFieldDetailForm; use SilverStripe\Forms\GridField\GridFieldDetailForm_ItemRequest; +use SilverStripe\Forms\HiddenField; use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\Category; use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\CategoryController; -use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\TestController; use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\GroupController; use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\PeopleGroup; use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\Person; +use SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\TestController; /** * @skipUpgrade @@ -36,6 +35,8 @@ class GridFieldDetailFormTest extends FunctionalTest GroupController::class, ]; + protected static $disable_themes = true; + public function testValidator() { $this->logInWithPermission('ADMIN'); diff --git a/tests/php/Forms/LookupFieldTest.php b/tests/php/Forms/LookupFieldTest.php index c41b1b141..1f53a8234 100644 --- a/tests/php/Forms/LookupFieldTest.php +++ b/tests/php/Forms/LookupFieldTest.php @@ -5,90 +5,85 @@ namespace SilverStripe\Forms\Tests; use SilverStripe\ORM\DataObject; use SilverStripe\Dev\SapphireTest; use SilverStripe\Forms\LookupField; +use SilverStripe\Security\Member; class LookupFieldTest extends SapphireTest { - protected static $fixture_file = 'LookupFieldTest.yml'; public function testNullValueWithNumericArraySource() { $source = array(1 => 'one', 2 => 'two', 3 => 'three'); - $f = new LookupField('test', 'test', $source); - $f->setValue(null); + $field = new LookupField('test', 'test', $source); + $field->setValue(null); + $result = trim($field->Field()->getValue()); - $this->assertEquals( - '(none)', - trim($f->Field()->getValue()) - ); + $this->assertContains('(none)', $result); + $this->assertContains('', $result); } public function testStringValueWithNumericArraySource() { $source = array(1 => 'one', 2 => 'two', 3 => 'three'); - $f = new LookupField('test', 'test', $source); - $f->setValue(1); - $this->assertEquals( - 'one', - trim($f->Field()->getValue()) - ); + $field = new LookupField('test', 'test', $source); + $field->setValue(1); + $result = trim($field->Field()->getValue()); + $this->assertContains('one', $result); + $this->assertContains('', $result); } public function testUnknownStringValueWithNumericArraySource() { $source = array(1 => 'one', 2 => 'two', 3 => 'three'); - $f = new LookupField('test', 'test', $source); - $f->setValue('w00t'); - $this->assertEquals( - 'w00t', - trim($f->Field()->getValue()) - ); + $field = new LookupField('test', 'test', $source); + $field->setValue('w00t'); + $result = trim($field->Field()->getValue()); + + $this->assertContains('w00t', $result); + $this->assertContains('', $result); } public function testArrayValueWithAssociativeArraySource() { // Array values (= multiple selections) might be set e.g. from ListboxField $source = array('one' => 'one val', 'two' => 'two val', 'three' => 'three val'); - $f = new LookupField('test', 'test', $source); - $f->setValue(array('one','two')); - $this->assertEquals( - 'one val, two val' - . '', - trim($f->Field()->getValue()) - ); + $field = new LookupField('test', 'test', $source); + $field->setValue(array('one','two')); + $result = trim($field->Field()->getValue()); + + $this->assertContains('one val, two val', $result); + $this->assertContains('', $result); } public function testArrayValueWithNumericArraySource() { // Array values (= multiple selections) might be set e.g. from ListboxField $source = array(1 => 'one', 2 => 'two', 3 => 'three'); - $f = new LookupField('test', 'test', $source); - $f->setValue(array(1,2)); - $this->assertEquals( - 'one, two', - trim($f->Field()->getValue()) - ); + $field = new LookupField('test', 'test', $source); + $field->setValue(array(1,2)); + $result = trim($field->Field()->getValue()); + + $this->assertContains('one, two', $result); + $this->assertContains('', $result); } public function testArrayValueWithSqlMapSource() { - $member1 = $this->objFromFixture('SilverStripe\\Security\\Member', 'member1'); - $member2 = $this->objFromFixture('SilverStripe\\Security\\Member', 'member2'); - $member3 = $this->objFromFixture('SilverStripe\\Security\\Member', 'member3'); + $member1 = $this->objFromFixture(Member::class, 'member1'); + $member2 = $this->objFromFixture(Member::class, 'member2'); + $member3 = $this->objFromFixture(Member::class, 'member3'); - $source = DataObject::get('SilverStripe\\Security\\Member'); - $f = new LookupField('test', 'test', $source->map('ID', 'FirstName')); - $f->setValue(array($member1->ID, $member2->ID)); + $source = DataObject::get(Member::class); + $field = new LookupField('test', 'test', $source->map('ID', 'FirstName')); + $field->setValue(array($member1->ID, $member2->ID)); + $result = trim($field->Field()->getValue()); - $this->assertEquals( - sprintf( - 'member1, member2' - . '', - $member1->ID, - $member2->ID - ), - trim($f->Field()->getValue()) - ); + $this->assertContains('member1, member2', $result); + $this->assertContains(sprintf( + '', + $member1->ID, + $member2->ID + ), $result); } public function testWithMultiDimensionalSource() @@ -105,23 +100,17 @@ class LookupFieldTest extends SapphireTest ) ); - $f = new LookupField('test', 'test', $choices); - $f->setValue(3); + $field = new LookupField('test', 'test', $choices); + $field->setValue(3); + $result = trim($field->Field()->getValue()); - $this->assertEquals( - 'Carrots', - trim($f->Field()->getValue()) - ); + $this->assertContains('Carrots', $result); + $this->assertContains('', $result); - $f->setValue( - array( - 3, 9 - ) - ); + $field->setValue([3, 9]); + $result = trim($field->Field()->getValue()); - $this->assertEquals( - 'Carrots, Vegan', - trim($f->Field()->getValue()) - ); + $this->assertContains('Carrots, Vegan', $result); + $this->assertContains('', $result); } } diff --git a/tests/php/Forms/SelectionGroupTest.php b/tests/php/Forms/SelectionGroupTest.php index 3c1d3e441..82ba88520 100644 --- a/tests/php/Forms/SelectionGroupTest.php +++ b/tests/php/Forms/SelectionGroupTest.php @@ -58,6 +58,7 @@ class SelectionGroupTest extends SapphireTest $field->setValue('two'); $parser = new CSSContentParser($field->FieldHolder()); + $listEls = $parser->getBySelector('li'); $listElOne = $listEls[0]; $listElTwo = $listEls[1]; diff --git a/tests/php/Forms/TreeDropdownFieldTest.php b/tests/php/Forms/TreeDropdownFieldTest.php index dfc49bbce..5b5e22fd1 100644 --- a/tests/php/Forms/TreeDropdownFieldTest.php +++ b/tests/php/Forms/TreeDropdownFieldTest.php @@ -196,15 +196,17 @@ class TreeDropdownFieldTest extends SapphireTest public function testReadonly() { $field = new TreeDropdownField('TestTree', 'Test tree', File::class); - $asdf = $this->objFromFixture(File::class, 'asdf'); - $field->setValue($asdf->ID); + $fileMock = $this->objFromFixture(File::class, 'asdf'); + $field->setValue($fileMock->ID); $readonlyField = $field->performReadonlyTransformation(); - $this->assertEquals( - <<<"HTML" -<Special & characters> -HTML - , - (string)$readonlyField->Field() + $result = (string) $readonlyField->Field(); + $this->assertContains( + '<Special & characters>', + $result + ); + $this->assertContains( + '', + $result ); } }