Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Cleaning up docblocks in `Form` helper test.

  • Loading branch information...
commit 9a429900621515121a23a19ea02acb44843dec02 1 parent 1902698
@nateabele nateabele authored
Showing with 5 additions and 15 deletions.
  1. +5 −15 libraries/lithium/tests/cases/template/helper/FormTest.php
View
20 libraries/lithium/tests/cases/template/helper/FormTest.php
@@ -40,8 +40,6 @@ class FormTest extends \lithium\test\Unit {
/**
* Initialize test by creating a new object instance with a default context.
- *
- * @return void
*/
public function setUp() {
$this->_routes = Router::get();
@@ -105,8 +103,6 @@ public function testFormCreation() {
/**
* Tests creating forms with non-browser compatible HTTP methods, required for REST interfaces.
- *
- * @return void
*/
public function testRestFormCreation() {
$result = $this->form->create(null, array('action' => 'delete', 'method' => 'delete'));
@@ -153,8 +149,6 @@ public function testFormCreationWithBinding() {
/**
* Ensures that password fields aren't rendered with pre-populated values from bound record or
* document objects.
- *
- * @return void
*/
public function testPasswordWithBindingValue() {
$this->form->create(new Record(array(
@@ -660,8 +654,6 @@ public function testFormField() {
/**
* Verifies that calls to `field()` with `'type' => 'hidden'` do not produce `<label />`s.
- *
- * @return void
*/
public function testHiddenFieldWithNoLabel() {
$result = $this->form->field('foo', array('type' => 'hidden'));
@@ -695,8 +687,6 @@ public function testFieldWithLabelShorthand() {
/**
* Demonstrates that the options for a `<label />` element can be passed through the `field()`
* method, using the label text as a key.
- *
- * @return void
*/
public function testFieldLabelWithOptions() {
$result = $this->form->field('name', array(
@@ -887,8 +877,6 @@ public function testErrorWithCustomConfiguration() {
/**
* Tests that the string template form `Form::field()` can be overridden.
- *
- * @return void
*/
public function testFieldTemplateOverride() {
$this->form->config(array('templates' => array('field' => '{:label}{:input}{:error}')));
@@ -899,6 +887,10 @@ public function testFieldTemplateOverride() {
));
}
+ /**
+ * Tests that the `field()` method properly renders a `<select />` element if the `'list'`
+ * option is passed.
+ */
public function testFieldAssumeSelectIfList() {
$result = $this->form->field('colors', array(
'list' => array('r' => 'red', 'g' => 'green', 'b' => 'blue')
@@ -937,8 +929,6 @@ public function testFieldInputIdWithFormId() {
/**
* Tests that inputs for nested objects can be assigned using dot syntax.
- *
- * @return void
*/
public function testNestedFieldAccess() {
$doc = new Document(array('data' => array('foo' => array('bar' => 'value'))));
@@ -955,7 +945,7 @@ public function testNestedFieldAccess() {
'label' => array('for' => 'FooBar'), 'Foo Bar', '/label',
'input' => array(
'type' => 'text', 'name' => 'foo[bar]', 'id' => 'FooBar', 'value' => 'value'
- ),
+ )
));
}
Please sign in to comment.
Something went wrong with that request. Please try again.