sminnee /
silverstripe-framework
| @@ 377-394 (lines=18) @@ | ||
| 374 | /** |
|
| 375 | * #2939 CheckboxSetField creates invalid HTML when required |
|
| 376 | */ |
|
| 377 | public function testNoAriaRequired() |
|
| 378 | { |
|
| 379 | $field = new CheckboxSetField('RequiredField', 'myRequiredField'); |
|
| 380 | ||
| 381 | $form = new Form( |
|
| 382 | Controller::curr(), |
|
| 383 | "form", |
|
| 384 | new FieldList($field), |
|
| 385 | new FieldList(), |
|
| 386 | new RequiredFields(["RequiredField"]) |
|
| 387 | ); |
|
| 388 | $this->assertTrue($field->Required()); |
|
| 389 | ||
| 390 | $attributes = $field->getAttributes(); |
|
| 391 | $this->assertFalse(array_key_exists("aria-required", $attributes)); |
|
| 392 | $this->assertFalse(array_key_exists("name", $attributes)); |
|
| 393 | $this->assertFalse(array_key_exists("required", $attributes)); |
|
| 394 | } |
|
| 395 | } |
|
| 396 | ||
| @@ 108-125 (lines=18) @@ | ||
| 105 | /** |
|
| 106 | * #2939 OptionSetField creates invalid HTML when required |
|
| 107 | */ |
|
| 108 | public function testNoAriaRequired() |
|
| 109 | { |
|
| 110 | $field = new OptionsetField('RequiredField', 'myRequiredField'); |
|
| 111 | ||
| 112 | $form = new Form( |
|
| 113 | Controller::curr(), |
|
| 114 | "form", |
|
| 115 | new FieldList($field), |
|
| 116 | new FieldList(), |
|
| 117 | new RequiredFields(["RequiredField"]) |
|
| 118 | ); |
|
| 119 | $this->assertTrue($field->Required()); |
|
| 120 | ||
| 121 | $attributes = $field->getAttributes(); |
|
| 122 | $this->assertFalse(array_key_exists("name", $attributes)); |
|
| 123 | $this->assertFalse(array_key_exists("required", $attributes)); |
|
| 124 | $this->assertTrue(array_key_exists("role", $attributes)); |
|
| 125 | } |
|
| 126 | } |
|
| 127 | ||
