diff --git a/browsertest-convert.php b/browsertest-convert.php new file mode 100644 index 0000000..a2c3dfb --- /dev/null +++ b/browsertest-convert.php @@ -0,0 +1,624 @@ +drupalGetMails(') !== FALSE) { + $file_content = str_replace(" extends BrowserTestBase {", " extends BrowserTestBase {\n use AssertMailTrait {\n getMails as drupalGetMails;\n }", $file_content); + $file_content = str_replace("use Drupal\\Tests\\BrowserTestBase;", "use Drupal\\Tests\\BrowserTestBase;\nuse Drupal\Core\Test\AssertMailTrait;", $file_content); + } + + $test_dir = dirname("$browsertest_dir/$file_name"); + if (!file_exists($test_dir)) { + mkdir($test_dir, 0777, TRUE); + } + + file_put_contents("$browsertest_dir/$file_name", $file_content); + + + // Keep existing base classes. + if (strpos($simpletest_file_path, 'Base.php') === FALSE) { + unlink($simpletest_file_path); + } + } + + + print "Tests converted for module $module_name.\n"; +} diff --git a/core/modules/action/src/Tests/ActionListTest.php b/core/modules/action/tests/src/Functional/ActionListTest.php similarity index 87% rename from core/modules/action/src/Tests/ActionListTest.php rename to core/modules/action/tests/src/Functional/ActionListTest.php index f533cdf..2510e86 100644 --- a/core/modules/action/src/Tests/ActionListTest.php +++ b/core/modules/action/tests/src/Functional/ActionListTest.php @@ -1,15 +1,15 @@ xpath() instead and check the values directly in the test. + */ + protected function assertFieldByXPath($xpath, $value = NULL, $message = '') { + $fields = $this->xpath($xpath); + + $this->assertFieldsByValue($fields, $value, $message); + } + + /** + * Asserts that a field does not exist or its value does not match, by XPath. + * + * @param string $xpath + * XPath used to find the field. + * @param string $value + * (optional) Value of the field, to assert that the field's value on the + * page does not match it. + * @param string $message + * (optional) A message to display with the assertion. Do not translate + * messages with t(). + * + * @deprecated Scheduled for removal in Drupal 9.0.0. + * Use $this->assertSession()->fieldNotExists() or + * $this->assertSession()->fieldValueNotEquals() instead. + */ + protected function assertNoFieldByXPath($xpath, $value = NULL, $message = '') { + $fields = $this->xpath($xpath); + + // If value specified then check array for match. + $found = TRUE; + if (isset($value)) { + $found = FALSE; + if ($fields) { + foreach ($fields as $field) { + if ($field->getAttribute('value') == $value) { + $found = TRUE; + } + } + } + } + return $this->assertFalse($fields && $found, $message); + } + + /** + * Asserts that a field exists in the current page with a given Xpath result. + * + * @param \Behat\Mink\Element\NodeElement[] $fields + * Xml elements. + * @param string $value + * (optional) Value of the field to assert. You may pass in NULL (default) to skip + * checking the actual value, while still checking that the field exists. + * @param string $message + * (optional) A message to display with the assertion. Do not translate + * messages with t(). + * + * @deprecated Scheduled for removal in Drupal 9.0.0. + * Iterate over the fields yourself instead and directly check the values in + * the test. + */ + protected function assertFieldsByValue($fields, $value = NULL, $message = '') { + // If value specified then check array for match. + $found = TRUE; + if (isset($value)) { + $found = FALSE; + if ($fields) { + foreach ($fields as $field) { + if ($field->getAttribute('value') == $value) { + // Input element with correct value. + $found = TRUE; + } + elseif ($field->find('xpath', '//option[@value = ' . (new Escaper())->escapeLiteral($value) . ' and @selected = "selected"]')) { + // Select element with an option. + $found = TRUE; + } + elseif ($field->getText() == $value) { + // Text area with correct text. + $found = TRUE; + } + } + } + } + $this->assertTrue($fields && $found, $message); + } + + /** * Passes if the raw text IS found escaped on the loaded page, fail otherwise. * * Raw text refers to the raw HTML that the page generated. diff --git a/core/tests/Drupal/FunctionalTests/BrowserTestBaseTest.php b/core/tests/Drupal/FunctionalTests/BrowserTestBaseTest.php index eb315af..6cba6b9 100644 --- a/core/tests/Drupal/FunctionalTests/BrowserTestBaseTest.php +++ b/core/tests/Drupal/FunctionalTests/BrowserTestBaseTest.php @@ -134,9 +134,9 @@ public function testPipeCharInLocator() { } /** - * Tests legacy asserts. + * Tests legacy text asserts. */ - public function testLegacyAsserts() { + public function testLegacyTextAsserts() { $this->drupalGet('test-encoded'); $dangerous = 'Bad html '; $sanitized = Html::escape($dangerous); @@ -144,4 +144,26 @@ public function testLegacyAsserts() { $this->assertText($sanitized); } + /** + * Tests legacy XPath asserts. + */ + public function testLegacyXPathAsserts() { + $account = $this->drupalCreateUser(['administer users'], 'test'); + $this->drupalLogin($account); + + $this->drupalGet('admin/people'); + $this->assertFieldsByValue($this->xpath("//h1[@class = 'page-title']")); + $this->assertFieldsByValue($this->xpath('//table/tbody/tr[2]/td[1]/span'), $account->getAccountName()); + $this->assertFieldByXPath('//table/tbody/tr[2]/td[1]/span', $account->getAccountName()); + + $this->drupalGet('user/' . $account->id() . '/edit'); + $this->assertFieldsByValue($this->xpath("//input[@id = 'edit-name']"), $account->getAccountName()); + $this->assertFieldByXPath("//input[@id = 'edit-name']", $account->getAccountName()); + $this->assertFieldsByValue($this->xpath("//select[@id = 'edit-timezone--2']"), 'Australia/Sydney'); + $this->assertFieldByXPath("//select[@id = 'edit-timezone--2']", 'Australia/Sydney'); + + $this->assertNoFieldByXPath('//notexisting'); + $this->assertNoFieldByXPath("//input[@id = 'edit-name']", 'wrong value'); + } + }