diff options
author | Dries Buytaert <dries@buytaert.net> | 2010-03-12 14:38:37 +0000 |
---|---|---|
committer | Dries Buytaert <dries@buytaert.net> | 2010-03-12 14:38:37 +0000 |
commit | 160b42812c2ec0a32fcdcd02f32d3f86e549fac7 (patch) | |
tree | df699a16df19f072114ada0a36bb19808bf39c55 /modules | |
parent | d8e62ff260991b911a05092d285aa6a31cdb8229 (diff) | |
download | brdo-160b42812c2ec0a32fcdcd02f32d3f86e549fac7.tar.gz brdo-160b42812c2ec0a32fcdcd02f32d3f86e549fac7.tar.bz2 |
- Patch #335035 by boombatower, Damien Tournoud, ksenzee, Berdir, Dave Reid, effulgentsia: Simpletest drupalPost() incorrectly submits input for disabled elements, causing unreliable testing of forms that contain them.
Diffstat (limited to 'modules')
-rw-r--r-- | modules/simpletest/drupal_web_test_case.php | 2 | ||||
-rw-r--r-- | modules/simpletest/tests/form.test | 2 | ||||
-rw-r--r-- | modules/translation/translation.test | 7 |
3 files changed, 7 insertions, 4 deletions
diff --git a/modules/simpletest/drupal_web_test_case.php b/modules/simpletest/drupal_web_test_case.php index 24a2acb81..f2622444d 100644 --- a/modules/simpletest/drupal_web_test_case.php +++ b/modules/simpletest/drupal_web_test_case.php @@ -1747,7 +1747,7 @@ class DrupalWebTestCase extends DrupalTestCase { */ protected function handleForm(&$post, &$edit, &$upload, $submit, $form) { // Retrieve the form elements. - $elements = $form->xpath('.//input|.//textarea|.//select'); + $elements = $form->xpath('.//input[not(@disabled)]|.//textarea[not(@disabled)]|.//select[not(@disabled)]'); $submit_matches = FALSE; foreach ($elements as $element) { // SimpleXML objects need string casting all the time. diff --git a/modules/simpletest/tests/form.test b/modules/simpletest/tests/form.test index 82a9b9e08..87dd698d8 100644 --- a/modules/simpletest/tests/form.test +++ b/modules/simpletest/tests/form.test @@ -172,7 +172,7 @@ class FormsTestCase extends DrupalWebTestCase { // Checkboxes values are not filtered out. $returned_values[$key] = array_filter($returned_values[$key]); } - $this->assertEqual($expected_value, $returned_values[$key], t('Default value for %type: expected %expected, returned %returned.', array('%type' => $key, '%expected' => var_export($expected_value, TRUE), '%returned' => var_export($returned_values[$key], TRUE)))); + $this->assertIdentical($expected_value, $returned_values[$key], t('Default value for %type: expected %expected, returned %returned.', array('%type' => $key, '%expected' => var_export($expected_value, TRUE), '%returned' => var_export($returned_values[$key], TRUE)))); } } } diff --git a/modules/translation/translation.test b/modules/translation/translation.test index 78d093e84..0ef64c275 100644 --- a/modules/translation/translation.test +++ b/modules/translation/translation.test @@ -110,11 +110,14 @@ class TranslationTestCase extends DrupalWebTestCase { $this->assertRaw(t('The language %language has been created and can now be used. More information is available on the <a href="@locale-help">help screen</a>.', array('%language' => $languages[$language_code]->name, '@locale-help' => url('admin/help/locale'))), t('Language has been created.')); } } + elseif ($this->xpath('//input[@type=\'checkbox\' and @name=\'enabled[' . $language_code . ']\' and @checked=\'checked\']')) { + // It's installed and enabled. No need to do anything. + $this->assertTrue(true, 'Language [' . $language_code . '] already installed and enabled.'); + } else { - // Ensure that it is enabled. + // It's installed but not enabled. Enable it. $this->assertTrue(true, 'Language [' . $language_code . '] already installed.'); $this->drupalPost(NULL, array('enabled[' . $language_code . ']' => TRUE), t('Save configuration')); - $this->assertRaw(t('Configuration saved.'), t('Language successfully enabled.')); } } |