From 24b70a280c7136b9d4ee4aa1746f7e5731c2c7dc Mon Sep 17 00:00:00 2001 From: Dries Date: Tue, 25 Sep 2012 08:41:28 -0400 Subject: [PATCH] Issue #1741338 by lazysoundsystem: Removing t() from asserts in simpletests in block module. --- .../lib/Drupal/block/Tests/BlockCacheTest.php | 38 +++++----- .../block/Tests/BlockHiddenRegionTest.php | 6 +- .../Drupal/block/Tests/BlockHtmlIdTest.php | 2 +- .../Drupal/block/Tests/BlockLanguageTest.php | 20 ++--- .../BlockTemplateSuggestionsUnitTest.php | 6 +- .../lib/Drupal/block/Tests/BlockTest.php | 76 +++++++++---------- .../block/Tests/NewDefaultThemeBlocksTest.php | 4 +- 7 files changed, 76 insertions(+), 76 deletions(-) diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockCacheTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockCacheTest.php index 90327f9b043..3a5776cf251 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockCacheTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockCacheTest.php @@ -64,20 +64,20 @@ class BlockCacheTest extends WebTestBase { variable_set('block_test_content', $current_content); $this->drupalLogin($this->normal_user); $this->drupalGet(''); - $this->assertText($current_content, t('Block content displays.')); + $this->assertText($current_content, 'Block content displays.'); // Change the content, but the cached copy should still be served. $old_content = $current_content; $current_content = $this->randomName(); variable_set('block_test_content', $current_content); $this->drupalGet(''); - $this->assertText($old_content, t('Block is served from the cache.')); + $this->assertText($old_content, 'Block is served from the cache.'); // Clear the cache and verify that the stale data is no longer there. cache_invalidate(array('content' => TRUE)); $this->drupalGet(''); - $this->assertNoText($old_content, t('Block cache clear removes stale cache data.')); - $this->assertText($current_content, t('Fresh block content is displayed after clearing the cache.')); + $this->assertNoText($old_content, 'Block cache clear removes stale cache data.'); + $this->assertText($current_content, 'Fresh block content is displayed after clearing the cache.'); // Test whether the cached data is served for the correct users. $old_content = $current_content; @@ -85,19 +85,19 @@ class BlockCacheTest extends WebTestBase { variable_set('block_test_content', $current_content); $this->drupalLogout(); $this->drupalGet(''); - $this->assertNoText($old_content, t('Anonymous user does not see content cached per-role for normal user.')); + $this->assertNoText($old_content, 'Anonymous user does not see content cached per-role for normal user.'); $this->drupalLogin($this->normal_user_alt); $this->drupalGet(''); - $this->assertText($old_content, t('User with the same roles sees per-role cached content.')); + $this->assertText($old_content, 'User with the same roles sees per-role cached content.'); $this->drupalLogin($this->admin_user); $this->drupalGet(''); - $this->assertNoText($old_content, t('Admin user does not see content cached per-role for normal user.')); + $this->assertNoText($old_content, 'Admin user does not see content cached per-role for normal user.'); $this->drupalLogin($this->normal_user); $this->drupalGet(''); - $this->assertText($old_content, t('Block is served from the per-role cache.')); + $this->assertText($old_content, 'Block is served from the per-role cache.'); } /** @@ -109,7 +109,7 @@ class BlockCacheTest extends WebTestBase { variable_set('block_test_content', $current_content); $this->drupalGet(''); - $this->assertText($current_content, t('Block content displays.')); + $this->assertText($current_content, 'Block content displays.'); $old_content = $current_content; $current_content = $this->randomName(); @@ -117,7 +117,7 @@ class BlockCacheTest extends WebTestBase { $this->drupalLogout(); $this->drupalGet('user'); - $this->assertText($old_content, t('Block content served from global cache.')); + $this->assertText($old_content, 'Block content served from global cache.'); } /** @@ -130,13 +130,13 @@ class BlockCacheTest extends WebTestBase { // If DRUPAL_NO_CACHE has no effect, the next request would be cached. $this->drupalGet(''); - $this->assertText($current_content, t('Block content displays.')); + $this->assertText($current_content, 'Block content displays.'); // A cached copy should not be served. $current_content = $this->randomName(); variable_set('block_test_content', $current_content); $this->drupalGet(''); - $this->assertText($current_content, t('DRUPAL_NO_CACHE prevents blocks from being cached.')); + $this->assertText($current_content, 'DRUPAL_NO_CACHE prevents blocks from being cached.'); } /** @@ -149,22 +149,22 @@ class BlockCacheTest extends WebTestBase { $this->drupalLogin($this->normal_user); $this->drupalGet(''); - $this->assertText($current_content, t('Block content displays.')); + $this->assertText($current_content, 'Block content displays.'); $old_content = $current_content; $current_content = $this->randomName(); variable_set('block_test_content', $current_content); $this->drupalGet(''); - $this->assertText($old_content, t('Block is served from per-user cache.')); + $this->assertText($old_content, 'Block is served from per-user cache.'); $this->drupalLogin($this->normal_user_alt); $this->drupalGet(''); - $this->assertText($current_content, t('Per-user block cache is not served for other users.')); + $this->assertText($current_content, 'Per-user block cache is not served for other users.'); $this->drupalLogin($this->normal_user); $this->drupalGet(''); - $this->assertText($old_content, t('Per-user block cache is persistent.')); + $this->assertText($old_content, 'Per-user block cache is persistent.'); } /** @@ -176,16 +176,16 @@ class BlockCacheTest extends WebTestBase { variable_set('block_test_content', $current_content); $this->drupalGet('node'); - $this->assertText($current_content, t('Block content displays on the node page.')); + $this->assertText($current_content, 'Block content displays on the node page.'); $old_content = $current_content; $current_content = $this->randomName(); variable_set('block_test_content', $current_content); $this->drupalGet('user'); - $this->assertNoText($old_content, t('Block content cached for the node page does not show up for the user page.')); + $this->assertNoText($old_content, 'Block content cached for the node page does not show up for the user page.'); $this->drupalGet('node'); - $this->assertText($old_content, t('Block content cached for the node page.')); + $this->assertText($old_content, 'Block content cached for the node page.'); } /** diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockHiddenRegionTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockHiddenRegionTest.php index 13a78af11c6..af5acb920ff 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockHiddenRegionTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockHiddenRegionTest.php @@ -59,7 +59,7 @@ class BlockHiddenRegionTest extends WebTestBase { // Ensure that the search form block is displayed. $this->drupalGet(''); - $this->assertText('Search', t('Block was displayed on the front page.')); + $this->assertText('Search', 'Block was displayed on the front page.'); // Enable "block_test_theme" and set it as the default theme. $theme = 'block_test_theme'; @@ -69,10 +69,10 @@ class BlockHiddenRegionTest extends WebTestBase { // Ensure that "block_test_theme" is set as the default theme. $this->drupalGet('admin/structure/block'); - $this->assertText('Block test theme(' . t('active tab') . ')', t('Default local task on blocks admin page is the block test theme.')); + $this->assertText('Block test theme(' . t('active tab') . ')', 'Default local task on blocks admin page is the block test theme.'); // Ensure that the search form block is displayed. $this->drupalGet(''); - $this->assertText('Search', t('Block was displayed on the front page.')); + $this->assertText('Search', 'Block was displayed on the front page.'); } } diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockHtmlIdTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockHtmlIdTest.php index fd2870293bb..705bc76a3be 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockHtmlIdTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockHtmlIdTest.php @@ -50,6 +50,6 @@ class BlockHtmlIdTest extends WebTestBase { */ function testHtmlId() { $this->drupalGet(''); - $this->assertRaw('block-block-test-test-html-id', t('HTML id for test block is valid.')); + $this->assertRaw('block-block-test-test-html-id', 'HTML id for test block is valid.'); } } diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockLanguageTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockLanguageTest.php index 49d6a76f801..6d0c6bcbe88 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockLanguageTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockLanguageTest.php @@ -44,11 +44,11 @@ class BlockLanguageTest extends WebTestBase { 'predefined_langcode' => 'fr', ); $this->drupalPost('admin/config/regional/language/add', $edit, t('Add language')); - $this->assertText('French', t('Language added successfully.')); + $this->assertText('French', 'Language added successfully.'); // Check if the visibility setting is available. $this->drupalGet('admin/structure/block/add'); - $this->assertField('langcodes[en]', t('Language visibility field is visible.')); + $this->assertField('langcodes[en]', 'Language visibility field is visible.'); // Create a new block. $info_name = $this->randomString(10); @@ -81,11 +81,11 @@ class BlockLanguageTest extends WebTestBase { // Check that a page has a block $this->drupalGet('', array('language' => language_load('en'))); - $this->assertText($body, t('The body of the custom block appears on the page.')); + $this->assertText($body, 'The body of the custom block appears on the page.'); // Check that a page doesn't has a block for the current language anymore $this->drupalGet('', array('language' => language_load('fr'))); - $this->assertNoText($body, t('The body of the custom block does not appear on the page.')); + $this->assertNoText($body, 'The body of the custom block does not appear on the page.'); } /** @@ -103,7 +103,7 @@ class BlockLanguageTest extends WebTestBase { 'predefined_langcode' => 'fr', ); $this->drupalPost('admin/config/regional/language/add', $edit, t('Add language')); - $this->assertText('French', t('Language added successfully.')); + $this->assertText('French', 'Language added successfully.'); // Create a new block. $info_name = $this->randomString(10); @@ -130,7 +130,7 @@ class BlockLanguageTest extends WebTestBase { ':module' => 'block', ':delta' => '1' ))->fetchField(); - $this->assertTrue($count == 1, t('The block language visibility has an entry in the database.')); + $this->assertTrue($count == 1, 'The block language visibility has an entry in the database.'); // Delete the language. $this->drupalPost('admin/config/regional/language/delete/fr', array(), t('Delete')); @@ -140,7 +140,7 @@ class BlockLanguageTest extends WebTestBase { ':module' => 'block', ':delta' => '1' ))->fetchField(); - $this->assertTrue($count == 0, t('The block language visibility do not have an entry in the database.')); + $this->assertTrue($count == 0, 'The block language visibility do not have an entry in the database.'); } /** @@ -158,7 +158,7 @@ class BlockLanguageTest extends WebTestBase { 'predefined_langcode' => 'fr', ); $this->drupalPost('admin/config/regional/language/add', $edit, t('Add language')); - $this->assertText('French', t('Language added successfully.')); + $this->assertText('French', 'Language added successfully.'); // Create a new block. $info_name = $this->randomString(10); @@ -185,7 +185,7 @@ class BlockLanguageTest extends WebTestBase { ':module' => 'block', ':delta' => '1' ))->fetchField(); - $this->assertTrue($count == 1, t('The block language visibility has an entry in the database.')); + $this->assertTrue($count == 1, 'The block language visibility has an entry in the database.'); // Delete the custom block. $this->drupalPost('admin/structure/block/manage/block/1/delete', array(), t('Delete')); @@ -195,6 +195,6 @@ class BlockLanguageTest extends WebTestBase { ':module' => 'block', ':delta' => '1' ))->fetchField(); - $this->assertTrue($count == 0, t('The block language visibility do not have an entry in the database.')); + $this->assertTrue($count == 0, 'The block language visibility do not have an entry in the database.'); } } diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockTemplateSuggestionsUnitTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockTemplateSuggestionsUnitTest.php index f30956b3f14..3b5b389b14f 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockTemplateSuggestionsUnitTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockTemplateSuggestionsUnitTest.php @@ -35,7 +35,7 @@ class BlockTemplateSuggestionsUnitTest extends UnitTestBase { $variables1['elements']['#block'] = $block1; $variables1['elements']['#children'] = ''; template_preprocess_block($variables1); - $this->assertEqual($variables1['theme_hook_suggestions'], array('block__footer', 'block__block', 'block__block__underscore_test'), t('Found expected block suggestions for delta with underscore')); + $this->assertEqual($variables1['theme_hook_suggestions'], array('block__footer', 'block__block', 'block__block__underscore_test'), 'Found expected block suggestions for delta with underscore'); // Define block delta with hyphens to be preprocessed. Hyphens should be // replaced with underscores. @@ -49,8 +49,8 @@ class BlockTemplateSuggestionsUnitTest extends UnitTestBase { // Test adding a class to the block content. $variables2['content_attributes']['class'][] = 'test-class'; template_preprocess_block($variables2); - $this->assertEqual($variables2['theme_hook_suggestions'], array('block__footer', 'block__block', 'block__block__hyphen_test'), t('Hyphens (-) in block delta were replaced by underscore (_)')); + $this->assertEqual($variables2['theme_hook_suggestions'], array('block__footer', 'block__block', 'block__block__hyphen_test'), 'Hyphens (-) in block delta were replaced by underscore (_)'); // Test that the default class and added class are available. - $this->assertEqual($variables2['content_attributes']['class'], array('test-class', 'content'), t('Default .content class added to block content_attributes_array')); + $this->assertEqual($variables2['content_attributes']['class'], array('test-class', 'content'), 'Default .content class added to block content_attributes_array'); } } diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockTest.php index f98da2bfb43..1b598e3ee4b 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockTest.php @@ -68,9 +68,9 @@ class BlockTest extends WebTestBase { // Confirm that the add block link appears on block overview pages. $this->drupalGet('admin/structure/block'); - $this->assertRaw(l('Add block', 'admin/structure/block/add'), t('Add block link is present on block overview page for default theme.')); + $this->assertRaw(l('Add block', 'admin/structure/block/add'), 'Add block link is present on block overview page for default theme.'); $this->drupalGet('admin/structure/block/list/seven'); - $this->assertRaw(l('Add block', 'admin/structure/block/list/seven/add'), t('Add block link is present on block overview page for non-default theme.')); + $this->assertRaw(l('Add block', 'admin/structure/block/list/seven/add'), 'Add block link is present on block overview page for non-default theme.'); // Confirm that hidden regions are not shown as options for block placement // when adding a new block. @@ -81,7 +81,7 @@ class BlockTest extends WebTestBase { if ($theme->status) { foreach ($theme->info['regions_hidden'] as $hidden_region) { $elements = $this->xpath('//select[@id=:id]//option[@value=:value]', array(':id' => 'edit-regions-' . $key, ':value' => $hidden_region)); - $this->assertFalse(isset($elements[0]), t('The hidden region @region is not available for @theme.', array('@region' => $hidden_region, '@theme' => $key))); + $this->assertFalse(isset($elements[0]), format_string('The hidden region @region is not available for @theme.', array('@region' => $hidden_region, '@theme' => $key))); } } } @@ -94,17 +94,17 @@ class BlockTest extends WebTestBase { $this->drupalPost('admin/structure/block/add', $custom_block, t('Save block')); // Confirm that the custom block has been created, and then query the created bid. - $this->assertText(t('The block has been created.'), t('Custom block successfully created.')); + $this->assertText(t('The block has been created.'), 'Custom block successfully created.'); $bid = db_query("SELECT bid FROM {block_custom} WHERE info = :info", array(':info' => $custom_block['info']))->fetchField(); // Check to see if the custom block was created by checking that it's in the database. - $this->assertNotNull($bid, t('Custom block found in database')); + $this->assertNotNull($bid, 'Custom block found in database'); // Check that block_block_view() returns the correct title and content. $data = block_block_view($bid); $format = db_query("SELECT format FROM {block_custom} WHERE bid = :bid", array(':bid' => $bid))->fetchField(); - $this->assertTrue(array_key_exists('subject', $data) && empty($data['subject']), t('block_block_view() provides an empty block subject, since custom blocks do not have default titles.')); - $this->assertEqual(check_markup($custom_block['body[value]'], $format), $data['content'], t('block_block_view() provides correct block content.')); + $this->assertTrue(array_key_exists('subject', $data) && empty($data['subject']), 'block_block_view() provides an empty block subject, since custom blocks do not have default titles.'); + $this->assertEqual(check_markup($custom_block['body[value]'], $format), $data['content'], 'block_block_view() provides correct block content.'); // Check whether the block can be moved to all available regions. $custom_block['module'] = 'block'; @@ -115,8 +115,8 @@ class BlockTest extends WebTestBase { // Verify presence of configure and delete links for custom block. $this->drupalGet('admin/structure/block'); - $this->assertLinkByHref('admin/structure/block/manage/block/' . $bid . '/configure', 0, t('Custom block configure link found.')); - $this->assertLinkByHref('admin/structure/block/manage/block/' . $bid . '/delete', 0, t('Custom block delete link found.')); + $this->assertLinkByHref('admin/structure/block/manage/block/' . $bid . '/configure', 0, 'Custom block configure link found.'); + $this->assertLinkByHref('admin/structure/block/manage/block/' . $bid . '/delete', 0, 'Custom block delete link found.'); // Set visibility only for authenticated users, to verify delete functionality. $edit = array(); @@ -126,10 +126,10 @@ class BlockTest extends WebTestBase { // Delete the created custom block & verify that it's been deleted and no longer appearing on the page. $this->clickLink(t('delete')); $this->drupalPost('admin/structure/block/manage/block/' . $bid . '/delete', array(), t('Delete')); - $this->assertRaw(t('The block %title has been removed.', array('%title' => $custom_block['info'])), t('Custom block successfully deleted.')); - $this->assertNoText(t($custom_block['title']), t('Custom block no longer appears on page.')); + $this->assertRaw(t('The block %title has been removed.', array('%title' => $custom_block['info'])), 'Custom block successfully deleted.'); + $this->assertNoText(t($custom_block['title']), 'Custom block no longer appears on page.'); $count = db_query("SELECT 1 FROM {block_role} WHERE module = :module AND delta = :delta", array(':module' => $custom_block['module'], ':delta' => $custom_block['delta']))->fetchField(); - $this->assertFalse($count, t('Table block_role being cleaned.')); + $this->assertFalse($count, 'Table block_role being cleaned.'); } /** @@ -153,20 +153,20 @@ class BlockTest extends WebTestBase { // Confirm that the custom block is being displayed using configured text format. $this->drupalGet('node'); - $this->assertRaw('

Full HTML

', t('Custom block successfully being displayed using Full HTML.')); + $this->assertRaw('

Full HTML

', 'Custom block successfully being displayed using Full HTML.'); // Confirm that a user without access to Full HTML can not see the body field, // but can still submit the form without errors. $block_admin = $this->drupalCreateUser(array('administer blocks')); $this->drupalLogin($block_admin); $this->drupalGet('admin/structure/block/manage/block/' . $bid . '/configure'); - $this->assertFieldByXPath("//textarea[@name='body[value]' and @disabled='disabled']", t('This field has been disabled because you do not have sufficient permissions to edit it.'), t('Body field contains denied message')); + $this->assertFieldByXPath("//textarea[@name='body[value]' and @disabled='disabled']", t('This field has been disabled because you do not have sufficient permissions to edit it.'), 'Body field contains denied message'); $this->drupalPost('admin/structure/block/manage/block/' . $bid . '/configure', array(), t('Save block')); $this->assertNoText(t('Ensure that each block description is unique.')); // Confirm that the custom block is still being displayed using configured text format. $this->drupalGet('node'); - $this->assertRaw('

Full HTML

', t('Custom block successfully being displayed using Full HTML.')); + $this->assertRaw('

Full HTML

', 'Custom block successfully being displayed using Full HTML.'); } /** @@ -205,21 +205,21 @@ class BlockTest extends WebTestBase { $this->moveBlockToRegion($block, $this->regions[1]); $this->drupalGet(''); - $this->assertText($title, t('Block was displayed on the front page.')); + $this->assertText($title, 'Block was displayed on the front page.'); $this->drupalGet('user'); - $this->assertNoText($title, t('Block was not displayed according to block visibility rules.')); + $this->assertNoText($title, 'Block was not displayed according to block visibility rules.'); $this->drupalGet('USER/' . $this->admin_user->uid); - $this->assertNoText($title, t('Block was not displayed according to block visibility rules regardless of path case.')); + $this->assertNoText($title, 'Block was not displayed according to block visibility rules regardless of path case.'); // Confirm that the block is not displayed to anonymous users. $this->drupalLogout(); $this->drupalGet(''); - $this->assertNoText($title, t('Block was not displayed to anonymous users.')); + $this->assertNoText($title, 'Block was not displayed to anonymous users.'); // Confirm that an empty block is not displayed. - $this->assertNoRaw('block-system-help', t('Empty block not displayed.')); + $this->assertNoRaw('block-system-help', 'Empty block not displayed.'); } /** @@ -254,15 +254,15 @@ class BlockTest extends WebTestBase { $this->drupalPost('admin/structure/block/manage/' . $block['module'] . '/' . $block['delta'] . '/configure', $edit, t('Save block')); $this->drupalGet(''); - $this->assertNoText($title, t('Block was not displayed according to block visibility rules.')); + $this->assertNoText($title, 'Block was not displayed according to block visibility rules.'); $this->drupalGet('user'); - $this->assertNoText($title, t('Block was not displayed according to block visibility rules regardless of path case.')); + $this->assertNoText($title, 'Block was not displayed according to block visibility rules regardless of path case.'); // Confirm that the block is not displayed to anonymous users. $this->drupalLogout(); $this->drupalGet(''); - $this->assertNoText($title, t('Block was not displayed to anonymous users.')); + $this->assertNoText($title, 'Block was not displayed to anonymous users.'); } /** @@ -300,7 +300,7 @@ class BlockTest extends WebTestBase { $this->drupalPost('user/' . $this->admin_user->uid . '/edit', $edit, t('Save')); $this->drupalGet(''); - $this->assertNoText($block['title'], t('Block was not displayed according to per user block visibility setting.')); + $this->assertNoText($block['title'], 'Block was not displayed according to per user block visibility setting.'); // Set the block to be customizable per user, hidden by default. $edit = array(); @@ -313,7 +313,7 @@ class BlockTest extends WebTestBase { $this->drupalPost('user/' . $this->admin_user->uid . '/edit', $edit, t('Save')); $this->drupalGet(''); - $this->assertText($block['title'], t('Block was displayed according to per user block visibility setting.')); + $this->assertText($block['title'], 'Block was displayed according to per user block visibility setting.'); } /** @@ -328,14 +328,14 @@ class BlockTest extends WebTestBase { // Set block title to confirm that interface works and override any custom titles. $this->drupalPost('admin/structure/block/manage/' . $block['module'] . '/' . $block['delta'] . '/configure', array('title' => $block['title']), t('Save block')); - $this->assertText(t('The block configuration has been saved.'), t('Block title set.')); + $this->assertText(t('The block configuration has been saved.'), 'Block title set.'); $bid = db_query("SELECT bid FROM {block} WHERE module = :module AND delta = :delta", array( ':module' => $block['module'], ':delta' => $block['delta'], ))->fetchField(); // Check to see if the block was created by checking that it's in the database. - $this->assertNotNull($bid, t('Block found in database')); + $this->assertNotNull($bid, 'Block found in database'); // Check whether the block can be moved to all available regions. foreach ($this->regions as $region) { @@ -348,21 +348,21 @@ class BlockTest extends WebTestBase { $this->drupalPost('admin/structure/block', $edit, t('Save blocks')); // Confirm that the block was moved to the proper region. - $this->assertText(t('The block settings have been updated.'), t('Block successfully move to disabled region.')); - $this->assertNoText(t($block['title']), t('Block no longer appears on page.')); + $this->assertText(t('The block settings have been updated.'), 'Block successfully move to disabled region.'); + $this->assertNoText(t($block['title']), 'Block no longer appears on page.'); // Confirm that the region's xpath is not available. $xpath = $this->buildXPathQuery('//div[@id=:id]/*', array(':id' => 'block-block-' . $bid)); - $this->assertNoFieldByXPath($xpath, FALSE, t('Custom block found in no regions.')); + $this->assertNoFieldByXPath($xpath, FALSE, 'Custom block found in no regions.'); // For convenience of developers, put the navigation block back. $edit = array(); $edit['blocks[' . $block['module'] . '_' . $block['delta'] . '][region]'] = $this->regions[1]; $this->drupalPost('admin/structure/block', $edit, t('Save blocks')); - $this->assertText(t('The block settings have been updated.'), t('Block successfully move to first sidebar region.')); + $this->assertText(t('The block settings have been updated.'), 'Block successfully move to first sidebar region.'); $this->drupalPost('admin/structure/block/manage/' . $block['module'] . '/' . $block['delta'] . '/configure', array('title' => 'Navigation'), t('Save block')); - $this->assertText(t('The block configuration has been saved.'), t('Block title set.')); + $this->assertText(t('The block configuration has been saved.'), 'Block title set.'); } function moveBlockToRegion($block, $region) { @@ -372,18 +372,18 @@ class BlockTest extends WebTestBase { $this->drupalPost('admin/structure/block', $edit, t('Save blocks')); // Confirm that the block was moved to the proper region. - $this->assertText(t('The block settings have been updated.'), t('Block successfully moved to %region_name region.', array( '%region_name' => $region))); + $this->assertText(t('The block settings have been updated.'), format_string('Block successfully moved to %region_name region.', array( '%region_name' => $region))); // Confirm that the block is being displayed. $this->drupalGet('node'); - $this->assertText(t($block['title']), t('Block successfully being displayed on the page.')); + $this->assertText(t($block['title']), 'Block successfully being displayed on the page.'); // Confirm that the custom block was found at the proper region. $xpath = $this->buildXPathQuery('//div[@class=:region-class]//div[@id=:block-id]/*', array( ':region-class' => 'region region-' . str_replace('_', '-', $region), ':block-id' => 'block-' . $block['module'] . '-' . $block['delta'], )); - $this->assertFieldByXPath($xpath, NULL, t('Custom block found in %region_name region.', array('%region_name' => $region))); + $this->assertFieldByXPath($xpath, NULL, format_string('Custom block found in %region_name region.', array('%region_name' => $region))); } /** @@ -391,14 +391,14 @@ class BlockTest extends WebTestBase { */ function testBlockRehash() { module_enable(array('block_test')); - $this->assertTrue(module_exists('block_test'), t('Test block module enabled.')); + $this->assertTrue(module_exists('block_test'), 'Test block module enabled.'); // Our new block should be inserted in the database when we visit the // block management page. $this->drupalGet('admin/structure/block'); // Our test block's caching should default to DRUPAL_CACHE_PER_ROLE. $current_caching = db_query("SELECT cache FROM {block} WHERE module = 'block_test' AND delta = 'test_cache'")->fetchField(); - $this->assertEqual($current_caching, DRUPAL_CACHE_PER_ROLE, t('Test block cache mode defaults to DRUPAL_CACHE_PER_ROLE.')); + $this->assertEqual($current_caching, DRUPAL_CACHE_PER_ROLE, 'Test block cache mode defaults to DRUPAL_CACHE_PER_ROLE.'); // Disable caching for this block. variable_set('block_test_caching', DRUPAL_NO_CACHE); @@ -406,6 +406,6 @@ class BlockTest extends WebTestBase { $this->resetAll(); // Verify that the database is updated with the new caching mode. $current_caching = db_query("SELECT cache FROM {block} WHERE module = 'block_test' AND delta = 'test_cache'")->fetchField(); - $this->assertEqual($current_caching, DRUPAL_NO_CACHE, t("Test block's database entry updated to DRUPAL_NO_CACHE.")); + $this->assertEqual($current_caching, DRUPAL_NO_CACHE, "Test block's database entry updated to DRUPAL_NO_CACHE."); } } diff --git a/core/modules/block/lib/Drupal/block/Tests/NewDefaultThemeBlocksTest.php b/core/modules/block/lib/Drupal/block/Tests/NewDefaultThemeBlocksTest.php index bf1ba7abe7e..c133f17c622 100644 --- a/core/modules/block/lib/Drupal/block/Tests/NewDefaultThemeBlocksTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/NewDefaultThemeBlocksTest.php @@ -44,7 +44,7 @@ class NewDefaultThemeBlocksTest extends WebTestBase { $themes['admin'] = $admin_theme; } $count = db_query_range('SELECT 1 FROM {block} WHERE theme NOT IN (:themes)', 0, 1, array(':themes' => $themes))->fetchField(); - $this->assertFalse($count, t('Only the default theme and the admin theme have blocks.')); + $this->assertFalse($count, 'Only the default theme and the admin theme have blocks.'); // Populate list of all blocks for matching against new theme. $blocks = array(); @@ -63,7 +63,7 @@ class NewDefaultThemeBlocksTest extends WebTestBase { $result = db_query('SELECT * FROM {block} WHERE theme = :theme', array(':theme' => $new_theme)); foreach ($result as $block) { unset($block->theme, $block->bid); - $this->assertEqual($blocks[$block->module][$block->delta], $block, t('Block %name matched', array('%name' => $block->module . '-' . $block->delta))); + $this->assertEqual($blocks[$block->module][$block->delta], $block, format_string('Block %name matched', array('%name' => $block->module . '-' . $block->delta))); } } }