diff --git a/core/modules/action/tests/src/Functional/ConfigurationTest.php b/core/modules/action/tests/src/Functional/ConfigurationTest.php index 23d0a38..0cbaa16 100644 --- a/core/modules/action/tests/src/Functional/ConfigurationTest.php +++ b/core/modules/action/tests/src/Functional/ConfigurationTest.php @@ -24,7 +24,7 @@ class ConfigurationTest extends BrowserTestBase { /** * Tests configuration of advanced actions through administration interface. */ - function testActionConfiguration() { + public function testActionConfiguration() { // Create a user with permission to view the actions administration pages. $user = $this->drupalCreateUser(array('administer actions')); $this->drupalLogin($user); diff --git a/core/modules/aggregator/src/Tests/AggregatorAdminTest.php b/core/modules/aggregator/src/Tests/AggregatorAdminTest.php index 249817f..887055b 100644 --- a/core/modules/aggregator/src/Tests/AggregatorAdminTest.php +++ b/core/modules/aggregator/src/Tests/AggregatorAdminTest.php @@ -59,7 +59,7 @@ public function testSettingsPage() { /** * Tests the overview page. */ - function testOverviewPage() { + public function testOverviewPage() { $feed = $this->createFeed($this->getRSS091Sample()); $this->drupalGet('admin/config/services/aggregator'); diff --git a/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php b/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php index 738bddf..230a743 100644 --- a/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php +++ b/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php @@ -21,7 +21,7 @@ class IpAddressBlockingTest extends BrowserTestBase { /** * Tests various user input to confirm correct validation and saving of data. */ - function testIPAddressValidation() { + public function testIPAddressValidation() { // Create user. $admin_user = $this->drupalCreateUser(array('ban IP addresses')); $this->drupalLogin($admin_user); diff --git a/core/modules/basic_auth/src/Tests/Authentication/BasicAuthTest.php b/core/modules/basic_auth/src/Tests/Authentication/BasicAuthTest.php index 22bb9ca..ca41a32 100644 --- a/core/modules/basic_auth/src/Tests/Authentication/BasicAuthTest.php +++ b/core/modules/basic_auth/src/Tests/Authentication/BasicAuthTest.php @@ -75,7 +75,7 @@ public function testBasicAuth() { /** * Test the global login flood control. */ - function testGlobalLoginFloodControl() { + public function testGlobalLoginFloodControl() { $this->config('user.flood') ->set('ip_limit', 2) // Set a high per-user limit out so that it is not relevant in the test. @@ -100,7 +100,7 @@ function testGlobalLoginFloodControl() { /** * Test the per-user login flood control. */ - function testPerUserLoginFloodControl() { + public function testPerUserLoginFloodControl() { $this->config('user.flood') // Set a high global limit out so that it is not relevant in the test. ->set('ip_limit', 4000) @@ -138,7 +138,7 @@ function testPerUserLoginFloodControl() { /** * Tests compatibility with locale/UI translation. */ - function testLocale() { + public function testLocale() { ConfigurableLanguage::createFromLangcode('de')->save(); $this->config('system.site')->set('default_langcode', 'de')->save(); @@ -154,7 +154,7 @@ function testLocale() { /** * Tests if a comprehensive message is displayed when the route is denied. */ - function testUnauthorizedErrorMessage() { + public function testUnauthorizedErrorMessage() { $account = $this->drupalCreateUser(); $url = Url::fromRoute('router_test.11'); diff --git a/core/modules/block/src/Tests/BlockAdminThemeTest.php b/core/modules/block/src/Tests/BlockAdminThemeTest.php index ab444d5..600fa05 100644 --- a/core/modules/block/src/Tests/BlockAdminThemeTest.php +++ b/core/modules/block/src/Tests/BlockAdminThemeTest.php @@ -21,7 +21,7 @@ class BlockAdminThemeTest extends WebTestBase { /** * Check for the accessibility of the admin theme on the block admin page. */ - function testAdminTheme() { + public function testAdminTheme() { // Create administrative user. $admin_user = $this->drupalCreateUser(array('administer blocks', 'administer themes')); $this->drupalLogin($admin_user); @@ -42,7 +42,7 @@ function testAdminTheme() { /** * Ensure contextual links are disabled in Seven theme. */ - function testSevenAdminTheme() { + public function testSevenAdminTheme() { // Create administrative user. $admin_user = $this->drupalCreateUser([ 'access administration pages', diff --git a/core/modules/block/src/Tests/BlockCacheTest.php b/core/modules/block/src/Tests/BlockCacheTest.php index f774539..fbae584 100644 --- a/core/modules/block/src/Tests/BlockCacheTest.php +++ b/core/modules/block/src/Tests/BlockCacheTest.php @@ -69,7 +69,7 @@ protected function setUp() { /** * Test "user.roles" cache context. */ - function testCachePerRole() { + public function testCachePerRole() { \Drupal::state()->set('block_test.cache_contexts', ['user.roles']); // Enable our test block. Set some content for it to display. @@ -116,7 +116,7 @@ function testCachePerRole() { /** * Test a cacheable block without any additional cache context. */ - function testCachePermissions() { + public function testCachePermissions() { // user.permissions is a required context, so a user with different // permissions will see a different version of the block. \Drupal::state()->set('block_test.cache_contexts', []); @@ -142,7 +142,7 @@ function testCachePermissions() { /** * Test non-cacheable block. */ - function testNoCache() { + public function testNoCache() { \Drupal::state()->set('block_test.cache_max_age', 0); $current_content = $this->randomMachineName(); @@ -162,7 +162,7 @@ function testNoCache() { /** * Test "user" cache context. */ - function testCachePerUser() { + public function testCachePerUser() { \Drupal::state()->set('block_test.cache_contexts', ['user']); $current_content = $this->randomMachineName(); @@ -191,7 +191,7 @@ function testCachePerUser() { /** * Test "url" cache context. */ - function testCachePerPage() { + public function testCachePerPage() { \Drupal::state()->set('block_test.cache_contexts', ['url']); $current_content = $this->randomMachineName(); diff --git a/core/modules/block/src/Tests/BlockFormInBlockTest.php b/core/modules/block/src/Tests/BlockFormInBlockTest.php index 73c7591..7676a6a 100644 --- a/core/modules/block/src/Tests/BlockFormInBlockTest.php +++ b/core/modules/block/src/Tests/BlockFormInBlockTest.php @@ -31,7 +31,7 @@ protected function setUp() { /** * Test to see if form in block's redirect isn't cached. */ - function testCachePerPage() { + public function testCachePerPage() { $form_values = ['email' => 'test@example.com']; // Go to "test-page" and test if the block is enabled. diff --git a/core/modules/block/src/Tests/BlockHtmlTest.php b/core/modules/block/src/Tests/BlockHtmlTest.php index baf5d4b..6d722c9 100644 --- a/core/modules/block/src/Tests/BlockHtmlTest.php +++ b/core/modules/block/src/Tests/BlockHtmlTest.php @@ -35,7 +35,7 @@ protected function setUp() { /** * Tests for valid HTML for a block. */ - function testHtml() { + public function testHtml() { $this->drupalGet(''); // Ensure that a block's ID is converted to an HTML valid ID, and that diff --git a/core/modules/block/src/Tests/BlockInvalidRegionTest.php b/core/modules/block/src/Tests/BlockInvalidRegionTest.php index 438a0c1..f371c60 100644 --- a/core/modules/block/src/Tests/BlockInvalidRegionTest.php +++ b/core/modules/block/src/Tests/BlockInvalidRegionTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Tests that blocks assigned to invalid regions work correctly. */ - function testBlockInInvalidRegion() { + public function testBlockInInvalidRegion() { // Enable a test block and place it in an invalid region. $block = $this->drupalPlaceBlock('test_html'); $block->setRegion('invalid_region'); diff --git a/core/modules/block/src/Tests/BlockRenderOrderTest.php b/core/modules/block/src/Tests/BlockRenderOrderTest.php index faf667e..de26292 100644 --- a/core/modules/block/src/Tests/BlockRenderOrderTest.php +++ b/core/modules/block/src/Tests/BlockRenderOrderTest.php @@ -31,7 +31,7 @@ protected function setUp() { /** * Tests the render order of the blocks. */ - function testBlockRenderOrder() { + public function testBlockRenderOrder() { // Enable test blocks and place them in the same region. $region = 'header'; $test_blocks = array( diff --git a/core/modules/block/src/Tests/BlockTemplateSuggestionsTest.php b/core/modules/block/src/Tests/BlockTemplateSuggestionsTest.php index d5ce8e6..26a3c0b 100644 --- a/core/modules/block/src/Tests/BlockTemplateSuggestionsTest.php +++ b/core/modules/block/src/Tests/BlockTemplateSuggestionsTest.php @@ -22,7 +22,7 @@ class BlockTemplateSuggestionsTest extends WebTestBase { /** * Tests template suggestions from block_theme_suggestions_block(). */ - function testBlockThemeHookSuggestions() { + public function testBlockThemeHookSuggestions() { // Define a block with a derivative to be preprocessed, which includes both // an underscore (not transformed) and a hyphen (transformed to underscore), // and generates possibilities for each level of derivative. diff --git a/core/modules/block/src/Tests/BlockTest.php b/core/modules/block/src/Tests/BlockTest.php index 039aa64..443b9f3 100644 --- a/core/modules/block/src/Tests/BlockTest.php +++ b/core/modules/block/src/Tests/BlockTest.php @@ -18,7 +18,7 @@ class BlockTest extends BlockTestBase { /** * Tests block visibility. */ - function testBlockVisibility() { + public function testBlockVisibility() { $block_name = 'system_powered_by_block'; // Create a random title for the block. $title = $this->randomMachineName(8); @@ -99,7 +99,7 @@ public function testBlockToggleVisibility() { /** * Test block visibility when leaving "pages" textarea empty. */ - function testBlockVisibilityListedEmpty() { + public function testBlockVisibilityListedEmpty() { $block_name = 'system_powered_by_block'; // Create a random title for the block. $title = $this->randomMachineName(8); @@ -174,7 +174,7 @@ public function testAddBlockFromLibrary() { /** * Test configuring and moving a module-define block to specific regions. */ - function testBlock() { + public function testBlock() { // Place page title block to test error messages. $this->drupalPlaceBlock('page_title_block'); @@ -263,7 +263,7 @@ public function testBlockThemeSelector() { /** * Test block display of theme titles. */ - function testThemeName() { + public function testThemeName() { // Enable the help block. $this->drupalPlaceBlock('help_block', array('region' => 'help')); $this->drupalPlaceBlock('local_tasks_block'); @@ -280,7 +280,7 @@ function testThemeName() { /** * Test block title display settings. */ - function testHideBlockTitle() { + public function testHideBlockTitle() { $block_name = 'system_powered_by_block'; // Create a random title for the block. $title = $this->randomMachineName(8); diff --git a/core/modules/block/src/Tests/BlockUiTest.php b/core/modules/block/src/Tests/BlockUiTest.php index 5ea62cd..2a76ae2 100644 --- a/core/modules/block/src/Tests/BlockUiTest.php +++ b/core/modules/block/src/Tests/BlockUiTest.php @@ -94,7 +94,7 @@ public function testBlockDemoUiPage() { /** * Test block admin page exists and functions correctly. */ - function testBlockAdminUiPage() { + public function testBlockAdminUiPage() { // Visit the blocks admin ui. $this->drupalGet('admin/structure/block'); // Look for the blocks table. diff --git a/core/modules/block/src/Tests/NewDefaultThemeBlocksTest.php b/core/modules/block/src/Tests/NewDefaultThemeBlocksTest.php index 9442df5..aad7a8b 100644 --- a/core/modules/block/src/Tests/NewDefaultThemeBlocksTest.php +++ b/core/modules/block/src/Tests/NewDefaultThemeBlocksTest.php @@ -21,7 +21,7 @@ class NewDefaultThemeBlocksTest extends WebTestBase { /** * Check the enabled Bartik blocks are correctly copied over. */ - function testNewDefaultThemeBlocks() { + public function testNewDefaultThemeBlocks() { $default_theme = $this->config('system.theme')->get('default'); // Add two instances of the user login block. diff --git a/core/modules/block/src/Tests/NonDefaultBlockAdminTest.php b/core/modules/block/src/Tests/NonDefaultBlockAdminTest.php index 218832d..191b571 100644 --- a/core/modules/block/src/Tests/NonDefaultBlockAdminTest.php +++ b/core/modules/block/src/Tests/NonDefaultBlockAdminTest.php @@ -30,7 +30,7 @@ protected function setUp() { /** * Test non-default theme admin. */ - function testNonDefaultBlockAdmin() { + public function testNonDefaultBlockAdmin() { $admin_user = $this->drupalCreateUser(array('administer blocks', 'administer themes')); $this->drupalLogin($admin_user); $new_theme = 'bartik'; diff --git a/core/modules/book/src/Tests/BookTest.php b/core/modules/book/src/Tests/BookTest.php index 7ebec63..f9271b2 100644 --- a/core/modules/book/src/Tests/BookTest.php +++ b/core/modules/book/src/Tests/BookTest.php @@ -151,7 +151,7 @@ public function testBookNavigationCacheContext() { /** * Tests saving the book outline on an empty book. */ - function testEmptyBook() { + public function testEmptyBook() { // Create a new empty book. $this->drupalLogin($this->bookAuthor); $book = $this->createBookNode('new'); @@ -166,7 +166,7 @@ function testEmptyBook() { /** * Tests book functionality through node interfaces. */ - function testBook() { + public function testBook() { // Create new book. $nodes = $this->createBook(); $book = $this->book; @@ -373,7 +373,7 @@ function createBookNode($book_nid, $parent = NULL) { /** * Tests book export ("printer-friendly version") functionality. */ - function testBookExport() { + public function testBookExport() { // Create a book. $nodes = $this->createBook(); @@ -418,7 +418,7 @@ function testBookExport() { /** * Tests the functionality of the book navigation block. */ - function testBookNavigationBlock() { + public function testBookNavigationBlock() { $this->drupalLogin($this->adminUser); // Enable the block. @@ -495,7 +495,7 @@ public function testGetTableOfContents() { /** * Tests the book navigation block when an access module is installed. */ - function testNavigationBlockOnAccessModuleInstalled() { + public function testNavigationBlockOnAccessModuleInstalled() { $this->drupalLogin($this->adminUser); $block = $this->drupalPlaceBlock('book_navigation', array('block_mode' => 'book pages')); @@ -526,7 +526,7 @@ function testNavigationBlockOnAccessModuleInstalled() { /** * Tests the access for deleting top-level book nodes. */ - function testBookDelete() { + public function testBookDelete() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $nodes = $this->createBook(); $this->drupalLogin($this->adminUser); diff --git a/core/modules/ckeditor/src/Tests/CKEditorAdminTest.php b/core/modules/ckeditor/src/Tests/CKEditorAdminTest.php index d10180d..9eeac18 100644 --- a/core/modules/ckeditor/src/Tests/CKEditorAdminTest.php +++ b/core/modules/ckeditor/src/Tests/CKEditorAdminTest.php @@ -48,7 +48,7 @@ protected function setUp() { /** * Tests configuring a text editor for an existing text format. */ - function testExistingFormat() { + public function testExistingFormat() { $ckeditor = $this->container->get('plugin.manager.editor')->createInstance('ckeditor'); $this->drupalLogin($this->adminUser); @@ -222,7 +222,7 @@ function testExistingFormat() { * This test only needs to ensure that the basics of the CKEditor * configuration form work; details are tested in testExistingFormat(). */ - function testNewFormat() { + public function testNewFormat() { $this->drupalLogin($this->adminUser); $this->drupalGet('admin/config/content/formats/add'); diff --git a/core/modules/ckeditor/src/Tests/CKEditorLoadingTest.php b/core/modules/ckeditor/src/Tests/CKEditorLoadingTest.php index e14b6e3..a24d387 100644 --- a/core/modules/ckeditor/src/Tests/CKEditorLoadingTest.php +++ b/core/modules/ckeditor/src/Tests/CKEditorLoadingTest.php @@ -74,7 +74,7 @@ protected function setUp() { /** * Tests loading of CKEditor CSS, JS and JS settings. */ - function testLoading() { + public function testLoading() { // The untrusted user: // - has access to 1 text format (plain_text); // - doesn't have access to the filtered_html text format, so: no text editor. @@ -158,7 +158,7 @@ function testLoading() { /** * Tests presence of essential configuration even without Internal's buttons. */ - protected function testLoadingWithoutInternalButtons() { + public function testLoadingWithoutInternalButtons() { // Change the CKEditor text editor configuration to only have link buttons. // This means: // - 0 buttons are from \Drupal\ckeditor\Plugin\CKEditorPlugin\Internal @@ -197,7 +197,7 @@ protected function testLoadingWithoutInternalButtons() { /** * Tests loading of theme's CKEditor stylesheets defined in the .info file. */ - function testExternalStylesheets() { + public function testExternalStylesheets() { $theme_handler = \Drupal::service('theme_handler'); // Case 1: Install theme which has an absolute external CSS URL. $theme_handler->install(['test_ckeditor_stylesheets_external']); diff --git a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php index 92c448b..c7b9ec2 100644 --- a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php +++ b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php @@ -50,7 +50,7 @@ protected function setUp() { /** * Tests the enabling of plugins. */ - function testEnabledPlugins() { + public function testEnabledPlugins() { $this->manager = $this->container->get('plugin.manager.ckeditor.plugin'); $editor = Editor::load('filtered_html'); @@ -132,7 +132,7 @@ function testEnabledPlugins() { /** * Tests the iframe instance CSS files of plugins. */ - function testCssFiles() { + public function testCssFiles() { $this->manager = $this->container->get('plugin.manager.ckeditor.plugin'); $editor = Editor::load('filtered_html'); diff --git a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php index 2bbd1e6..5009504 100644 --- a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php +++ b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php @@ -68,7 +68,7 @@ protected function setUp() { /** * Tests CKEditor::getJSSettings(). */ - function testGetJSSettings() { + public function testGetJSSettings() { $editor = Editor::load('filtered_html'); // Default toolbar. @@ -216,7 +216,7 @@ function testGetJSSettings() { /** * Tests CKEditor::buildToolbarJSSetting(). */ - function testBuildToolbarJSSetting() { + public function testBuildToolbarJSSetting() { $editor = Editor::load('filtered_html'); // Default toolbar. @@ -247,7 +247,7 @@ function testBuildToolbarJSSetting() { /** * Tests CKEditor::buildContentsCssJSSetting(). */ - function testBuildContentsCssJSSetting() { + public function testBuildContentsCssJSSetting() { $editor = Editor::load('filtered_html'); // Default toolbar. @@ -284,7 +284,7 @@ function testBuildContentsCssJSSetting() { /** * Tests Internal::getConfig(). */ - function testInternalGetConfig() { + public function testInternalGetConfig() { $editor = Editor::load('filtered_html'); $internal_plugin = $this->container->get('plugin.manager.ckeditor.plugin')->createInstance('internal'); @@ -305,7 +305,7 @@ function testInternalGetConfig() { /** * Tests StylesCombo::getConfig(). */ - function testStylesComboGetConfig() { + public function testStylesComboGetConfig() { $editor = Editor::load('filtered_html'); $stylescombo_plugin = $this->container->get('plugin.manager.ckeditor.plugin')->createInstance('stylescombo'); @@ -364,7 +364,7 @@ function testStylesComboGetConfig() { /** * Tests language list availability in CKEditor. */ - function testLanguages() { + public function testLanguages() { // Get CKEditor supported language codes and spot-check. $this->enableModules(array('language')); $this->installConfig(array('language')); @@ -389,7 +389,7 @@ function testLanguages() { /** * Tests that CKEditor plugins participate in JS translation. */ - function testJSTranslation() { + public function testJSTranslation() { $this->enableModules(array('language', 'locale')); $this->installSchema('locale', 'locales_source'); $this->installSchema('locale', 'locales_location'); diff --git a/core/modules/color/src/Tests/ColorConfigSchemaTest.php b/core/modules/color/src/Tests/ColorConfigSchemaTest.php index 941e520..e548f1c 100644 --- a/core/modules/color/src/Tests/ColorConfigSchemaTest.php +++ b/core/modules/color/src/Tests/ColorConfigSchemaTest.php @@ -40,7 +40,7 @@ protected function setUp() { /** * Tests whether the color config schema is valid. */ - function testValidColorConfigSchema() { + public function testValidColorConfigSchema() { $settings_path = 'admin/appearance/settings/bartik'; $edit['scheme'] = ''; $edit['palette[bg]'] = '#123456'; diff --git a/core/modules/color/src/Tests/ColorSafePreviewTest.php b/core/modules/color/src/Tests/ColorSafePreviewTest.php index 813a92e..933d18d 100644 --- a/core/modules/color/src/Tests/ColorSafePreviewTest.php +++ b/core/modules/color/src/Tests/ColorSafePreviewTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Ensures color preview.html is sanitized. */ - function testColorPreview() { + public function testColorPreview() { // Install the color test theme. \Drupal::service('theme_handler')->install(['color_test_theme']); $this->drupalLogin($this->bigUser); diff --git a/core/modules/color/src/Tests/ColorTest.php b/core/modules/color/src/Tests/ColorTest.php index 578f7fb..540eddc 100644 --- a/core/modules/color/src/Tests/ColorTest.php +++ b/core/modules/color/src/Tests/ColorTest.php @@ -84,7 +84,7 @@ protected function setUp() { /** * Tests the Color module functionality. */ - function testColor() { + public function testColor() { foreach ($this->themes as $theme => $test_values) { $this->_testColor($theme, $test_values); } @@ -151,7 +151,7 @@ function _testColor($theme, $test_values) { /** * Tests whether the provided color is valid. */ - function testValidColor() { + public function testValidColor() { $this->config('system.theme') ->set('default', 'bartik') ->save(); @@ -176,7 +176,7 @@ function testValidColor() { /** * Test whether the custom logo is used in the color preview. */ - function testLogoSettingOverride() { + public function testLogoSettingOverride() { $this->drupalLogin($this->bigUser); $edit = array( 'default_logo' => FALSE, @@ -192,7 +192,7 @@ function testLogoSettingOverride() { /** * Test whether the scheme can be set, viewed anonymously and reset. */ - function testOverrideAndResetScheme() { + public function testOverrideAndResetScheme() { $settings_path = 'admin/appearance/settings/bartik'; $this->config('system.theme') ->set('default', 'bartik') diff --git a/core/modules/comment/src/Tests/CommentActionsTest.php b/core/modules/comment/src/Tests/CommentActionsTest.php index 220be20..992da8a 100644 --- a/core/modules/comment/src/Tests/CommentActionsTest.php +++ b/core/modules/comment/src/Tests/CommentActionsTest.php @@ -22,7 +22,7 @@ class CommentActionsTest extends CommentTestBase { /** * Tests comment publish and unpublish actions. */ - function testCommentPublishUnpublishActions() { + public function testCommentPublishUnpublishActions() { $this->drupalLogin($this->webUser); $comment_text = $this->randomMachineName(); $subject = $this->randomMachineName(); @@ -42,7 +42,7 @@ function testCommentPublishUnpublishActions() { /** * Tests the unpublish comment by keyword action. */ - function testCommentUnpublishByKeyword() { + public function testCommentUnpublishByKeyword() { $this->drupalLogin($this->adminUser); $keyword_1 = $this->randomMachineName(); $keyword_2 = $this->randomMachineName(); diff --git a/core/modules/comment/src/Tests/CommentAdminTest.php b/core/modules/comment/src/Tests/CommentAdminTest.php index 5ae57cb..1727074 100644 --- a/core/modules/comment/src/Tests/CommentAdminTest.php +++ b/core/modules/comment/src/Tests/CommentAdminTest.php @@ -21,7 +21,7 @@ protected function setUp() { /** * Test comment approval functionality through admin/content/comment. */ - function testApprovalAdminInterface() { + public function testApprovalAdminInterface() { // Set anonymous comments to require approval. user_role_change_permissions(RoleInterface::ANONYMOUS_ID, array( 'access comments' => TRUE, @@ -103,7 +103,7 @@ function testApprovalAdminInterface() { /** * Tests comment approval functionality through the node interface. */ - function testApprovalNodeInterface() { + public function testApprovalNodeInterface() { // Set anonymous comments to require approval. user_role_change_permissions(RoleInterface::ANONYMOUS_ID, array( 'access comments' => TRUE, diff --git a/core/modules/comment/src/Tests/CommentAnonymousTest.php b/core/modules/comment/src/Tests/CommentAnonymousTest.php index 660d754..42438a1 100644 --- a/core/modules/comment/src/Tests/CommentAnonymousTest.php +++ b/core/modules/comment/src/Tests/CommentAnonymousTest.php @@ -30,7 +30,7 @@ protected function setUp() { /** * Tests anonymous comment functionality. */ - function testAnonymous() { + public function testAnonymous() { $this->drupalLogin($this->adminUser); $this->setCommentAnonymous(COMMENT_ANONYMOUS_MAYNOT_CONTACT); $this->drupalLogout(); diff --git a/core/modules/comment/src/Tests/CommentBlockTest.php b/core/modules/comment/src/Tests/CommentBlockTest.php index de92238..3b48bb8 100644 --- a/core/modules/comment/src/Tests/CommentBlockTest.php +++ b/core/modules/comment/src/Tests/CommentBlockTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests the recent comments block. */ - function testRecentCommentBlock() { + public function testRecentCommentBlock() { $this->drupalLogin($this->adminUser); $block = $this->drupalPlaceBlock('views_block:comments_recent-block_1'); diff --git a/core/modules/comment/src/Tests/CommentCSSTest.php b/core/modules/comment/src/Tests/CommentCSSTest.php index f015f3f..b651e63 100644 --- a/core/modules/comment/src/Tests/CommentCSSTest.php +++ b/core/modules/comment/src/Tests/CommentCSSTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Tests CSS classes on comments. */ - function testCommentClasses() { + public function testCommentClasses() { // Create all permutations for comments, users, and nodes. $parameters = array( 'node_uid' => array(0, $this->webUser->id()), diff --git a/core/modules/comment/src/Tests/CommentFieldsTest.php b/core/modules/comment/src/Tests/CommentFieldsTest.php index 46de1be..bfa6c1f 100644 --- a/core/modules/comment/src/Tests/CommentFieldsTest.php +++ b/core/modules/comment/src/Tests/CommentFieldsTest.php @@ -24,7 +24,7 @@ class CommentFieldsTest extends CommentTestBase { /** * Tests that the default 'comment_body' field is correctly added. */ - function testCommentDefaultFields() { + public function testCommentDefaultFields() { // Do not make assumptions on default node types created by the test // installation profile, and create our own. $this->drupalCreateContentType(array('type' => 'test_node_type')); @@ -183,7 +183,7 @@ public function testCommentFieldCreate() { /** * Tests that comment module works when installed after a content module. */ - function testCommentInstallAfterContentModule() { + public function testCommentInstallAfterContentModule() { // Create a user to do module administration. $this->adminUser = $this->drupalCreateUser(array('access administration pages', 'administer modules')); $this->drupalLogin($this->adminUser); diff --git a/core/modules/comment/src/Tests/CommentLanguageTest.php b/core/modules/comment/src/Tests/CommentLanguageTest.php index c60f827..6856c20 100644 --- a/core/modules/comment/src/Tests/CommentLanguageTest.php +++ b/core/modules/comment/src/Tests/CommentLanguageTest.php @@ -76,7 +76,7 @@ protected function setUp() { /** * Test that comment language is properly set. */ - function testCommentLanguage() { + public function testCommentLanguage() { // Create two nodes, one for english and one for french, and comment each // node using both english and french as content language by changing URL diff --git a/core/modules/comment/src/Tests/CommentNodeAccessTest.php b/core/modules/comment/src/Tests/CommentNodeAccessTest.php index e69bfde..0ad6f31 100644 --- a/core/modules/comment/src/Tests/CommentNodeAccessTest.php +++ b/core/modules/comment/src/Tests/CommentNodeAccessTest.php @@ -43,7 +43,7 @@ protected function setUp() { /** * Test that threaded comments can be viewed. */ - function testThreadedCommentView() { + public function testThreadedCommentView() { // Set comments to have subject required and preview disabled. $this->drupalLogin($this->adminUser); $this->setCommentPreview(DRUPAL_DISABLED); diff --git a/core/modules/comment/src/Tests/CommentNodeChangesTest.php b/core/modules/comment/src/Tests/CommentNodeChangesTest.php index 9fed4df..11c6853 100644 --- a/core/modules/comment/src/Tests/CommentNodeChangesTest.php +++ b/core/modules/comment/src/Tests/CommentNodeChangesTest.php @@ -16,7 +16,7 @@ class CommentNodeChangesTest extends CommentTestBase { /** * Tests that comments are deleted with the node. */ - function testNodeDeletion() { + public function testNodeDeletion() { $this->drupalLogin($this->webUser); $comment = $this->postComment($this->node, $this->randomMachineName(), $this->randomMachineName()); $this->assertTrue($comment->id(), 'The comment could be loaded.'); diff --git a/core/modules/comment/src/Tests/CommentNonNodeTest.php b/core/modules/comment/src/Tests/CommentNonNodeTest.php index 1b37c1f..da0afe0 100644 --- a/core/modules/comment/src/Tests/CommentNonNodeTest.php +++ b/core/modules/comment/src/Tests/CommentNonNodeTest.php @@ -249,7 +249,7 @@ function getUnapprovedComment($subject) { /** * Tests anonymous comment functionality. */ - function testCommentFunctionality() { + public function testCommentFunctionality() { $limited_user = $this->drupalCreateUser(array( 'administer entity_test fields' )); diff --git a/core/modules/comment/src/Tests/CommentPagerTest.php b/core/modules/comment/src/Tests/CommentPagerTest.php index 760e25f..9fdecb6 100644 --- a/core/modules/comment/src/Tests/CommentPagerTest.php +++ b/core/modules/comment/src/Tests/CommentPagerTest.php @@ -15,7 +15,7 @@ class CommentPagerTest extends CommentTestBase { /** * Confirms comment paging works correctly with flat and threaded comments. */ - function testCommentPaging() { + public function testCommentPaging() { $this->drupalLogin($this->adminUser); // Set comment variables. @@ -93,7 +93,7 @@ function testCommentPaging() { /** * Tests comment ordering and threading. */ - function testCommentOrderingThreading() { + public function testCommentOrderingThreading() { $this->drupalLogin($this->adminUser); // Set comment variables. @@ -192,7 +192,7 @@ function assertCommentOrder(array $comments, array $expected_order) { /** * Tests calculation of first page with new comment. */ - function testCommentNewPageIndicator() { + public function testCommentNewPageIndicator() { $this->drupalLogin($this->adminUser); // Set comment variables. @@ -272,7 +272,7 @@ function testCommentNewPageIndicator() { /** * Confirms comment paging works correctly with two pagers. */ - function testTwoPagers() { + public function testTwoPagers() { // Add another field to article content-type. $this->addDefaultCommentField('node', 'article', 'comment_2'); // Set default to display comment list with unique pager id. diff --git a/core/modules/comment/src/Tests/CommentPreviewTest.php b/core/modules/comment/src/Tests/CommentPreviewTest.php index 837a78a..cc5da05 100644 --- a/core/modules/comment/src/Tests/CommentPreviewTest.php +++ b/core/modules/comment/src/Tests/CommentPreviewTest.php @@ -26,7 +26,7 @@ class CommentPreviewTest extends CommentTestBase { /** * Tests comment preview. */ - function testCommentPreview() { + public function testCommentPreview() { // As admin user, configure comment settings. $this->drupalLogin($this->adminUser); $this->setCommentPreview(DRUPAL_OPTIONAL); @@ -123,7 +123,7 @@ public function testCommentPreviewDuplicateSubmission() { /** * Tests comment edit, preview, and save. */ - function testCommentEditPreviewSave() { + public function testCommentEditPreviewSave() { $web_user = $this->drupalCreateUser(array('access comments', 'post comments', 'skip comment approval', 'edit own comments')); $this->drupalLogin($this->adminUser); $this->setCommentPreview(DRUPAL_OPTIONAL); diff --git a/core/modules/comment/src/Tests/CommentRssTest.php b/core/modules/comment/src/Tests/CommentRssTest.php index ec673df..ea892a9 100644 --- a/core/modules/comment/src/Tests/CommentRssTest.php +++ b/core/modules/comment/src/Tests/CommentRssTest.php @@ -42,7 +42,7 @@ protected function setUp() { /** * Tests comments as part of an RSS feed. */ - function testCommentRss() { + public function testCommentRss() { // Find comment in RSS feed. $this->drupalLogin($this->webUser); $this->postComment($this->node, $this->randomMachineName(), $this->randomMachineName()); diff --git a/core/modules/comment/src/Tests/CommentStatisticsTest.php b/core/modules/comment/src/Tests/CommentStatisticsTest.php index 0c370d1..891e954 100644 --- a/core/modules/comment/src/Tests/CommentStatisticsTest.php +++ b/core/modules/comment/src/Tests/CommentStatisticsTest.php @@ -38,7 +38,7 @@ protected function setUp() { /** * Tests the node comment statistics. */ - function testCommentNodeCommentStatistics() { + public function testCommentNodeCommentStatistics() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); // Set comments to have subject and preview disabled. $this->drupalLogin($this->adminUser); diff --git a/core/modules/comment/src/Tests/CommentThreadingTest.php b/core/modules/comment/src/Tests/CommentThreadingTest.php index c8ea93e..7c3150c 100644 --- a/core/modules/comment/src/Tests/CommentThreadingTest.php +++ b/core/modules/comment/src/Tests/CommentThreadingTest.php @@ -13,7 +13,7 @@ class CommentThreadingTest extends CommentTestBase { /** * Tests the comment threading. */ - function testCommentThreading() { + public function testCommentThreading() { // Set comments to have a subject with preview disabled. $this->drupalLogin($this->adminUser); $this->setCommentPreview(DRUPAL_DISABLED); diff --git a/core/modules/comment/src/Tests/CommentTokenReplaceTest.php b/core/modules/comment/src/Tests/CommentTokenReplaceTest.php index eff5bf5..c6663fa 100644 --- a/core/modules/comment/src/Tests/CommentTokenReplaceTest.php +++ b/core/modules/comment/src/Tests/CommentTokenReplaceTest.php @@ -19,7 +19,7 @@ class CommentTokenReplaceTest extends CommentTestBase { /** * Creates a comment, then tests the tokens generated from it. */ - function testCommentTokenReplacement() { + public function testCommentTokenReplacement() { $token_service = \Drupal::token(); $language_interface = \Drupal::languageManager()->getCurrentLanguage(); $url_options = array( diff --git a/core/modules/comment/src/Tests/CommentTranslationUITest.php b/core/modules/comment/src/Tests/CommentTranslationUITest.php index b6fc4b9..52a8145 100644 --- a/core/modules/comment/src/Tests/CommentTranslationUITest.php +++ b/core/modules/comment/src/Tests/CommentTranslationUITest.php @@ -177,7 +177,7 @@ protected function doTestAuthoringInfo() { /** * Tests translate link on comment content admin page. */ - function testTranslateLinkCommentAdminPage() { + public function testTranslateLinkCommentAdminPage() { $this->adminUser = $this->drupalCreateUser(array_merge(parent::getTranslatorPermissions(), array('access administration pages', 'administer comments', 'skip comment approval'))); $this->drupalLogin($this->adminUser); diff --git a/core/modules/comment/src/Tests/CommentUninstallTest.php b/core/modules/comment/src/Tests/CommentUninstallTest.php index 4d86df9..fb92b4f 100644 --- a/core/modules/comment/src/Tests/CommentUninstallTest.php +++ b/core/modules/comment/src/Tests/CommentUninstallTest.php @@ -36,7 +36,7 @@ protected function setUp() { * * @throws \Drupal\Core\Extension\ModuleUninstallValidatorException */ - function testCommentUninstallWithField() { + public function testCommentUninstallWithField() { // Ensure that the field exists before uninstallation. $field_storage = FieldStorageConfig::loadByName('comment', 'comment_body'); $this->assertNotNull($field_storage, 'The comment_body field exists.'); @@ -55,7 +55,7 @@ function testCommentUninstallWithField() { /** * Tests if uninstallation succeeds if the field has been deleted beforehand. */ - function testCommentUninstallWithoutField() { + public function testCommentUninstallWithoutField() { // Manually delete the comment_body field before module uninstallation. $field_storage = FieldStorageConfig::loadByName('comment', 'comment_body'); $this->assertNotNull($field_storage, 'The comment_body field exists.'); diff --git a/core/modules/comment/src/Tests/Views/ArgumentUserUIDTest.php b/core/modules/comment/src/Tests/Views/ArgumentUserUIDTest.php index 63a0082..3d3901d 100644 --- a/core/modules/comment/src/Tests/Views/ArgumentUserUIDTest.php +++ b/core/modules/comment/src/Tests/Views/ArgumentUserUIDTest.php @@ -20,7 +20,7 @@ class ArgumentUserUIDTest extends CommentTestBase { */ public static $testViews = array('test_comment_user_uid'); - function testCommentUserUIDTest() { + public function testCommentUserUIDTest() { // Add an additional comment which is not created by the user. $new_user = User::create(['name' => 'new user']); $new_user->save(); diff --git a/core/modules/comment/src/Tests/Views/FilterUserUIDTest.php b/core/modules/comment/src/Tests/Views/FilterUserUIDTest.php index ee8281b..a5198d0 100644 --- a/core/modules/comment/src/Tests/Views/FilterUserUIDTest.php +++ b/core/modules/comment/src/Tests/Views/FilterUserUIDTest.php @@ -22,7 +22,7 @@ class FilterUserUIDTest extends CommentTestBase { */ public static $testViews = array('test_comment_user_uid'); - function testCommentUserUIDTest() { + public function testCommentUserUIDTest() { $view = Views::getView('test_comment_user_uid'); $view->setDisplay(); $view->removeHandler('default', 'argument', 'uid_touch'); diff --git a/core/modules/config/src/Tests/ConfigDependencyWebTest.php b/core/modules/config/src/Tests/ConfigDependencyWebTest.php index 5eb5206..ff09e4f 100644 --- a/core/modules/config/src/Tests/ConfigDependencyWebTest.php +++ b/core/modules/config/src/Tests/ConfigDependencyWebTest.php @@ -29,7 +29,7 @@ class ConfigDependencyWebTest extends WebTestBase { * * @see \Drupal\Core\Config\Entity\ConfigDependencyDeleteFormTrait */ - function testConfigDependencyDeleteFormTrait() { + public function testConfigDependencyDeleteFormTrait() { $this->drupalLogin($this->drupalCreateUser(['administer site configuration'])); /** @var \Drupal\Core\Config\Entity\ConfigEntityStorage $storage */ diff --git a/core/modules/config/src/Tests/ConfigEntityListMultilingualTest.php b/core/modules/config/src/Tests/ConfigEntityListMultilingualTest.php index f5c12d8..722c251 100644 --- a/core/modules/config/src/Tests/ConfigEntityListMultilingualTest.php +++ b/core/modules/config/src/Tests/ConfigEntityListMultilingualTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Tests the listing UI with different language scenarios. */ - function testListUI() { + public function testListUI() { // Log in as an administrative user to access the full menu trail. $this->drupalLogin($this->drupalCreateUser(array('access administration pages', 'administer site configuration'))); diff --git a/core/modules/config/src/Tests/ConfigEntityListTest.php b/core/modules/config/src/Tests/ConfigEntityListTest.php index 9d31de0..c4480f3 100644 --- a/core/modules/config/src/Tests/ConfigEntityListTest.php +++ b/core/modules/config/src/Tests/ConfigEntityListTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Tests entity list builder methods. */ - function testList() { + public function testList() { $controller = \Drupal::entityManager()->getListBuilder('config_test'); // Test getStorage() method. @@ -147,7 +147,7 @@ function testList() { /** * Tests the listing UI. */ - function testListUI() { + public function testListUI() { // Log in as an administrative user to access the full menu trail. $this->drupalLogin($this->drupalCreateUser(array('access administration pages', 'administer site configuration'))); diff --git a/core/modules/config/src/Tests/ConfigEntityStatusUITest.php b/core/modules/config/src/Tests/ConfigEntityStatusUITest.php index 95dd9da..6e7e42d 100644 --- a/core/modules/config/src/Tests/ConfigEntityStatusUITest.php +++ b/core/modules/config/src/Tests/ConfigEntityStatusUITest.php @@ -21,7 +21,7 @@ class ConfigEntityStatusUITest extends WebTestBase { /** * Tests status operations. */ - function testCRUD() { + public function testCRUD() { $this->drupalLogin($this->drupalCreateUser(['administer site configuration'])); $id = strtolower($this->randomMachineName()); diff --git a/core/modules/config/src/Tests/ConfigEntityTest.php b/core/modules/config/src/Tests/ConfigEntityTest.php index a8a8f0f..871a17d 100644 --- a/core/modules/config/src/Tests/ConfigEntityTest.php +++ b/core/modules/config/src/Tests/ConfigEntityTest.php @@ -32,7 +32,7 @@ class ConfigEntityTest extends WebTestBase { /** * Tests CRUD operations. */ - function testCRUD() { + public function testCRUD() { $default_langcode = \Drupal::languageManager()->getDefaultLanguage()->getId(); // Verify default properties on a newly created empty entity. $empty = entity_create('config_test'); @@ -230,7 +230,7 @@ function testCRUD() { /** * Tests CRUD operations through the UI. */ - function testCRUDUI() { + public function testCRUDUI() { $this->drupalLogin($this->drupalCreateUser(['administer site configuration'])); $id = strtolower($this->randomMachineName()); diff --git a/core/modules/config/src/Tests/ConfigExportUITest.php b/core/modules/config/src/Tests/ConfigExportUITest.php index 095c585..5489934 100644 --- a/core/modules/config/src/Tests/ConfigExportUITest.php +++ b/core/modules/config/src/Tests/ConfigExportUITest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Tests export of configuration. */ - function testExport() { + public function testExport() { // Verify the export page with export submit button is available. $this->drupalGet('admin/config/development/configuration/full/export'); $this->assertFieldById('edit-submit', t('Export')); diff --git a/core/modules/config/src/Tests/ConfigImportUITest.php b/core/modules/config/src/Tests/ConfigImportUITest.php index afa9b99..cb81458 100644 --- a/core/modules/config/src/Tests/ConfigImportUITest.php +++ b/core/modules/config/src/Tests/ConfigImportUITest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Tests importing configuration. */ - function testImport() { + public function testImport() { $name = 'system.site'; $dynamic_name = 'config_test.dynamic.new'; /** @var \Drupal\Core\Config\StorageInterface $sync */ @@ -221,7 +221,7 @@ function testImport() { /** * Tests concurrent importing of configuration. */ - function testImportLock() { + public function testImportLock() { // Create updated configuration object. $new_site_name = 'Config import test ' . $this->randomString(); $this->prepareSiteNameUpdate($new_site_name); @@ -248,7 +248,7 @@ function testImportLock() { /** * Tests verification of site UUID before importing configuration. */ - function testImportSiteUuidValidation() { + public function testImportSiteUuidValidation() { $sync = \Drupal::service('config.storage.sync'); // Create updated configuration object. $config_data = $this->config('system.site')->get(); @@ -265,7 +265,7 @@ function testImportSiteUuidValidation() { /** * Tests the screen that shows differences between active and sync. */ - function testImportDiff() { + public function testImportDiff() { $sync = $this->container->get('config.storage.sync'); $config_name = 'config_test.system'; $change_key = 'foo'; @@ -388,7 +388,7 @@ function prepareSiteNameUpdate($new_site_name) { /** * Tests an import that results in an error. */ - function testImportErrorLog() { + public function testImportErrorLog() { $name_primary = 'config_test.dynamic.primary'; $name_secondary = 'config_test.dynamic.secondary'; $sync = $this->container->get('config.storage.sync'); diff --git a/core/modules/config/src/Tests/ConfigImportUploadTest.php b/core/modules/config/src/Tests/ConfigImportUploadTest.php index 9836d77..cd51655 100644 --- a/core/modules/config/src/Tests/ConfigImportUploadTest.php +++ b/core/modules/config/src/Tests/ConfigImportUploadTest.php @@ -35,7 +35,7 @@ protected function setUp() { /** * Tests importing configuration. */ - function testImport() { + public function testImport() { // Verify access to the config upload form. $this->drupalGet('admin/config/development/configuration/full/import'); $this->assertResponse(200); diff --git a/core/modules/config/src/Tests/ConfigInstallProfileOverrideTest.php b/core/modules/config/src/Tests/ConfigInstallProfileOverrideTest.php index c2cea6d..9c007c7 100644 --- a/core/modules/config/src/Tests/ConfigInstallProfileOverrideTest.php +++ b/core/modules/config/src/Tests/ConfigInstallProfileOverrideTest.php @@ -28,7 +28,7 @@ class ConfigInstallProfileOverrideTest extends WebTestBase { /** * Tests install profile config changes. */ - function testInstallProfileConfigOverwrite() { + public function testInstallProfileConfigOverwrite() { $config_name = 'system.cron'; // The expected configuration from the system module. $expected_original_data = array( diff --git a/core/modules/config/src/Tests/ConfigInstallWebTest.php b/core/modules/config/src/Tests/ConfigInstallWebTest.php index fc6cee4..20b38ac 100644 --- a/core/modules/config/src/Tests/ConfigInstallWebTest.php +++ b/core/modules/config/src/Tests/ConfigInstallWebTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests module re-installation. */ - function testIntegrationModuleReinstallation() { + public function testIntegrationModuleReinstallation() { $default_config = 'config_integration_test.settings'; $default_configuration_entity = 'config_test.dynamic.config_integration_test'; diff --git a/core/modules/config/src/Tests/ConfigLanguageOverrideWebTest.php b/core/modules/config/src/Tests/ConfigLanguageOverrideWebTest.php index f966e5f..0403a8f 100644 --- a/core/modules/config/src/Tests/ConfigLanguageOverrideWebTest.php +++ b/core/modules/config/src/Tests/ConfigLanguageOverrideWebTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Tests translating the site name. */ - function testSiteNameTranslation() { + public function testSiteNameTranslation() { $adminUser = $this->drupalCreateUser(array('administer site configuration', 'administer languages')); $this->drupalLogin($adminUser); diff --git a/core/modules/contact/src/Tests/ContactAuthenticatedUserTest.php b/core/modules/contact/src/Tests/ContactAuthenticatedUserTest.php index eb7c527..0103200 100644 --- a/core/modules/contact/src/Tests/ContactAuthenticatedUserTest.php +++ b/core/modules/contact/src/Tests/ContactAuthenticatedUserTest.php @@ -21,7 +21,7 @@ class ContactAuthenticatedUserTest extends WebTestBase { /** * Tests that name and email fields are not present for authenticated users. */ - function testContactSiteWideTextfieldsLoggedInTestCase() { + public function testContactSiteWideTextfieldsLoggedInTestCase() { $this->drupalLogin($this->drupalCreateUser(array('access site-wide contact form'))); $this->drupalGet('contact'); diff --git a/core/modules/contact/src/Tests/ContactPersonalTest.php b/core/modules/contact/src/Tests/ContactPersonalTest.php index 936aed5..d1135c6 100644 --- a/core/modules/contact/src/Tests/ContactPersonalTest.php +++ b/core/modules/contact/src/Tests/ContactPersonalTest.php @@ -58,7 +58,7 @@ protected function setUp() { /** * Tests that mails for contact messages are correctly sent. */ - function testSendPersonalContactMessage() { + public function testSendPersonalContactMessage() { // Ensure that the web user's email needs escaping. $mail = $this->webUser->getUsername() . '&escaped@example.com'; $this->webUser->setEmail($mail)->save(); @@ -103,7 +103,7 @@ function testSendPersonalContactMessage() { /** * Tests access to the personal contact form. */ - function testPersonalContactAccess() { + public function testPersonalContactAccess() { // Test allowed access to admin user's contact form. $this->drupalLogin($this->webUser); $this->drupalGet('user/' . $this->adminUser->id() . '/contact'); @@ -223,7 +223,7 @@ function testPersonalContactAccess() { /** * Tests the personal contact form flood protection. */ - function testPersonalContactFlood() { + public function testPersonalContactFlood() { $flood_limit = 3; $this->config('contact.settings')->set('flood.limit', $flood_limit)->save(); @@ -248,7 +248,7 @@ function testPersonalContactFlood() { /** * Tests the personal contact form based access when an admin adds users. */ - function testAdminContact() { + public function testAdminContact() { user_role_grant_permissions(RoleInterface::ANONYMOUS_ID, array('access user contact forms')); $this->checkContactAccess(200); $this->checkContactAccess(403, FALSE); diff --git a/core/modules/contact/src/Tests/ContactSitewideTest.php b/core/modules/contact/src/Tests/ContactSitewideTest.php index b049f25..dac7b37 100644 --- a/core/modules/contact/src/Tests/ContactSitewideTest.php +++ b/core/modules/contact/src/Tests/ContactSitewideTest.php @@ -41,7 +41,7 @@ protected function setUp() { /** * Tests configuration options and the site-wide contact form. */ - function testSiteWideContact() { + public function testSiteWideContact() { // Create and log in administrative user. $admin_user = $this->drupalCreateUser(array( 'access site-wide contact form', @@ -303,7 +303,7 @@ function testSiteWideContact() { /** * Tests auto-reply on the site-wide contact form. */ - function testAutoReply() { + public function testAutoReply() { // Create and log in administrative user. $admin_user = $this->drupalCreateUser(array('access site-wide contact form', 'administer contact forms', 'administer permissions', 'administer users')); $this->drupalLogin($admin_user); diff --git a/core/modules/content_translation/src/Tests/ContentTranslationSettingsTest.php b/core/modules/content_translation/src/Tests/ContentTranslationSettingsTest.php index d7490c0..fb34e89 100644 --- a/core/modules/content_translation/src/Tests/ContentTranslationSettingsTest.php +++ b/core/modules/content_translation/src/Tests/ContentTranslationSettingsTest.php @@ -43,7 +43,7 @@ protected function setUp() { /** * Tests that the settings UI works as expected. */ - function testSettingsUI() { + public function testSettingsUI() { // Check for the content_translation_menu_links_discovered_alter() changes. $this->drupalGet('admin/config'); $this->assertLink('Content language and translation'); @@ -200,7 +200,7 @@ function testSettingsUI() { /** * Tests the language settings checkbox on account settings page. */ - function testAccountLanguageSettingsUI() { + public function testAccountLanguageSettingsUI() { // Make sure the checkbox is available and not checked by default. $this->drupalGet('admin/config/people/accounts'); $this->assertField('language[content_translation]'); @@ -245,7 +245,7 @@ protected function assertSettings($entity_type, $bundle, $enabled, $edit) { /** * Tests that field setting depends on bundle translatability. */ - function testFieldTranslatableSettingsUI() { + public function testFieldTranslatableSettingsUI() { // At least one field needs to be translatable to enable article for // translation. Create an extra field to be used for this purpose. We use // the UI to test our form alterations. @@ -280,7 +280,7 @@ function testFieldTranslatableSettingsUI() { /** * Tests the translatable settings checkbox for untranslatable entities. */ - function testNonTranslatableTranslationSettingsUI() { + public function testNonTranslatableTranslationSettingsUI() { $this->drupalGet('admin/config/regional/content-language'); $this->assertNoField('settings[entity_test][entity_test][translatable]'); } diff --git a/core/modules/content_translation/src/Tests/ContentTranslationSyncImageTest.php b/core/modules/content_translation/src/Tests/ContentTranslationSyncImageTest.php index 5f881a7..4753803 100644 --- a/core/modules/content_translation/src/Tests/ContentTranslationSyncImageTest.php +++ b/core/modules/content_translation/src/Tests/ContentTranslationSyncImageTest.php @@ -82,7 +82,7 @@ protected function getEditorPermissions() { /** * Tests image field field synchronization. */ - function testImageFieldSync() { + public function testImageFieldSync() { // Check that the alt and title fields are enabled for the image field. $this->drupalLogin($this->editor); $this->drupalGet('entity_test_mul/structure/' . $this->entityTypeId . '/fields/' . $this->entityTypeId . '.' . $this->entityTypeId . '.' . $this->fieldName); diff --git a/core/modules/content_translation/src/Tests/ContentTranslationUISkipTest.php b/core/modules/content_translation/src/Tests/ContentTranslationUISkipTest.php index 2f8be23..1db0a18 100644 --- a/core/modules/content_translation/src/Tests/ContentTranslationUISkipTest.php +++ b/core/modules/content_translation/src/Tests/ContentTranslationUISkipTest.php @@ -21,7 +21,7 @@ class ContentTranslationUISkipTest extends WebTestBase { /** * Tests the content_translation_ui_skip key functionality. */ - function testUICheckSkip() { + public function testUICheckSkip() { $admin_user = $this->drupalCreateUser(array( 'translate any entity', 'administer content translation', diff --git a/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php b/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php index e9fff8d..f09a2a1 100644 --- a/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php +++ b/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php @@ -52,7 +52,7 @@ /** * Tests the basic translation UI. */ - function testTranslationUI() { + public function testTranslationUI() { $this->doTestBasicTranslation(); $this->doTestTranslationOverview(); $this->doTestOutdatedStatus(); diff --git a/core/modules/content_translation/src/Tests/ContentTranslationWorkflowsTest.php b/core/modules/content_translation/src/Tests/ContentTranslationWorkflowsTest.php index f3c4c53..97b1b7f 100644 --- a/core/modules/content_translation/src/Tests/ContentTranslationWorkflowsTest.php +++ b/core/modules/content_translation/src/Tests/ContentTranslationWorkflowsTest.php @@ -76,7 +76,7 @@ protected function setupEntity() { /** * Test simple and editorial translation workflows. */ - function testWorkflows() { + public function testWorkflows() { // Test workflows for the editor. $expected_status = [ 'edit' => 200, diff --git a/core/modules/content_translation/tests/src/Kernel/ContentTranslationConfigImportTest.php b/core/modules/content_translation/tests/src/Kernel/ContentTranslationConfigImportTest.php index f650270..120d362 100644 --- a/core/modules/content_translation/tests/src/Kernel/ContentTranslationConfigImportTest.php +++ b/core/modules/content_translation/tests/src/Kernel/ContentTranslationConfigImportTest.php @@ -58,7 +58,7 @@ protected function setUp() { /** * Tests config import updates. */ - function testConfigImportUpdates() { + public function testConfigImportUpdates() { $entity_type_id = 'entity_test_mul'; $config_id = $entity_type_id . '.' . $entity_type_id; $config_name = 'language.content_settings.' . $config_id; diff --git a/core/modules/content_translation/tests/src/Kernel/ContentTranslationSettingsApiTest.php b/core/modules/content_translation/tests/src/Kernel/ContentTranslationSettingsApiTest.php index 291cdf9..f1ff3dc 100644 --- a/core/modules/content_translation/tests/src/Kernel/ContentTranslationSettingsApiTest.php +++ b/core/modules/content_translation/tests/src/Kernel/ContentTranslationSettingsApiTest.php @@ -29,7 +29,7 @@ protected function setUp() { /** * Tests that enabling translation via the API triggers schema updates. */ - function testSettingsApi() { + public function testSettingsApi() { $this->container->get('content_translation.manager')->setEnabled('entity_test_mul', 'entity_test_mul', TRUE); $result = db_field_exists('entity_test_mul_property_data', 'content_translation_source') && diff --git a/core/modules/contextual/src/Tests/ContextualDynamicContextTest.php b/core/modules/contextual/src/Tests/ContextualDynamicContextTest.php index e5d2c85..fd2f49b 100644 --- a/core/modules/contextual/src/Tests/ContextualDynamicContextTest.php +++ b/core/modules/contextual/src/Tests/ContextualDynamicContextTest.php @@ -64,7 +64,7 @@ protected function setUp() { * Ensures that contextual link placeholders always exist, even if the user is * not allowed to use contextual links. */ - function testDifferentPermissions() { + public function testDifferentPermissions() { $this->drupalLogin($this->editorUser); // Create three nodes in the following order: diff --git a/core/modules/contextual/tests/src/Kernel/ContextualUnitTest.php b/core/modules/contextual/tests/src/Kernel/ContextualUnitTest.php index 680303d..b3b4cf1 100644 --- a/core/modules/contextual/tests/src/Kernel/ContextualUnitTest.php +++ b/core/modules/contextual/tests/src/Kernel/ContextualUnitTest.php @@ -110,7 +110,7 @@ function _contextual_links_id_testcases() { /** * Tests _contextual_links_to_id(). */ - function testContextualLinksToId() { + public function testContextualLinksToId() { $tests = $this->_contextual_links_id_testcases(); foreach ($tests as $test) { $this->assertIdentical(_contextual_links_to_id($test['links']), $test['id']); @@ -120,7 +120,7 @@ function testContextualLinksToId() { /** * Tests _contextual_id_to_links(). */ - function testContextualIdToLinks() { + public function testContextualIdToLinks() { $tests = $this->_contextual_links_id_testcases(); foreach ($tests as $test) { $this->assertIdentical(_contextual_id_to_links($test['id']), $test['links']); diff --git a/core/modules/datetime/src/Tests/DateTimeFieldTest.php b/core/modules/datetime/src/Tests/DateTimeFieldTest.php index bf3ba07..e754a0b 100644 --- a/core/modules/datetime/src/Tests/DateTimeFieldTest.php +++ b/core/modules/datetime/src/Tests/DateTimeFieldTest.php @@ -114,7 +114,7 @@ protected function setUp() { /** * Tests date field functionality. */ - function testDateField() { + public function testDateField() { $field_name = $this->fieldStorage->getName(); // Display creation form. @@ -244,7 +244,7 @@ function testDateField() { /** * Tests date and time field. */ - function testDatetimeField() { + public function testDatetimeField() { $field_name = $this->fieldStorage->getName(); // Change the field to a datetime field. $this->fieldStorage->setSetting('datetime_type', 'datetime'); @@ -384,7 +384,7 @@ function testDatetimeField() { /** * Tests Date List Widget functionality. */ - function testDatelistWidget() { + public function testDatelistWidget() { $field_name = $this->fieldStorage->getName(); // Ensure field is set to a date only field. @@ -626,7 +626,7 @@ protected function datelistDataProvider() { /** * Test default value functionality. */ - function testDefaultValue() { + public function testDefaultValue() { // Create a test content type. $this->drupalCreateContentType(array('type' => 'date_content')); @@ -728,7 +728,7 @@ function testDefaultValue() { /** * Test that invalid values are caught and marked as invalid. */ - function testInvalidField() { + public function testInvalidField() { // Change the field to a datetime field. $this->fieldStorage->setSetting('datetime_type', 'datetime'); $this->fieldStorage->save(); diff --git a/core/modules/dblog/src/Tests/ConnectionFailureTest.php b/core/modules/dblog/src/Tests/ConnectionFailureTest.php index d800224..d4376e0 100644 --- a/core/modules/dblog/src/Tests/ConnectionFailureTest.php +++ b/core/modules/dblog/src/Tests/ConnectionFailureTest.php @@ -17,7 +17,7 @@ class ConnectionFailureTest extends WebTestBase { /** * Tests logging of connection failures. */ - function testConnectionFailureLogging() { + public function testConnectionFailureLogging() { $logger = \Drupal::service('logger.factory'); // MySQL errors like "1153 - Got a packet bigger than 'max_allowed_packet' diff --git a/core/modules/dblog/src/Tests/DbLogTest.php b/core/modules/dblog/src/Tests/DbLogTest.php index 90cb6c2..d865ae6 100644 --- a/core/modules/dblog/src/Tests/DbLogTest.php +++ b/core/modules/dblog/src/Tests/DbLogTest.php @@ -58,7 +58,7 @@ protected function setUp() { * Database Logging module functionality through both the admin and user * interfaces. */ - function testDbLog() { + public function testDbLog() { // Log in the admin user. $this->drupalLogin($this->adminUser); diff --git a/core/modules/editor/src/Tests/EditorSecurityTest.php b/core/modules/editor/src/Tests/EditorSecurityTest.php index c771439..71174bd 100644 --- a/core/modules/editor/src/Tests/EditorSecurityTest.php +++ b/core/modules/editor/src/Tests/EditorSecurityTest.php @@ -222,7 +222,7 @@ protected function setUp() { * * Tests 8 scenarios. Tests only with a text editor that is not XSS-safe. */ - function testInitialSecurity() { + public function testInitialSecurity() { $expected = array( array( 'node_id' => 1, @@ -302,7 +302,7 @@ function testInitialSecurity() { * format and contains a '; @@ -56,7 +56,7 @@ function testSearchLabelXSS() { /** * Tests the failed search text, and various other text on the search page. */ - function testSearchText() { + public function testSearchText() { $this->drupalLogin($this->searchingUser); $this->drupalGet('search/node'); $this->assertText(t('Enter your keywords')); diff --git a/core/modules/search/src/Tests/SearchPreprocessLangcodeTest.php b/core/modules/search/src/Tests/SearchPreprocessLangcodeTest.php index 38cb350..8f8d69c 100644 --- a/core/modules/search/src/Tests/SearchPreprocessLangcodeTest.php +++ b/core/modules/search/src/Tests/SearchPreprocessLangcodeTest.php @@ -38,7 +38,7 @@ protected function setUp() { /** * Tests that hook_search_preprocess() returns the correct langcode. */ - function testPreprocessLangcode() { + public function testPreprocessLangcode() { // Create a node. $this->node = $this->drupalCreateNode(array('body' => array(array()), 'langcode' => 'en')); @@ -63,7 +63,7 @@ function testPreprocessLangcode() { /** * Tests stemming for hook_search_preprocess(). */ - function testPreprocessStemming() { + public function testPreprocessStemming() { // Create a node. $this->node = $this->drupalCreateNode(array( 'title' => 'we are testing', diff --git a/core/modules/search/src/Tests/SearchQueryAlterTest.php b/core/modules/search/src/Tests/SearchQueryAlterTest.php index 0cb473f..93c82d6 100644 --- a/core/modules/search/src/Tests/SearchQueryAlterTest.php +++ b/core/modules/search/src/Tests/SearchQueryAlterTest.php @@ -18,7 +18,7 @@ class SearchQueryAlterTest extends SearchTestBase { /** * Tests that the query alter works. */ - function testQueryAlter() { + public function testQueryAlter() { // Log in with sufficient privileges. $this->drupalLogin($this->drupalCreateUser(array('create page content', 'search content'))); diff --git a/core/modules/search/src/Tests/SearchSimplifyTest.php b/core/modules/search/src/Tests/SearchSimplifyTest.php index bdb96c9..689dc1b 100644 --- a/core/modules/search/src/Tests/SearchSimplifyTest.php +++ b/core/modules/search/src/Tests/SearchSimplifyTest.php @@ -12,7 +12,7 @@ class SearchSimplifyTest extends SearchTestBase { /** * Tests that all Unicode characters simplify correctly. */ - function testSearchSimplifyUnicode() { + public function testSearchSimplifyUnicode() { // This test uses a file that was constructed so that the even lines are // boundary characters, and the odd lines are valid word characters. (It // was generated as a sequence of all the Unicode characters, and then the @@ -64,7 +64,7 @@ function testSearchSimplifyUnicode() { /** * Tests that search_simplify() does the right thing with punctuation. */ - function testSearchSimplifyPunctuation() { + public function testSearchSimplifyPunctuation() { $cases = array( array('20.03/94-28,876', '20039428876', 'Punctuation removed from numbers'), array('great...drupal--module', 'great drupal module', 'Multiple dot and dashes are word boundaries'), diff --git a/core/modules/search/src/Tests/SearchTokenizerTest.php b/core/modules/search/src/Tests/SearchTokenizerTest.php index f8b3ccb..02321ef 100644 --- a/core/modules/search/src/Tests/SearchTokenizerTest.php +++ b/core/modules/search/src/Tests/SearchTokenizerTest.php @@ -18,7 +18,7 @@ class SearchTokenizerTest extends SearchTestBase { * character classes are tokenized properly. See PREG_CLASS_CKJ for more * information. */ - function testTokenizer() { + public function testTokenizer() { // Set the minimum word size to 1 (to split all CJK characters) and make // sure CJK tokenizing is turned on. $this->config('search.settings') @@ -106,7 +106,7 @@ function testTokenizer() { * This is just a sanity check - it verifies that strings of letters are * not tokenized. */ - function testNoTokenizer() { + public function testNoTokenizer() { // Set the minimum word size to 1 (to split all CJK characters) and make // sure CJK tokenizing is turned on. $this->config('search.settings') diff --git a/core/modules/search/tests/src/Kernel/SearchExcerptTest.php b/core/modules/search/tests/src/Kernel/SearchExcerptTest.php index 85c0917..7788336 100644 --- a/core/modules/search/tests/src/Kernel/SearchExcerptTest.php +++ b/core/modules/search/tests/src/Kernel/SearchExcerptTest.php @@ -27,7 +27,7 @@ class SearchExcerptTest extends KernelTestBase { * contains either highlighted keywords or the original marked * up string if no keywords matched the string. */ - function testSearchExcerpt() { + public function testSearchExcerpt() { // Make some text with entities and tags. $text = 'The quick brown fox & jumps

over

the lazy dog'; $expected = 'The quick brown fox & jumps over the lazy dog'; @@ -74,7 +74,7 @@ function testSearchExcerpt() { * search_simplify(). This test passes keywords that match simplified words * and compares them with strings that contain the original unsimplified word. */ - function testSearchExcerptSimplified() { + public function testSearchExcerptSimplified() { $start_time = microtime(TRUE); $lorem1 = 'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Etiam vitae arcu at leo cursus laoreet. Curabitur dui tortor, adipiscing malesuada tempor in, bibendum ac diam. Cras non tellus a libero pellentesque condimentum. What is a Drupalism? Suspendisse ac lacus libero. Ut non est vel nisl faucibus interdum nec sed leo. Pellentesque sem risus, vulputate eu semper eget, auctor in libero.'; diff --git a/core/modules/search/tests/src/Kernel/SearchMatchTest.php b/core/modules/search/tests/src/Kernel/SearchMatchTest.php index 0a17f71..587cbd8 100644 --- a/core/modules/search/tests/src/Kernel/SearchMatchTest.php +++ b/core/modules/search/tests/src/Kernel/SearchMatchTest.php @@ -37,7 +37,7 @@ protected function setUp() { /** * Test search indexing. */ - function testMatching() { + public function testMatching() { $this->_setup(); $this->_testQueries(); } diff --git a/core/modules/serialization/src/Tests/EntityResolverTest.php b/core/modules/serialization/src/Tests/EntityResolverTest.php index 6fca3c7..c36c3ba 100644 --- a/core/modules/serialization/src/Tests/EntityResolverTest.php +++ b/core/modules/serialization/src/Tests/EntityResolverTest.php @@ -54,7 +54,7 @@ protected function setUp() { /** * Test that fields referencing UUIDs can be denormalized. */ - function testUuidEntityResolver() { + public function testUuidEntityResolver() { // Create an entity to get the UUID from. $entity = EntityTestMulRev::create(array('type' => 'entity_test_mulrev')); $entity->set('name', 'foobar'); diff --git a/core/modules/shortcut/src/Tests/ShortcutSetsTest.php b/core/modules/shortcut/src/Tests/ShortcutSetsTest.php index 3a04757..ccd460c 100644 --- a/core/modules/shortcut/src/Tests/ShortcutSetsTest.php +++ b/core/modules/shortcut/src/Tests/ShortcutSetsTest.php @@ -30,7 +30,7 @@ protected function setUp() { /** * Tests creating a shortcut set. */ - function testShortcutSetAdd() { + public function testShortcutSetAdd() { $this->drupalGet('admin/config/user-interface/shortcut'); $this->clickLink(t('Add shortcut set')); $edit = array( @@ -47,7 +47,7 @@ function testShortcutSetAdd() { /** * Tests editing a shortcut set. */ - function testShortcutSetEdit() { + public function testShortcutSetEdit() { $set = $this->set; $shortcuts = $set->getShortcuts(); @@ -100,7 +100,7 @@ function testShortcutSetEdit() { /** * Tests switching a user's own shortcut set. */ - function testShortcutSetSwitchOwn() { + public function testShortcutSetSwitchOwn() { $new_set = $this->generateShortcutSet($this->randomMachineName()); // Attempt to switch the default shortcut set to the newly created shortcut @@ -114,7 +114,7 @@ function testShortcutSetSwitchOwn() { /** * Tests switching another user's shortcut set. */ - function testShortcutSetAssign() { + public function testShortcutSetAssign() { $new_set = $this->generateShortcutSet($this->randomMachineName()); \Drupal::entityManager()->getStorage('shortcut_set')->assignUser($new_set, $this->shortcutUser); @@ -125,7 +125,7 @@ function testShortcutSetAssign() { /** * Tests switching a user's shortcut set and creating one at the same time. */ - function testShortcutSetSwitchCreate() { + public function testShortcutSetSwitchCreate() { $edit = array( 'set' => 'new', 'id' => strtolower($this->randomMachineName()), @@ -140,7 +140,7 @@ function testShortcutSetSwitchCreate() { /** * Tests switching a user's shortcut set without providing a new set name. */ - function testShortcutSetSwitchNoSetName() { + public function testShortcutSetSwitchNoSetName() { $edit = array('set' => 'new'); $this->drupalPostForm('user/' . $this->adminUser->id() . '/shortcuts', $edit, t('Change set')); $this->assertText(t('The new set label is required.')); @@ -152,7 +152,7 @@ function testShortcutSetSwitchNoSetName() { /** * Tests renaming a shortcut set. */ - function testShortcutSetRename() { + public function testShortcutSetRename() { $set = $this->set; $new_label = $this->randomMachineName(); @@ -166,7 +166,7 @@ function testShortcutSetRename() { /** * Tests unassigning a shortcut set. */ - function testShortcutSetUnassign() { + public function testShortcutSetUnassign() { $new_set = $this->generateShortcutSet($this->randomMachineName()); $shortcut_set_storage = \Drupal::entityManager()->getStorage('shortcut_set'); @@ -180,7 +180,7 @@ function testShortcutSetUnassign() { /** * Tests deleting a shortcut set. */ - function testShortcutSetDelete() { + public function testShortcutSetDelete() { $new_set = $this->generateShortcutSet($this->randomMachineName()); $this->drupalPostForm('admin/config/user-interface/shortcut/manage/' . $new_set->id() . '/delete', array(), t('Delete')); @@ -191,7 +191,7 @@ function testShortcutSetDelete() { /** * Tests deleting the default shortcut set. */ - function testShortcutSetDeleteDefault() { + public function testShortcutSetDeleteDefault() { $this->drupalGet('admin/config/user-interface/shortcut/manage/default/delete'); $this->assertResponse(403); } @@ -199,7 +199,7 @@ function testShortcutSetDeleteDefault() { /** * Tests creating a new shortcut set with a defined set name. */ - function testShortcutSetCreateWithSetName() { + public function testShortcutSetCreateWithSetName() { $random_name = $this->randomMachineName(); $new_set = $this->generateShortcutSet($random_name, $random_name); $sets = ShortcutSet::loadMultiple(); diff --git a/core/modules/simpletest/src/Tests/BrokenSetUpTest.php b/core/modules/simpletest/src/Tests/BrokenSetUpTest.php index e988f4d..779e432 100644 --- a/core/modules/simpletest/src/Tests/BrokenSetUpTest.php +++ b/core/modules/simpletest/src/Tests/BrokenSetUpTest.php @@ -71,7 +71,7 @@ protected function tearDown() { /** * Runs this test case from within the simpletest child site. */ - function testMethod() { + public function testMethod() { // If the test is being run from the main site, run it again from the web // interface within the simpletest child site. if (!$this->isInChildSite()) { diff --git a/core/modules/simpletest/src/Tests/BrowserTest.php b/core/modules/simpletest/src/Tests/BrowserTest.php index f98fe48..c8e28ce 100644 --- a/core/modules/simpletest/src/Tests/BrowserTest.php +++ b/core/modules/simpletest/src/Tests/BrowserTest.php @@ -37,7 +37,7 @@ protected function setUp() { /** * Test \Drupal\simpletest\WebTestBase::getAbsoluteUrl(). */ - function testGetAbsoluteUrl() { + public function testGetAbsoluteUrl() { $url = 'user/login'; $this->drupalGet($url); @@ -58,7 +58,7 @@ function testGetAbsoluteUrl() { /** * Tests XPath escaping. */ - function testXPathEscaping() { + public function testXPathEscaping() { $testpage = <<< EOF diff --git a/core/modules/simpletest/src/Tests/FolderTest.php b/core/modules/simpletest/src/Tests/FolderTest.php index d43c638..c69b9f7 100644 --- a/core/modules/simpletest/src/Tests/FolderTest.php +++ b/core/modules/simpletest/src/Tests/FolderTest.php @@ -19,7 +19,7 @@ class FolderTest extends WebTestBase { */ public static $modules = array('image'); - function testFolderSetup() { + public function testFolderSetup() { $directory = file_default_scheme() . '://styles'; $this->assertTrue(file_prepare_directory($directory, FALSE), 'Directory created.'); } diff --git a/core/modules/simpletest/src/Tests/InstallationProfileModuleTestsTest.php b/core/modules/simpletest/src/Tests/InstallationProfileModuleTestsTest.php index a1063e7..56c195b 100644 --- a/core/modules/simpletest/src/Tests/InstallationProfileModuleTestsTest.php +++ b/core/modules/simpletest/src/Tests/InstallationProfileModuleTestsTest.php @@ -49,7 +49,7 @@ protected function setUp() { /** * Tests existence of test case located in an installation profile module. */ - function testInstallationProfileTests() { + public function testInstallationProfileTests() { $this->drupalGet('admin/config/development/testing'); $this->assertText('Drupal\drupal_system_listing_compatible_test\Tests\SystemListingCompatibleTest'); $edit = array( diff --git a/core/modules/simpletest/src/Tests/KernelTestBaseTest.php b/core/modules/simpletest/src/Tests/KernelTestBaseTest.php index 15ecf1f..fded4b1 100644 --- a/core/modules/simpletest/src/Tests/KernelTestBaseTest.php +++ b/core/modules/simpletest/src/Tests/KernelTestBaseTest.php @@ -50,7 +50,7 @@ function simpletest_test_stub_settings_function() {} /** * Tests expected behavior of setUp(). */ - function testSetUp() { + public function testSetUp() { $modules = array('entity_test'); $table = 'entity_test'; @@ -79,7 +79,7 @@ function testSetUp() { /** * Tests expected load behavior of enableModules(). */ - function testEnableModulesLoad() { + public function testEnableModulesLoad() { $module = 'field_test'; // Verify that the module does not exist yet. @@ -103,7 +103,7 @@ function testEnableModulesLoad() { /** * Tests expected installation behavior of enableModules(). */ - function testEnableModulesInstall() { + public function testEnableModulesInstall() { $module = 'module_test'; $table = 'module_test'; @@ -134,7 +134,7 @@ function testEnableModulesInstall() { /** * Tests installing modules with DependencyInjection services. */ - function testEnableModulesInstallContainer() { + public function testEnableModulesInstallContainer() { // Install Node module. $this->enableModules(array('user', 'field', 'node')); @@ -151,7 +151,7 @@ function testEnableModulesInstallContainer() { /** * Tests expected behavior of installSchema(). */ - function testInstallSchema() { + public function testInstallSchema() { $module = 'entity_test'; $table = 'entity_test_example'; // Verify that we can install a table from the module schema. @@ -200,7 +200,7 @@ function testInstallSchema() { /** * Tests expected behavior of installEntitySchema(). */ - function testInstallEntitySchema() { + public function testInstallEntitySchema() { $entity = 'entity_test'; // The entity_test Entity has a field that depends on the User module. $this->enableModules(array('user')); @@ -212,7 +212,7 @@ function testInstallEntitySchema() { /** * Tests expected behavior of installConfig(). */ - function testInstallConfig() { + public function testInstallConfig() { // The user module has configuration that depends on system. $this->enableModules(array('system')); $module = 'user'; @@ -237,7 +237,7 @@ function testInstallConfig() { /** * Tests that the module list is retained after enabling/installing/disabling. */ - function testEnableModulesFixedList() { + public function testEnableModulesFixedList() { // Install system module. $this->container->get('module_installer')->install(array('system', 'menu_link_content')); $entity_manager = \Drupal::entityManager(); @@ -290,7 +290,7 @@ function testEnableModulesFixedList() { /** * Tests that ThemeManager works right after loading a module. */ - function testEnableModulesTheme() { + public function testEnableModulesTheme() { /** @var \Drupal\Core\Render\RendererInterface $renderer */ $renderer = $this->container->get('renderer'); $original_element = $element = array( @@ -311,7 +311,7 @@ function testEnableModulesTheme() { /** * Tests that there is no theme by default. */ - function testNoThemeByDefault() { + public function testNoThemeByDefault() { $themes = $this->config('core.extension')->get('theme'); $this->assertEqual($themes, array()); diff --git a/core/modules/simpletest/src/Tests/MailCaptureTest.php b/core/modules/simpletest/src/Tests/MailCaptureTest.php index 700c757..6a80eb5 100644 --- a/core/modules/simpletest/src/Tests/MailCaptureTest.php +++ b/core/modules/simpletest/src/Tests/MailCaptureTest.php @@ -14,7 +14,7 @@ class MailCaptureTest extends WebTestBase { /** * Test to see if the wrapper function is executed correctly. */ - function testMailSend() { + public function testMailSend() { // Create an email. $subject = $this->randomString(64); $body = $this->randomString(128); diff --git a/core/modules/simpletest/src/Tests/MissingDependentModuleUnitTest.php b/core/modules/simpletest/src/Tests/MissingDependentModuleUnitTest.php index 9c707b8..a8eb0de 100644 --- a/core/modules/simpletest/src/Tests/MissingDependentModuleUnitTest.php +++ b/core/modules/simpletest/src/Tests/MissingDependentModuleUnitTest.php @@ -15,7 +15,7 @@ class MissingDependentModuleUnitTest extends WebTestBase { /** * Ensure that this test will not be loaded despite its dependency. */ - function testFail() { + public function testFail() { $this->fail('Running test with missing required module.'); } diff --git a/core/modules/simpletest/src/Tests/OtherInstallationProfileTestsTest.php b/core/modules/simpletest/src/Tests/OtherInstallationProfileTestsTest.php index b4263f1..ed82386 100644 --- a/core/modules/simpletest/src/Tests/OtherInstallationProfileTestsTest.php +++ b/core/modules/simpletest/src/Tests/OtherInstallationProfileTestsTest.php @@ -50,7 +50,7 @@ protected function setUp() { /** * Tests that tests located in another installation profile appear. */ - function testOtherInstallationProfile() { + public function testOtherInstallationProfile() { // Assert the existence of a test in a different installation profile than // the current. $this->drupalGet('admin/config/development/testing'); diff --git a/core/modules/simpletest/src/Tests/SimpleTestTest.php b/core/modules/simpletest/src/Tests/SimpleTestTest.php index 19ab17f..b9d6f33 100644 --- a/core/modules/simpletest/src/Tests/SimpleTestTest.php +++ b/core/modules/simpletest/src/Tests/SimpleTestTest.php @@ -109,7 +109,7 @@ class: Drupal\Core\Cache\MemoryBackendFactory /** * Ensures the tests selected through the web interface are run and displayed. */ - function testWebTestRunner() { + public function testWebTestRunner() { $this->passMessage = t('SimpleTest pass.'); $this->failMessage = t('SimpleTest fail.'); $this->validPermission = 'access administration pages'; diff --git a/core/modules/simpletest/src/Tests/TimeZoneTest.php b/core/modules/simpletest/src/Tests/TimeZoneTest.php index 2f33eaa..9a98c79 100644 --- a/core/modules/simpletest/src/Tests/TimeZoneTest.php +++ b/core/modules/simpletest/src/Tests/TimeZoneTest.php @@ -28,7 +28,7 @@ protected function setUp() { /** * Tests that user accounts have the default time zone set. */ - function testAccountTimeZones() { + public function testAccountTimeZones() { $expected = 'Australia/Sydney'; $this->assertEqual($this->rootUser->getTimeZone(), $expected, 'Root user has correct time zone.'); $this->assertEqual($this->adminUser->getTimeZone(), $expected, 'Admin user has correct time zone.'); diff --git a/core/modules/simpletest/src/Tests/UserHelpersTest.php b/core/modules/simpletest/src/Tests/UserHelpersTest.php index c3fb00b..cef787b 100644 --- a/core/modules/simpletest/src/Tests/UserHelpersTest.php +++ b/core/modules/simpletest/src/Tests/UserHelpersTest.php @@ -14,7 +14,7 @@ class UserHelpersTest extends WebTestBase { /** * Tests WebTestBase::drupalUserIsLoggedIn(). */ - function testDrupalUserIsLoggedIn() { + public function testDrupalUserIsLoggedIn() { $first_user = $this->drupalCreateUser(); $second_user = $this->drupalCreateUser(); diff --git a/core/modules/simpletest/tests/src/Unit/SimpletestPhpunitRunCommandTest.php b/core/modules/simpletest/tests/src/Unit/SimpletestPhpunitRunCommandTest.php index 2afea1c..6e688b6 100644 --- a/core/modules/simpletest/tests/src/Unit/SimpletestPhpunitRunCommandTest.php +++ b/core/modules/simpletest/tests/src/Unit/SimpletestPhpunitRunCommandTest.php @@ -15,7 +15,7 @@ */ class SimpletestPhpunitRunCommandTest extends UnitTestCase { - function testSimpletestPhpUnitRunCommand() { + public function testSimpletestPhpUnitRunCommand() { include_once __DIR__ . '/../../fixtures/simpletest_phpunit_run_command_test.php'; $app_root = __DIR__ . '/../../../../../..'; include_once "$app_root/core/modules/simpletest/simpletest.module"; diff --git a/core/modules/statistics/src/Tests/StatisticsAdminTest.php b/core/modules/statistics/src/Tests/StatisticsAdminTest.php index 565925c..8c46787 100644 --- a/core/modules/statistics/src/Tests/StatisticsAdminTest.php +++ b/core/modules/statistics/src/Tests/StatisticsAdminTest.php @@ -59,7 +59,7 @@ protected function setUp() { /** * Verifies that the statistics settings page works. */ - function testStatisticsSettings() { + public function testStatisticsSettings() { $config = $this->config('statistics.settings'); $this->assertFalse($config->get('count_content_views'), 'Count content view log is disabled by default.'); @@ -102,7 +102,7 @@ function testStatisticsSettings() { /** * Tests that when a node is deleted, the node counter is deleted too. */ - function testDeleteNode() { + public function testDeleteNode() { $this->config('statistics.settings')->set('count_content_views', 1)->save(); $this->drupalGet('node/' . $this->testNode->id()); @@ -133,7 +133,7 @@ function testDeleteNode() { /** * Tests that cron clears day counts and expired access logs. */ - function testExpiredLogs() { + public function testExpiredLogs() { $this->config('statistics.settings') ->set('count_content_views', 1) ->save(); diff --git a/core/modules/statistics/src/Tests/StatisticsLoggingTest.php b/core/modules/statistics/src/Tests/StatisticsLoggingTest.php index 7cfec7c..2d03f0a 100644 --- a/core/modules/statistics/src/Tests/StatisticsLoggingTest.php +++ b/core/modules/statistics/src/Tests/StatisticsLoggingTest.php @@ -90,7 +90,7 @@ protected function setUp() { /** * Verifies node hit counter logging and script placement. */ - function testLogging() { + public function testLogging() { $path = 'node/' . $this->node->id(); $module_path = drupal_get_path('module', 'statistics'); $stats_path = base_path() . $module_path . '/statistics.php'; diff --git a/core/modules/statistics/src/Tests/StatisticsReportsTest.php b/core/modules/statistics/src/Tests/StatisticsReportsTest.php index 9c0d26c..be8d00e 100644 --- a/core/modules/statistics/src/Tests/StatisticsReportsTest.php +++ b/core/modules/statistics/src/Tests/StatisticsReportsTest.php @@ -12,7 +12,7 @@ class StatisticsReportsTest extends StatisticsTestBase { /** * Tests the "popular content" block. */ - function testPopularContentBlock() { + public function testPopularContentBlock() { // Clear the block cache to load the Statistics module's block definitions. $this->container->get('plugin.manager.block')->clearCachedDefinitions(); diff --git a/core/modules/statistics/src/Tests/StatisticsTokenReplaceTest.php b/core/modules/statistics/src/Tests/StatisticsTokenReplaceTest.php index b7d22b8..b48b2f7 100644 --- a/core/modules/statistics/src/Tests/StatisticsTokenReplaceTest.php +++ b/core/modules/statistics/src/Tests/StatisticsTokenReplaceTest.php @@ -12,7 +12,7 @@ class StatisticsTokenReplaceTest extends StatisticsTestBase { /** * Creates a node, then tests the statistics tokens generated from it. */ - function testStatisticsTokenReplacement() { + public function testStatisticsTokenReplacement() { $language_interface = \Drupal::languageManager()->getCurrentLanguage(); // Create user and node. diff --git a/core/modules/syslog/src/Tests/SyslogTest.php b/core/modules/syslog/src/Tests/SyslogTest.php index a40847c..d8c8806 100644 --- a/core/modules/syslog/src/Tests/SyslogTest.php +++ b/core/modules/syslog/src/Tests/SyslogTest.php @@ -21,7 +21,7 @@ class SyslogTest extends WebTestBase { /** * Tests the syslog settings page. */ - function testSettings() { + public function testSettings() { $admin_user = $this->drupalCreateUser(array('administer site configuration')); $this->drupalLogin($admin_user); diff --git a/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php b/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php index 4ca7390..8e2bfd9 100644 --- a/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php +++ b/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php @@ -17,7 +17,7 @@ protected function setUp() { /** * Submits forms with select and checkbox elements via Ajax. */ - function testSimpleAjaxFormValue() { + public function testSimpleAjaxFormValue() { $this->drupalGet('/ajax_forms_test_get_form'); $this->assertText('Test group'); $this->assertText('AJAX checkbox in a group'); diff --git a/core/modules/system/src/Tests/Ajax/CommandsTest.php b/core/modules/system/src/Tests/Ajax/CommandsTest.php index 7e26fa3..eb56cd3 100644 --- a/core/modules/system/src/Tests/Ajax/CommandsTest.php +++ b/core/modules/system/src/Tests/Ajax/CommandsTest.php @@ -32,7 +32,7 @@ class CommandsTest extends AjaxTestBase { /** * Tests the various Ajax Commands. */ - function testAjaxCommands() { + public function testAjaxCommands() { $form_path = 'ajax_forms_test_ajax_commands_form'; $web_user = $this->drupalCreateUser(array('access content')); $this->drupalLogin($web_user); diff --git a/core/modules/system/src/Tests/Ajax/ElementValidationTest.php b/core/modules/system/src/Tests/Ajax/ElementValidationTest.php index 5133ce3..efbfbba 100644 --- a/core/modules/system/src/Tests/Ajax/ElementValidationTest.php +++ b/core/modules/system/src/Tests/Ajax/ElementValidationTest.php @@ -16,7 +16,7 @@ class ElementValidationTest extends AjaxTestBase { * filled in, and we want to see if the activation of the "drivertext" * Ajax-enabled field fails due to the required field being empty. */ - function testAjaxElementValidation() { + public function testAjaxElementValidation() { $edit = array('drivertext' => t('some dumb text')); // Post with 'drivertext' as the triggering element. diff --git a/core/modules/system/src/Tests/Ajax/FormValuesTest.php b/core/modules/system/src/Tests/Ajax/FormValuesTest.php index 0a23db1..33ff5ee 100644 --- a/core/modules/system/src/Tests/Ajax/FormValuesTest.php +++ b/core/modules/system/src/Tests/Ajax/FormValuesTest.php @@ -19,7 +19,7 @@ protected function setUp() { /** * Submits forms with select and checkbox elements via Ajax. */ - function testSimpleAjaxFormValue() { + public function testSimpleAjaxFormValue() { // Verify form values of a select element. foreach (array('red', 'green', 'blue') as $item) { $edit = array( diff --git a/core/modules/system/src/Tests/Ajax/MultiFormTest.php b/core/modules/system/src/Tests/Ajax/MultiFormTest.php index e07204c..738975a 100644 --- a/core/modules/system/src/Tests/Ajax/MultiFormTest.php +++ b/core/modules/system/src/Tests/Ajax/MultiFormTest.php @@ -50,7 +50,7 @@ protected function setUp() { /** * Tests that pages with the 'node_page_form' included twice work correctly. */ - function testMultiForm() { + public function testMultiForm() { // HTML IDs for elements within the field are potentially modified with // each Ajax submission, but these variables are stable and help target the // desired elements. diff --git a/core/modules/system/src/Tests/Batch/PageTest.php b/core/modules/system/src/Tests/Batch/PageTest.php index f9948ed..d94b26f 100644 --- a/core/modules/system/src/Tests/Batch/PageTest.php +++ b/core/modules/system/src/Tests/Batch/PageTest.php @@ -21,7 +21,7 @@ class PageTest extends WebTestBase { /** * Tests that the batch API progress page uses the correct theme. */ - function testBatchProgressPageTheme() { + public function testBatchProgressPageTheme() { // Make sure that the page which starts the batch (an administrative page) // is using a different theme than would normally be used by the batch API. $this->container->get('theme_handler')->install(array('seven', 'bartik')); @@ -46,7 +46,7 @@ function testBatchProgressPageTheme() { /** * Tests that the batch API progress page shows the title correctly. */ - function testBatchProgressPageTitle() { + public function testBatchProgressPageTitle() { // Visit an administrative page that runs a test batch, and check that the // title shown during batch execution (which the batch callback function // saved as a variable) matches the theme used on the administrative page. @@ -59,7 +59,7 @@ function testBatchProgressPageTitle() { /** * Tests that the progress messages are correct. */ - function testBatchProgressMessages() { + public function testBatchProgressMessages() { // Go to the initial step only. $this->maximumMetaRefreshCount = 0; $this->drupalGet('batch-test/test-title'); diff --git a/core/modules/system/src/Tests/Batch/ProcessingTest.php b/core/modules/system/src/Tests/Batch/ProcessingTest.php index 66c0fc6..043d381 100644 --- a/core/modules/system/src/Tests/Batch/ProcessingTest.php +++ b/core/modules/system/src/Tests/Batch/ProcessingTest.php @@ -22,7 +22,7 @@ class ProcessingTest extends WebTestBase { /** * Tests batches triggered outside of form submission. */ - function testBatchNoForm() { + public function testBatchNoForm() { // Displaying the page triggers batch 1. $this->drupalGet('batch-test/no-form'); $this->assertBatchMessages($this->_resultMessages('batch_1'), 'Batch for step 2 performed successfully.'); @@ -33,7 +33,7 @@ function testBatchNoForm() { /** * Tests batches that redirect in the batch finished callback. */ - function testBatchRedirectFinishedCallback() { + public function testBatchRedirectFinishedCallback() { // Displaying the page triggers batch 1. $this->drupalGet('batch-test/finish-redirect'); $this->assertBatchMessages($this->_resultMessages('batch_1'), 'Batch for step 2 performed successfully.'); @@ -45,7 +45,7 @@ function testBatchRedirectFinishedCallback() { /** * Tests batches defined in a form submit handler. */ - function testBatchForm() { + public function testBatchForm() { // Batch 0: no operation. $edit = array('batch' => 'batch_0'); $this->drupalPostForm('batch-test', $edit, 'Submit'); @@ -89,7 +89,7 @@ function testBatchForm() { /** * Tests batches defined in a multistep form. */ - function testBatchFormMultistep() { + public function testBatchFormMultistep() { $this->drupalGet('batch-test/multistep'); $this->assertNoEscaped('<', 'No escaped markup is present.'); $this->assertText('step 1', 'Form is displayed in step 1.'); @@ -112,7 +112,7 @@ function testBatchFormMultistep() { /** * Tests batches defined in different submit handlers on the same form. */ - function testBatchFormMultipleBatches() { + public function testBatchFormMultipleBatches() { // Batches 1, 2 and 3 are triggered in sequence by different submit // handlers. Each submit handler modify the submitted 'value'. $value = rand(0, 255); @@ -131,7 +131,7 @@ function testBatchFormMultipleBatches() { * * Same as above, but the form is submitted through drupal_form_execute(). */ - function testBatchFormProgrammatic() { + public function testBatchFormProgrammatic() { // Batches 1, 2 and 3 are triggered in sequence by different submit // handlers. Each submit handler modify the submitted 'value'. $value = rand(0, 255); @@ -147,7 +147,7 @@ function testBatchFormProgrammatic() { /** * Test form submission during a batch operation. */ - function testDrupalFormSubmitInBatch() { + public function testDrupalFormSubmitInBatch() { // Displaying the page triggers a batch that programmatically submits a // form. $value = rand(0, 255); @@ -160,7 +160,7 @@ function testDrupalFormSubmitInBatch() { * * @see https://www.drupal.org/node/600836 */ - function testBatchLargePercentage() { + public function testBatchLargePercentage() { // Displaying the page triggers batch 5. $this->drupalGet('batch-test/large-percentage'); $this->assertBatchMessages($this->_resultMessages('batch_5'), 'Batch for step 2 performed successfully.'); diff --git a/core/modules/system/src/Tests/Bootstrap/DrupalSetMessageTest.php b/core/modules/system/src/Tests/Bootstrap/DrupalSetMessageTest.php index 29ca02f..1e1b93a 100644 --- a/core/modules/system/src/Tests/Bootstrap/DrupalSetMessageTest.php +++ b/core/modules/system/src/Tests/Bootstrap/DrupalSetMessageTest.php @@ -21,7 +21,7 @@ class DrupalSetMessageTest extends WebTestBase { /** * Tests drupal_set_message(). */ - function testDrupalSetMessage() { + public function testDrupalSetMessage() { // The page at system-test/drupal-set-message sets two messages and then // removes the first before it is displayed. $this->drupalGet('system-test/drupal-set-message'); diff --git a/core/modules/system/src/Tests/Cache/ClearTest.php b/core/modules/system/src/Tests/Cache/ClearTest.php index 3a3fe90..770295d 100644 --- a/core/modules/system/src/Tests/Cache/ClearTest.php +++ b/core/modules/system/src/Tests/Cache/ClearTest.php @@ -21,7 +21,7 @@ protected function setUp() { /** * Tests drupal_flush_all_caches(). */ - function testFlushAllCaches() { + public function testFlushAllCaches() { // Create cache entries for each flushed cache bin. $bins = Cache::getBins(); $this->assertTrue($bins, 'Cache::getBins() returned bins to flush.'); diff --git a/core/modules/system/src/Tests/Cache/GenericCacheBackendUnitTestBase.php b/core/modules/system/src/Tests/Cache/GenericCacheBackendUnitTestBase.php index 6c3d98f..1dbb1b0 100644 --- a/core/modules/system/src/Tests/Cache/GenericCacheBackendUnitTestBase.php +++ b/core/modules/system/src/Tests/Cache/GenericCacheBackendUnitTestBase.php @@ -494,7 +494,7 @@ public function testDeleteAll() { * Test Drupal\Core\Cache\CacheBackendInterface::invalidate() and * Drupal\Core\Cache\CacheBackendInterface::invalidateMultiple(). */ - function testInvalidate() { + public function testInvalidate() { $backend = $this->getCacheBackend(); $backend->set('test1', 1); $backend->set('test2', 2); @@ -526,7 +526,7 @@ function testInvalidate() { /** * Tests Drupal\Core\Cache\CacheBackendInterface::invalidateTags(). */ - function testInvalidateTags() { + public function testInvalidateTags() { $backend = $this->getCacheBackend(); // Create two cache entries with the same tag and tag value. diff --git a/core/modules/system/src/Tests/Common/AddFeedTest.php b/core/modules/system/src/Tests/Common/AddFeedTest.php index b539281..e0d668c 100644 --- a/core/modules/system/src/Tests/Common/AddFeedTest.php +++ b/core/modules/system/src/Tests/Common/AddFeedTest.php @@ -15,7 +15,7 @@ class AddFeedTest extends WebTestBase { /** * Tests attaching feeds with paths, URLs, and titles. */ - function testBasicFeedAddNoTitle() { + public function testBasicFeedAddNoTitle() { $path = $this->randomMachineName(12); $external_url = 'http://' . $this->randomMachineName(12) . '/' . $this->randomMachineName(12); $fully_qualified_local_url = Url::fromUri('base:' . $this->randomMachineName(12), array('absolute' => TRUE))->toString(); @@ -82,7 +82,7 @@ function urlToRSSLinkPattern($url, $title = '') { * * @see https://www.drupal.org/node/1211668 */ - function testFeedIconEscaping() { + public function testFeedIconEscaping() { $variables = array( '#theme' => 'feed_icon', '#url' => 'node', diff --git a/core/modules/system/src/Tests/Common/AlterTest.php b/core/modules/system/src/Tests/Common/AlterTest.php index 86bed7d..7e61dc7 100644 --- a/core/modules/system/src/Tests/Common/AlterTest.php +++ b/core/modules/system/src/Tests/Common/AlterTest.php @@ -21,7 +21,7 @@ class AlterTest extends WebTestBase { /** * Tests if the theme has been altered. */ - function testDrupalAlter() { + public function testDrupalAlter() { // This test depends on Bartik, so make sure that it is always the current // active theme. \Drupal::service('theme_handler')->install(array('bartik')); diff --git a/core/modules/system/src/Tests/Common/EarlyRenderingControllerTest.php b/core/modules/system/src/Tests/Common/EarlyRenderingControllerTest.php index 196d463..1afdd4e 100644 --- a/core/modules/system/src/Tests/Common/EarlyRenderingControllerTest.php +++ b/core/modules/system/src/Tests/Common/EarlyRenderingControllerTest.php @@ -25,7 +25,7 @@ class EarlyRenderingControllerTest extends WebTestBase { /** * Tests theme preprocess functions being able to attach assets. */ - function testEarlyRendering() { + public function testEarlyRendering() { // Render array: non-early & early. $this->drupalGet(Url::fromRoute('early_rendering_controller_test.render_array')); $this->assertResponse(200); diff --git a/core/modules/system/src/Tests/Common/FormatDateTest.php b/core/modules/system/src/Tests/Common/FormatDateTest.php index 36589ef..8e5637d 100644 --- a/core/modules/system/src/Tests/Common/FormatDateTest.php +++ b/core/modules/system/src/Tests/Common/FormatDateTest.php @@ -50,7 +50,7 @@ protected function setUp() { /** * Tests admin-defined formats in format_date(). */ - function testAdminDefinedFormatDate() { + public function testAdminDefinedFormatDate() { // Create and log in an admin user. $this->drupalLogin($this->drupalCreateUser(array('administer site configuration'))); @@ -80,7 +80,7 @@ function testAdminDefinedFormatDate() { /** * Tests the format_date() function. */ - function testFormatDate() { + public function testFormatDate() { $timestamp = strtotime('2007-03-26T00:00:00+00:00'); $this->assertIdentical(format_date($timestamp, 'custom', 'l, d-M-y H:i:s T', 'America/Los_Angeles', 'en'), 'Sunday, 25-Mar-07 17:00:00 PDT', 'Test all parameters.'); $this->assertIdentical(format_date($timestamp, 'custom', 'l, d-M-y H:i:s T', 'America/Los_Angeles', self::LANGCODE), 'domingo, 25-Mar-07 17:00:00 PDT', 'Test translated format.'); diff --git a/core/modules/system/src/Tests/Common/PageRenderTest.php b/core/modules/system/src/Tests/Common/PageRenderTest.php index f07ece6..756e264 100644 --- a/core/modules/system/src/Tests/Common/PageRenderTest.php +++ b/core/modules/system/src/Tests/Common/PageRenderTest.php @@ -14,7 +14,7 @@ class PageRenderTest extends KernelTestBase { /** * Tests hook_page_attachments() exceptions. */ - function testHookPageAttachmentsExceptions() { + public function testHookPageAttachmentsExceptions() { $this->enableModules(['common_test', 'system']); \Drupal::service('router.builder')->rebuild(); @@ -24,7 +24,7 @@ function testHookPageAttachmentsExceptions() { /** * Tests hook_page_attachments_alter() exceptions. */ - function testHookPageAlter() { + public function testHookPageAlter() { $this->enableModules(['common_test', 'system']); \Drupal::service('router.builder')->rebuild(); diff --git a/core/modules/system/src/Tests/Common/RenderWebTest.php b/core/modules/system/src/Tests/Common/RenderWebTest.php index fde7fa6..b2c46ea 100644 --- a/core/modules/system/src/Tests/Common/RenderWebTest.php +++ b/core/modules/system/src/Tests/Common/RenderWebTest.php @@ -24,7 +24,7 @@ class RenderWebTest extends WebTestBase { /** * Asserts the cache context for the wrapper format is always present. */ - function testWrapperFormatCacheContext() { + public function testWrapperFormatCacheContext() { $this->drupalGet('common-test/type-link-active-class'); $this->assertIdentical(0, strpos($this->getRawContent(), "\nassertIdentical('text/html; charset=UTF-8', $this->drupalGetHeader('Content-Type')); @@ -43,7 +43,7 @@ function testWrapperFormatCacheContext() { * Tests rendering form elements without passing through * \Drupal::formBuilder()->doBuildForm(). */ - function testDrupalRenderFormElements() { + public function testDrupalRenderFormElements() { // Define a series of form elements. $element = array( '#type' => 'button', diff --git a/core/modules/system/src/Tests/Common/SimpleTestErrorCollectorTest.php b/core/modules/system/src/Tests/Common/SimpleTestErrorCollectorTest.php index 1be7e94..c42e157 100644 --- a/core/modules/system/src/Tests/Common/SimpleTestErrorCollectorTest.php +++ b/core/modules/system/src/Tests/Common/SimpleTestErrorCollectorTest.php @@ -31,7 +31,7 @@ class SimpleTestErrorCollectorTest extends WebTestBase { /** * Tests that simpletest collects errors from the tested site. */ - function testErrorCollect() { + public function testErrorCollect() { $this->collectedErrors = array(); $this->drupalGet('error-test/generate-warnings-with-report'); $this->assertEqual(count($this->collectedErrors), 3, 'Three errors were collected'); diff --git a/core/modules/system/src/Tests/Common/SystemListingTest.php b/core/modules/system/src/Tests/Common/SystemListingTest.php index cf29be4..7262823 100644 --- a/core/modules/system/src/Tests/Common/SystemListingTest.php +++ b/core/modules/system/src/Tests/Common/SystemListingTest.php @@ -14,7 +14,7 @@ class SystemListingTest extends KernelTestBase { /** * Tests that files in different directories take precedence as expected. */ - function testDirectoryPrecedence() { + public function testDirectoryPrecedence() { // Define the module files we will search for, and the directory precedence // we expect. $expected_directories = array( diff --git a/core/modules/system/src/Tests/Common/UrlTest.php b/core/modules/system/src/Tests/Common/UrlTest.php index 062e3e4..eeea836 100644 --- a/core/modules/system/src/Tests/Common/UrlTest.php +++ b/core/modules/system/src/Tests/Common/UrlTest.php @@ -25,7 +25,7 @@ class UrlTest extends WebTestBase { /** * Confirms that invalid URLs are filtered in link generating functions. */ - function testLinkXSS() { + public function testLinkXSS() { // Test \Drupal::l(). $text = $this->randomMachineName(); $path = ""; @@ -42,7 +42,7 @@ function testLinkXSS() { /** * Tests that #type=link bubbles outbound route/path processors' metadata. */ - function testLinkBubbleableMetadata() { + public function testLinkBubbleableMetadata() { $cases = [ ['Regular link', 'internal:/user', [], ['contexts' => [], 'tags' => [], 'max-age' => Cache::PERMANENT], []], ['Regular link, absolute', 'internal:/user', ['absolute' => TRUE], ['contexts' => ['url.site'], 'tags' => [], 'max-age' => Cache::PERMANENT], []], @@ -71,7 +71,7 @@ function testLinkBubbleableMetadata() { /** * Tests that default and custom attributes are handled correctly on links. */ - function testLinkAttributes() { + public function testLinkAttributes() { /** @var \Drupal\Core\Render\RendererInterface $renderer */ $renderer = $this->container->get('renderer'); @@ -160,7 +160,7 @@ function testLinkAttributes() { /** * Tests that link functions support render arrays as 'text'. */ - function testLinkRenderArrayText() { + public function testLinkRenderArrayText() { /** @var \Drupal\Core\Render\RendererInterface $renderer */ $renderer = $this->container->get('renderer'); @@ -211,7 +211,7 @@ private function hasAttribute($attribute, $link, $class) { /** * Tests UrlHelper::filterQueryParameters(). */ - function testDrupalGetQueryParameters() { + public function testDrupalGetQueryParameters() { $original = array( 'a' => 1, 'b' => array( @@ -247,7 +247,7 @@ function testDrupalGetQueryParameters() { /** * Tests UrlHelper::parse(). */ - function testDrupalParseUrl() { + public function testDrupalParseUrl() { // Relative, absolute, and external URLs, without/with explicit script path, // without/with Drupal path. foreach (array('', '/', 'https://www.drupal.org/') as $absolute) { @@ -285,7 +285,7 @@ function testDrupalParseUrl() { /** * Tests external URL handling. */ - function testExternalUrls() { + public function testExternalUrls() { $test_url = 'https://www.drupal.org/'; // Verify external URL can contain a fragment. diff --git a/core/modules/system/src/Tests/Condition/ConditionFormTest.php b/core/modules/system/src/Tests/Condition/ConditionFormTest.php index 21d0fcb..371d8af 100644 --- a/core/modules/system/src/Tests/Condition/ConditionFormTest.php +++ b/core/modules/system/src/Tests/Condition/ConditionFormTest.php @@ -21,7 +21,7 @@ class ConditionFormTest extends WebTestBase { /** * Submit the condition_node_type_test_form to test condition forms. */ - function testConfigForm() { + public function testConfigForm() { $this->drupalCreateContentType(array('type' => 'page', 'name' => 'Page')); $this->drupalCreateContentType(array('type' => 'article', 'name' => 'Article')); diff --git a/core/modules/system/src/Tests/Database/SelectPagerDefaultTest.php b/core/modules/system/src/Tests/Database/SelectPagerDefaultTest.php index cdfeab8..f56dcb0 100644 --- a/core/modules/system/src/Tests/Database/SelectPagerDefaultTest.php +++ b/core/modules/system/src/Tests/Database/SelectPagerDefaultTest.php @@ -16,7 +16,7 @@ class SelectPagerDefaultTest extends DatabaseWebTestBase { * Note that we have to make an HTTP request to a test page handler * because the pager depends on GET parameters. */ - function testEvenPagerQuery() { + public function testEvenPagerQuery() { // To keep the test from being too brittle, we determine up front // what the page count should be dynamically, and pass the control // information forward to the actual query on the other side of the @@ -50,7 +50,7 @@ function testEvenPagerQuery() { * Note that we have to make an HTTP request to a test page handler * because the pager depends on GET parameters. */ - function testOddPagerQuery() { + public function testOddPagerQuery() { // To keep the test from being too brittle, we determine up front // what the page count should be dynamically, and pass the control // information forward to the actual query on the other side of the @@ -83,7 +83,7 @@ function testOddPagerQuery() { * * This is a regression test for #467984. */ - function testInnerPagerQuery() { + public function testInnerPagerQuery() { $query = db_select('test', 't') ->extend('Drupal\Core\Database\Query\PagerSelectExtender'); $query @@ -105,7 +105,7 @@ function testInnerPagerQuery() { * * This is a regression test for #467984. */ - function testHavingPagerQuery() { + public function testHavingPagerQuery() { $query = db_select('test', 't') ->extend('Drupal\Core\Database\Query\PagerSelectExtender'); $query @@ -124,7 +124,7 @@ function testHavingPagerQuery() { /** * Confirms that every pager gets a valid, non-overlapping element ID. */ - function testElementNumbers() { + public function testElementNumbers() { $request = Request::createFromGlobals(); $request->query->replace(array( diff --git a/core/modules/system/src/Tests/Database/SelectTableSortDefaultTest.php b/core/modules/system/src/Tests/Database/SelectTableSortDefaultTest.php index 7ccbbb5..1a5e490 100644 --- a/core/modules/system/src/Tests/Database/SelectTableSortDefaultTest.php +++ b/core/modules/system/src/Tests/Database/SelectTableSortDefaultTest.php @@ -15,7 +15,7 @@ class SelectTableSortDefaultTest extends DatabaseWebTestBase { * Note that we have to make an HTTP request to a test page handler * because the pager depends on GET parameters. */ - function testTableSortQuery() { + public function testTableSortQuery() { $sorts = array( array('field' => t('Task ID'), 'sort' => 'desc', 'first' => 'perform at superbowl', 'last' => 'eat'), array('field' => t('Task ID'), 'sort' => 'asc', 'first' => 'eat', 'last' => 'perform at superbowl'), @@ -43,7 +43,7 @@ function testTableSortQuery() { * If a tablesort's orderByHeader is called before another orderBy, then its * header happens first. */ - function testTableSortQueryFirst() { + public function testTableSortQueryFirst() { $sorts = array( array('field' => t('Task ID'), 'sort' => 'desc', 'first' => 'perform at superbowl', 'last' => 'eat'), array('field' => t('Task ID'), 'sort' => 'asc', 'first' => 'eat', 'last' => 'perform at superbowl'), @@ -71,7 +71,7 @@ function testTableSortQueryFirst() { * Specifically that no sort is set in a tableselect, and that header links * are correct. */ - function testTableSortDefaultSort() { + public function testTableSortDefaultSort() { $this->drupalGet('database_test/tablesort_default_sort'); // Verify that the table was displayed. Just the header is checked for diff --git a/core/modules/system/src/Tests/Database/TemporaryQueryTest.php b/core/modules/system/src/Tests/Database/TemporaryQueryTest.php index 1bfa754..9553884 100644 --- a/core/modules/system/src/Tests/Database/TemporaryQueryTest.php +++ b/core/modules/system/src/Tests/Database/TemporaryQueryTest.php @@ -26,7 +26,7 @@ function countTableRows($table_name) { /** * Confirms that temporary tables work and are limited to one request. */ - function testTemporaryQuery() { + public function testTemporaryQuery() { $this->drupalGet('database_test/db_query_temporary'); $data = json_decode($this->getRawContent()); if ($data) { diff --git a/core/modules/system/src/Tests/Datetime/DrupalDateTimeTest.php b/core/modules/system/src/Tests/Datetime/DrupalDateTimeTest.php index 219bc34..f6c4fd9 100644 --- a/core/modules/system/src/Tests/Datetime/DrupalDateTimeTest.php +++ b/core/modules/system/src/Tests/Datetime/DrupalDateTimeTest.php @@ -101,7 +101,7 @@ public function testDateTimezone() { /** * Tests the ability to override the time zone in the format method. */ - function testTimezoneFormat() { + public function testTimezoneFormat() { // Create a date in UTC $date = DrupalDateTime::createFromTimestamp(87654321, 'UTC'); diff --git a/core/modules/system/src/Tests/DrupalKernel/ContentNegotiationTest.php b/core/modules/system/src/Tests/DrupalKernel/ContentNegotiationTest.php index dcd88ec..1d08582 100644 --- a/core/modules/system/src/Tests/DrupalKernel/ContentNegotiationTest.php +++ b/core/modules/system/src/Tests/DrupalKernel/ContentNegotiationTest.php @@ -19,7 +19,7 @@ class ContentNegotiationTest extends WebTestBase { * * @see https://www.drupal.org/node/1716790 */ - function testBogusAcceptHeader() { + public function testBogusAcceptHeader() { $tests = array( // See https://bugs.webkit.org/show_bug.cgi?id=27267. 'Firefox 3.5 (2009)' => 'text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8', diff --git a/core/modules/system/src/Tests/Entity/EntityFormTest.php b/core/modules/system/src/Tests/Entity/EntityFormTest.php index 4e0ced5..8c96b14 100644 --- a/core/modules/system/src/Tests/Entity/EntityFormTest.php +++ b/core/modules/system/src/Tests/Entity/EntityFormTest.php @@ -31,7 +31,7 @@ protected function setUp() { /** * Tests basic form CRUD functionality. */ - function testFormCRUD() { + public function testFormCRUD() { // All entity variations have to have the same results. foreach (entity_test_entity_types() as $entity_type) { $this->doTestFormCRUD($entity_type); @@ -53,7 +53,7 @@ public function testMultilingualFormCRUD() { * * @see entity_test_entity_form_display_alter() */ - function testEntityFormDisplayAlter() { + public function testEntityFormDisplayAlter() { $this->drupalGet('entity_test/add'); $altered_field = $this->xpath('//input[@name="field_test_text[0][value]" and @size="42"]'); $this->assertTrue(count($altered_field) === 1, 'The altered field has the correct size value.'); diff --git a/core/modules/system/src/Tests/Entity/EntityTranslationFormTest.php b/core/modules/system/src/Tests/Entity/EntityTranslationFormTest.php index d64e019..62fc425 100644 --- a/core/modules/system/src/Tests/Entity/EntityTranslationFormTest.php +++ b/core/modules/system/src/Tests/Entity/EntityTranslationFormTest.php @@ -43,7 +43,7 @@ protected function setUp() { /** * Tests entity form language. */ - function testEntityFormLanguage() { + public function testEntityFormLanguage() { $this->drupalCreateContentType(array('type' => 'page', 'name' => 'Basic page')); $web_user = $this->drupalCreateUser(array('create page content', 'edit own page content', 'administer content types')); diff --git a/core/modules/system/src/Tests/Entity/EntityViewControllerTest.php b/core/modules/system/src/Tests/Entity/EntityViewControllerTest.php index d9f36ff..a0d169b 100644 --- a/core/modules/system/src/Tests/Entity/EntityViewControllerTest.php +++ b/core/modules/system/src/Tests/Entity/EntityViewControllerTest.php @@ -41,7 +41,7 @@ protected function setUp() { /** * Tests EntityViewController. */ - function testEntityViewController() { + public function testEntityViewController() { $get_label_markup = function($label) { return '

' . $label . '
diff --git a/core/modules/system/src/Tests/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php b/core/modules/system/src/Tests/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php index ceab801..ddc52fe 100644 --- a/core/modules/system/src/Tests/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php +++ b/core/modules/system/src/Tests/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php @@ -134,7 +134,7 @@ public function testMultipleUpdates() { /** * Tests that entity updates are correctly reported in the status report page. */ - function testStatusReport() { + public function testStatusReport() { // Create a test entity. $entity = EntityTest::create(['name' => $this->randomString(), 'user_id' => mt_rand()]); $entity->save(); diff --git a/core/modules/system/src/Tests/File/ConfigTest.php b/core/modules/system/src/Tests/File/ConfigTest.php index 2313e26..d23bb6e 100644 --- a/core/modules/system/src/Tests/File/ConfigTest.php +++ b/core/modules/system/src/Tests/File/ConfigTest.php @@ -19,7 +19,7 @@ protected function setUp(){ /** * Tests file configuration page. */ - function testFileConfigurationPage() { + public function testFileConfigurationPage() { $this->drupalGet('admin/config/media/file-system'); // Set the file paths to non-default values. diff --git a/core/modules/system/src/Tests/File/FileSaveHtaccessLoggingTest.php b/core/modules/system/src/Tests/File/FileSaveHtaccessLoggingTest.php index 5827e24..49ff766 100644 --- a/core/modules/system/src/Tests/File/FileSaveHtaccessLoggingTest.php +++ b/core/modules/system/src/Tests/File/FileSaveHtaccessLoggingTest.php @@ -17,7 +17,7 @@ class FileSaveHtaccessLoggingTest extends WebTestBase { /** * Tests file_save_htaccess(). */ - function testHtaccessSave() { + public function testHtaccessSave() { // Prepare test directories. $private = $this->publicFilesDirectory . '/test/private'; diff --git a/core/modules/system/src/Tests/FileTransfer/FileTransferTest.php b/core/modules/system/src/Tests/FileTransfer/FileTransferTest.php index e8a6789..623aed6 100644 --- a/core/modules/system/src/Tests/FileTransfer/FileTransferTest.php +++ b/core/modules/system/src/Tests/FileTransfer/FileTransferTest.php @@ -66,7 +66,7 @@ function _writeDirectory($base, $files = array()) { } } - function testJail() { + public function testJail() { $source = $this->_buildFakeModule(); // This convoluted piece of code is here because our testing framework does diff --git a/core/modules/system/src/Tests/Form/AlterTest.php b/core/modules/system/src/Tests/Form/AlterTest.php index ae30722..4c493d7 100644 --- a/core/modules/system/src/Tests/Form/AlterTest.php +++ b/core/modules/system/src/Tests/Form/AlterTest.php @@ -22,7 +22,7 @@ class AlterTest extends WebTestBase { /** * Tests execution order of hook_form_alter() and hook_form_FORM_ID_alter(). */ - function testExecutionOrder() { + public function testExecutionOrder() { $this->drupalGet('form-test/alter'); // Ensure that the order is first by module, then for a given module, the // id-specific one after the generic one. diff --git a/core/modules/system/src/Tests/Form/ArbitraryRebuildTest.php b/core/modules/system/src/Tests/Form/ArbitraryRebuildTest.php index 3702d35..63f604f 100644 --- a/core/modules/system/src/Tests/Form/ArbitraryRebuildTest.php +++ b/core/modules/system/src/Tests/Form/ArbitraryRebuildTest.php @@ -48,7 +48,7 @@ protected function setUp() { /** * Tests a basic rebuild with the user registration form. */ - function testUserRegistrationRebuild() { + public function testUserRegistrationRebuild() { $edit = array( 'name' => 'foo', 'mail' => 'bar@example.com', @@ -62,7 +62,7 @@ function testUserRegistrationRebuild() { /** * Tests a rebuild caused by a multiple value field. */ - function testUserRegistrationMultipleField() { + public function testUserRegistrationMultipleField() { $edit = array( 'name' => 'foo', 'mail' => 'bar@example.com', diff --git a/core/modules/system/src/Tests/Form/CheckboxTest.php b/core/modules/system/src/Tests/Form/CheckboxTest.php index ebeeeea..5b2659b 100644 --- a/core/modules/system/src/Tests/Form/CheckboxTest.php +++ b/core/modules/system/src/Tests/Form/CheckboxTest.php @@ -19,7 +19,7 @@ class CheckboxTest extends WebTestBase { */ public static $modules = array('form_test'); - function testFormCheckbox() { + public function testFormCheckbox() { // Ensure that the checked state is determined and rendered correctly for // tricky combinations of default and return values. foreach (array(FALSE, NULL, TRUE, 0, '0', '', 1, '1', 'foobar', '1foobar') as $default_value) { diff --git a/core/modules/system/src/Tests/Form/ConfirmFormTest.php b/core/modules/system/src/Tests/Form/ConfirmFormTest.php index 312aa2e..aa965fb 100644 --- a/core/modules/system/src/Tests/Form/ConfirmFormTest.php +++ b/core/modules/system/src/Tests/Form/ConfirmFormTest.php @@ -20,7 +20,7 @@ class ConfirmFormTest extends WebTestBase { */ public static $modules = array('form_test'); - function testConfirmForm() { + public function testConfirmForm() { // Test the building of the form. $this->drupalGet('form-test/confirm-form'); $site_name = $this->config('system.site')->get('name'); diff --git a/core/modules/system/src/Tests/Form/ElementTest.php b/core/modules/system/src/Tests/Form/ElementTest.php index 684e13a..a230d5e 100644 --- a/core/modules/system/src/Tests/Form/ElementTest.php +++ b/core/modules/system/src/Tests/Form/ElementTest.php @@ -21,7 +21,7 @@ class ElementTest extends WebTestBase { /** * Tests placeholder text for elements that support placeholders. */ - function testPlaceHolderText() { + public function testPlaceHolderText() { $this->drupalGet('form-test/placeholder-text'); $expected = 'placeholder-text'; // Test to make sure non-textarea elements have the proper placeholder text. @@ -44,7 +44,7 @@ function testPlaceHolderText() { /** * Tests expansion of #options for #type checkboxes and radios. */ - function testOptions() { + public function testOptions() { $this->drupalGet('form-test/checkboxes-radios'); // Verify that all options appear in their defined order. @@ -91,7 +91,7 @@ function testOptions() { /** * Tests wrapper ids for checkboxes and radios. */ - function testWrapperIds() { + public function testWrapperIds() { $this->drupalGet('form-test/checkboxes-radios'); // Verify that wrapper id is different from element id. @@ -106,7 +106,7 @@ function testWrapperIds() { /** * Tests button classes. */ - function testButtonClasses() { + public function testButtonClasses() { $this->drupalGet('form-test/button-class'); // Just contains(@class, "button") won't do because then // "button--foo" would contain "button". Instead, check @@ -120,7 +120,7 @@ function testButtonClasses() { /** * Tests the #group property. */ - function testGroupElements() { + public function testGroupElements() { $this->drupalGet('form-test/group-details'); $elements = $this->xpath('//div[@class="details-wrapper"]//div[@class="details-wrapper"]//label'); $this->assertTrue(count($elements) == 1); diff --git a/core/modules/system/src/Tests/Form/ElementsLabelsTest.php b/core/modules/system/src/Tests/Form/ElementsLabelsTest.php index 5965dc25..99d9bfd 100644 --- a/core/modules/system/src/Tests/Form/ElementsLabelsTest.php +++ b/core/modules/system/src/Tests/Form/ElementsLabelsTest.php @@ -22,7 +22,7 @@ class ElementsLabelsTest extends WebTestBase { * Test form elements, labels, title attributes and required marks output * correctly and have the correct label option class if needed. */ - function testFormLabels() { + public function testFormLabels() { $this->drupalGet('form_test/form-labels'); // Check that the checkbox/radio processing is not interfering with @@ -99,7 +99,7 @@ function testFormLabels() { /** * Tests different display options for form element descriptions. */ - function testFormDescriptions() { + public function testFormDescriptions() { $this->drupalGet('form_test/form-descriptions'); // Check #description placement with #description_display='after'. @@ -126,7 +126,7 @@ function testFormDescriptions() { /** * Test forms in theme-less environments. */ - function testFormsInThemeLessEnvironments() { + public function testFormsInThemeLessEnvironments() { $form = $this->getFormWithLimitedProperties(); $render_service = $this->container->get('renderer'); // This should not throw any notices. diff --git a/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php b/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php index 34a558c..0e0fdc9 100644 --- a/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php +++ b/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php @@ -22,7 +22,7 @@ class ElementsTableSelectTest extends WebTestBase { /** * Test the display of checkboxes when #multiple is TRUE. */ - function testMultipleTrue() { + public function testMultipleTrue() { $this->drupalGet('form_test/tableselect/multiple-true'); @@ -40,7 +40,7 @@ function testMultipleTrue() { /** * Test the presence of ajax functionality for all options. */ - function testAjax() { + public function testAjax() { $rows = array('row1', 'row2', 'row3'); // Test checkboxes (#multiple == TRUE). foreach ($rows as $row) { @@ -61,7 +61,7 @@ function testAjax() { /** * Test the display of radios when #multiple is FALSE. */ - function testMultipleFalse() { + public function testMultipleFalse() { $this->drupalGet('form_test/tableselect/multiple-false'); $this->assertNoText(t('Empty text.'), 'Empty text should not be displayed.'); @@ -78,7 +78,7 @@ function testMultipleFalse() { /** * Tests the display when #colspan is set. */ - function testTableselectColSpan() { + public function testTableselectColSpan() { $this->drupalGet('form_test/tableselect/colspan'); $this->assertText(t('Three'), 'Presence of the third column'); @@ -103,7 +103,7 @@ function testTableselectColSpan() { /** * Test the display of the #empty text when #options is an empty array. */ - function testEmptyText() { + public function testEmptyText() { $this->drupalGet('form_test/tableselect/empty-text'); $this->assertText(t('Empty text.'), 'Empty text should be displayed.'); } @@ -111,7 +111,7 @@ function testEmptyText() { /** * Test the submission of single and multiple values when #multiple is TRUE. */ - function testMultipleTrueSubmit() { + public function testMultipleTrueSubmit() { // Test a submission with one checkbox checked. $edit = array(); @@ -136,7 +136,7 @@ function testMultipleTrueSubmit() { /** * Test submission of values when #multiple is FALSE. */ - function testMultipleFalseSubmit() { + public function testMultipleFalseSubmit() { $edit['tableselect'] = 'row1'; $this->drupalPostForm('form_test/tableselect/multiple-false', $edit, 'Submit'); $this->assertText(t('Submitted: row1'), 'Selected radio button'); @@ -145,7 +145,7 @@ function testMultipleFalseSubmit() { /** * Test the #js_select property. */ - function testAdvancedSelect() { + public function testAdvancedSelect() { // When #multiple = TRUE a Select all checkbox should be displayed by default. $this->drupalGet('form_test/tableselect/advanced-select/multiple-true-default'); $this->assertFieldByXPath('//th[@class="select-all"]', NULL, 'Display a "Select all" checkbox by default when #multiple is TRUE.'); @@ -166,7 +166,7 @@ function testAdvancedSelect() { /** * Test the whether the option checker gives an error on invalid tableselect values for checkboxes. */ - function testMultipleTrueOptionchecker() { + public function testMultipleTrueOptionchecker() { list($header, $options) = _form_test_tableselect_get_data(); @@ -190,7 +190,7 @@ function testMultipleTrueOptionchecker() { /** * Test the whether the option checker gives an error on invalid tableselect values for radios. */ - function testMultipleFalseOptionchecker() { + public function testMultipleFalseOptionchecker() { list($header, $options) = _form_test_tableselect_get_data(); diff --git a/core/modules/system/src/Tests/Form/ElementsVerticalTabsTest.php b/core/modules/system/src/Tests/Form/ElementsVerticalTabsTest.php index 3de1867..71bb6b9 100644 --- a/core/modules/system/src/Tests/Form/ElementsVerticalTabsTest.php +++ b/core/modules/system/src/Tests/Form/ElementsVerticalTabsTest.php @@ -47,7 +47,7 @@ protected function setUp() { * * Otherwise, collapse.js adds "SHOW" or "HIDE" labels to the tabs. */ - function testJavaScriptOrdering() { + public function testJavaScriptOrdering() { $this->drupalGet('form_test/vertical-tabs'); $position1 = strpos($this->content, 'core/misc/vertical-tabs.js'); $position2 = strpos($this->content, 'core/misc/collapse.js'); @@ -57,7 +57,7 @@ function testJavaScriptOrdering() { /** * Ensures that vertical tab markup is not shown if user has no tab access. */ - function testWrapperNotShownWhenEmpty() { + public function testWrapperNotShownWhenEmpty() { // Test admin user can see vertical tabs and wrapper. $this->drupalGet('form_test/vertical-tabs'); $wrapper = $this->xpath("//div[@data-vertical-tabs-panes]"); @@ -73,7 +73,7 @@ function testWrapperNotShownWhenEmpty() { /** * Ensures that default vertical tab is correctly selected. */ - function testDefaultTab() { + public function testDefaultTab() { $this->drupalGet('form_test/vertical-tabs'); $this->assertFieldByName('vertical_tabs__active_tab', 'edit-tab3', t('The default vertical tab is correctly selected.')); } @@ -81,7 +81,7 @@ function testDefaultTab() { /** * Ensures that vertical tab form values are cleaned. */ - function testDefaultTabCleaned() { + public function testDefaultTabCleaned() { $values = Json::decode($this->drupalPostForm('form_test/form-state-values-clean', [], t('Submit'))); $this->assertFalse(isset($values['vertical_tabs__active_tab']), SafeMarkup::format('%element was removed.', ['%element' => 'vertical_tabs__active_tab'])); } diff --git a/core/modules/system/src/Tests/Form/EmailTest.php b/core/modules/system/src/Tests/Form/EmailTest.php index e53247f..9644874 100644 --- a/core/modules/system/src/Tests/Form/EmailTest.php +++ b/core/modules/system/src/Tests/Form/EmailTest.php @@ -24,7 +24,7 @@ class EmailTest extends WebTestBase { /** * Tests that #type 'email' fields are properly validated. */ - function testFormEmail() { + public function testFormEmail() { $edit = array(); $edit['email'] = 'invalid'; $edit['email_required'] = ' '; diff --git a/core/modules/system/src/Tests/Form/FormObjectTest.php b/core/modules/system/src/Tests/Form/FormObjectTest.php index 38db211..99f298e 100644 --- a/core/modules/system/src/Tests/Form/FormObjectTest.php +++ b/core/modules/system/src/Tests/Form/FormObjectTest.php @@ -37,7 +37,7 @@ protected function setUp() { * * @see \Drupal\form_test\EventSubscriber\FormTestEventSubscriber::onKernelRequest() */ - function testObjectFormCallback() { + public function testObjectFormCallback() { $config_factory = $this->container->get('config.factory'); $this->drupalGet('form-test/object-builder'); diff --git a/core/modules/system/src/Tests/Form/FormTest.php b/core/modules/system/src/Tests/Form/FormTest.php index cf73836..dd26233 100644 --- a/core/modules/system/src/Tests/Form/FormTest.php +++ b/core/modules/system/src/Tests/Form/FormTest.php @@ -48,7 +48,7 @@ protected function setUp() { * * If the form field is found in $form_state->getErrors() then the test pass. */ - function testRequiredFields() { + public function testRequiredFields() { // Originates from https://www.drupal.org/node/117748. // Sets of empty strings and arrays. $empty_strings = array('""' => "", '"\n"' => "\n", '" "' => " ", '"\t"' => "\t", '" \n\t "' => " \n\t ", '"\n\n\n\n\n"' => "\n\n\n\n\n"); @@ -163,7 +163,7 @@ function testRequiredFields() { * * @see \Drupal\form_test\Form\FormTestValidateRequiredForm */ - function testRequiredCheckboxesRadio() { + public function testRequiredCheckboxesRadio() { $form = \Drupal::formBuilder()->getForm('\Drupal\form_test\Form\FormTestValidateRequiredForm'); // Attempt to submit the form with no required fields set. @@ -312,7 +312,7 @@ public function testGetFormsCsrfToken() { * * @see \Drupal\form_test\Form\FormTestValidateRequiredNoTitleForm */ - function testRequiredTextfieldNoTitle() { + public function testRequiredTextfieldNoTitle() { // Attempt to submit the form with no required field set. $edit = array(); $this->drupalPostForm('form-test/validate-required-no-title', $edit, 'Submit'); @@ -339,7 +339,7 @@ function testRequiredTextfieldNoTitle() { * * @see _form_test_checkbox() */ - function testCheckboxProcessing() { + public function testCheckboxProcessing() { // First, try to submit without the required checkbox. $edit = array(); $this->drupalPostForm('form-test/checkbox', $edit, t('Submit')); @@ -367,7 +367,7 @@ function testCheckboxProcessing() { /** * Tests validation of #type 'select' elements. */ - function testSelect() { + public function testSelect() { $form = \Drupal::formBuilder()->getForm('Drupal\form_test\Form\FormTestSelectForm'); $this->drupalGet('form-test/select'); @@ -444,7 +444,7 @@ function testSelect() { /** * Tests a select element when #options is not set. */ - function testEmptySelect() { + public function testEmptySelect() { $this->drupalGet('form-test/empty-select'); $this->assertFieldByXPath("//select[1]", NULL, 'Select element found.'); $this->assertNoFieldByXPath("//select[1]/option", NULL, 'No option element found.'); @@ -453,7 +453,7 @@ function testEmptySelect() { /** * Tests validation of #type 'number' and 'range' elements. */ - function testNumber() { + public function testNumber() { $form = \Drupal::formBuilder()->getForm('\Drupal\form_test\Form\FormTestNumberForm'); // Array with all the error messages to be checked. @@ -517,7 +517,7 @@ function testNumber() { /** * Tests default value handling of #type 'range' elements. */ - function testRange() { + public function testRange() { $values = json_decode($this->drupalPostForm('form-test/range', array(), 'Submit')); $this->assertEqual($values->with_default_value, 18); $this->assertEqual($values->float, 10.5); @@ -531,7 +531,7 @@ function testRange() { /** * Tests validation of #type 'color' elements. */ - function testColorValidation() { + public function testColorValidation() { // Keys are inputs, values are expected results. $values = array( '' => '#000000', @@ -568,7 +568,7 @@ function testColorValidation() { * * @see _form_test_disabled_elements() */ - function testDisabledElements() { + public function testDisabledElements() { // Get the raw form in its original state. $form_state = new FormState(); $form = (new FormTestDisabledElementsForm())->buildForm(array(), $form_state); @@ -653,7 +653,7 @@ function assertFormValuesDefault($values, $form) { * * @see _form_test_disabled_elements() */ - function testDisabledMarkup() { + public function testDisabledMarkup() { $this->drupalGet('form-test/disabled-elements'); $form = \Drupal::formBuilder()->getForm('\Drupal\form_test\Form\FormTestDisabledElementsForm'); $type_map = array( @@ -715,7 +715,7 @@ function testDisabledMarkup() { * * @see _form_test_input_forgery() */ - function testInputForgery() { + public function testInputForgery() { $this->drupalGet('form-test/input-forgery'); $checkbox = $this->xpath('//input[@name="checkboxes[two]"]'); $checkbox[0]['value'] = 'FORGERY'; @@ -726,7 +726,7 @@ function testInputForgery() { /** * Tests required attribute. */ - function testRequiredAttribute() { + public function testRequiredAttribute() { $this->drupalGet('form-test/required-attribute'); $expected = 'required'; // Test to make sure the elements have the proper required attribute. diff --git a/core/modules/system/src/Tests/Form/LanguageSelectElementTest.php b/core/modules/system/src/Tests/Form/LanguageSelectElementTest.php index 7b3d73c..0039a1a 100644 --- a/core/modules/system/src/Tests/Form/LanguageSelectElementTest.php +++ b/core/modules/system/src/Tests/Form/LanguageSelectElementTest.php @@ -25,7 +25,7 @@ class LanguageSelectElementTest extends WebTestBase { /** * Tests that the options printed by the language select element are correct. */ - function testLanguageSelectElementOptions() { + public function testLanguageSelectElementOptions() { // Add some languages. ConfigurableLanguage::create(array( 'id' => 'aaa', @@ -68,7 +68,7 @@ function testLanguageSelectElementOptions() { * * This happens when the language module is disabled. */ - function testHiddenLanguageSelectElement() { + public function testHiddenLanguageSelectElement() { // Disable the language module, so that the language select field will not // be rendered. $this->container->get('module_installer')->uninstall(array('language')); diff --git a/core/modules/system/src/Tests/Form/ProgrammaticTest.php b/core/modules/system/src/Tests/Form/ProgrammaticTest.php index 5fbefb5..6d33e94 100644 --- a/core/modules/system/src/Tests/Form/ProgrammaticTest.php +++ b/core/modules/system/src/Tests/Form/ProgrammaticTest.php @@ -22,7 +22,7 @@ class ProgrammaticTest extends WebTestBase { /** * Test the programmatic form submission workflow. */ - function testSubmissionWorkflow() { + public function testSubmissionWorkflow() { // Backup the current batch status and reset it to avoid conflicts while // processing the dummy form submit handler. $current_batch = $batch =& batch_get(); diff --git a/core/modules/system/src/Tests/Form/RebuildTest.php b/core/modules/system/src/Tests/Form/RebuildTest.php index bee15fe..7a43bb8 100644 --- a/core/modules/system/src/Tests/Form/RebuildTest.php +++ b/core/modules/system/src/Tests/Form/RebuildTest.php @@ -42,7 +42,7 @@ protected function setUp() { /** * Tests preservation of values. */ - function testRebuildPreservesValues() { + public function testRebuildPreservesValues() { $edit = array( 'checkbox_1_default_off' => TRUE, 'checkbox_1_default_on' => FALSE, @@ -67,7 +67,7 @@ function testRebuildPreservesValues() { * The 'action' attribute of a form should not change after an Ajax submission * followed by a non-Ajax submission, which triggers a validation error. */ - function testPreserveFormActionAfterAJAX() { + public function testPreserveFormActionAfterAJAX() { // Create a multi-valued field for 'page' nodes to use for Ajax testing. $field_name = 'field_ajax_test'; FieldStorageConfig::create(array( diff --git a/core/modules/system/src/Tests/Form/RedirectTest.php b/core/modules/system/src/Tests/Form/RedirectTest.php index 6b6d360..aa7c21e 100644 --- a/core/modules/system/src/Tests/Form/RedirectTest.php +++ b/core/modules/system/src/Tests/Form/RedirectTest.php @@ -21,7 +21,7 @@ class RedirectTest extends WebTestBase { /** * Tests form redirection. */ - function testRedirect() { + public function testRedirect() { $path = 'form-test/redirect'; $options = array('query' => array('foo' => 'bar')); $options['absolute'] = TRUE; diff --git a/core/modules/system/src/Tests/Form/StateValuesCleanAdvancedTest.php b/core/modules/system/src/Tests/Form/StateValuesCleanAdvancedTest.php index eb53244..1fea902 100644 --- a/core/modules/system/src/Tests/Form/StateValuesCleanAdvancedTest.php +++ b/core/modules/system/src/Tests/Form/StateValuesCleanAdvancedTest.php @@ -28,7 +28,7 @@ class StateValuesCleanAdvancedTest extends WebTestBase { /** * Tests \Drupal\Core\Form\FormState::cleanValues(). */ - function testFormStateValuesCleanAdvanced() { + public function testFormStateValuesCleanAdvanced() { // Get an image for uploading. $image_files = $this->drupalGetTestFiles('image'); diff --git a/core/modules/system/src/Tests/Form/StateValuesCleanTest.php b/core/modules/system/src/Tests/Form/StateValuesCleanTest.php index 7761bd8..28c63c3 100644 --- a/core/modules/system/src/Tests/Form/StateValuesCleanTest.php +++ b/core/modules/system/src/Tests/Form/StateValuesCleanTest.php @@ -24,7 +24,7 @@ class StateValuesCleanTest extends WebTestBase { /** * Tests \Drupal\Core\Form\FormState::cleanValues(). */ - function testFormStateValuesClean() { + public function testFormStateValuesClean() { $values = Json::decode($this->drupalPostForm('form_test/form-state-values-clean', array(), t('Submit'))); // Setup the expected result. diff --git a/core/modules/system/src/Tests/Form/StorageTest.php b/core/modules/system/src/Tests/Form/StorageTest.php index d29f36b..79b4f50 100644 --- a/core/modules/system/src/Tests/Form/StorageTest.php +++ b/core/modules/system/src/Tests/Form/StorageTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Tests using the form in a usual way. */ - function testForm() { + public function testForm() { $this->drupalGet('form_test/form-storage'); $this->assertText('Form constructions: 1'); @@ -60,7 +60,7 @@ function testForm() { /** * Tests using the form after calling $form_state->setCached(). */ - function testFormCached() { + public function testFormCached() { $this->drupalGet('form_test/form-storage', array('query' => array('cache' => 1))); $this->assertText('Form constructions: 1'); @@ -87,7 +87,7 @@ function testFormCached() { /** * Tests validation when form storage is used. */ - function testValidation() { + public function testValidation() { $this->drupalPostForm('form_test/form-storage', array('title' => '', 'value' => 'value_is_set'), 'Continue submit'); $this->assertPattern('/value_is_set/', 'The input values have been kept.'); } @@ -102,7 +102,7 @@ function testValidation() { * during form validation, while another, required element in the form * triggers a form validation error. */ - function testCachedFormStorageValidation() { + public function testCachedFormStorageValidation() { // Request the form with 'cache' query parameter to enable form caching. $this->drupalGet('form_test/form-storage', array('query' => array('cache' => 1))); diff --git a/core/modules/system/src/Tests/Form/SystemConfigFormTest.php b/core/modules/system/src/Tests/Form/SystemConfigFormTest.php index 69a7f7e..55107b0 100644 --- a/core/modules/system/src/Tests/Form/SystemConfigFormTest.php +++ b/core/modules/system/src/Tests/Form/SystemConfigFormTest.php @@ -21,7 +21,7 @@ class SystemConfigFormTest extends WebTestBase { /** * Tests the SystemConfigFormTestBase class. */ - function testSystemConfigForm() { + public function testSystemConfigForm() { $this->drupalGet('form-test/system-config-form'); $element = $this->xpath('//div[@id = :id]/input[contains(@class, :class)]', array(':id' => 'edit-actions', ':class' => 'button--primary')); $this->assertTrue($element, 'The primary action submit button was found.'); diff --git a/core/modules/system/src/Tests/Form/TriggeringElementTest.php b/core/modules/system/src/Tests/Form/TriggeringElementTest.php index 5d8a180..8950664 100644 --- a/core/modules/system/src/Tests/Form/TriggeringElementTest.php +++ b/core/modules/system/src/Tests/Form/TriggeringElementTest.php @@ -23,7 +23,7 @@ class TriggeringElementTest extends WebTestBase { * information is included in the POST data, as is sometimes the case when * the ENTER key is pressed in a textfield in Internet Explorer. */ - function testNoButtonInfoInPost() { + public function testNoButtonInfoInPost() { $path = 'form-test/clicked-button'; $edit = array(); $form_html_id = 'form-test-clicked-button'; @@ -70,7 +70,7 @@ function testNoButtonInfoInPost() { * Test that the triggering element does not get set to a button with * #access=FALSE. */ - function testAttemptAccessControlBypass() { + public function testAttemptAccessControlBypass() { $path = 'form-test/clicked-button'; $form_html_id = 'form-test-clicked-button'; diff --git a/core/modules/system/src/Tests/Form/UrlTest.php b/core/modules/system/src/Tests/Form/UrlTest.php index 4bf6f60..8278493 100644 --- a/core/modules/system/src/Tests/Form/UrlTest.php +++ b/core/modules/system/src/Tests/Form/UrlTest.php @@ -24,7 +24,7 @@ class UrlTest extends WebTestBase { /** * Tests that #type 'url' fields are properly validated and trimmed. */ - function testFormUrl() { + public function testFormUrl() { $edit = array(); $edit['url'] = 'http://'; $edit['url_required'] = ' '; diff --git a/core/modules/system/src/Tests/Form/ValidationTest.php b/core/modules/system/src/Tests/Form/ValidationTest.php index 35b340b..267cd43 100644 --- a/core/modules/system/src/Tests/Form/ValidationTest.php +++ b/core/modules/system/src/Tests/Form/ValidationTest.php @@ -22,7 +22,7 @@ class ValidationTest extends WebTestBase { /** * Tests #element_validate and #validate. */ - function testValidate() { + public function testValidate() { $this->drupalGet('form-test/validate'); // Verify that #element_validate handlers can alter the form and submitted // form values. @@ -72,7 +72,7 @@ function testValidate() { /** * Tests that a form with a disabled CSRF token can be validated. */ - function testDisabledToken() { + public function testDisabledToken() { $this->drupalPostForm('form-test/validate-no-token', [], 'Save'); $this->assertText('The form_test_validate_no_token form has been submitted successfully.'); } @@ -80,7 +80,7 @@ function testDisabledToken() { /** * Tests partial form validation through #limit_validation_errors. */ - function testValidateLimitErrors() { + public function testValidateLimitErrors() { $edit = array( 'test' => 'invalid', 'test_numeric_index[0]' => 'invalid', @@ -141,7 +141,7 @@ function testValidateLimitErrors() { /** * Tests #pattern validation. */ - function testPatternValidation() { + public function testPatternValidation() { $textfield_error = t('%name field is not in the right format.', array('%name' => 'One digit followed by lowercase letters')); $tel_error = t('%name field is not in the right format.', array('%name' => 'Everything except numbers')); $password_error = t('%name field is not in the right format.', array('%name' => 'Password')); @@ -202,7 +202,7 @@ function testPatternValidation() { * * @see \Drupal\form_test\Form\FormTestValidateRequiredForm */ - function testCustomRequiredError() { + public function testCustomRequiredError() { $form = \Drupal::formBuilder()->getForm('\Drupal\form_test\Form\FormTestValidateRequiredForm'); // Verify that a custom #required error can be set. diff --git a/core/modules/system/src/Tests/Image/ToolkitSetupFormTest.php b/core/modules/system/src/Tests/Image/ToolkitSetupFormTest.php index 66989de..f5cbc9a 100644 --- a/core/modules/system/src/Tests/Image/ToolkitSetupFormTest.php +++ b/core/modules/system/src/Tests/Image/ToolkitSetupFormTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Test Image toolkit setup form. */ - function testToolkitSetupForm() { + public function testToolkitSetupForm() { // Get form. $this->drupalGet('admin/config/media/image-toolkit'); diff --git a/core/modules/system/src/Tests/Image/ToolkitTest.php b/core/modules/system/src/Tests/Image/ToolkitTest.php index 8848323..72e9762 100644 --- a/core/modules/system/src/Tests/Image/ToolkitTest.php +++ b/core/modules/system/src/Tests/Image/ToolkitTest.php @@ -12,7 +12,7 @@ class ToolkitTest extends ToolkitTestBase { * Check that ImageToolkitManager::getAvailableToolkits() only returns * available toolkits. */ - function testGetAvailableToolkits() { + public function testGetAvailableToolkits() { $manager = $this->container->get('image.toolkit.manager'); $toolkits = $manager->getAvailableToolkits(); $this->assertTrue(isset($toolkits['test']), 'The working toolkit was returned.'); @@ -24,7 +24,7 @@ function testGetAvailableToolkits() { /** * Tests Image's methods. */ - function testLoad() { + public function testLoad() { $image = $this->getImage(); $this->assertTrue(is_object($image), 'Returned an object.'); $this->assertEqual($image->getToolkitId(), 'test', 'Image had toolkit set.'); @@ -34,7 +34,7 @@ function testLoad() { /** * Test the image_save() function. */ - function testSave() { + public function testSave() { $this->assertFalse($this->image->save(), 'Function returned the expected value.'); $this->assertToolkitOperationsCalled(array('save')); } @@ -42,7 +42,7 @@ function testSave() { /** * Test the image_apply() function. */ - function testApply() { + public function testApply() { $data = array('p1' => 1, 'p2' => TRUE, 'p3' => 'text'); $this->assertTrue($this->image->apply('my_operation', $data), 'Function returned the expected value.'); @@ -58,7 +58,7 @@ function testApply() { /** * Test the image_apply() function. */ - function testApplyNoParameters() { + public function testApplyNoParameters() { $this->assertTrue($this->image->apply('my_operation'), 'Function returned the expected value.'); // Check that apply was called and with the correct parameters. diff --git a/core/modules/system/src/Tests/Installer/SiteNameTest.php b/core/modules/system/src/Tests/Installer/SiteNameTest.php index ec4c0e1..081dfb1 100644 --- a/core/modules/system/src/Tests/Installer/SiteNameTest.php +++ b/core/modules/system/src/Tests/Installer/SiteNameTest.php @@ -31,7 +31,7 @@ protected function installParameters() { /** * Tests that the desired site name appears on the page after installation. */ - function testSiteName() { + public function testSiteName() { $this->drupalGet(''); $this->assertRaw($this->siteName, 'The site name that was set during the installation appears on the front page after installation.'); } diff --git a/core/modules/system/src/Tests/Menu/BreadcrumbTest.php b/core/modules/system/src/Tests/Menu/BreadcrumbTest.php index f756300..1797ccd 100644 --- a/core/modules/system/src/Tests/Menu/BreadcrumbTest.php +++ b/core/modules/system/src/Tests/Menu/BreadcrumbTest.php @@ -58,7 +58,7 @@ protected function setUp() { /** * Tests breadcrumbs on node and administrative paths. */ - function testBreadCrumbs() { + public function testBreadCrumbs() { // Prepare common base breadcrumb elements. $home = array('' => 'Home'); $admin = $home + array('admin' => t('Administration')); diff --git a/core/modules/system/src/Tests/Module/ClassLoaderTest.php b/core/modules/system/src/Tests/Module/ClassLoaderTest.php index 5d1db03..5b060bd 100644 --- a/core/modules/system/src/Tests/Module/ClassLoaderTest.php +++ b/core/modules/system/src/Tests/Module/ClassLoaderTest.php @@ -21,7 +21,7 @@ class ClassLoaderTest extends WebTestBase { * * @see \Drupal\module_autoload_test\SomeClass */ - function testClassLoading() { + public function testClassLoading() { // Enable the module_test and module_autoload_test modules. \Drupal::service('module_installer')->install(array('module_test', 'module_autoload_test'), FALSE); $this->resetAll(); @@ -37,7 +37,7 @@ function testClassLoading() { * * @see \Drupal\module_autoload_test\SomeClass */ - function testClassLoadingDisabledModules() { + public function testClassLoadingDisabledModules() { // Ensure that module_autoload_test is disabled. $this->container->get('module_installer')->uninstall(array('module_autoload_test'), FALSE); $this->resetAll(); diff --git a/core/modules/system/src/Tests/Module/DependencyTest.php b/core/modules/system/src/Tests/Module/DependencyTest.php index 8b219b2..d6c1858 100644 --- a/core/modules/system/src/Tests/Module/DependencyTest.php +++ b/core/modules/system/src/Tests/Module/DependencyTest.php @@ -12,7 +12,7 @@ class DependencyTest extends ModuleTestBase { /** * Checks functionality of project namespaces for dependencies. */ - function testProjectNamespaceForDependencies() { + public function testProjectNamespaceForDependencies() { $edit = array( 'modules[Core][filter][enable]' => TRUE, ); @@ -28,7 +28,7 @@ function testProjectNamespaceForDependencies() { /** * Attempts to enable the Content Translation module without Language enabled. */ - function testEnableWithoutDependency() { + public function testEnableWithoutDependency() { // Attempt to enable Content Translation without Language enabled. $edit = array(); $edit['modules[Multilingual][content_translation][enable]'] = 'content_translation'; @@ -52,7 +52,7 @@ function testEnableWithoutDependency() { /** * Attempts to enable a module with a missing dependency. */ - function testMissingModules() { + public function testMissingModules() { // Test that the system_dependencies_test module is marked // as missing a dependency. $this->drupalGet('admin/modules'); @@ -64,7 +64,7 @@ function testMissingModules() { /** * Tests enabling a module that depends on an incompatible version of a module. */ - function testIncompatibleModuleVersionDependency() { + public function testIncompatibleModuleVersionDependency() { // Test that the system_incompatible_module_version_dependencies_test is // marked as having an incompatible dependency. $this->drupalGet('admin/modules'); @@ -79,7 +79,7 @@ function testIncompatibleModuleVersionDependency() { /** * Tests enabling a module that depends on a module with an incompatible core version. */ - function testIncompatibleCoreVersionDependency() { + public function testIncompatibleCoreVersionDependency() { // Test that the system_incompatible_core_version_dependencies_test is // marked as having an incompatible dependency. $this->drupalGet('admin/modules'); @@ -93,7 +93,7 @@ function testIncompatibleCoreVersionDependency() { /** * Tests enabling a module that depends on a module which fails hook_requirements(). */ - function testEnableRequirementsFailureDependency() { + public function testEnableRequirementsFailureDependency() { \Drupal::service('module_installer')->install(array('comment')); $this->assertModules(array('requirements1_test'), FALSE); @@ -120,7 +120,7 @@ function testEnableRequirementsFailureDependency() { * * Dependencies should be enabled before their dependents. */ - function testModuleEnableOrder() { + public function testModuleEnableOrder() { \Drupal::service('module_installer')->install(array('module_test'), FALSE); $this->resetAll(); $this->assertModules(array('module_test'), TRUE); @@ -153,7 +153,7 @@ function testModuleEnableOrder() { /** * Tests attempting to uninstall a module that has installed dependents. */ - function testUninstallDependents() { + public function testUninstallDependents() { // Enable the forum module. $edit = array('modules[Core][forum][enable]' => 'forum'); $this->drupalPostForm('admin/modules', $edit, t('Install')); diff --git a/core/modules/system/src/Tests/Module/HookRequirementsTest.php b/core/modules/system/src/Tests/Module/HookRequirementsTest.php index ccbbff9..c25b66c 100644 --- a/core/modules/system/src/Tests/Module/HookRequirementsTest.php +++ b/core/modules/system/src/Tests/Module/HookRequirementsTest.php @@ -11,7 +11,7 @@ class HookRequirementsTest extends ModuleTestBase { /** * Assert that a module cannot be installed if it fails hook_requirements(). */ - function testHookRequirementsFailure() { + public function testHookRequirementsFailure() { $this->assertModules(array('requirements1_test'), FALSE); // Attempt to install the requirements1_test module. diff --git a/core/modules/system/src/Tests/Module/RequiredTest.php b/core/modules/system/src/Tests/Module/RequiredTest.php index 004854b..96d2742 100644 --- a/core/modules/system/src/Tests/Module/RequiredTest.php +++ b/core/modules/system/src/Tests/Module/RequiredTest.php @@ -11,7 +11,7 @@ class RequiredTest extends ModuleTestBase { /** * Assert that core required modules cannot be disabled. */ - function testDisableRequired() { + public function testDisableRequired() { $module_info = system_get_info('module'); $this->drupalGet('admin/modules'); foreach ($module_info as $module => $info) { diff --git a/core/modules/system/src/Tests/Module/UninstallTest.php b/core/modules/system/src/Tests/Module/UninstallTest.php index 9cecf8f..b719149 100644 --- a/core/modules/system/src/Tests/Module/UninstallTest.php +++ b/core/modules/system/src/Tests/Module/UninstallTest.php @@ -26,7 +26,7 @@ class UninstallTest extends WebTestBase { /** * Tests the hook_modules_uninstalled() of the user module. */ - function testUserPermsUninstalled() { + public function testUserPermsUninstalled() { // Uninstalls the module_test module, so hook_modules_uninstalled() // is executed. $this->container->get('module_installer')->uninstall(array('module_test')); @@ -38,7 +38,7 @@ function testUserPermsUninstalled() { /** * Tests the Uninstall page and Uninstall confirmation page. */ - function testUninstallPage() { + public function testUninstallPage() { $account = $this->drupalCreateUser(array('administer modules')); $this->drupalLogin($account); diff --git a/core/modules/system/src/Tests/Module/VersionTest.php b/core/modules/system/src/Tests/Module/VersionTest.php index 031d4ba..e5c8a91 100644 --- a/core/modules/system/src/Tests/Module/VersionTest.php +++ b/core/modules/system/src/Tests/Module/VersionTest.php @@ -12,7 +12,7 @@ class VersionTest extends ModuleTestBase { /** * Test version dependencies. */ - function testModuleVersions() { + public function testModuleVersions() { $dependencies = array( // Alternating between being compatible and incompatible with 8.x-2.4-beta3. // The first is always a compatible. diff --git a/core/modules/system/src/Tests/Pager/PagerTest.php b/core/modules/system/src/Tests/Pager/PagerTest.php index 1000b1c..e3ba22a 100644 --- a/core/modules/system/src/Tests/Pager/PagerTest.php +++ b/core/modules/system/src/Tests/Pager/PagerTest.php @@ -45,7 +45,7 @@ protected function setUp() { /** * Tests markup and CSS classes of pager links. */ - function testActiveClass() { + public function testActiveClass() { // Verify first page. $this->drupalGet('admin/reports/dblog'); $current_page = 0; @@ -67,7 +67,7 @@ function testActiveClass() { /** * Test proper functioning of the query parameters and the pager cache context. */ - protected function testPagerQueryParametersAndCacheContext() { + public function testPagerQueryParametersAndCacheContext() { // First page. $this->drupalGet('pager-test/query-parameters'); $this->assertText(t('Pager calls: 0'), 'Initial call to pager shows 0 calls.'); diff --git a/core/modules/system/src/Tests/Path/UrlAlterFunctionalTest.php b/core/modules/system/src/Tests/Path/UrlAlterFunctionalTest.php index 743b74b..29655a0 100644 --- a/core/modules/system/src/Tests/Path/UrlAlterFunctionalTest.php +++ b/core/modules/system/src/Tests/Path/UrlAlterFunctionalTest.php @@ -23,7 +23,7 @@ class UrlAlterFunctionalTest extends WebTestBase { /** * Test that URL altering works and that it occurs in the correct order. */ - function testUrlAlter() { + public function testUrlAlter() { // Ensure that the url_alias table exists after Drupal installation. $this->assertTrue(Database::getConnection()->schema()->tableExists('url_alias'), 'The url_alias table exists after Drupal installation.'); diff --git a/core/modules/system/src/Tests/Render/DisplayVariantTest.php b/core/modules/system/src/Tests/Render/DisplayVariantTest.php index 09f54d9..6e8ea72 100644 --- a/core/modules/system/src/Tests/Render/DisplayVariantTest.php +++ b/core/modules/system/src/Tests/Render/DisplayVariantTest.php @@ -21,7 +21,7 @@ class DisplayVariantTest extends WebTestBase { /** * Tests selecting the variant and passing configuration. */ - function testPageDisplayVariantSelectionEvent() { + public function testPageDisplayVariantSelectionEvent() { // Tests that our display variant was selected, and that its configuration // was passed correctly. If the configuration wasn't passed, we'd get an // error page here. diff --git a/core/modules/system/src/Tests/Session/SessionTest.php b/core/modules/system/src/Tests/Session/SessionTest.php index 139fba5..c3db296 100644 --- a/core/modules/system/src/Tests/Session/SessionTest.php +++ b/core/modules/system/src/Tests/Session/SessionTest.php @@ -24,7 +24,7 @@ class SessionTest extends WebTestBase { * Tests for \Drupal\Core\Session\WriteSafeSessionHandler::setSessionWritable() * ::isSessionWritable and \Drupal\Core\Session\SessionManager::regenerate(). */ - function testSessionSaveRegenerate() { + public function testSessionSaveRegenerate() { $session_handler = $this->container->get('session_handler.write_safe'); $this->assertTrue($session_handler->isSessionWritable(), 'session_handler->isSessionWritable() initially returns TRUE.'); $session_handler->setSessionWritable(FALSE); @@ -74,7 +74,7 @@ function testSessionSaveRegenerate() { /** * Test data persistence via the session_test module callbacks. */ - function testDataPersistence() { + public function testDataPersistence() { $user = $this->drupalCreateUser(array()); // Enable sessions. $this->sessionReset($user->id()); @@ -147,7 +147,7 @@ public function testSessionPersistenceOnLogin() { /** * Test that empty anonymous sessions are destroyed. */ - function testEmptyAnonymousSession() { + public function testEmptyAnonymousSession() { // Disable the dynamic_page_cache module; it'd cause session_test's debug // output (that is added in // SessionTestSubscriber::onKernelResponseSessionTest()) to not be added. @@ -210,7 +210,7 @@ function testEmptyAnonymousSession() { /** * Test that sessions are only saved when necessary. */ - function testSessionWrite() { + public function testSessionWrite() { $user = $this->drupalCreateUser(array()); $this->drupalLogin($user); @@ -258,7 +258,7 @@ function testSessionWrite() { /** * Test that empty session IDs are not allowed. */ - function testEmptySessionID() { + public function testEmptySessionID() { $user = $this->drupalCreateUser(array()); $this->drupalLogin($user); $this->drupalGet('session-test/is-logged-in'); diff --git a/core/modules/system/src/Tests/System/AccessDeniedTest.php b/core/modules/system/src/Tests/System/AccessDeniedTest.php index 81f9fad..2468394 100644 --- a/core/modules/system/src/Tests/System/AccessDeniedTest.php +++ b/core/modules/system/src/Tests/System/AccessDeniedTest.php @@ -36,7 +36,7 @@ protected function setUp() { user_role_grant_permissions(RoleInterface::AUTHENTICATED_ID, array('access user profiles')); } - function testAccessDenied() { + public function testAccessDenied() { $this->drupalGet('admin'); $this->assertText(t('Access denied'), 'Found the default 403 page'); $this->assertResponse(403); diff --git a/core/modules/system/src/Tests/System/AdminTest.php b/core/modules/system/src/Tests/System/AdminTest.php index 8921ad0..f562b0f 100644 --- a/core/modules/system/src/Tests/System/AdminTest.php +++ b/core/modules/system/src/Tests/System/AdminTest.php @@ -51,7 +51,7 @@ protected function setUp() { /** * Tests output on administrative listing pages. */ - function testAdminPages() { + public function testAdminPages() { // Go to Administration. $this->drupalGet('admin'); @@ -143,7 +143,7 @@ protected function getTopLevelMenuLinks() { /** * Test compact mode. */ - function testCompactMode() { + public function testCompactMode() { // The front page defaults to 'user/login', which redirects to 'user/{user}' // for authenticated users. We cannot use '', since this does not // match the redirected url. diff --git a/core/modules/system/src/Tests/System/CronRunTest.php b/core/modules/system/src/Tests/System/CronRunTest.php index ec35b72..87ca88a 100644 --- a/core/modules/system/src/Tests/System/CronRunTest.php +++ b/core/modules/system/src/Tests/System/CronRunTest.php @@ -21,7 +21,7 @@ class CronRunTest extends WebTestBase { /** * Test cron runs. */ - function testCronRun() { + public function testCronRun() { // Run cron anonymously without any cron key. $this->drupalGet('cron'); $this->assertResponse(404); @@ -43,7 +43,7 @@ function testCronRun() { * In these tests we do not use REQUEST_TIME to track start time, because we * need the exact time when cron is triggered. */ - function testAutomatedCron() { + public function testAutomatedCron() { // Test with a logged in user; anonymous users likely don't cause Drupal to // fully bootstrap, because of the internal page cache or an external // reverse proxy. Reuse this user for disabling cron later in the test. @@ -83,7 +83,7 @@ function testAutomatedCron() { /** * Make sure exceptions thrown on hook_cron() don't affect other modules. */ - function testCronExceptions() { + public function testCronExceptions() { \Drupal::state()->delete('common_test.cron'); // The common_test module throws an exception. If it isn't caught, the tests // won't finish successfully. @@ -96,7 +96,7 @@ function testCronExceptions() { /** * Make sure the cron UI reads from the state storage. */ - function testCronUI() { + public function testCronUI() { $admin_user = $this->drupalCreateUser(array('administer site configuration')); $this->drupalLogin($admin_user); $this->drupalGet('admin/config/system/cron'); diff --git a/core/modules/system/src/Tests/System/DateTimeTest.php b/core/modules/system/src/Tests/System/DateTimeTest.php index 9ca050f..9007b3f 100644 --- a/core/modules/system/src/Tests/System/DateTimeTest.php +++ b/core/modules/system/src/Tests/System/DateTimeTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Test time zones and DST handling. */ - function testTimeZoneHandling() { + public function testTimeZoneHandling() { // Setup date/time settings for Honolulu time. $config = $this->config('system.date') ->set('timezone.default', 'Pacific/Honolulu') @@ -76,7 +76,7 @@ function testTimeZoneHandling() { /** * Test date format configuration. */ - function testDateFormatConfiguration() { + public function testDateFormatConfiguration() { // Confirm 'no custom date formats available' message appears. $this->drupalGet('admin/config/regional/date-time'); @@ -167,7 +167,7 @@ function testDateFormatConfiguration() { /** * Test handling case with invalid data in selectors (like February, 31st). */ - function testEnteringDateTimeViaSelectors() { + public function testEnteringDateTimeViaSelectors() { $this->drupalCreateContentType(array('type' => 'page_with_date', 'name' => 'Page with date')); diff --git a/core/modules/system/src/Tests/System/ErrorHandlerTest.php b/core/modules/system/src/Tests/System/ErrorHandlerTest.php index 113d495..53800de 100644 --- a/core/modules/system/src/Tests/System/ErrorHandlerTest.php +++ b/core/modules/system/src/Tests/System/ErrorHandlerTest.php @@ -22,7 +22,7 @@ class ErrorHandlerTest extends WebTestBase { /** * Test the error handler. */ - function testErrorHandler() { + public function testErrorHandler() { $config = $this->config('system.logging'); $error_notice = array( '%type' => 'Notice', @@ -121,7 +121,7 @@ function testErrorHandler() { /** * Test the exception handler. */ - function testExceptionHandler() { + public function testExceptionHandler() { // Ensure the test error log is empty before these tests. $this->assertNoErrorsLogged(); diff --git a/core/modules/system/src/Tests/System/IndexPhpTest.php b/core/modules/system/src/Tests/System/IndexPhpTest.php index 72811a8..9bf8efd 100644 --- a/core/modules/system/src/Tests/System/IndexPhpTest.php +++ b/core/modules/system/src/Tests/System/IndexPhpTest.php @@ -17,7 +17,7 @@ protected function setUp() { /** * Test index.php handling. */ - function testIndexPhpHandling() { + public function testIndexPhpHandling() { $index_php = $GLOBALS['base_url'] . '/index.php'; $this->drupalGet($index_php, array('external' => TRUE)); diff --git a/core/modules/system/src/Tests/System/MainContentFallbackTest.php b/core/modules/system/src/Tests/System/MainContentFallbackTest.php index 551e4c7..107ff26 100644 --- a/core/modules/system/src/Tests/System/MainContentFallbackTest.php +++ b/core/modules/system/src/Tests/System/MainContentFallbackTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Test availability of main content: Drupal falls back to SimplePageVariant. */ - function testMainContentFallback() { + public function testMainContentFallback() { $edit = array(); // Uninstall the block module. $edit['uninstall[block]'] = 'block'; diff --git a/core/modules/system/src/Tests/System/PageNotFoundTest.php b/core/modules/system/src/Tests/System/PageNotFoundTest.php index f289f0e..38adfe7 100644 --- a/core/modules/system/src/Tests/System/PageNotFoundTest.php +++ b/core/modules/system/src/Tests/System/PageNotFoundTest.php @@ -34,7 +34,7 @@ protected function setUp() { user_role_grant_permissions(RoleInterface::AUTHENTICATED_ID, array('access user profiles')); } - function testPageNotFound() { + public function testPageNotFound() { $this->drupalLogin($this->adminUser); $this->drupalGet($this->randomMachineName(10)); $this->assertText(t('Page not found'), 'Found the default 404 page'); diff --git a/core/modules/system/src/Tests/System/PageTitleTest.php b/core/modules/system/src/Tests/System/PageTitleTest.php index 848eacc..2c8574f 100644 --- a/core/modules/system/src/Tests/System/PageTitleTest.php +++ b/core/modules/system/src/Tests/System/PageTitleTest.php @@ -40,7 +40,7 @@ protected function setUp() { /** * Tests the handling of HTML in node titles. */ - function testTitleTags() { + public function testTitleTags() { $title = "string with HTML"; // Generate node content. $edit = array( @@ -60,7 +60,7 @@ function testTitleTags() { /** * Test if the title of the site is XSS proof. */ - function testTitleXSS() { + public function testTitleXSS() { // Set some title with JavaScript and HTML chars to escape. $title = ' & < > " \' '; $title_filtered = Html::escape($title); diff --git a/core/modules/system/src/Tests/System/ResponseGeneratorTest.php b/core/modules/system/src/Tests/System/ResponseGeneratorTest.php index d0d22a2..ce9a25d 100644 --- a/core/modules/system/src/Tests/System/ResponseGeneratorTest.php +++ b/core/modules/system/src/Tests/System/ResponseGeneratorTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Test to see if generator header is added. */ - function testGeneratorHeaderAdded() { + public function testGeneratorHeaderAdded() { $node = $this->drupalCreateNode(); diff --git a/core/modules/system/src/Tests/System/RetrieveFileTest.php b/core/modules/system/src/Tests/System/RetrieveFileTest.php index 8f6fe4a..f7974b3 100644 --- a/core/modules/system/src/Tests/System/RetrieveFileTest.php +++ b/core/modules/system/src/Tests/System/RetrieveFileTest.php @@ -13,7 +13,7 @@ class RetrieveFileTest extends WebTestBase { /** * Invokes system_retrieve_file() in several scenarios. */ - function testFileRetrieving() { + public function testFileRetrieving() { // Test 404 handling by trying to fetch a randomly named file. drupal_mkdir($sourcedir = 'public://' . $this->randomMachineName()); $filename = 'Файл для тестирования ' . $this->randomMachineName(); diff --git a/core/modules/system/src/Tests/System/ShutdownFunctionsTest.php b/core/modules/system/src/Tests/System/ShutdownFunctionsTest.php index 99839ab..9c2081d 100644 --- a/core/modules/system/src/Tests/System/ShutdownFunctionsTest.php +++ b/core/modules/system/src/Tests/System/ShutdownFunctionsTest.php @@ -29,7 +29,7 @@ protected function tearDown() { /** * Test shutdown functions. */ - function testShutdownFunctions() { + public function testShutdownFunctions() { $arg1 = $this->randomMachineName(); $arg2 = $this->randomMachineName(); $this->drupalGet('system-test/shutdown-functions/' . $arg1 . '/' . $arg2); diff --git a/core/modules/system/src/Tests/System/SiteMaintenanceTest.php b/core/modules/system/src/Tests/System/SiteMaintenanceTest.php index 49aff66..92cc477 100644 --- a/core/modules/system/src/Tests/System/SiteMaintenanceTest.php +++ b/core/modules/system/src/Tests/System/SiteMaintenanceTest.php @@ -38,7 +38,7 @@ protected function setUp() { /** * Verifies site maintenance mode functionality. */ - protected function testSiteMaintenance() { + public function testSiteMaintenance() { $this->drupalGet(Url::fromRoute('user.page')); // JS should be aggregated, so drupal.js is not in the page source. $links = $this->xpath('//script[contains(@src, :href)]', array(':href' => '/core/misc/drupal.js')); diff --git a/core/modules/system/src/Tests/System/SystemAuthorizeTest.php b/core/modules/system/src/Tests/System/SystemAuthorizeTest.php index 5e278d7..25edf17 100644 --- a/core/modules/system/src/Tests/System/SystemAuthorizeTest.php +++ b/core/modules/system/src/Tests/System/SystemAuthorizeTest.php @@ -43,7 +43,7 @@ function drupalGetAuthorizePHP($page_title = 'system-test-auth') { /** * Tests the FileTransfer hooks */ - function testFileTransferHooks() { + public function testFileTransferHooks() { $page_title = $this->randomMachineName(16); $this->drupalGetAuthorizePHP($page_title); $this->assertTitle(strtr('@title | Drupal', array('@title' => $page_title)), 'authorize.php page title is correct.'); diff --git a/core/modules/system/src/Tests/System/ThemeTest.php b/core/modules/system/src/Tests/System/ThemeTest.php index e01f28a..a49f3aa 100644 --- a/core/modules/system/src/Tests/System/ThemeTest.php +++ b/core/modules/system/src/Tests/System/ThemeTest.php @@ -41,7 +41,7 @@ protected function setUp() { /** * Test the theme settings form. */ - function testThemeSettings() { + public function testThemeSettings() { // Ensure invalid theme settings form URLs return a proper 404. $this->drupalGet('admin/appearance/settings/bartik'); $this->assertResponse(404, 'The theme settings form URL for a uninstalled theme could not be found.'); @@ -218,7 +218,7 @@ function testThemeSettings() { /** * Test the theme settings logo form. */ - function testThemeSettingsLogo() { + public function testThemeSettingsLogo() { // Visit Bartik's theme settings page to replace the logo. $this->container->get('theme_handler')->install(['bartik']); $this->drupalGet('admin/appearance/settings/bartik'); @@ -241,7 +241,7 @@ function testThemeSettingsLogo() { /** * Test the administration theme functionality. */ - function testAdministrationTheme() { + public function testAdministrationTheme() { $this->container->get('theme_handler')->install(array('seven')); // Install an administration theme and show it on the node admin pages. @@ -300,7 +300,7 @@ function testAdministrationTheme() { /** * Test switching the default theme. */ - function testSwitchDefaultTheme() { + public function testSwitchDefaultTheme() { /** @var \Drupal\Core\Extension\ThemeHandlerInterface $theme_handler */ $theme_handler = \Drupal::service('theme_handler'); // First, install Stark and set it as the default theme programmatically. @@ -330,7 +330,7 @@ function testSwitchDefaultTheme() { * Include test for themes that have a missing base theme somewhere further up * the chain than the immediate base theme. */ - function testInvalidTheme() { + public function testInvalidTheme() { // theme_page_test_system_info_alter() un-hides all hidden themes. $this->container->get('module_installer')->install(array('theme_page_test')); // Clear the system_list() and theme listing cache to pick up the change. @@ -348,7 +348,7 @@ function testInvalidTheme() { /** * Test uninstalling of themes works. */ - function testUninstallingThemes() { + public function testUninstallingThemes() { // Install Bartik and set it as the default theme. \Drupal::service('theme_handler')->install(array('bartik')); // Set up seven as the admin theme. @@ -408,7 +408,7 @@ function testUninstallingThemes() { /** * Tests installing a theme and setting it as default. */ - function testInstallAndSetAsDefault() { + public function testInstallAndSetAsDefault() { $this->drupalGet('admin/appearance'); // Bartik is uninstalled in the test profile and has the third "Install and // set as default" link. diff --git a/core/modules/system/src/Tests/System/TokenScanTest.php b/core/modules/system/src/Tests/System/TokenScanTest.php index 89f2abd..9c3e7d9 100644 --- a/core/modules/system/src/Tests/System/TokenScanTest.php +++ b/core/modules/system/src/Tests/System/TokenScanTest.php @@ -14,7 +14,7 @@ class TokenScanTest extends WebTestBase { /** * Scans dummy text, then tests the output. */ - function testTokenScan() { + public function testTokenScan() { // Define text with valid and not valid, fake and existing token-like // strings. $text = 'First a [valid:simple], but dummy token, and a dummy [valid:token with: spaces].'; diff --git a/core/modules/system/src/Tests/Theme/EngineNyanCatTest.php b/core/modules/system/src/Tests/Theme/EngineNyanCatTest.php index 0eb43b7..e0fde83 100644 --- a/core/modules/system/src/Tests/Theme/EngineNyanCatTest.php +++ b/core/modules/system/src/Tests/Theme/EngineNyanCatTest.php @@ -26,7 +26,7 @@ protected function setUp() { /** * Ensures a theme's template is overridable based on the 'template' filename. */ - function testTemplateOverride() { + public function testTemplateOverride() { $this->config('system.theme') ->set('default', 'test_theme_nyan_cat_engine') ->save(); diff --git a/core/modules/system/src/Tests/Theme/EngineTwigTest.php b/core/modules/system/src/Tests/Theme/EngineTwigTest.php index a6b7acf..56ece78 100644 --- a/core/modules/system/src/Tests/Theme/EngineTwigTest.php +++ b/core/modules/system/src/Tests/Theme/EngineTwigTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Tests that the Twig engine handles PHP data correctly. */ - function testTwigVariableDataTypes() { + public function testTwigVariableDataTypes() { $this->config('system.theme') ->set('default', 'test_theme') ->save(); diff --git a/core/modules/system/src/Tests/Theme/EntityFilteringThemeTest.php b/core/modules/system/src/Tests/Theme/EntityFilteringThemeTest.php index cc4af2a..214a88e 100644 --- a/core/modules/system/src/Tests/Theme/EntityFilteringThemeTest.php +++ b/core/modules/system/src/Tests/Theme/EntityFilteringThemeTest.php @@ -121,7 +121,7 @@ protected function setUp() { /** * Checks each themed entity for XSS filtering in available themes. */ - function testThemedEntity() { + public function testThemedEntity() { // Check paths where various view modes of the entities are rendered. $paths = array( 'user', diff --git a/core/modules/system/src/Tests/Theme/FastTest.php b/core/modules/system/src/Tests/Theme/FastTest.php index a05666f..851d628 100644 --- a/core/modules/system/src/Tests/Theme/FastTest.php +++ b/core/modules/system/src/Tests/Theme/FastTest.php @@ -26,7 +26,7 @@ protected function setUp() { /** * Tests access to user autocompletion and verify the correct results. */ - function testUserAutocomplete() { + public function testUserAutocomplete() { $this->drupalLogin($this->account); $this->drupalGet('user/autocomplete', array('query' => array('q' => $this->account->getUsername()))); $this->assertRaw($this->account->getUsername()); diff --git a/core/modules/system/src/Tests/Theme/FunctionsTest.php b/core/modules/system/src/Tests/Theme/FunctionsTest.php index fa5a2ac..82ebd5f 100644 --- a/core/modules/system/src/Tests/Theme/FunctionsTest.php +++ b/core/modules/system/src/Tests/Theme/FunctionsTest.php @@ -26,7 +26,7 @@ class FunctionsTest extends WebTestBase { /** * Tests item-list.html.twig. */ - function testItemList() { + public function testItemList() { // Verify that empty items produce no output. $variables = array(); $expected = ''; @@ -166,7 +166,7 @@ function testItemList() { /** * Tests links.html.twig. */ - function testLinks() { + public function testLinks() { // Turn off the query for the // \Drupal\Core\Utility\LinkGeneratorInterface::generate() method to compare // the active link correctly. @@ -287,7 +287,7 @@ function testLinks() { /** * Tests links.html.twig using links with indexed keys. */ - function testIndexedKeyedLinks() { + public function testIndexedKeyedLinks() { // Turn off the query for the // \Drupal\Core\Utility\LinkGeneratorInterface::generate() method to compare // the active link correctly. @@ -408,7 +408,7 @@ function testIndexedKeyedLinks() { /** * Test the use of drupal_pre_render_links() on a nested array of links. */ - function testDrupalPreRenderLinks() { + public function testDrupalPreRenderLinks() { // Define the base array to be rendered, containing a variety of different // kinds of links. $base_array = array( @@ -506,7 +506,7 @@ function testDrupalPreRenderLinks() { /** * Tests theme_image(). */ - function testImage() { + public function testImage() { // Test that data URIs work with theme_image(). $variables = array(); $variables['uri'] = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAYAAACNbyblAAAAHElEQVQI12P4//8/w38GIAXDIBKE0DHxgljNBAAO9TXL0Y4OHwAAAABJRU5ErkJggg=='; diff --git a/core/modules/system/src/Tests/Theme/HtmlAttributesTest.php b/core/modules/system/src/Tests/Theme/HtmlAttributesTest.php index f103caa..8b3e9f0 100644 --- a/core/modules/system/src/Tests/Theme/HtmlAttributesTest.php +++ b/core/modules/system/src/Tests/Theme/HtmlAttributesTest.php @@ -21,7 +21,7 @@ class HtmlAttributesTest extends WebTestBase { /** * Tests that attributes in the 'html' and 'body' elements can be altered. */ - function testThemeHtmlAttributes() { + public function testThemeHtmlAttributes() { $this->drupalGet(''); $attributes = $this->xpath('/html[@theme_test_html_attribute="theme test html attribute value"]'); $this->assertTrue(count($attributes) == 1, "Attribute set in the 'html' element via hook_preprocess_HOOK() found."); diff --git a/core/modules/system/src/Tests/Theme/ThemeEarlyInitializationTest.php b/core/modules/system/src/Tests/Theme/ThemeEarlyInitializationTest.php index ddbb770..f7e8d8c 100644 --- a/core/modules/system/src/Tests/Theme/ThemeEarlyInitializationTest.php +++ b/core/modules/system/src/Tests/Theme/ThemeEarlyInitializationTest.php @@ -22,7 +22,7 @@ class ThemeEarlyInitializationTest extends WebTestBase { /** * Test that the theme system can generate output in a request listener. */ - function testRequestListener() { + public function testRequestListener() { $this->drupalGet('theme-test/request-listener'); // Verify that themed output generated in the request listener appears. $this->assertRaw('Themed output generated in a KernelEvents::REQUEST listener'); diff --git a/core/modules/system/src/Tests/Theme/ThemeInfoTest.php b/core/modules/system/src/Tests/Theme/ThemeInfoTest.php index 043986a..a84d032 100644 --- a/core/modules/system/src/Tests/Theme/ThemeInfoTest.php +++ b/core/modules/system/src/Tests/Theme/ThemeInfoTest.php @@ -53,7 +53,7 @@ protected function setUp() { /** * Tests stylesheets-remove. */ - function testStylesheets() { + public function testStylesheets() { $this->themeHandler->install(array('test_basetheme', 'test_subtheme')); $this->config('system.theme') ->set('default', 'test_subtheme') diff --git a/core/modules/system/src/Tests/Theme/ThemeSuggestionsAlterTest.php b/core/modules/system/src/Tests/Theme/ThemeSuggestionsAlterTest.php index 75e52c9..348be07 100644 --- a/core/modules/system/src/Tests/Theme/ThemeSuggestionsAlterTest.php +++ b/core/modules/system/src/Tests/Theme/ThemeSuggestionsAlterTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Tests that hooks to provide theme suggestions work. */ - function testTemplateSuggestions() { + public function testTemplateSuggestions() { $this->drupalGet('theme-test/suggestion-provided'); $this->assertText('Template for testing suggestions provided by the module declaring the theme hook.'); @@ -44,7 +44,7 @@ function testTemplateSuggestions() { /** * Tests hook_theme_suggestions_alter(). */ - function testGeneralSuggestionsAlter() { + public function testGeneralSuggestionsAlter() { $this->drupalGet('theme-test/general-suggestion-alter'); $this->assertText('Original template for testing hook_theme_suggestions_alter().'); @@ -66,7 +66,7 @@ function testGeneralSuggestionsAlter() { /** * Tests that theme suggestion alter hooks work for templates. */ - function testTemplateSuggestionsAlter() { + public function testTemplateSuggestionsAlter() { $this->drupalGet('theme-test/suggestion-alter'); $this->assertText('Original template for testing hook_theme_suggestions_HOOK_alter().'); @@ -88,7 +88,7 @@ function testTemplateSuggestionsAlter() { /** * Tests that theme suggestion alter hooks work for specific theme calls. */ - function testSpecificSuggestionsAlter() { + public function testSpecificSuggestionsAlter() { // Test that the default template is rendered. $this->drupalGet('theme-test/specific-suggestion-alter'); $this->assertText('Template for testing specific theme calls.'); @@ -113,7 +113,7 @@ function testSpecificSuggestionsAlter() { /** * Tests that theme suggestion alter hooks work for theme functions. */ - function testThemeFunctionSuggestionsAlter() { + public function testThemeFunctionSuggestionsAlter() { $this->drupalGet('theme-test/function-suggestion-alter'); $this->assertText('Original theme function.'); @@ -157,7 +157,7 @@ public function testSuggestionsAlterInclude() { * hook_theme_suggestions_alter() should fire before * hook_theme_suggestions_HOOK_alter() within an extension (module or theme). */ - function testExecutionOrder() { + public function testExecutionOrder() { // Install our test theme and module. $this->config('system.theme') ->set('default', 'test_theme') diff --git a/core/modules/system/src/Tests/Theme/ThemeTest.php b/core/modules/system/src/Tests/Theme/ThemeTest.php index 4eae7ae..ead2230 100644 --- a/core/modules/system/src/Tests/Theme/ThemeTest.php +++ b/core/modules/system/src/Tests/Theme/ThemeTest.php @@ -39,7 +39,7 @@ protected function setUp() { * - the render element's #attributes * - any attributes set in the template's preprocessing function */ - function testAttributeMerging() { + public function testAttributeMerging() { $theme_test_render_element = array( 'elements' => array( '#attributes' => array('data-foo' => 'bar'), @@ -54,7 +54,7 @@ function testAttributeMerging() { /** * Test that ThemeManager renders the expected data types. */ - function testThemeDataTypes() { + public function testThemeDataTypes() { // theme_test_false is an implemented theme hook so \Drupal::theme() service // should return a string or an object that implements MarkupInterface, // even though the theme function itself can return anything. @@ -79,7 +79,7 @@ function testThemeDataTypes() { /** * Test function theme_get_suggestions() for SA-CORE-2009-003. */ - function testThemeSuggestions() { + public function testThemeSuggestions() { // Set the front page as something random otherwise the CLI // test runner fails. $this->config('system.site')->set('page.front', '/nobody-home')->save(); @@ -110,7 +110,7 @@ function testThemeSuggestions() { * separate file, so this test also ensures that that file is correctly loaded * when needed. */ - function testPreprocessForSuggestions() { + public function testPreprocessForSuggestions() { // Test with both an unprimed and primed theme registry. drupal_theme_rebuild(); for ($i = 0; $i < 2; $i++) { @@ -142,7 +142,7 @@ public function testThemeOnNonHtmlRequest() { /** * Ensure page-front template suggestion is added when on front page. */ - function testFrontPageThemeSuggestion() { + public function testFrontPageThemeSuggestion() { // Set the current route to user.login because theme_get_suggestions() will // query it to see if we are on the front page. $request = Request::create('/user/login'); @@ -161,7 +161,7 @@ function testFrontPageThemeSuggestion() { * * @see test_theme.info.yml */ - function testCSSOverride() { + public function testCSSOverride() { // Reuse the same page as in testPreprocessForSuggestions(). We're testing // what is output to the HTML HEAD based on what is in a theme's .info.yml // file, so it doesn't matter what page we get, as long as it is themed with @@ -186,7 +186,7 @@ function testCSSOverride() { /** * Ensures a themes template is overridable based on the 'template' filename. */ - function testTemplateOverride() { + public function testTemplateOverride() { $this->config('system.theme') ->set('default', 'test_theme') ->save(); @@ -197,7 +197,7 @@ function testTemplateOverride() { /** * Ensures a theme template can override a theme function. */ - function testFunctionOverride() { + public function testFunctionOverride() { $this->drupalGet('theme-test/function-template-overridden'); $this->assertText('Success: Template overrides theme function.', 'Theme function overridden by test_theme template.'); } @@ -205,7 +205,7 @@ function testFunctionOverride() { /** * Test the listInfo() function. */ - function testListThemes() { + public function testListThemes() { $theme_handler = $this->container->get('theme_handler'); $theme_handler->install(array('test_subtheme')); $themes = $theme_handler->listInfo(); @@ -230,7 +230,7 @@ function testListThemes() { /** * Tests child element rendering for 'render element' theme hooks. */ - function testDrupalRenderChildren() { + public function testDrupalRenderChildren() { $element = array( '#theme' => 'theme_test_render_element_children', 'child' => array( @@ -251,7 +251,7 @@ function testDrupalRenderChildren() { /** * Tests theme can provide classes. */ - function testClassLoading() { + public function testClassLoading() { new ThemeClass(); } @@ -270,7 +270,7 @@ public function testFindThemeTemplates() { * Some modules check the page array in template_preprocess_html(), so we * ensure that it has not been rendered prematurely. */ - function testPreprocessHtml() { + public function testPreprocessHtml() { $this->drupalGet(''); $attributes = $this->xpath('/html/body[@theme_test_page_variable="Page variable is an array."]'); $this->assertTrue(count($attributes) == 1, 'In template_preprocess_html(), the page variable is still an array (not rendered yet).'); diff --git a/core/modules/system/src/Tests/Theme/TwigDebugMarkupTest.php b/core/modules/system/src/Tests/Theme/TwigDebugMarkupTest.php index 1783be6..23ae9f4 100644 --- a/core/modules/system/src/Tests/Theme/TwigDebugMarkupTest.php +++ b/core/modules/system/src/Tests/Theme/TwigDebugMarkupTest.php @@ -21,7 +21,7 @@ class TwigDebugMarkupTest extends WebTestBase { /** * Tests debug markup added to Twig template output. */ - function testTwigDebugMarkup() { + public function testTwigDebugMarkup() { /** @var \Drupal\Core\Render\RendererInterface $renderer */ $renderer = $this->container->get('renderer'); $extension = twig_extension(); diff --git a/core/modules/system/src/Tests/Theme/TwigExtensionTest.php b/core/modules/system/src/Tests/Theme/TwigExtensionTest.php index 4be1f73..38156f9 100644 --- a/core/modules/system/src/Tests/Theme/TwigExtensionTest.php +++ b/core/modules/system/src/Tests/Theme/TwigExtensionTest.php @@ -26,7 +26,7 @@ protected function setUp() { /** * Tests that the provided Twig extension loads the service appropriately. */ - function testTwigExtensionLoaded() { + public function testTwigExtensionLoaded() { $twigService = \Drupal::service('twig'); $ext = $twigService->getExtension('twig_extension_test.test_extension'); $this->assertEqual(get_class($ext), 'Drupal\twig_extension_test\TwigExtension\TestExtension', 'TestExtension loaded successfully.'); @@ -35,7 +35,7 @@ function testTwigExtensionLoaded() { /** * Tests that the Twig extension's filter produces expected output. */ - function testTwigExtensionFilter() { + public function testTwigExtensionFilter() { $this->config('system.theme') ->set('default', 'test_theme') ->save(); @@ -49,7 +49,7 @@ function testTwigExtensionFilter() { /** * Tests that the Twig extension's function produces expected output. */ - function testTwigExtensionFunction() { + public function testTwigExtensionFunction() { $this->config('system.theme') ->set('default', 'test_theme') ->save(); diff --git a/core/modules/system/src/Tests/Theme/TwigSettingsTest.php b/core/modules/system/src/Tests/Theme/TwigSettingsTest.php index b776639..4e17bc7 100644 --- a/core/modules/system/src/Tests/Theme/TwigSettingsTest.php +++ b/core/modules/system/src/Tests/Theme/TwigSettingsTest.php @@ -22,7 +22,7 @@ class TwigSettingsTest extends WebTestBase { /** * Ensures Twig template auto reload setting can be overridden. */ - function testTwigAutoReloadOverride() { + public function testTwigAutoReloadOverride() { // Enable auto reload and rebuild the service container. $parameters = $this->container->getParameter('twig.config'); $parameters['auto_reload'] = TRUE; @@ -44,7 +44,7 @@ function testTwigAutoReloadOverride() { /** * Ensures Twig engine debug setting can be overridden. */ - function testTwigDebugOverride() { + public function testTwigDebugOverride() { // Enable debug and rebuild the service container. $parameters = $this->container->getParameter('twig.config'); $parameters['debug'] = TRUE; @@ -74,7 +74,7 @@ function testTwigDebugOverride() { /** * Ensures Twig template cache setting can be overridden. */ - function testTwigCacheOverride() { + public function testTwigCacheOverride() { $extension = twig_extension(); $theme_handler = $this->container->get('theme_handler'); $theme_handler->install(array('test_theme')); diff --git a/core/modules/system/src/Tests/Update/DependencyHookInvocationTest.php b/core/modules/system/src/Tests/Update/DependencyHookInvocationTest.php index a9723c2..102d8df 100644 --- a/core/modules/system/src/Tests/Update/DependencyHookInvocationTest.php +++ b/core/modules/system/src/Tests/Update/DependencyHookInvocationTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Test the structure of the array returned by hook_update_dependencies(). */ - function testHookUpdateDependencies() { + public function testHookUpdateDependencies() { $update_dependencies = update_retrieve_dependencies(); $this->assertTrue($update_dependencies['update_test_0'][8001]['update_test_1'] == 8001, 'An update function that has a dependency on two separate modules has the first dependency recorded correctly.'); $this->assertTrue($update_dependencies['update_test_0'][8001]['update_test_2'] == 8002, 'An update function that has a dependency on two separate modules has the second dependency recorded correctly.'); diff --git a/core/modules/system/src/Tests/Update/DependencyMissingTest.php b/core/modules/system/src/Tests/Update/DependencyMissingTest.php index 9764b4c..31f2854 100644 --- a/core/modules/system/src/Tests/Update/DependencyMissingTest.php +++ b/core/modules/system/src/Tests/Update/DependencyMissingTest.php @@ -25,7 +25,7 @@ protected function setUp() { require_once \Drupal::root() . '/core/includes/update.inc'; } - function testMissingUpdate() { + public function testMissingUpdate() { $starting_updates = array( 'update_test_2' => 8001, ); diff --git a/core/modules/system/src/Tests/Update/DependencyOrderingTest.php b/core/modules/system/src/Tests/Update/DependencyOrderingTest.php index bb3ae37..c7edb88 100644 --- a/core/modules/system/src/Tests/Update/DependencyOrderingTest.php +++ b/core/modules/system/src/Tests/Update/DependencyOrderingTest.php @@ -26,7 +26,7 @@ protected function setUp() { /** * Test that updates within a single module run in the correct order. */ - function testUpdateOrderingSingleModule() { + public function testUpdateOrderingSingleModule() { $starting_updates = array( 'update_test_1' => 8001, ); @@ -42,7 +42,7 @@ function testUpdateOrderingSingleModule() { /** * Test that dependencies between modules are resolved correctly. */ - function testUpdateOrderingModuleInterdependency() { + public function testUpdateOrderingModuleInterdependency() { $starting_updates = array( 'update_test_2' => 8001, 'update_test_3' => 8001, diff --git a/core/modules/system/src/Tests/Update/InvalidUpdateHookTest.php b/core/modules/system/src/Tests/Update/InvalidUpdateHookTest.php index ce5a006..1f282c7 100644 --- a/core/modules/system/src/Tests/Update/InvalidUpdateHookTest.php +++ b/core/modules/system/src/Tests/Update/InvalidUpdateHookTest.php @@ -41,7 +41,7 @@ protected function setUp() { $this->updateUser = $this->drupalCreateUser(array('administer software updates')); } - function testInvalidUpdateHook() { + public function testInvalidUpdateHook() { // Confirm that a module with hook_update_8000() cannot be updated. $this->drupalLogin($this->updateUser); $this->drupalGet($this->updateUrl); diff --git a/core/modules/system/src/Tests/Update/UpdateScriptTest.php b/core/modules/system/src/Tests/Update/UpdateScriptTest.php index d788636..990bca6 100644 --- a/core/modules/system/src/Tests/Update/UpdateScriptTest.php +++ b/core/modules/system/src/Tests/Update/UpdateScriptTest.php @@ -49,7 +49,7 @@ protected function setUp() { /** * Tests access to the update script. */ - function testUpdateAccess() { + public function testUpdateAccess() { // Try accessing update.php without the proper permission. $regular_user = $this->drupalCreateUser(); $this->drupalLogin($regular_user); @@ -93,7 +93,7 @@ function testUpdateAccess() { /** * Tests that requirements warnings and errors are correctly displayed. */ - function testRequirements() { + public function testRequirements() { $update_script_test_config = $this->config('update_script_test.settings'); $this->drupalLogin($this->updateUser); @@ -146,7 +146,7 @@ function testRequirements() { /** * Tests the effect of using the update script on the theme system. */ - function testThemeSystem() { + public function testThemeSystem() { // Since visiting update.php triggers a rebuild of the theme system from an // unusual maintenance mode environment, we check that this rebuild did not // put any incorrect information about the themes into the database. @@ -160,7 +160,7 @@ function testThemeSystem() { /** * Tests update.php when there are no updates to apply. */ - function testNoUpdateFunctionality() { + public function testNoUpdateFunctionality() { // Click through update.php with 'administer software updates' permission. $this->drupalLogin($this->updateUser); $this->drupalGet($this->updateUrl, array('external' => TRUE)); @@ -186,7 +186,7 @@ function testNoUpdateFunctionality() { /** * Tests update.php after performing a successful update. */ - function testSuccessfulUpdateFunctionality() { + public function testSuccessfulUpdateFunctionality() { $initial_maintenance_mode = $this->container->get('state')->get('system.maintenance_mode'); $this->assertFalse($initial_maintenance_mode, 'Site is not in maintenance mode.'); $this->updateScriptTest($initial_maintenance_mode); @@ -220,7 +220,7 @@ function testSuccessfulUpdateFunctionality() { /** * Tests update.php while in maintenance mode. */ - function testMaintenanceModeUpdateFunctionality() { + public function testMaintenanceModeUpdateFunctionality() { $this->container->get('state') ->set('system.maintenance_mode', TRUE); $initial_maintenance_mode = $this->container->get('state') @@ -235,7 +235,7 @@ function testMaintenanceModeUpdateFunctionality() { /** * Tests perfoming updates with update.php in a multilingual environment. */ - function testSuccessfulMultilingualUpdateFunctionality() { + public function testSuccessfulMultilingualUpdateFunctionality() { // Add some custom languages. foreach (array('aa', 'bb') as $language_code) { ConfigurableLanguage::create(array( diff --git a/core/modules/system/src/Tests/Update/UpdatesWith7xTest.php b/core/modules/system/src/Tests/Update/UpdatesWith7xTest.php index 9782d91..1d66f47 100644 --- a/core/modules/system/src/Tests/Update/UpdatesWith7xTest.php +++ b/core/modules/system/src/Tests/Update/UpdatesWith7xTest.php @@ -36,7 +36,7 @@ protected function setUp() { $this->updateUser = $this->drupalCreateUser(array('administer software updates')); } - function testWith7x() { + public function testWith7x() { // Ensure that the minimum schema version is 8000, despite 7200 update // hooks and a 7XXX hook_update_last_removed(). $this->assertEqual(drupal_get_installed_schema_version('update_test_with_7x'), 8000); diff --git a/core/modules/system/tests/modules/batch_test/src/Controller/BatchTestController.php b/core/modules/system/tests/modules/batch_test/src/Controller/BatchTestController.php index 811c7fc..1c749c1 100644 --- a/core/modules/system/tests/modules/batch_test/src/Controller/BatchTestController.php +++ b/core/modules/system/tests/modules/batch_test/src/Controller/BatchTestController.php @@ -95,7 +95,7 @@ public function testFinishRedirect() { * @return array * Render array containing markup. */ - function testProgrammatic($value = 1) { + public function testProgrammatic($value = 1) { $form_state = (new FormState())->setValues([ 'value' => $value, ]); diff --git a/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php b/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php index b46f374..6cd97a0 100644 --- a/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php +++ b/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php @@ -3,7 +3,7 @@ namespace Drupal\module_autoload_test; class SomeClass { - function testMethod() { + public function testMethod() { return 'Drupal\\module_autoload_test\\SomeClass::testMethod() was invoked.'; } diff --git a/core/modules/system/tests/src/Kernel/Extension/ModuleHandlerTest.php b/core/modules/system/tests/src/Kernel/Extension/ModuleHandlerTest.php index 9beac7a..5941bba 100644 --- a/core/modules/system/tests/src/Kernel/Extension/ModuleHandlerTest.php +++ b/core/modules/system/tests/src/Kernel/Extension/ModuleHandlerTest.php @@ -49,7 +49,7 @@ public function register(ContainerBuilder $container) { /** * The basic functionality of retrieving enabled modules. */ - function testModuleList() { + public function testModuleList() { $module_list = ['system']; $this->assertModuleList($module_list, 'Initial'); @@ -103,7 +103,7 @@ protected function assertModuleList(array $expected_values, $condition) { * @see module_test_system_info_alter() * @see https://www.drupal.org/files/issues/dep.gv__0.png */ - function testDependencyResolution() { + public function testDependencyResolution() { $this->enableModules(array('module_test')); $this->assertTrue($this->moduleHandler()->moduleExists('module_test'), 'Test module is enabled.'); @@ -182,7 +182,7 @@ function testDependencyResolution() { /** * Tests uninstalling a module that is a "dependency" of a profile. */ - function testUninstallProfileDependency() { + public function testUninstallProfileDependency() { $profile = 'minimal'; $dependency = 'dblog'; $this->setSetting('install_profile', $profile); @@ -215,7 +215,7 @@ function testUninstallProfileDependency() { /** * Tests uninstalling a module that has content. */ - function testUninstallContentDependency() { + public function testUninstallContentDependency() { $this->enableModules(array('module_test', 'entity_test', 'text', 'user', 'help')); $this->assertTrue($this->moduleHandler()->moduleExists('entity_test'), 'Test module is enabled.'); $this->assertTrue($this->moduleHandler()->moduleExists('module_test'), 'Test module is enabled.'); @@ -269,7 +269,7 @@ function testUninstallContentDependency() { /** * Tests whether the correct module metadata is returned. */ - function testModuleMetaData() { + public function testModuleMetaData() { // Generate the list of available modules. $modules = system_rebuild_module_data(); // Check that the mtime field exists for the system module. @@ -300,7 +300,7 @@ public function testModuleStreamWrappers() { /** * Tests whether the correct theme metadata is returned. */ - function testThemeMetaData() { + public function testThemeMetaData() { // Generate the list of available themes. $themes = \Drupal::service('theme_handler')->rebuildThemeData(); // Check that the mtime field exists for the bartik theme. diff --git a/core/modules/system/tests/src/Kernel/System/InfoAlterTest.php b/core/modules/system/tests/src/Kernel/System/InfoAlterTest.php index 8e6f79e..c178caf 100644 --- a/core/modules/system/tests/src/Kernel/System/InfoAlterTest.php +++ b/core/modules/system/tests/src/Kernel/System/InfoAlterTest.php @@ -20,7 +20,7 @@ class InfoAlterTest extends KernelTestBase { * hook_system_info_alter() is enabled. Also tests if core *_list() functions * return freshly altered info. */ - function testSystemInfoAlter() { + public function testSystemInfoAlter() { \Drupal::state()->set('module_required_test.hook_system_info_alter', TRUE); $info = system_rebuild_module_data(); $this->assertFalse(isset($info['node']->info['required']), 'Before the module_required_test is installed the node module is not required.'); diff --git a/core/modules/taxonomy/src/Tests/EfqTest.php b/core/modules/taxonomy/src/Tests/EfqTest.php index 572ae8f..7907506 100644 --- a/core/modules/taxonomy/src/Tests/EfqTest.php +++ b/core/modules/taxonomy/src/Tests/EfqTest.php @@ -25,7 +25,7 @@ protected function setUp() { /** * Tests that a basic taxonomy entity query works. */ - function testTaxonomyEfq() { + public function testTaxonomyEfq() { $terms = array(); for ($i = 0; $i < 5; $i++) { $term = $this->createTerm($this->vocabulary); diff --git a/core/modules/taxonomy/src/Tests/LegacyTest.php b/core/modules/taxonomy/src/Tests/LegacyTest.php index b842226..6774462 100644 --- a/core/modules/taxonomy/src/Tests/LegacyTest.php +++ b/core/modules/taxonomy/src/Tests/LegacyTest.php @@ -51,7 +51,7 @@ protected function setUp() { /** * Test taxonomy functionality with nodes prior to 1970. */ - function testTaxonomyLegacyNode() { + public function testTaxonomyLegacyNode() { // Posts an article with a taxonomy term and a date prior to 1970. $date = new DrupalDateTime('1969-01-01 00:00:00'); $edit = array(); diff --git a/core/modules/taxonomy/src/Tests/LoadMultipleTest.php b/core/modules/taxonomy/src/Tests/LoadMultipleTest.php index 349516c..252f650 100644 --- a/core/modules/taxonomy/src/Tests/LoadMultipleTest.php +++ b/core/modules/taxonomy/src/Tests/LoadMultipleTest.php @@ -20,7 +20,7 @@ protected function setUp() { * Create a vocabulary and some taxonomy terms, ensuring they're loaded * correctly using entity_load_multiple(). */ - function testTaxonomyTermMultipleLoad() { + public function testTaxonomyTermMultipleLoad() { // Create a vocabulary. $vocabulary = $this->createVocabulary(); diff --git a/core/modules/taxonomy/src/Tests/RssTest.php b/core/modules/taxonomy/src/Tests/RssTest.php index 10e47af..afcecd3 100644 --- a/core/modules/taxonomy/src/Tests/RssTest.php +++ b/core/modules/taxonomy/src/Tests/RssTest.php @@ -66,7 +66,7 @@ protected function setUp() { * * Create a node and assert that taxonomy terms appear in rss.xml. */ - function testTaxonomyRss() { + public function testTaxonomyRss() { // Create two taxonomy terms. $term1 = $this->createTerm($this->vocabulary); diff --git a/core/modules/taxonomy/src/Tests/TaxonomyTermIndentationTest.php b/core/modules/taxonomy/src/Tests/TaxonomyTermIndentationTest.php index 6043d77..08a95c1 100644 --- a/core/modules/taxonomy/src/Tests/TaxonomyTermIndentationTest.php +++ b/core/modules/taxonomy/src/Tests/TaxonomyTermIndentationTest.php @@ -32,7 +32,7 @@ protected function setUp() { /** * Tests term indentation. */ - function testTermIndentation() { + public function testTermIndentation() { // Create three taxonomy terms. $term1 = $this->createTerm($this->vocabulary); $term2 = $this->createTerm($this->vocabulary); diff --git a/core/modules/taxonomy/src/Tests/TermEntityReferenceTest.php b/core/modules/taxonomy/src/Tests/TermEntityReferenceTest.php index de77654..4c1da53 100644 --- a/core/modules/taxonomy/src/Tests/TermEntityReferenceTest.php +++ b/core/modules/taxonomy/src/Tests/TermEntityReferenceTest.php @@ -26,7 +26,7 @@ class TermEntityReferenceTest extends TaxonomyTestBase { * field to limit the target vocabulary to one of them, ensuring that * the restriction applies. */ - function testSelectionTestVocabularyRestriction() { + public function testSelectionTestVocabularyRestriction() { // Create two vocabularies. $vocabulary = $this->createVocabulary(); diff --git a/core/modules/taxonomy/src/Tests/TermIndexTest.php b/core/modules/taxonomy/src/Tests/TermIndexTest.php index 28e816c..a03a306 100644 --- a/core/modules/taxonomy/src/Tests/TermIndexTest.php +++ b/core/modules/taxonomy/src/Tests/TermIndexTest.php @@ -87,7 +87,7 @@ protected function setUp() { /** * Tests that the taxonomy index is maintained properly. */ - function testTaxonomyIndex() { + public function testTaxonomyIndex() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); // Create terms in the vocabulary. $term_1 = $this->createTerm($this->vocabulary); @@ -197,7 +197,7 @@ function testTaxonomyIndex() { /** * Tests that there is a link to the parent term on the child term page. */ - function testTaxonomyTermHierarchyBreadcrumbs() { + public function testTaxonomyTermHierarchyBreadcrumbs() { // Create two taxonomy terms and set term2 as the parent of term1. $term1 = $this->createTerm($this->vocabulary); $term2 = $this->createTerm($this->vocabulary); diff --git a/core/modules/taxonomy/src/Tests/TermLanguageTest.php b/core/modules/taxonomy/src/Tests/TermLanguageTest.php index e894ad2..fee3b49 100644 --- a/core/modules/taxonomy/src/Tests/TermLanguageTest.php +++ b/core/modules/taxonomy/src/Tests/TermLanguageTest.php @@ -39,7 +39,7 @@ protected function setUp() { } } - function testTermLanguage() { + public function testTermLanguage() { // Configure the vocabulary to not hide the language selector. $edit = array( 'default_language[language_alterable]' => TRUE, @@ -73,7 +73,7 @@ function testTermLanguage() { $this->assertOptionSelected('edit-langcode-0-value', $edit['langcode[0][value]'], 'The term language was correctly selected.'); } - function testDefaultTermLanguage() { + public function testDefaultTermLanguage() { // Configure the vocabulary to not hide the language selector, and make the // default language of the terms fixed. $edit = array( diff --git a/core/modules/taxonomy/src/Tests/TermTest.php b/core/modules/taxonomy/src/Tests/TermTest.php index 403c28c..f48ae05 100644 --- a/core/modules/taxonomy/src/Tests/TermTest.php +++ b/core/modules/taxonomy/src/Tests/TermTest.php @@ -76,7 +76,7 @@ protected function setUp() { /** * Test terms in a single and multiple hierarchy. */ - function testTaxonomyTermHierarchy() { + public function testTaxonomyTermHierarchy() { // Create two taxonomy terms. $term1 = $this->createTerm($this->vocabulary); $term2 = $this->createTerm($this->vocabulary); @@ -116,7 +116,7 @@ function testTaxonomyTermHierarchy() { /** * Tests that many terms with parents show on each page */ - function testTaxonomyTermChildTerms() { + public function testTaxonomyTermChildTerms() { // Set limit to 10 terms per page. Set variable to 9 so 10 terms appear. $this->config('taxonomy.settings')->set('terms_per_page_admin', '9')->save(); $term1 = $this->createTerm($this->vocabulary); @@ -171,7 +171,7 @@ function testTaxonomyTermChildTerms() { * * Save & edit a node and assert that taxonomy terms are saved/loaded properly. */ - function testTaxonomyNode() { + public function testTaxonomyNode() { // Create two taxonomy terms. $term1 = $this->createTerm($this->vocabulary); $term2 = $this->createTerm($this->vocabulary); @@ -210,7 +210,7 @@ function testTaxonomyNode() { /** * Test term creation with a free-tagging vocabulary from the node form. */ - function testNodeTermCreationAndDeletion() { + public function testNodeTermCreationAndDeletion() { // Enable tags in the vocabulary. $field = $this->field; entity_get_form_display($field->getTargetEntityTypeId(), $field->getTargetBundle(), 'default') @@ -303,7 +303,7 @@ function testNodeTermCreationAndDeletion() { /** * Save, edit and delete a term using the user interface. */ - function testTermInterface() { + public function testTermInterface() { \Drupal::service('module_installer')->install(array('views')); $edit = array( 'name[0][value]' => $this->randomMachineName(12), @@ -380,7 +380,7 @@ function testTermInterface() { /** * Save, edit and delete a term using the user interface. */ - function testTermReorder() { + public function testTermReorder() { $this->createTerm($this->vocabulary); $this->createTerm($this->vocabulary); $this->createTerm($this->vocabulary); @@ -437,7 +437,7 @@ function testTermReorder() { /** * Test saving a term with multiple parents through the UI. */ - function testTermMultipleParentsInterface() { + public function testTermMultipleParentsInterface() { // Add a new term to the vocabulary so that we can have multiple parents. $parent = $this->createTerm($this->vocabulary); @@ -466,7 +466,7 @@ function testTermMultipleParentsInterface() { /** * Test taxonomy_term_load_multiple_by_name(). */ - function testTaxonomyGetTermByName() { + public function testTaxonomyGetTermByName() { $term = $this->createTerm($this->vocabulary); // Load the term with the exact name. @@ -526,7 +526,7 @@ function testTaxonomyGetTermByName() { /** * Tests that editing and saving a node with no changes works correctly. */ - function testReSavingTags() { + public function testReSavingTags() { // Enable tags in the vocabulary. $field = $this->field; entity_get_form_display($field->getTargetEntityTypeId(), $field->getTargetBundle(), 'default') diff --git a/core/modules/taxonomy/src/Tests/TermTranslationTest.php b/core/modules/taxonomy/src/Tests/TermTranslationTest.php index 91b13b1..f4a1dbc 100644 --- a/core/modules/taxonomy/src/Tests/TermTranslationTest.php +++ b/core/modules/taxonomy/src/Tests/TermTranslationTest.php @@ -87,7 +87,7 @@ public function testTranslatedBreadcrumbs() { /** * Test translation of terms are showed in the node. */ - protected function testTermsTranslation() { + public function testTermsTranslation() { // Set the display of the term reference field on the article content type // to "Check boxes/radio buttons". diff --git a/core/modules/taxonomy/src/Tests/TermTranslationUITest.php b/core/modules/taxonomy/src/Tests/TermTranslationUITest.php index a396544..9ab7d8b 100644 --- a/core/modules/taxonomy/src/Tests/TermTranslationUITest.php +++ b/core/modules/taxonomy/src/Tests/TermTranslationUITest.php @@ -100,7 +100,7 @@ public function testTranslationUI() { /** * Tests translate link on vocabulary term list. */ - function testTranslateLinkVocabularyAdminPage() { + public function testTranslateLinkVocabularyAdminPage() { $this->drupalLogin($this->drupalCreateUser(array_merge(parent::getTranslatorPermissions(), ['access administration pages', 'administer taxonomy']))); $values = array( diff --git a/core/modules/taxonomy/src/Tests/ThemeTest.php b/core/modules/taxonomy/src/Tests/ThemeTest.php index 54ad6e4..0288009 100644 --- a/core/modules/taxonomy/src/Tests/ThemeTest.php +++ b/core/modules/taxonomy/src/Tests/ThemeTest.php @@ -29,7 +29,7 @@ protected function setUp() { /** * Test the theme used when adding, viewing and editing taxonomy terms. */ - function testTaxonomyTermThemes() { + public function testTaxonomyTermThemes() { // Adding a term to a vocabulary is considered an administrative action and // should use the administrative theme. $vocabulary = $this->createVocabulary(); diff --git a/core/modules/taxonomy/src/Tests/TokenReplaceTest.php b/core/modules/taxonomy/src/Tests/TokenReplaceTest.php index f3c6c5d..e9f3cb8 100644 --- a/core/modules/taxonomy/src/Tests/TokenReplaceTest.php +++ b/core/modules/taxonomy/src/Tests/TokenReplaceTest.php @@ -56,7 +56,7 @@ protected function setUp() { /** * Creates some terms and a node, then tests the tokens generated from them. */ - function testTaxonomyTokenReplacement() { + public function testTaxonomyTokenReplacement() { $token_service = \Drupal::token(); $language_interface = \Drupal::languageManager()->getCurrentLanguage(); diff --git a/core/modules/taxonomy/src/Tests/Views/RelationshipNodeTermDataTest.php b/core/modules/taxonomy/src/Tests/Views/RelationshipNodeTermDataTest.php index c38c45c..cb09631 100644 --- a/core/modules/taxonomy/src/Tests/Views/RelationshipNodeTermDataTest.php +++ b/core/modules/taxonomy/src/Tests/Views/RelationshipNodeTermDataTest.php @@ -18,7 +18,7 @@ class RelationshipNodeTermDataTest extends TaxonomyTestBase { */ public static $testViews = array('test_taxonomy_node_term_data'); - function testViewsHandlerRelationshipNodeTermData() { + public function testViewsHandlerRelationshipNodeTermData() { $view = Views::getView('test_taxonomy_node_term_data'); // Tests \Drupal\taxonomy\Plugin\views\relationship\NodeTermData::calculateDependencies(). $expected = [ diff --git a/core/modules/taxonomy/src/Tests/Views/TaxonomyFieldTidTest.php b/core/modules/taxonomy/src/Tests/Views/TaxonomyFieldTidTest.php index 6480491..778b6ae 100644 --- a/core/modules/taxonomy/src/Tests/Views/TaxonomyFieldTidTest.php +++ b/core/modules/taxonomy/src/Tests/Views/TaxonomyFieldTidTest.php @@ -19,7 +19,7 @@ class TaxonomyFieldTidTest extends TaxonomyTestBase { */ public static $testViews = array('test_taxonomy_tid_field'); - function testViewsHandlerTidField() { + public function testViewsHandlerTidField() { /** @var \Drupal\Core\Render\RendererInterface $renderer */ $renderer = \Drupal::service('renderer'); diff --git a/core/modules/taxonomy/src/Tests/VocabularyCrudTest.php b/core/modules/taxonomy/src/Tests/VocabularyCrudTest.php index 914b7f2..165dc57 100644 --- a/core/modules/taxonomy/src/Tests/VocabularyCrudTest.php +++ b/core/modules/taxonomy/src/Tests/VocabularyCrudTest.php @@ -32,7 +32,7 @@ protected function setUp() { /** * Test deleting a taxonomy that contains terms. */ - function testTaxonomyVocabularyDeleteWithTerms() { + public function testTaxonomyVocabularyDeleteWithTerms() { // Delete any existing vocabularies. foreach (Vocabulary::loadMultiple() as $vocabulary) { $vocabulary->delete(); @@ -65,7 +65,7 @@ function testTaxonomyVocabularyDeleteWithTerms() { /** * Ensure that the vocabulary static reset works correctly. */ - function testTaxonomyVocabularyLoadStaticReset() { + public function testTaxonomyVocabularyLoadStaticReset() { $original_vocabulary = Vocabulary::load($this->vocabulary->id()); $this->assertTrue(is_object($original_vocabulary), 'Vocabulary loaded successfully.'); $this->assertEqual($this->vocabulary->label(), $original_vocabulary->label(), 'Vocabulary loaded successfully.'); @@ -89,7 +89,7 @@ function testTaxonomyVocabularyLoadStaticReset() { /** * Tests for loading multiple vocabularies. */ - function testTaxonomyVocabularyLoadMultiple() { + public function testTaxonomyVocabularyLoadMultiple() { // Delete any existing vocabularies. foreach (Vocabulary::loadMultiple() as $vocabulary) { @@ -141,7 +141,7 @@ function testTaxonomyVocabularyLoadMultiple() { /** * Test uninstall and reinstall of the taxonomy module. */ - function testUninstallReinstall() { + public function testUninstallReinstall() { // Field storages and fields attached to taxonomy term bundles should be // removed when the module is uninstalled. $field_name = Unicode::strtolower($this->randomMachineName() . '_field_name'); diff --git a/core/modules/taxonomy/src/Tests/VocabularyLanguageTest.php b/core/modules/taxonomy/src/Tests/VocabularyLanguageTest.php index 3326e51..ee88612 100644 --- a/core/modules/taxonomy/src/Tests/VocabularyLanguageTest.php +++ b/core/modules/taxonomy/src/Tests/VocabularyLanguageTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests language settings for vocabularies. */ - function testVocabularyLanguage() { + public function testVocabularyLanguage() { $this->drupalGet('admin/structure/taxonomy/add'); // Check that we have the language selector available. @@ -67,7 +67,7 @@ function testVocabularyLanguage() { /** * Tests term language settings for vocabulary terms are saved and updated. */ - function testVocabularyDefaultLanguageForTerms() { + public function testVocabularyDefaultLanguageForTerms() { // Add a new vocabulary and check that the default language settings are for // the terms are saved. $edit = array( diff --git a/core/modules/taxonomy/src/Tests/VocabularyPermissionsTest.php b/core/modules/taxonomy/src/Tests/VocabularyPermissionsTest.php index 9a5e8c0..2211211 100644 --- a/core/modules/taxonomy/src/Tests/VocabularyPermissionsTest.php +++ b/core/modules/taxonomy/src/Tests/VocabularyPermissionsTest.php @@ -18,7 +18,7 @@ protected function setUp() { /** * Create, edit and delete a taxonomy term via the user interface. */ - function testVocabularyPermissionsTaxonomyTerm() { + public function testVocabularyPermissionsTaxonomyTerm() { // Vocabulary used for creating, removing and editing terms. $vocabulary = $this->createVocabulary(); diff --git a/core/modules/taxonomy/src/Tests/VocabularyTranslationTest.php b/core/modules/taxonomy/src/Tests/VocabularyTranslationTest.php index 7ad7609..aaa6e92 100644 --- a/core/modules/taxonomy/src/Tests/VocabularyTranslationTest.php +++ b/core/modules/taxonomy/src/Tests/VocabularyTranslationTest.php @@ -32,7 +32,7 @@ protected function setUp() { /** * Tests language settings for vocabularies. */ - function testVocabularyLanguage() { + public function testVocabularyLanguage() { $this->drupalGet('admin/structure/taxonomy/add'); // Check that the field to enable content translation is available. diff --git a/core/modules/taxonomy/src/Tests/VocabularyUiTest.php b/core/modules/taxonomy/src/Tests/VocabularyUiTest.php index 3066ef0..ec4b8db 100644 --- a/core/modules/taxonomy/src/Tests/VocabularyUiTest.php +++ b/core/modules/taxonomy/src/Tests/VocabularyUiTest.php @@ -31,7 +31,7 @@ protected function setUp() { /** * Create, edit and delete a vocabulary via the user interface. */ - function testVocabularyInterface() { + public function testVocabularyInterface() { // Visit the main taxonomy administration page. $this->drupalGet('admin/structure/taxonomy'); @@ -81,7 +81,7 @@ function testVocabularyInterface() { /** * Changing weights on the vocabulary overview with two or more vocabularies. */ - function testTaxonomyAdminChangingWeights() { + public function testTaxonomyAdminChangingWeights() { // Create some vocabularies. for ($i = 0; $i < 10; $i++) { $this->createVocabulary(); @@ -110,7 +110,7 @@ function testTaxonomyAdminChangingWeights() { /** * Test the vocabulary overview with no vocabularies. */ - function testTaxonomyAdminNoVocabularies() { + public function testTaxonomyAdminNoVocabularies() { // Delete all vocabularies. $vocabularies = Vocabulary::loadMultiple(); foreach ($vocabularies as $key => $vocabulary) { @@ -126,7 +126,7 @@ function testTaxonomyAdminNoVocabularies() { /** * Deleting a vocabulary. */ - function testTaxonomyAdminDeletingVocabulary() { + public function testTaxonomyAdminDeletingVocabulary() { // Create a vocabulary. $vid = Unicode::strtolower($this->randomMachineName()); $edit = array( diff --git a/core/modules/telephone/src/Tests/TelephoneFieldTest.php b/core/modules/telephone/src/Tests/TelephoneFieldTest.php index f7452b0..b9e591f 100644 --- a/core/modules/telephone/src/Tests/TelephoneFieldTest.php +++ b/core/modules/telephone/src/Tests/TelephoneFieldTest.php @@ -44,7 +44,7 @@ protected function setUp() { /** * Helper function for testTelephoneField(). */ - function testTelephoneField() { + public function testTelephoneField() { // Add the telephone field to the article content type. FieldStorageConfig::create(array( diff --git a/core/modules/text/src/Tests/TextFieldTest.php b/core/modules/text/src/Tests/TextFieldTest.php index 993346a..e0d4114 100644 --- a/core/modules/text/src/Tests/TextFieldTest.php +++ b/core/modules/text/src/Tests/TextFieldTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Test text field validation. */ - function testTextFieldValidation() { + public function testTextFieldValidation() { // Create a field with settings to validate. $max_length = 3; $field_name = Unicode::strtolower($this->randomMachineName()); @@ -69,7 +69,7 @@ function testTextFieldValidation() { /** * Test required long text with file upload. */ - function testRequiredLongTextWithFileUpload() { + public function testRequiredLongTextWithFileUpload() { // Create a text field. $text_field_name = 'text_long'; $field_storage = FieldStorageConfig::create(array( @@ -128,7 +128,7 @@ function testRequiredLongTextWithFileUpload() { /** * Test widgets. */ - function testTextfieldWidgets() { + public function testTextfieldWidgets() { $this->_testTextfieldWidgets('text', 'text_textfield'); $this->_testTextfieldWidgets('text_long', 'text_textarea'); } @@ -136,7 +136,7 @@ function testTextfieldWidgets() { /** * Test widgets + 'formatted_text' setting. */ - function testTextfieldWidgetsFormatted() { + public function testTextfieldWidgetsFormatted() { $this->_testTextfieldWidgetsFormatted('text', 'text_textfield'); $this->_testTextfieldWidgetsFormatted('text_long', 'text_textarea'); } diff --git a/core/modules/text/tests/src/Kernel/TextSummaryTest.php b/core/modules/text/tests/src/Kernel/TextSummaryTest.php index abc0b68..5434f80 100644 --- a/core/modules/text/tests/src/Kernel/TextSummaryTest.php +++ b/core/modules/text/tests/src/Kernel/TextSummaryTest.php @@ -25,7 +25,7 @@ protected function setUp() { * subsequent sentences are not. This edge case is documented at * https://www.drupal.org/node/180425. */ - function testFirstSentenceQuestion() { + public function testFirstSentenceQuestion() { $text = 'A question? A sentence. Another sentence.'; $expected = 'A question? A sentence.'; $this->assertTextSummary($text, $expected, NULL, 30); @@ -34,7 +34,7 @@ function testFirstSentenceQuestion() { /** * Test summary with long example. */ - function testLongSentence() { + public function testLongSentence() { $text = 'Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ' . // 125 'Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. ' . // 108 'Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. ' . // 103 @@ -49,7 +49,7 @@ function testLongSentence() { /** * Test various summary length edge cases. */ - function testLength() { + public function testLength() { FilterFormat::create(array( 'format' => 'autop', 'filters' => array( diff --git a/core/modules/toolbar/src/Tests/ToolbarAdminMenuTest.php b/core/modules/toolbar/src/Tests/ToolbarAdminMenuTest.php index 56b9d69..b4d0a2d 100644 --- a/core/modules/toolbar/src/Tests/ToolbarAdminMenuTest.php +++ b/core/modules/toolbar/src/Tests/ToolbarAdminMenuTest.php @@ -97,7 +97,7 @@ protected function setUp() { * Tests the toolbar_modules_installed() and toolbar_modules_uninstalled() hook * implementations. */ - function testModuleStatusChangeSubtreesHashCacheClear() { + public function testModuleStatusChangeSubtreesHashCacheClear() { // Uninstall a module. $edit = array(); $edit['uninstall[taxonomy]'] = TRUE; @@ -124,7 +124,7 @@ function testModuleStatusChangeSubtreesHashCacheClear() { /** * Tests toolbar cache tags implementation. */ - function testMenuLinkUpdateSubtreesHashCacheClear() { + public function testMenuLinkUpdateSubtreesHashCacheClear() { // The ID of a (any) admin menu link. $admin_menu_link_id = 'system.admin_config_development'; @@ -144,7 +144,7 @@ function testMenuLinkUpdateSubtreesHashCacheClear() { * Exercises the toolbar_user_role_update() and toolbar_user_update() hook * implementations. */ - function testUserRoleUpdateSubtreesHashCacheClear() { + public function testUserRoleUpdateSubtreesHashCacheClear() { // Find the new role ID. $all_rids = $this->adminUser->getRoles(); unset($all_rids[array_search(RoleInterface::AUTHENTICATED_ID, $all_rids)]); @@ -208,7 +208,7 @@ function testUserRoleUpdateSubtreesHashCacheClear() { * Tests that changes to a user account by another user clears the changed * account's toolbar cached, not the user's who took the action. */ - function testNonCurrentUserAccountUpdates() { + public function testNonCurrentUserAccountUpdates() { $admin_user_id = $this->adminUser->id(); $this->hash = $this->getSubtreesHash(); @@ -243,7 +243,7 @@ function testNonCurrentUserAccountUpdates() { /** * Tests that toolbar cache is cleared when string translations are made. */ - function testLocaleTranslationSubtreesHashCacheClear() { + public function testLocaleTranslationSubtreesHashCacheClear() { $admin_user = $this->adminUser; // User to translate and delete string. $translate_user = $this->drupalCreateUser(array('translate interface', 'access administration pages')); @@ -324,7 +324,7 @@ function testLocaleTranslationSubtreesHashCacheClear() { /** * Tests that the 'toolbar/subtrees/{hash}' is reachable and correct. */ - function testSubtreesJsonRequest() { + public function testSubtreesJsonRequest() { $admin_user = $this->adminUser; $this->drupalLogin($admin_user); // Request a new page to refresh the drupalSettings object. @@ -339,7 +339,7 @@ function testSubtreesJsonRequest() { /** * Test that subtrees hashes vary by the language of the page. */ - function testLanguageSwitching() { + public function testLanguageSwitching() { // Create a new language with the langcode 'xx'. $langcode = 'xx'; $language = ConfigurableLanguage::createFromLangcode($langcode); diff --git a/core/modules/toolbar/src/Tests/ToolbarHookToolbarTest.php b/core/modules/toolbar/src/Tests/ToolbarHookToolbarTest.php index 7cff043..a32ee34 100644 --- a/core/modules/toolbar/src/Tests/ToolbarHookToolbarTest.php +++ b/core/modules/toolbar/src/Tests/ToolbarHookToolbarTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests for a tab and tray provided by a module implementing hook_toolbar(). */ - function testHookToolbar() { + public function testHookToolbar() { $this->drupalGet('test-page'); $this->assertResponse(200); diff --git a/core/modules/toolbar/src/Tests/ToolbarMenuTranslationTest.php b/core/modules/toolbar/src/Tests/ToolbarMenuTranslationTest.php index 8fa2bed5..c0f0e0e 100644 --- a/core/modules/toolbar/src/Tests/ToolbarMenuTranslationTest.php +++ b/core/modules/toolbar/src/Tests/ToolbarMenuTranslationTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests that toolbar classes don't change when adding a translation. */ - function testToolbarClasses() { + public function testToolbarClasses() { $langcode = 'es'; // Add Spanish. diff --git a/core/modules/tracker/src/Tests/TrackerNodeAccessTest.php b/core/modules/tracker/src/Tests/TrackerNodeAccessTest.php index 29492bc..f60ea4b 100644 --- a/core/modules/tracker/src/Tests/TrackerNodeAccessTest.php +++ b/core/modules/tracker/src/Tests/TrackerNodeAccessTest.php @@ -35,7 +35,7 @@ protected function setUp() { /** * Ensure private node on /tracker is only visible to users with permission. */ - function testTrackerNodeAccess() { + public function testTrackerNodeAccess() { // Create user with node test view permission. $access_user = $this->drupalCreateUser(array('node test view', 'access user profiles')); diff --git a/core/modules/tracker/src/Tests/TrackerTest.php b/core/modules/tracker/src/Tests/TrackerTest.php index a927e89..4b005ea 100644 --- a/core/modules/tracker/src/Tests/TrackerTest.php +++ b/core/modules/tracker/src/Tests/TrackerTest.php @@ -63,7 +63,7 @@ protected function setUp() { /** * Tests for the presence of nodes on the global tracker listing. */ - function testTrackerAll() { + public function testTrackerAll() { $this->drupalLogin($this->user); $unpublished = $this->drupalCreateNode(array( @@ -129,7 +129,7 @@ function testTrackerAll() { /** * Tests for the presence of nodes on a user's tracker listing. */ - function testTrackerUser() { + public function testTrackerUser() { $this->drupalLogin($this->user); $unpublished = $this->drupalCreateNode(array( @@ -223,7 +223,7 @@ function testTrackerUser() { /** * Tests the metadata for the "new"/"updated" indicators. */ - function testTrackerHistoryMetadata() { + public function testTrackerHistoryMetadata() { $this->drupalLogin($this->user); // Create a page node. @@ -274,7 +274,7 @@ function testTrackerHistoryMetadata() { /** * Tests for ordering on a users tracker listing when comments are posted. */ - function testTrackerOrderingNewComments() { + public function testTrackerOrderingNewComments() { $this->drupalLogin($this->user); $node_one = $this->drupalCreateNode(array( @@ -340,7 +340,7 @@ function testTrackerOrderingNewComments() { /** * Tests that existing nodes are indexed by cron. */ - function testTrackerCronIndexing() { + public function testTrackerCronIndexing() { $this->drupalLogin($this->user); // Create 3 nodes. @@ -393,7 +393,7 @@ function testTrackerCronIndexing() { /** * Tests that publish/unpublish works at admin/content/node. */ - function testTrackerAdminUnpublish() { + public function testTrackerAdminUnpublish() { \Drupal::service('module_installer')->install(array('views')); \Drupal::service('router.builder')->rebuild(); $admin_user = $this->drupalCreateUser(array('access content overview', 'administer nodes', 'bypass node access')); diff --git a/core/modules/update/src/Tests/UpdateContribTest.php b/core/modules/update/src/Tests/UpdateContribTest.php index 88cf378..67e5b40 100644 --- a/core/modules/update/src/Tests/UpdateContribTest.php +++ b/core/modules/update/src/Tests/UpdateContribTest.php @@ -29,7 +29,7 @@ protected function setUp() { /** * Tests when there is no available release data for a contrib module. */ - function testNoReleasesAvailable() { + public function testNoReleasesAvailable() { $system_info = array( '#all' => array( 'version' => '8.0.0', @@ -60,7 +60,7 @@ function testNoReleasesAvailable() { /** * Tests the basic functionality of a contrib module on the status report. */ - function testUpdateContribBasic() { + public function testUpdateContribBasic() { $project_link = \Drupal::l(t('AAA Update test'), Url::fromUri('http://example.com/project/aaa_update_test')); $system_info = array( '#all' => array( @@ -122,7 +122,7 @@ function testUpdateContribBasic() { * if you sort alphabetically by module name (which is the order we see things * inside system_rebuild_module_data() for example). */ - function testUpdateContribOrder() { + public function testUpdateContribOrder() { // We want core to be version 8.0.0. $system_info = array( '#all' => array( @@ -184,7 +184,7 @@ function testUpdateContribOrder() { /** * Tests that subthemes are notified about security updates for base themes. */ - function testUpdateBaseThemeSecurityUpdate() { + public function testUpdateBaseThemeSecurityUpdate() { // @todo https://www.drupal.org/node/2338175 base themes have to be // installed. // Only install the subtheme, not the base theme. @@ -226,7 +226,7 @@ function testUpdateBaseThemeSecurityUpdate() { * @todo https://www.drupal.org/node/2338175 extensions can not be hidden and * base themes have to be installed. */ - function testUpdateShowDisabledThemes() { + public function testUpdateShowDisabledThemes() { $update_settings = $this->config('update.settings'); // Make sure all the update_test_* themes are disabled. $extension_config = $this->config('core.extension'); @@ -291,7 +291,7 @@ function testUpdateShowDisabledThemes() { /** * Tests updates with a hidden base theme. */ - function testUpdateHiddenBaseTheme() { + public function testUpdateHiddenBaseTheme() { module_load_include('compare.inc', 'update'); // Install the subtheme. @@ -322,7 +322,7 @@ function testUpdateHiddenBaseTheme() { /** * Makes sure that if we fetch from a broken URL, sane things happen. */ - function testUpdateBrokenFetchURL() { + public function testUpdateBrokenFetchURL() { $system_info = array( '#all' => array( 'version' => '8.0.0', @@ -386,7 +386,7 @@ function testUpdateBrokenFetchURL() { * hook_update_status_alter() to try to mark this as missing a security * update, then assert if we see the appropriate warnings on the right pages. */ - function testHookUpdateStatusAlter() { + public function testHookUpdateStatusAlter() { $update_test_config = $this->config('update_test.settings'); $update_admin_user = $this->drupalCreateUser(array('administer site configuration', 'administer software updates')); $this->drupalLogin($update_admin_user); diff --git a/core/modules/update/src/Tests/UpdateCoreTest.php b/core/modules/update/src/Tests/UpdateCoreTest.php index baecde8..0b25e35 100644 --- a/core/modules/update/src/Tests/UpdateCoreTest.php +++ b/core/modules/update/src/Tests/UpdateCoreTest.php @@ -44,7 +44,7 @@ protected function setSystemInfo($version) { /** * Tests the Update Manager module when no updates are available. */ - function testNoUpdatesAvailable() { + public function testNoUpdatesAvailable() { foreach (array(0, 1) as $minor_version) { foreach (array(0, 1) as $patch_version) { foreach (array('-alpha1', '-beta1', '') as $extra_version) { @@ -63,7 +63,7 @@ function testNoUpdatesAvailable() { /** * Tests the Update Manager module when one normal update is available. */ - function testNormalUpdateAvailable() { + public function testNormalUpdateAvailable() { $this->setSystemInfo('8.0.0'); // Ensure that the update check requires a token. @@ -130,7 +130,7 @@ function testNormalUpdateAvailable() { /** * Tests the Update Manager module when a major update is available. */ - function testMajorUpdateAvailable() { + public function testMajorUpdateAvailable() { foreach (array(0, 1) as $minor_version) { foreach (array(0, 1) as $patch_version) { foreach (array('-alpha1', '-beta1', '') as $extra_version) { @@ -156,7 +156,7 @@ function testMajorUpdateAvailable() { /** * Tests the Update Manager module when a security update is available. */ - function testSecurityUpdateAvailable() { + public function testSecurityUpdateAvailable() { foreach (array(0, 1) as $minor_version) { $this->setSystemInfo("8.$minor_version.0"); $this->refreshUpdateStatus(array('drupal' => "$minor_version.2-sec")); @@ -174,7 +174,7 @@ function testSecurityUpdateAvailable() { /** * Ensures proper results where there are date mismatches among modules. */ - function testDatestampMismatch() { + public function testDatestampMismatch() { $system_info = array( '#all' => array( // We need to think we're running a -dev snapshot to see dates. @@ -197,7 +197,7 @@ function testDatestampMismatch() { /** * Checks that running cron updates the list of available updates. */ - function testModulePageRunCron() { + public function testModulePageRunCron() { $this->setSystemInfo('8.0.0'); $this->config('update.settings') ->set('fetch.url', Url::fromRoute('update_test.update_test')->setAbsolute()->toString()) @@ -214,7 +214,7 @@ function testModulePageRunCron() { /** * Checks the messages at admin/modules when the site is up to date. */ - function testModulePageUpToDate() { + public function testModulePageUpToDate() { $this->setSystemInfo('8.0.0'); // Instead of using refreshUpdateStatus(), set these manually. $this->config('update.settings') @@ -235,7 +235,7 @@ function testModulePageUpToDate() { /** * Checks the messages at admin/modules when an update is missing. */ - function testModulePageRegularUpdate() { + public function testModulePageRegularUpdate() { $this->setSystemInfo('8.0.0'); // Instead of using refreshUpdateStatus(), set these manually. $this->config('update.settings') @@ -256,7 +256,7 @@ function testModulePageRegularUpdate() { /** * Checks the messages at admin/modules when a security update is missing. */ - function testModulePageSecurityUpdate() { + public function testModulePageSecurityUpdate() { $this->setSystemInfo('8.0.0'); // Instead of using refreshUpdateStatus(), set these manually. $this->config('update.settings') @@ -295,7 +295,7 @@ function testModulePageSecurityUpdate() { /** * Tests the Update Manager module when the update server returns 503 errors. */ - function testServiceUnavailable() { + public function testServiceUnavailable() { $this->refreshUpdateStatus(array(), '503-error'); // Ensure that no "Warning: SimpleXMLElement..." parse errors are found. $this->assertNoText('SimpleXMLElement'); @@ -305,7 +305,7 @@ function testServiceUnavailable() { /** * Tests that exactly one fetch task per project is created and not more. */ - function testFetchTasks() { + public function testFetchTasks() { $projecta = array( 'name' => 'aaa_update_test', ); @@ -331,7 +331,7 @@ function testFetchTasks() { /** * Checks language module in core package at admin/reports/updates. */ - function testLanguageModuleUpdate() { + public function testLanguageModuleUpdate() { $this->setSystemInfo('8.0.0'); // Instead of using refreshUpdateStatus(), set these manually. $this->config('update.settings') diff --git a/core/modules/update/src/Tests/UpdateDeleteFileIfStaleTest.php b/core/modules/update/src/Tests/UpdateDeleteFileIfStaleTest.php index 63c2889..0977156 100644 --- a/core/modules/update/src/Tests/UpdateDeleteFileIfStaleTest.php +++ b/core/modules/update/src/Tests/UpdateDeleteFileIfStaleTest.php @@ -26,7 +26,7 @@ protected function setUp() { /** * Tests the deletion of stale files. */ - function testUpdateDeleteFileIfStale() { + public function testUpdateDeleteFileIfStale() { $file_name = file_unmanaged_save_data($this->randomMachineName()); $this->assertNotNull($file_name); diff --git a/core/modules/update/src/Tests/UpdateUploadTest.php b/core/modules/update/src/Tests/UpdateUploadTest.php index 39b6944..3aa3b9d 100644 --- a/core/modules/update/src/Tests/UpdateUploadTest.php +++ b/core/modules/update/src/Tests/UpdateUploadTest.php @@ -129,7 +129,7 @@ public function testUploadModule() { /** * Ensures that archiver extensions are properly merged in the UI. */ - function testFileNameExtensionMerging() { + public function testFileNameExtensionMerging() { $this->drupalGet('admin/modules/install'); // Make sure the bogus extension supported by update_test.module is there. $this->assertPattern('/file extensions are supported:.*update-test-extension/', "Found 'update-test-extension' extension."); @@ -140,7 +140,7 @@ function testFileNameExtensionMerging() { /** * Checks the messages on update manager pages when missing a security update. */ - function testUpdateManagerCoreSecurityUpdateMessages() { + public function testUpdateManagerCoreSecurityUpdateMessages() { $setting = array( '#all' => array( 'version' => '8.0.0', diff --git a/core/modules/user/src/Tests/UserAccountLinksTest.php b/core/modules/user/src/Tests/UserAccountLinksTest.php index 6d7d723..37006ce 100644 --- a/core/modules/user/src/Tests/UserAccountLinksTest.php +++ b/core/modules/user/src/Tests/UserAccountLinksTest.php @@ -31,7 +31,7 @@ protected function setUp() { /** * Tests the secondary menu. */ - function testSecondaryMenu() { + public function testSecondaryMenu() { // Create a regular user. $user = $this->drupalCreateUser(array()); @@ -71,7 +71,7 @@ function testSecondaryMenu() { /** * Tests disabling the 'My account' link. */ - function testDisabledAccountLink() { + public function testDisabledAccountLink() { // Create an admin user and log in. $this->drupalLogin($this->drupalCreateUser(array('access administration pages', 'administer menu'))); @@ -110,7 +110,7 @@ function testDisabledAccountLink() { /** * Tests page title is set correctly on user account tabs. */ - function testAccountPageTitles() { + public function testAccountPageTitles() { // Default page titles are suffixed with the site name - Drupal. $title_suffix = ' | Drupal'; diff --git a/core/modules/user/src/Tests/UserAdminLanguageTest.php b/core/modules/user/src/Tests/UserAdminLanguageTest.php index 03baa04..94b3ad2 100644 --- a/core/modules/user/src/Tests/UserAdminLanguageTest.php +++ b/core/modules/user/src/Tests/UserAdminLanguageTest.php @@ -44,7 +44,7 @@ protected function setUp() { /** * Tests that admin language is not configurable in single language sites. */ - function testUserAdminLanguageConfigurationNotAvailableWithOnlyOneLanguage() { + public function testUserAdminLanguageConfigurationNotAvailableWithOnlyOneLanguage() { $this->drupalLogin($this->adminUser); $this->setLanguageNegotiation(); $path = 'user/' . $this->adminUser->id() . '/edit'; @@ -56,7 +56,7 @@ function testUserAdminLanguageConfigurationNotAvailableWithOnlyOneLanguage() { /** * Tests that admin language negotiation is configurable only if enabled. */ - function testUserAdminLanguageConfigurationAvailableWithAdminLanguageNegotiation() { + public function testUserAdminLanguageConfigurationAvailableWithAdminLanguageNegotiation() { $this->drupalLogin($this->adminUser); $this->addCustomLanguage(); $path = 'user/' . $this->adminUser->id() . '/edit'; @@ -83,7 +83,7 @@ function testUserAdminLanguageConfigurationAvailableWithAdminLanguageNegotiation * have a setting for pages they cannot access, so they should not be able to * set a language for those pages. */ - function testUserAdminLanguageConfigurationAvailableIfAdminLanguageNegotiationIsEnabled() { + public function testUserAdminLanguageConfigurationAvailableIfAdminLanguageNegotiationIsEnabled() { $this->drupalLogin($this->adminUser); // Adds a new language, because with only one language, setting won't show. $this->addCustomLanguage(); @@ -103,7 +103,7 @@ function testUserAdminLanguageConfigurationAvailableIfAdminLanguageNegotiationIs /** * Tests the actual language negotiation. */ - function testActualNegotiation() { + public function testActualNegotiation() { $this->drupalLogin($this->adminUser); $this->addCustomLanguage(); $this->setLanguageNegotiation(); diff --git a/core/modules/user/src/Tests/UserAdminTest.php b/core/modules/user/src/Tests/UserAdminTest.php index 60fd56b..71b64f3 100644 --- a/core/modules/user/src/Tests/UserAdminTest.php +++ b/core/modules/user/src/Tests/UserAdminTest.php @@ -22,7 +22,7 @@ class UserAdminTest extends WebTestBase { /** * Registers a user and deletes it. */ - function testUserAdmin() { + public function testUserAdmin() { $config = $this->config('user.settings'); $user_a = $this->drupalCreateUser(); $user_a->name = 'User A'; @@ -149,7 +149,7 @@ function testUserAdmin() { /** * Tests the alternate notification email address for user mails. */ - function testNotificationEmailAddress() { + public function testNotificationEmailAddress() { // Test that the Notification Email address field is on the config page. $admin_user = $this->drupalCreateUser(array('administer users', 'administer account settings')); $this->drupalLogin($admin_user); diff --git a/core/modules/user/src/Tests/UserBlocksTest.php b/core/modules/user/src/Tests/UserBlocksTest.php index 40e63d8..579f0af 100644 --- a/core/modules/user/src/Tests/UserBlocksTest.php +++ b/core/modules/user/src/Tests/UserBlocksTest.php @@ -37,7 +37,7 @@ protected function setUp() { /** * Tests that user login block is hidden from user/login. */ - function testUserLoginBlockVisibility() { + public function testUserLoginBlockVisibility() { // Array keyed list where key being the URL address and value being expected // visibility as boolean type. $paths = [ @@ -61,7 +61,7 @@ function testUserLoginBlockVisibility() { /** * Test the user login block. */ - function testUserLoginBlock() { + public function testUserLoginBlock() { // Create a user with some permission that anonymous users lack. $user = $this->drupalCreateUser(array('administer permissions')); @@ -92,7 +92,7 @@ function testUserLoginBlock() { /** * Test the Who's Online block. */ - function testWhosOnlineBlock() { + public function testWhosOnlineBlock() { $block = $this->drupalPlaceBlock('views_block:who_s_online-who_s_online_block'); // Generate users. diff --git a/core/modules/user/src/Tests/UserCancelTest.php b/core/modules/user/src/Tests/UserCancelTest.php index 4ce4d91..00d0f9a 100644 --- a/core/modules/user/src/Tests/UserCancelTest.php +++ b/core/modules/user/src/Tests/UserCancelTest.php @@ -33,7 +33,7 @@ protected function setUp() { /** * Attempt to cancel account without permission. */ - function testUserCancelWithoutPermission() { + public function testUserCancelWithoutPermission() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); $user_storage = $this->container->get('entity.manager')->getStorage('user'); @@ -94,7 +94,7 @@ public function testUserCancelChangePermission() { * This should never be possible, or the site owner would become unable to * administer the site. */ - function testUserCancelUid1() { + public function testUserCancelUid1() { $user_storage = $this->container->get('entity.manager')->getStorage('user'); \Drupal::service('module_installer')->install(array('views')); @@ -135,7 +135,7 @@ function testUserCancelUid1() { /** * Attempt invalid account cancellations. */ - function testUserCancelInvalid() { + public function testUserCancelInvalid() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); $user_storage = $this->container->get('entity.manager')->getStorage('user'); @@ -183,7 +183,7 @@ function testUserCancelInvalid() { /** * Disable account and keep all content. */ - function testUserBlock() { + public function testUserBlock() { $this->config('user.settings')->set('cancel_method', 'user_cancel_block')->save(); $user_storage = $this->container->get('entity.manager')->getStorage('user'); @@ -221,7 +221,7 @@ function testUserBlock() { /** * Disable account and unpublish all content. */ - function testUserBlockUnpublish() { + public function testUserBlockUnpublish() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $this->config('user.settings')->set('cancel_method', 'user_cancel_block_unpublish')->save(); // Create comment field on page. @@ -291,7 +291,7 @@ function testUserBlockUnpublish() { /** * Delete account and anonymize all content. */ - function testUserAnonymize() { + public function testUserAnonymize() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); // Create comment field on page. @@ -418,7 +418,7 @@ public function testUserAnonymizeBatch() { /** * Delete account and remove all content. */ - function testUserDelete() { + public function testUserDelete() { $node_storage = $this->container->get('entity.manager')->getStorage('node'); $this->config('user.settings')->set('cancel_method', 'user_cancel_delete')->save(); \Drupal::service('module_installer')->install(array('comment')); @@ -489,7 +489,7 @@ function testUserDelete() { /** * Create an administrative user and delete another user. */ - function testUserCancelByAdmin() { + public function testUserCancelByAdmin() { $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); // Create a regular user. @@ -514,7 +514,7 @@ function testUserCancelByAdmin() { /** * Tests deletion of a user account without an email address. */ - function testUserWithoutEmailCancelByAdmin() { + public function testUserWithoutEmailCancelByAdmin() { $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); // Create a regular user. @@ -542,7 +542,7 @@ function testUserWithoutEmailCancelByAdmin() { /** * Create an administrative user and mass-delete other users. */ - function testMassUserCancelByAdmin() { + public function testMassUserCancelByAdmin() { \Drupal::service('module_installer')->install(array('views')); \Drupal::service('router.builder')->rebuild(); $this->config('user.settings')->set('cancel_method', 'user_cancel_reassign')->save(); diff --git a/core/modules/user/src/Tests/UserDeleteTest.php b/core/modules/user/src/Tests/UserDeleteTest.php index dbb3002..eaf84f9 100644 --- a/core/modules/user/src/Tests/UserDeleteTest.php +++ b/core/modules/user/src/Tests/UserDeleteTest.php @@ -15,7 +15,7 @@ class UserDeleteTest extends WebTestBase { /** * Test deleting multiple users. */ - function testUserDeleteMultiple() { + public function testUserDeleteMultiple() { // Create a few users with permissions, so roles will be created. $user_a = $this->drupalCreateUser(array('access user profiles')); $user_b = $this->drupalCreateUser(array('access user profiles')); diff --git a/core/modules/user/src/Tests/UserEditTest.php b/core/modules/user/src/Tests/UserEditTest.php index 225b194..f6448c8 100644 --- a/core/modules/user/src/Tests/UserEditTest.php +++ b/core/modules/user/src/Tests/UserEditTest.php @@ -14,7 +14,7 @@ class UserEditTest extends WebTestBase { /** * Test user edit page. */ - function testUserEdit() { + public function testUserEdit() { // Test user edit functionality. $user1 = $this->drupalCreateUser(array('change own username')); $user2 = $this->drupalCreateUser(array()); @@ -124,7 +124,7 @@ public function testUserWith0Password() { /** * Tests editing of a user account without an email address. */ - function testUserWithoutEmailEdit() { + public function testUserWithoutEmailEdit() { // Test that an admin can edit users without an email address. $admin = $this->drupalCreateUser(array('administer users')); $this->drupalLogin($admin); diff --git a/core/modules/user/src/Tests/UserEditedOwnAccountTest.php b/core/modules/user/src/Tests/UserEditedOwnAccountTest.php index 6c058bf..66ddd73 100644 --- a/core/modules/user/src/Tests/UserEditedOwnAccountTest.php +++ b/core/modules/user/src/Tests/UserEditedOwnAccountTest.php @@ -18,7 +18,7 @@ class UserEditedOwnAccountTest extends WebTestBase { */ public static $modules = array('user_form_test'); - function testUserEditedOwnAccount() { + public function testUserEditedOwnAccount() { // Change account setting 'Who can register accounts?' to Administrators // only. $this->config('user.settings')->set('register', USER_REGISTER_ADMINISTRATORS_ONLY)->save(); diff --git a/core/modules/user/src/Tests/UserEntityCallbacksTest.php b/core/modules/user/src/Tests/UserEntityCallbacksTest.php index 944b352..a970544 100644 --- a/core/modules/user/src/Tests/UserEntityCallbacksTest.php +++ b/core/modules/user/src/Tests/UserEntityCallbacksTest.php @@ -44,7 +44,7 @@ protected function setUp() { /** * Test label callback. */ - function testLabelCallback() { + public function testLabelCallback() { $this->assertEqual($this->account->label(), $this->account->getUsername(), 'The username should be used as label'); // Setup a random anonymous name to be sure the name is used. diff --git a/core/modules/user/src/Tests/UserLanguageCreationTest.php b/core/modules/user/src/Tests/UserLanguageCreationTest.php index 08ca707..0801cf4 100644 --- a/core/modules/user/src/Tests/UserLanguageCreationTest.php +++ b/core/modules/user/src/Tests/UserLanguageCreationTest.php @@ -23,7 +23,7 @@ class UserLanguageCreationTest extends WebTestBase { /** * Functional test for language handling during user creation. */ - function testLocalUserCreation() { + public function testLocalUserCreation() { // User to add and remove language and create new users. $admin_user = $this->drupalCreateUser(array('administer languages', 'access administration pages', 'administer users')); $this->drupalLogin($admin_user); diff --git a/core/modules/user/src/Tests/UserLanguageTest.php b/core/modules/user/src/Tests/UserLanguageTest.php index 5c5f73f..ed9f88f 100644 --- a/core/modules/user/src/Tests/UserLanguageTest.php +++ b/core/modules/user/src/Tests/UserLanguageTest.php @@ -22,7 +22,7 @@ class UserLanguageTest extends WebTestBase { /** * Test if user can change their default language. */ - function testUserLanguageConfiguration() { + public function testUserLanguageConfiguration() { // User to add and remove language. $admin_user = $this->drupalCreateUser(array('administer languages', 'access administration pages')); // User to change their default language. diff --git a/core/modules/user/src/Tests/UserLoginTest.php b/core/modules/user/src/Tests/UserLoginTest.php index af00c74..c30e069 100644 --- a/core/modules/user/src/Tests/UserLoginTest.php +++ b/core/modules/user/src/Tests/UserLoginTest.php @@ -15,7 +15,7 @@ class UserLoginTest extends WebTestBase { /** * Tests login with destination. */ - function testLoginCacheTagsAndDestination() { + public function testLoginCacheTagsAndDestination() { $this->drupalGet('user/login'); // The user login form says "Enter your username.", hence it // depends on config:system.site, and its cache tags should be present. @@ -31,7 +31,7 @@ function testLoginCacheTagsAndDestination() { /** * Test the global login flood control. */ - function testGlobalLoginFloodControl() { + public function testGlobalLoginFloodControl() { $this->config('user.flood') ->set('ip_limit', 10) // Set a high per-user limit out so that it is not relevant in the test. @@ -68,7 +68,7 @@ function testGlobalLoginFloodControl() { /** * Test the per-user login flood control. */ - function testPerUserLoginFloodControl() { + public function testPerUserLoginFloodControl() { $this->config('user.flood') // Set a high global limit out so that it is not relevant in the test. ->set('ip_limit', 4000) @@ -108,7 +108,7 @@ function testPerUserLoginFloodControl() { /** * Test that user password is re-hashed upon login after changing $count_log2. */ - function testPasswordRehashOnLogin() { + public function testPasswordRehashOnLogin() { // Determine default log2 for phpass hashing algorithm $default_count_log2 = 16; diff --git a/core/modules/user/src/Tests/UserPasswordResetTest.php b/core/modules/user/src/Tests/UserPasswordResetTest.php index 0fa89e5..3494608 100644 --- a/core/modules/user/src/Tests/UserPasswordResetTest.php +++ b/core/modules/user/src/Tests/UserPasswordResetTest.php @@ -67,7 +67,7 @@ protected function setUp() { /** * Tests password reset functionality. */ - function testUserPasswordReset() { + public function testUserPasswordReset() { // Try to reset the password for an invalid account. $this->drupalGet('user/password'); @@ -234,7 +234,7 @@ public function testUserResetPasswordTextboxFilled() { /** * Make sure that users cannot forge password reset URLs of other users. */ - function testResetImpersonation() { + public function testResetImpersonation() { // Create two identical user accounts except for the user name. They must // have the same empty password, so we can't use $this->drupalCreateUser(). $edit = array(); diff --git a/core/modules/user/src/Tests/UserPermissionsTest.php b/core/modules/user/src/Tests/UserPermissionsTest.php index 5e3ecd5..33aeb09 100644 --- a/core/modules/user/src/Tests/UserPermissionsTest.php +++ b/core/modules/user/src/Tests/UserPermissionsTest.php @@ -42,7 +42,7 @@ protected function setUp() { /** * Test changing user permissions through the permissions page. */ - function testUserPermissionChanges() { + public function testUserPermissionChanges() { $permissions_hash_generator = $this->container->get('user_permissions_hash_generator'); $storage = $this->container->get('entity.manager')->getStorage('user_role'); @@ -92,7 +92,7 @@ function testUserPermissionChanges() { /** * Test assigning of permissions for the administrator role. */ - function testAdministratorRole() { + public function testAdministratorRole() { $this->drupalLogin($this->adminUser); $this->drupalGet('admin/config/people/accounts'); @@ -135,7 +135,7 @@ function testAdministratorRole() { /** * Verify proper permission changes by user_role_change_permissions(). */ - function testUserRoleChangePermissions() { + public function testUserRoleChangePermissions() { $permissions_hash_generator = $this->container->get('user_permissions_hash_generator'); $rid = $this->rid; diff --git a/core/modules/user/src/Tests/UserPictureTest.php b/core/modules/user/src/Tests/UserPictureTest.php index 3f8db42..2d39d23 100644 --- a/core/modules/user/src/Tests/UserPictureTest.php +++ b/core/modules/user/src/Tests/UserPictureTest.php @@ -44,7 +44,7 @@ protected function setUp() { /** * Tests creation, display, and deletion of user pictures. */ - function testCreateDeletePicture() { + public function testCreateDeletePicture() { $this->drupalLogin($this->webUser); // Save a new picture. @@ -81,7 +81,7 @@ function testCreateDeletePicture() { /** * Tests embedded users on node pages. */ - function testPictureOnNodeComment() { + public function testPictureOnNodeComment() { $this->drupalLogin($this->webUser); // Save a new picture. diff --git a/core/modules/user/src/Tests/UserRegistrationTest.php b/core/modules/user/src/Tests/UserRegistrationTest.php index 0831362..0930d48 100644 --- a/core/modules/user/src/Tests/UserRegistrationTest.php +++ b/core/modules/user/src/Tests/UserRegistrationTest.php @@ -23,7 +23,7 @@ class UserRegistrationTest extends WebTestBase { */ public static $modules = array('field_test'); - function testRegistrationWithEmailVerification() { + public function testRegistrationWithEmailVerification() { $config = $this->config('user.settings'); // Require email verification. $config->set('verify_mail', TRUE)->save(); @@ -62,7 +62,7 @@ function testRegistrationWithEmailVerification() { $this->assertFalse($new_user->isActive(), 'New account is blocked until approved by an administrator.'); } - function testRegistrationWithoutEmailVerification() { + public function testRegistrationWithoutEmailVerification() { $config = $this->config('user.settings'); // Don't require email verification and allow registration by site visitors // without administrator approval. @@ -128,7 +128,7 @@ function testRegistrationWithoutEmailVerification() { $this->assertText(t('Member for'), 'User can log in after administrator approval.'); } - function testRegistrationEmailDuplicates() { + public function testRegistrationEmailDuplicates() { // Don't require email verification and allow registration by site visitors // without administrator approval. $this->config('user.settings') @@ -223,7 +223,7 @@ public function testUuidFormState() { $this->assertTrue($user_storage->loadByProperties(['name' => $edit['name']])); } - function testRegistrationDefaultValues() { + public function testRegistrationDefaultValues() { // Don't require email verification and allow registration by site visitors // without administrator approval. $config_user_settings = $this->config('user.settings') @@ -287,7 +287,7 @@ public function testUniqueFields() { /** * Tests Field API fields on user registration forms. */ - function testRegistrationWithUserFields() { + public function testRegistrationWithUserFields() { // Create a field on 'user' entity type. $field_storage = FieldStorageConfig::create(array( 'field_name' => 'test_user_field', diff --git a/core/modules/user/src/Tests/UserRoleAdminTest.php b/core/modules/user/src/Tests/UserRoleAdminTest.php index 182da93..05e68ca 100644 --- a/core/modules/user/src/Tests/UserRoleAdminTest.php +++ b/core/modules/user/src/Tests/UserRoleAdminTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Test adding, renaming and deleting roles. */ - function testRoleAdministration() { + public function testRoleAdministration() { $this->drupalLogin($this->adminUser); $default_langcode = \Drupal::languageManager()->getDefaultLanguage()->getId(); // Test presence of tab. @@ -98,7 +98,7 @@ function testRoleAdministration() { /** * Test user role weight change operation and ordering. */ - function testRoleWeightOrdering() { + public function testRoleWeightOrdering() { $this->drupalLogin($this->adminUser); $roles = user_roles(); $weight = count($roles); diff --git a/core/modules/user/src/Tests/UserRolesAssignmentTest.php b/core/modules/user/src/Tests/UserRolesAssignmentTest.php index 485bd2f..9852b90 100644 --- a/core/modules/user/src/Tests/UserRolesAssignmentTest.php +++ b/core/modules/user/src/Tests/UserRolesAssignmentTest.php @@ -21,7 +21,7 @@ protected function setUp() { * Tests that a user can be assigned a role and that the role can be removed * again. */ - function testAssignAndRemoveRole() { + public function testAssignAndRemoveRole() { $rid = $this->drupalCreateRole(array('administer users')); $account = $this->drupalCreateUser(); @@ -42,7 +42,7 @@ function testAssignAndRemoveRole() { * Tests that when creating a user the role can be assigned. And that it can * be removed again. */ - function testCreateUserWithRole() { + public function testCreateUserWithRole() { $rid = $this->drupalCreateRole(array('administer users')); // Create a new user and add the role at the same time. $edit = array( diff --git a/core/modules/user/src/Tests/UserSaveTest.php b/core/modules/user/src/Tests/UserSaveTest.php index 6a454ee..b7185cb 100644 --- a/core/modules/user/src/Tests/UserSaveTest.php +++ b/core/modules/user/src/Tests/UserSaveTest.php @@ -15,7 +15,7 @@ class UserSaveTest extends WebTestBase { /** * Test creating a user with arbitrary uid. */ - function testUserImport() { + public function testUserImport() { // User ID must be a number that is not in the database. $uids = \Drupal::entityManager()->getStorage('user')->getQuery() @@ -48,7 +48,7 @@ function testUserImport() { /** * Ensures that an existing password is unset after the user was saved. */ - function testExistingPasswordRemoval() { + public function testExistingPasswordRemoval() { /** @var \Drupal\user\Entity\User $user */ $user = User::create(['name' => $this->randomMachineName()]); $user->save(); diff --git a/core/modules/user/src/Tests/UserSearchTest.php b/core/modules/user/src/Tests/UserSearchTest.php index d154030..7768e0f 100644 --- a/core/modules/user/src/Tests/UserSearchTest.php +++ b/core/modules/user/src/Tests/UserSearchTest.php @@ -19,7 +19,7 @@ class UserSearchTest extends WebTestBase { */ public static $modules = array('search'); - function testUserSearch() { + public function testUserSearch() { // Verify that a user without 'administer users' permission cannot search // for users by email address. Additionally, ensure that the username has a // plus sign to ensure searching works with that. diff --git a/core/modules/user/src/Tests/UserTimeZoneTest.php b/core/modules/user/src/Tests/UserTimeZoneTest.php index 3229d31..05b711f 100644 --- a/core/modules/user/src/Tests/UserTimeZoneTest.php +++ b/core/modules/user/src/Tests/UserTimeZoneTest.php @@ -22,7 +22,7 @@ class UserTimeZoneTest extends WebTestBase { /** * Tests the display of dates and time when user-configurable time zones are set. */ - function testUserTimeZone() { + public function testUserTimeZone() { // Setup date/time settings for Los Angeles time. $this->config('system.date') ->set('timezone.user.configurable', 1) diff --git a/core/modules/user/src/Tests/UserTokenReplaceTest.php b/core/modules/user/src/Tests/UserTokenReplaceTest.php index b8cfde0..294562a 100644 --- a/core/modules/user/src/Tests/UserTokenReplaceTest.php +++ b/core/modules/user/src/Tests/UserTokenReplaceTest.php @@ -34,7 +34,7 @@ protected function setUp() { /** * Creates a user, then tests the tokens generated from it. */ - function testUserTokenReplacement() { + public function testUserTokenReplacement() { $token_service = \Drupal::token(); $language_interface = \Drupal::languageManager()->getCurrentLanguage(); $url_options = array( diff --git a/core/modules/user/src/Tests/Views/AccessPermissionTest.php b/core/modules/user/src/Tests/Views/AccessPermissionTest.php index b8ef5a9..9f233ff 100644 --- a/core/modules/user/src/Tests/Views/AccessPermissionTest.php +++ b/core/modules/user/src/Tests/Views/AccessPermissionTest.php @@ -24,7 +24,7 @@ class AccessPermissionTest extends AccessTestBase { /** * Tests perm access plugin. */ - function testAccessPerm() { + public function testAccessPerm() { $view = Views::getView('test_access_perm'); $view->setDisplay(); diff --git a/core/modules/user/src/Tests/Views/AccessRoleTest.php b/core/modules/user/src/Tests/Views/AccessRoleTest.php index 02a83a7..11ab713 100644 --- a/core/modules/user/src/Tests/Views/AccessRoleTest.php +++ b/core/modules/user/src/Tests/Views/AccessRoleTest.php @@ -25,7 +25,7 @@ class AccessRoleTest extends AccessTestBase { /** * Tests role access plugin. */ - function testAccessRole() { + public function testAccessRole() { /** @var \Drupal\views\ViewEntityInterface $view */ $view = \Drupal::entityManager()->getStorage('view')->load('test_access_role'); $display = &$view->getDisplay('default'); diff --git a/core/modules/user/src/Tests/Views/ArgumentValidateTest.php b/core/modules/user/src/Tests/Views/ArgumentValidateTest.php index 0a49142..1714f1e 100644 --- a/core/modules/user/src/Tests/Views/ArgumentValidateTest.php +++ b/core/modules/user/src/Tests/Views/ArgumentValidateTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests the User (ID) argument validator. */ - function testArgumentValidateUserUid() { + public function testArgumentValidateUserUid() { $account = $this->account; $view = Views::getView('test_view_argument_validate_user'); diff --git a/core/modules/user/tests/src/Kernel/UserAccountFormFieldsTest.php b/core/modules/user/tests/src/Kernel/UserAccountFormFieldsTest.php index 4a83c41..2b9eca3 100644 --- a/core/modules/user/tests/src/Kernel/UserAccountFormFieldsTest.php +++ b/core/modules/user/tests/src/Kernel/UserAccountFormFieldsTest.php @@ -23,7 +23,7 @@ class UserAccountFormFieldsTest extends KernelTestBase { /** * Tests the root user account form section in the "Configure site" form. */ - function testInstallConfigureForm() { + public function testInstallConfigureForm() { require_once \Drupal::root() . '/core/includes/install.core.inc'; require_once \Drupal::root() . '/core/includes/install.inc'; $install_state = install_state_defaults(); @@ -45,7 +45,7 @@ function testInstallConfigureForm() { /** * Tests the user registration form. */ - function testUserRegistrationForm() { + public function testUserRegistrationForm() { // Install default configuration; required for AccountFormController. $this->installConfig(array('user')); @@ -69,7 +69,7 @@ function testUserRegistrationForm() { /** * Tests the user edit form. */ - function testUserEditForm() { + public function testUserEditForm() { // Install default configuration; required for AccountFormController. $this->installConfig(array('user')); diff --git a/core/modules/user/tests/src/Kernel/UserActionConfigSchemaTest.php b/core/modules/user/tests/src/Kernel/UserActionConfigSchemaTest.php index 3426349..2195e3c 100644 --- a/core/modules/user/tests/src/Kernel/UserActionConfigSchemaTest.php +++ b/core/modules/user/tests/src/Kernel/UserActionConfigSchemaTest.php @@ -26,7 +26,7 @@ class UserActionConfigSchemaTest extends KernelTestBase { /** * Tests whether the user action config schema are valid. */ - function testValidUserActionConfigSchema() { + public function testValidUserActionConfigSchema() { $rid = strtolower($this->randomMachineName(8)); Role::create(array('id' => $rid))->save(); diff --git a/core/modules/user/tests/src/Kernel/UserEntityReferenceTest.php b/core/modules/user/tests/src/Kernel/UserEntityReferenceTest.php index 8958c54..b88d471 100644 --- a/core/modules/user/tests/src/Kernel/UserEntityReferenceTest.php +++ b/core/modules/user/tests/src/Kernel/UserEntityReferenceTest.php @@ -54,7 +54,7 @@ protected function setUp() { /** * Tests user selection by roles. */ - function testUserSelectionByRole() { + public function testUserSelectionByRole() { $field_definition = FieldConfig::loadByName('user', 'user', 'user_reference'); $handler_settings = $field_definition->getSetting('handler_settings'); $handler_settings['filter']['role'] = array( diff --git a/core/modules/user/tests/src/Kernel/UserFieldsTest.php b/core/modules/user/tests/src/Kernel/UserFieldsTest.php index f0fb532..e02e619 100644 --- a/core/modules/user/tests/src/Kernel/UserFieldsTest.php +++ b/core/modules/user/tests/src/Kernel/UserFieldsTest.php @@ -36,7 +36,7 @@ protected function setUp() { /** * Tests account's available fields. */ - function testUserFields() { + public function testUserFields() { // Create the user to test the user fields. $user = User::create([ 'name' => 'foobar', diff --git a/core/modules/user/tests/src/Kernel/UserSaveStatusTest.php b/core/modules/user/tests/src/Kernel/UserSaveStatusTest.php index 00a29fc..8aa6573 100644 --- a/core/modules/user/tests/src/Kernel/UserSaveStatusTest.php +++ b/core/modules/user/tests/src/Kernel/UserSaveStatusTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Test SAVED_NEW and SAVED_UPDATED statuses for user entity type. */ - function testUserSaveStatus() { + public function testUserSaveStatus() { // Create a new user. $values = array( 'uid' => 1, diff --git a/core/modules/user/tests/src/Kernel/UserValidationTest.php b/core/modules/user/tests/src/Kernel/UserValidationTest.php index b27b00f..140736b 100644 --- a/core/modules/user/tests/src/Kernel/UserValidationTest.php +++ b/core/modules/user/tests/src/Kernel/UserValidationTest.php @@ -39,7 +39,7 @@ protected function setUp() { /** * Tests user name validation. */ - function testUsernames() { + public function testUsernames() { $test_cases = array( // '' => array('', 'assert'), 'foo' => array('Valid username', 'assertNull'), 'FOO' => array('Valid username', 'assertNull'), @@ -69,7 +69,7 @@ function testUsernames() { /** * Runs entity validation checks. */ - function testValidation() { + public function testValidation() { $user = User::create(array( 'name' => 'test', 'mail' => 'test@example.com', diff --git a/core/modules/views/src/Tests/FieldApiDataTest.php b/core/modules/views/src/Tests/FieldApiDataTest.php index 086f179..cb0d7df 100644 --- a/core/modules/views/src/Tests/FieldApiDataTest.php +++ b/core/modules/views/src/Tests/FieldApiDataTest.php @@ -50,7 +50,7 @@ protected function setUp() { * * We check data structure for both node and node revision tables. */ - function testViewsData() { + public function testViewsData() { $table_mapping = \Drupal::entityManager()->getStorage('node')->getTableMapping(); $field_storage = $this->fieldStorages[0]; $current_table = $table_mapping->getDedicatedDataTableName($field_storage); diff --git a/core/modules/views/src/Tests/Handler/ArgumentStringTest.php b/core/modules/views/src/Tests/Handler/ArgumentStringTest.php index e47b1a3..798c794 100644 --- a/core/modules/views/src/Tests/Handler/ArgumentStringTest.php +++ b/core/modules/views/src/Tests/Handler/ArgumentStringTest.php @@ -28,7 +28,7 @@ class ArgumentStringTest extends HandlerTestBase { /** * Tests the glossary feature. */ - function testGlossary() { + public function testGlossary() { // Setup some nodes, one with a, two with b and three with c. $counter = 1; foreach (array('a', 'b', 'c') as $char) { diff --git a/core/modules/views/src/Tests/Plugin/AccessTest.php b/core/modules/views/src/Tests/Plugin/AccessTest.php index 1bcf300..41b5f71 100644 --- a/core/modules/views/src/Tests/Plugin/AccessTest.php +++ b/core/modules/views/src/Tests/Plugin/AccessTest.php @@ -61,7 +61,7 @@ protected function setUp() { /** * Tests none access plugin. */ - function testAccessNone() { + public function testAccessNone() { $view = Views::getView('test_access_none'); $view->setDisplay(); @@ -78,7 +78,7 @@ function testAccessNone() { * * @see \Drupal\views_test\Plugin\views\access\StaticTest */ - function testStaticAccessPlugin() { + public function testStaticAccessPlugin() { $view = Views::getView('test_access_static'); $view->setDisplay(); diff --git a/core/modules/views/src/Tests/Plugin/NumericFormatPluralTest.php b/core/modules/views/src/Tests/Plugin/NumericFormatPluralTest.php index bb06328..70f9745 100644 --- a/core/modules/views/src/Tests/Plugin/NumericFormatPluralTest.php +++ b/core/modules/views/src/Tests/Plugin/NumericFormatPluralTest.php @@ -37,7 +37,7 @@ protected function setUp() { /** * Test plural formatting setting on a numeric views handler. */ - function testNumericFormatPlural() { + public function testNumericFormatPlural() { // Create a file. $file = $this->createFile(); diff --git a/core/modules/views/src/Tests/Plugin/PagerTest.php b/core/modules/views/src/Tests/Plugin/PagerTest.php index f922748..4aab501 100644 --- a/core/modules/views/src/Tests/Plugin/PagerTest.php +++ b/core/modules/views/src/Tests/Plugin/PagerTest.php @@ -285,7 +285,7 @@ public function testRenderNullPager() { /** * Test the api functions on the view object. */ - function testPagerApi() { + public function testPagerApi() { $view = Views::getView('test_pager_full'); $view->setDisplay(); // On the first round don't initialize the pager. diff --git a/core/modules/views/src/Tests/Plugin/StyleTest.php b/core/modules/views/src/Tests/Plugin/StyleTest.php index 6773c83..901b194 100644 --- a/core/modules/views/src/Tests/Plugin/StyleTest.php +++ b/core/modules/views/src/Tests/Plugin/StyleTest.php @@ -87,7 +87,7 @@ public function testStyle() { $this->assertTrue(strpos($output, $random_text) !== FALSE, 'Make sure that the rendering of the style plugin appears in the output of the view.'); } - function testGrouping() { + public function testGrouping() { $this->_testGrouping(FALSE); $this->_testGrouping(TRUE); } @@ -256,7 +256,7 @@ function _testGrouping($stripped = FALSE) { /** * Tests custom css classes. */ - function testCustomRowClasses() { + public function testCustomRowClasses() { $view = Views::getView('test_view'); $view->setDisplay(); diff --git a/core/modules/views/src/Tests/Wizard/BasicTest.php b/core/modules/views/src/Tests/Wizard/BasicTest.php index a8eedf2..69a6d6a 100644 --- a/core/modules/views/src/Tests/Wizard/BasicTest.php +++ b/core/modules/views/src/Tests/Wizard/BasicTest.php @@ -20,7 +20,7 @@ protected function setUp() { $this->drupalPlaceBlock('page_title_block'); } - function testViewsWizardAndListing() { + public function testViewsWizardAndListing() { $this->drupalCreateContentType(array('type' => 'article')); $this->drupalCreateContentType(array('type' => 'page')); diff --git a/core/modules/views/src/Tests/Wizard/ItemsPerPageTest.php b/core/modules/views/src/Tests/Wizard/ItemsPerPageTest.php index 7a2f430..033632b 100644 --- a/core/modules/views/src/Tests/Wizard/ItemsPerPageTest.php +++ b/core/modules/views/src/Tests/Wizard/ItemsPerPageTest.php @@ -19,7 +19,7 @@ protected function setUp() { /** * Tests the number of items per page. */ - function testItemsPerPage() { + public function testItemsPerPage() { $this->drupalCreateContentType(array('type' => 'article')); // Create articles, each with a different creation time so that we can do a diff --git a/core/modules/views/src/Tests/Wizard/MenuTest.php b/core/modules/views/src/Tests/Wizard/MenuTest.php index 552ff50..f637dc4 100644 --- a/core/modules/views/src/Tests/Wizard/MenuTest.php +++ b/core/modules/views/src/Tests/Wizard/MenuTest.php @@ -15,7 +15,7 @@ class MenuTest extends WizardTestBase { /** * Tests the menu functionality. */ - function testMenus() { + public function testMenus() { $this->drupalPlaceBlock('system_menu_block:main'); // Create a view with a page display and a menu link in the Main Menu. diff --git a/core/modules/views/src/Tests/Wizard/SortingTest.php b/core/modules/views/src/Tests/Wizard/SortingTest.php index 6172930..6456b31 100644 --- a/core/modules/views/src/Tests/Wizard/SortingTest.php +++ b/core/modules/views/src/Tests/Wizard/SortingTest.php @@ -18,7 +18,7 @@ protected function setUp() { /** * Tests the sorting functionality. */ - function testSorting() { + public function testSorting() { // Create nodes, each with a different creation time so that we can do a // meaningful sort. $this->drupalCreateContentType(array('type' => 'page')); diff --git a/core/modules/views/src/Tests/Wizard/TaggedWithTest.php b/core/modules/views/src/Tests/Wizard/TaggedWithTest.php index 939473f..c602bbb 100644 --- a/core/modules/views/src/Tests/Wizard/TaggedWithTest.php +++ b/core/modules/views/src/Tests/Wizard/TaggedWithTest.php @@ -114,7 +114,7 @@ protected function setUp() { /** * Tests the "tagged with" functionality. */ - function testTaggedWith() { + public function testTaggedWith() { // In this test we will only create nodes that have an instance of the tag // field. $node_add_path = 'node/add/' . $this->nodeTypeWithTags->id(); @@ -180,7 +180,7 @@ function testTaggedWith() { /** * Tests that the "tagged with" form element only shows for node types that support it. */ - function testTaggedWithByNodeType() { + public function testTaggedWithByNodeType() { // The tagging field is associated with one of our node types only. So the // "tagged with" form element on the view wizard should appear on the form // by default (when the wizard is configured to display all content) and diff --git a/core/modules/views/tests/src/Kernel/Handler/FieldCounterTest.php b/core/modules/views/tests/src/Kernel/Handler/FieldCounterTest.php index 12f68ed..7ac1144 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FieldCounterTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FieldCounterTest.php @@ -26,7 +26,7 @@ class FieldCounterTest extends ViewsKernelTestBase { */ public static $testViews = array('test_view'); - function testSimple() { + public function testSimple() { $view = Views::getView('test_view'); $view->setDisplay(); $view->displayHandlers->get('default')->overrideOption('fields', array( @@ -87,7 +87,7 @@ function testSimple() { /** * @todo: Write tests for pager. */ - function testPager() { + public function testPager() { } } diff --git a/core/modules/views/tests/src/Kernel/Handler/FieldKernelTest.php b/core/modules/views/tests/src/Kernel/Handler/FieldKernelTest.php index a57e478..8d7ae60 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FieldKernelTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FieldKernelTest.php @@ -351,7 +351,7 @@ public function testExclude() { /** * Tests everything related to empty output of a field. */ - function testEmpty() { + public function testEmpty() { $this->_testHideIfEmpty(); $this->_testEmptyText(); } @@ -693,7 +693,7 @@ function _testEmptyText() { /** * Tests views_handler_field::isValueEmpty(). */ - function testIsValueEmpty() { + public function testIsValueEmpty() { $view = Views::getView('test_view'); $view->initHandlers(); $field = $view->field['name']; diff --git a/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php b/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php index a4bd6c8..e307fa7 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php @@ -36,7 +36,7 @@ function viewsData() { return $data; } - function testEqual() { + public function testEqual() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -82,7 +82,7 @@ public function testEqualGroupedExposed() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testNotEqual() { + public function testNotEqual() { $view = Views::getView('test_view'); $view->setDisplay(); diff --git a/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php b/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php index e8c4977..b487392 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php @@ -81,7 +81,7 @@ protected function getBasicPageView() { return $view; } - function testFilterStringEqual() { + public function testFilterStringEqual() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -106,7 +106,7 @@ function testFilterStringEqual() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedEqual() { + public function testFilterStringGroupedExposedEqual() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -128,7 +128,7 @@ function testFilterStringGroupedExposedEqual() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringNotEqual() { + public function testFilterStringNotEqual() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -162,7 +162,7 @@ function testFilterStringNotEqual() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedNotEqual() { + public function testFilterStringGroupedExposedNotEqual() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -194,7 +194,7 @@ function testFilterStringGroupedExposedNotEqual() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringContains() { + public function testFilterStringContains() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -220,7 +220,7 @@ function testFilterStringContains() { } - function testFilterStringGroupedExposedContains() { + public function testFilterStringGroupedExposedContains() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -243,7 +243,7 @@ function testFilterStringGroupedExposedContains() { } - function testFilterStringWord() { + public function testFilterStringWord() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -296,7 +296,7 @@ function testFilterStringWord() { } - function testFilterStringGroupedExposedWord() { + public function testFilterStringGroupedExposedWord() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -338,7 +338,7 @@ function testFilterStringGroupedExposedWord() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringStarts() { + public function testFilterStringStarts() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -363,7 +363,7 @@ function testFilterStringStarts() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedStarts() { + public function testFilterStringGroupedExposedStarts() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -384,7 +384,7 @@ function testFilterStringGroupedExposedStarts() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringNotStarts() { + public function testFilterStringNotStarts() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -416,7 +416,7 @@ function testFilterStringNotStarts() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedNotStarts() { + public function testFilterStringGroupedExposedNotStarts() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -444,7 +444,7 @@ function testFilterStringGroupedExposedNotStarts() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringEnds() { + public function testFilterStringEnds() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -472,7 +472,7 @@ function testFilterStringEnds() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedEnds() { + public function testFilterStringGroupedExposedEnds() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -496,7 +496,7 @@ function testFilterStringGroupedExposedEnds() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringNotEnds() { + public function testFilterStringNotEnds() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -525,7 +525,7 @@ function testFilterStringNotEnds() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedNotEnds() { + public function testFilterStringGroupedExposedNotEnds() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -550,7 +550,7 @@ function testFilterStringGroupedExposedNotEnds() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringNot() { + public function testFilterStringNot() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -580,7 +580,7 @@ function testFilterStringNot() { } - function testFilterStringGroupedExposedNot() { + public function testFilterStringGroupedExposedNot() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -606,7 +606,7 @@ function testFilterStringGroupedExposedNot() { } - function testFilterStringShorter() { + public function testFilterStringShorter() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -634,7 +634,7 @@ function testFilterStringShorter() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedShorter() { + public function testFilterStringGroupedExposedShorter() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -657,7 +657,7 @@ function testFilterStringGroupedExposedShorter() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringLonger() { + public function testFilterStringLonger() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -682,7 +682,7 @@ function testFilterStringLonger() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedLonger() { + public function testFilterStringGroupedExposedLonger() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -703,7 +703,7 @@ function testFilterStringGroupedExposedLonger() { } - function testFilterStringEmpty() { + public function testFilterStringEmpty() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -727,7 +727,7 @@ function testFilterStringEmpty() { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - function testFilterStringGroupedExposedEmpty() { + public function testFilterStringGroupedExposedEmpty() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); diff --git a/core/modules/views/tests/src/Kernel/ModuleTest.php b/core/modules/views/tests/src/Kernel/ModuleTest.php index ddf3347..d27b366 100644 --- a/core/modules/views/tests/src/Kernel/ModuleTest.php +++ b/core/modules/views/tests/src/Kernel/ModuleTest.php @@ -208,7 +208,7 @@ public function testLoadFunctions() { /** * Tests view enable and disable procedural wrapper functions. */ - function testStatusFunctions() { + public function testStatusFunctions() { $view = Views::getView('test_view_status')->storage; $this->assertFalse($view->status(), 'The view status is disabled.'); diff --git a/core/modules/views/tests/src/Kernel/Plugin/ArgumentValidatorTest.php b/core/modules/views/tests/src/Kernel/Plugin/ArgumentValidatorTest.php index cac7985..34d426d 100644 --- a/core/modules/views/tests/src/Kernel/Plugin/ArgumentValidatorTest.php +++ b/core/modules/views/tests/src/Kernel/Plugin/ArgumentValidatorTest.php @@ -20,7 +20,7 @@ class ArgumentValidatorTest extends ViewsKernelTestBase { */ public static $testViews = ['test_view_argument_validate_numeric', 'test_view']; - function testArgumentValidateNumeric() { + public function testArgumentValidateNumeric() { $view = Views::getView('test_view_argument_validate_numeric'); $view->initHandlers(); $this->assertFalse($view->argument['null']->validateArgument($this->randomString())); diff --git a/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php b/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php index 31805d8..4b18c96 100644 --- a/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php +++ b/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php @@ -224,7 +224,7 @@ public function testTimeResultCachingWithPager() { * * @see views_plugin_cache_time */ - function testNoneResultCaching() { + public function testNoneResultCaching() { // Create a basic result which just 2 results. $view = Views::getView('test_cache'); $view->setDisplay(); @@ -261,7 +261,7 @@ function testNoneResultCaching() { /** * Tests css/js storage and restoring mechanism. */ - function testHeaderStorage() { + public function testHeaderStorage() { // Create a view with output caching enabled. // Some hook_views_pre_render in views_test_data.module adds the test css/js file. // so they should be added to the css/js storage. diff --git a/core/modules/views/tests/src/Kernel/Plugin/StyleHtmlListTest.php b/core/modules/views/tests/src/Kernel/Plugin/StyleHtmlListTest.php index f1f1fb1..19fc15f 100644 --- a/core/modules/views/tests/src/Kernel/Plugin/StyleHtmlListTest.php +++ b/core/modules/views/tests/src/Kernel/Plugin/StyleHtmlListTest.php @@ -23,7 +23,7 @@ class StyleHtmlListTest extends ViewsKernelTestBase { /** * Make sure that the HTML list style markup is correct. */ - function testDefaultRowClasses() { + public function testDefaultRowClasses() { $view = Views::getView('test_style_html_list'); $output = $view->preview(); $output = \Drupal::service('renderer')->renderRoot($output); diff --git a/core/modules/views/tests/src/Kernel/Plugin/StyleUnformattedTest.php b/core/modules/views/tests/src/Kernel/Plugin/StyleUnformattedTest.php index 14e5895..284a571 100644 --- a/core/modules/views/tests/src/Kernel/Plugin/StyleUnformattedTest.php +++ b/core/modules/views/tests/src/Kernel/Plugin/StyleUnformattedTest.php @@ -21,7 +21,7 @@ class StyleUnformattedTest extends StyleTestBase { /** * Make sure that the default css classes works as expected. */ - function testDefaultRowClasses() { + public function testDefaultRowClasses() { $view = Views::getView('test_view'); $view->setDisplay(); $output = $view->preview(); diff --git a/core/modules/views/tests/src/Kernel/TokenReplaceTest.php b/core/modules/views/tests/src/Kernel/TokenReplaceTest.php index 9377ff4..77d9487 100644 --- a/core/modules/views/tests/src/Kernel/TokenReplaceTest.php +++ b/core/modules/views/tests/src/Kernel/TokenReplaceTest.php @@ -29,7 +29,7 @@ protected function setUp($import_test_views = TRUE) { /** * Tests core token replacements generated from a view. */ - function testTokenReplacement() { + public function testTokenReplacement() { $token_handler = \Drupal::token(); $view = Views::getView('test_tokens'); $view->setDisplay('page_1'); @@ -74,7 +74,7 @@ function testTokenReplacement() { /** * Tests core token replacements generated from a view without results. */ - function testTokenReplacementNoResults() { + public function testTokenReplacementNoResults() { $token_handler = \Drupal::token(); $view = Views::getView('test_tokens'); $view->setDisplay('page_2'); diff --git a/core/modules/views/tests/src/Kernel/ViewStorageTest.php b/core/modules/views/tests/src/Kernel/ViewStorageTest.php index fa264bd..b244c9a 100644 --- a/core/modules/views/tests/src/Kernel/ViewStorageTest.php +++ b/core/modules/views/tests/src/Kernel/ViewStorageTest.php @@ -57,7 +57,7 @@ class ViewStorageTest extends ViewsKernelTestBase { /** * Tests CRUD operations. */ - function testConfigurationEntityCRUD() { + public function testConfigurationEntityCRUD() { // Get the configuration entity type and controller. $this->entityType = \Drupal::entityManager()->getDefinition('view'); $this->controller = $this->container->get('entity.manager')->getStorage('view'); diff --git a/core/modules/views_ui/src/Tests/AnalyzeTest.php b/core/modules/views_ui/src/Tests/AnalyzeTest.php index 6cc9203..559c3f4 100644 --- a/core/modules/views_ui/src/Tests/AnalyzeTest.php +++ b/core/modules/views_ui/src/Tests/AnalyzeTest.php @@ -37,7 +37,7 @@ protected function setUp() { /** * Tests that analyze works in general. */ - function testAnalyzeBasic() { + public function testAnalyzeBasic() { $this->drupalLogin($this->admin); $this->drupalGet('admin/structure/views/view/test_view/edit'); diff --git a/core/modules/views_ui/src/Tests/DefaultViewsTest.php b/core/modules/views_ui/src/Tests/DefaultViewsTest.php index 3b3f2a7..59ab4a1 100644 --- a/core/modules/views_ui/src/Tests/DefaultViewsTest.php +++ b/core/modules/views_ui/src/Tests/DefaultViewsTest.php @@ -30,7 +30,7 @@ protected function setUp() { /** * Tests default views. */ - function testDefaultViews() { + public function testDefaultViews() { // Make sure the view starts off as disabled (does not appear on the listing // page). $edit_href = 'admin/structure/views/view/glossary'; @@ -159,7 +159,7 @@ function testDefaultViews() { /** * Tests that enabling views moves them to the correct table. */ - function testSplitListing() { + public function testSplitListing() { // Build a re-usable xpath query. $xpath = '//div[@id="views-entity-list"]/div[@class = :status]/table//tr[@title = :title]'; $arguments = array( diff --git a/core/modules/views_ui/src/Tests/ExposedFormUITest.php b/core/modules/views_ui/src/Tests/ExposedFormUITest.php index fd70978..41f0b07 100644 --- a/core/modules/views_ui/src/Tests/ExposedFormUITest.php +++ b/core/modules/views_ui/src/Tests/ExposedFormUITest.php @@ -33,7 +33,7 @@ protected function setUp() { /** * Tests the admin interface of exposed filter and sort items. */ - function testExposedAdminUi() { + public function testExposedAdminUi() { $edit = array(); $this->drupalGet('admin/structure/views/nojs/handler/test_exposed_admin_ui/default/filter/type'); diff --git a/core/modules/views_ui/src/Tests/GroupByTest.php b/core/modules/views_ui/src/Tests/GroupByTest.php index 8d0ae3a..6667315 100644 --- a/core/modules/views_ui/src/Tests/GroupByTest.php +++ b/core/modules/views_ui/src/Tests/GroupByTest.php @@ -21,7 +21,7 @@ class GroupByTest extends UITestBase { * * @todo This should check the change of the settings as well. */ - function testGroupBySave() { + public function testGroupBySave() { $this->drupalGet('admin/structure/views/view/test_views_groupby_save/edit'); $edit_groupby_url = 'admin/structure/views/nojs/handler-group/test_views_groupby_save/default/field/id'; diff --git a/core/modules/views_ui/src/Tests/OverrideDisplaysTest.php b/core/modules/views_ui/src/Tests/OverrideDisplaysTest.php index 98576ad..7ba6ed5 100644 --- a/core/modules/views_ui/src/Tests/OverrideDisplaysTest.php +++ b/core/modules/views_ui/src/Tests/OverrideDisplaysTest.php @@ -18,7 +18,7 @@ protected function setUp() { /** * Tests that displays can be overridden via the UI. */ - function testOverrideDisplays() { + public function testOverrideDisplays() { // Create a basic view that shows all content, with a page and a block // display. $view['label'] = $this->randomMachineName(16); @@ -77,7 +77,7 @@ function testOverrideDisplays() { /** * Tests that the wizard correctly sets up default and overridden displays. */ - function testWizardMixedDefaultOverriddenDisplays() { + public function testWizardMixedDefaultOverriddenDisplays() { // Create a basic view with a page, block, and feed. Give the page and feed // identical titles, but give the block a different one, so we expect the // page and feed to inherit their titles from the default display, but the @@ -172,7 +172,7 @@ function testWizardMixedDefaultOverriddenDisplays() { /** * Tests that the revert to all displays select-option works as expected. */ - function testRevertAllDisplays() { + public function testRevertAllDisplays() { // Create a basic view with a page, block. // Because there is both a title on page and block we expect the title on // the block be overridden. diff --git a/core/modules/views_ui/src/Tests/PreviewTest.php b/core/modules/views_ui/src/Tests/PreviewTest.php index 3b4dd9b..c2f50c5 100644 --- a/core/modules/views_ui/src/Tests/PreviewTest.php +++ b/core/modules/views_ui/src/Tests/PreviewTest.php @@ -45,7 +45,7 @@ public function testPreviewContextual() { /** * Tests arguments in the preview form. */ - function testPreviewUI() { + public function testPreviewUI() { $this->drupalGet('admin/structure/views/view/test_preview/edit'); $this->assertResponse(200); diff --git a/core/modules/views_ui/src/Tests/SettingsTest.php b/core/modules/views_ui/src/Tests/SettingsTest.php index 30397f4..8583102 100644 --- a/core/modules/views_ui/src/Tests/SettingsTest.php +++ b/core/modules/views_ui/src/Tests/SettingsTest.php @@ -27,7 +27,7 @@ protected function setUp() { /** * Tests the settings for the edit ui. */ - function testEditUI() { + public function testEditUI() { $this->drupalLogin($this->adminUser); // Test the settings tab exists. diff --git a/core/profiles/minimal/src/Tests/MinimalTest.php b/core/profiles/minimal/src/Tests/MinimalTest.php index 056565e..396df0f 100644 --- a/core/profiles/minimal/src/Tests/MinimalTest.php +++ b/core/profiles/minimal/src/Tests/MinimalTest.php @@ -16,7 +16,7 @@ class MinimalTest extends WebTestBase { /** * Tests Minimal installation profile. */ - function testMinimal() { + public function testMinimal() { $this->drupalGet(''); // Check the login block is present. $this->assertLink(t('Create new account')); diff --git a/core/profiles/standard/tests/src/Functional/StandardTest.php b/core/profiles/standard/tests/src/Functional/StandardTest.php index 4f9e756..c671eba 100644 --- a/core/profiles/standard/tests/src/Functional/StandardTest.php +++ b/core/profiles/standard/tests/src/Functional/StandardTest.php @@ -31,7 +31,7 @@ class StandardTest extends BrowserTestBase { /** * Tests Standard installation profile. */ - function testStandard() { + public function testStandard() { $this->drupalGet(''); $this->assertLink(t('Contact')); $this->clickLink(t('Contact')); diff --git a/core/profiles/testing/modules/drupal_system_listing_compatible_test/src/Tests/SystemListingCompatibleTest.php b/core/profiles/testing/modules/drupal_system_listing_compatible_test/src/Tests/SystemListingCompatibleTest.php index 332b000..a9f606d 100644 --- a/core/profiles/testing/modules/drupal_system_listing_compatible_test/src/Tests/SystemListingCompatibleTest.php +++ b/core/profiles/testing/modules/drupal_system_listing_compatible_test/src/Tests/SystemListingCompatibleTest.php @@ -35,7 +35,7 @@ class SystemListingCompatibleTest extends WebTestBase { /** * Non-empty test* method required to executed the test case class. */ - function testSystemListing() { + public function testSystemListing() { $this->pass(__CLASS__ . ' test executed.'); } diff --git a/core/tests/Drupal/KernelTests/Core/Asset/AttachedAssetsTest.php b/core/tests/Drupal/KernelTests/Core/Asset/AttachedAssetsTest.php index 72e7b6f..24cec73 100644 --- a/core/tests/Drupal/KernelTests/Core/Asset/AttachedAssetsTest.php +++ b/core/tests/Drupal/KernelTests/Core/Asset/AttachedAssetsTest.php @@ -55,7 +55,7 @@ protected function setUp() { /** * Tests that default CSS and JavaScript is empty. */ - function testDefault() { + public function testDefault() { $assets = new AttachedAssets(); $this->assertEqual(array(), $this->assetResolver->getCssAssets($assets, FALSE), 'Default CSS is empty.'); list($js_assets_header, $js_assets_footer) = $this->assetResolver->getJsAssets($assets, FALSE); @@ -66,7 +66,7 @@ function testDefault() { /** * Tests non-existing libraries. */ - function testLibraryUnknown() { + public function testLibraryUnknown() { $build['#attached']['library'][] = 'core/unknown'; $assets = AttachedAssets::createFromRenderArray($build); @@ -76,7 +76,7 @@ function testLibraryUnknown() { /** * Tests adding a CSS and a JavaScript file. */ - function testAddFiles() { + public function testAddFiles() { $build['#attached']['library'][] = 'common_test/files'; $assets = AttachedAssets::createFromRenderArray($build); @@ -97,7 +97,7 @@ function testAddFiles() { /** * Tests adding JavaScript settings. */ - function testAddJsSettings() { + public function testAddJsSettings() { // Add a file in order to test default settings. $build['#attached']['library'][] = 'core/drupalSettings'; $assets = AttachedAssets::createFromRenderArray($build); @@ -116,7 +116,7 @@ function testAddJsSettings() { /** * Tests adding external CSS and JavaScript files. */ - function testAddExternalFiles() { + public function testAddExternalFiles() { $build['#attached']['library'][] = 'common_test/external'; $assets = AttachedAssets::createFromRenderArray($build); @@ -136,7 +136,7 @@ function testAddExternalFiles() { /** * Tests adding JavaScript files with additional attributes. */ - function testAttributes() { + public function testAttributes() { $build['#attached']['library'][] = 'common_test/js-attributes'; $assets = AttachedAssets::createFromRenderArray($build); @@ -152,7 +152,7 @@ function testAttributes() { /** * Tests that attributes are maintained when JS aggregation is enabled. */ - function testAggregatedAttributes() { + public function testAggregatedAttributes() { $build['#attached']['library'][] = 'common_test/js-attributes'; $assets = AttachedAssets::createFromRenderArray($build); @@ -168,7 +168,7 @@ function testAggregatedAttributes() { /** * Integration test for CSS/JS aggregation. */ - function testAggregation() { + public function testAggregation() { $build['#attached']['library'][] = 'core/drupal.timezone'; $build['#attached']['library'][] = 'core/drupal.vertical-tabs'; $assets = AttachedAssets::createFromRenderArray($build); @@ -186,7 +186,7 @@ function testAggregation() { /** * Tests JavaScript settings. */ - function testSettings() { + public function testSettings() { $build = array(); $build['#attached']['library'][] = 'core/drupalSettings'; // Nonsensical value to verify if it's possible to override path settings. @@ -226,7 +226,7 @@ function testSettings() { /** * Tests JS assets depending on the 'core/' virtual library. */ - function testHeaderHTML() { + public function testHeaderHTML() { $build['#attached']['library'][] = 'common_test/js-header'; $assets = AttachedAssets::createFromRenderArray($build); @@ -242,7 +242,7 @@ function testHeaderHTML() { /** * Tests that for assets with cache = FALSE, Drupal sets preprocess = FALSE. */ - function testNoCache() { + public function testNoCache() { $build['#attached']['library'][] = 'common_test/no-cache'; $assets = AttachedAssets::createFromRenderArray($build); @@ -255,7 +255,7 @@ function testNoCache() { * * @see \Drupal\Core\Render\Element\HtmlTag::preRenderConditionalComments() */ - function testBrowserConditionalComments() { + public function testBrowserConditionalComments() { $default_query_string = $this->container->get('state')->get('system.css_js_query_string') ?: '0'; $build['#attached']['library'][] = 'common_test/browsers'; @@ -274,7 +274,7 @@ function testBrowserConditionalComments() { /** * Tests JavaScript versioning. */ - function testVersionQueryString() { + public function testVersionQueryString() { $build['#attached']['library'][] = 'core/backbone'; $build['#attached']['library'][] = 'core/domready'; $assets = AttachedAssets::createFromRenderArray($build); @@ -289,7 +289,7 @@ function testVersionQueryString() { /** * Tests JavaScript and CSS asset ordering. */ - function testRenderOrder() { + public function testRenderOrder() { $build['#attached']['library'][] = 'common_test/order'; $assets = AttachedAssets::createFromRenderArray($build); @@ -366,7 +366,7 @@ function testRenderOrder() { /** * Tests rendering the JavaScript with a file's weight above jQuery's. */ - function testRenderDifferentWeight() { + public function testRenderDifferentWeight() { // If a library contains assets A and B, and A is listed first, then B can // still make itself appear first by defining a lower weight. $build['#attached']['library'][] = 'core/jquery'; @@ -386,7 +386,7 @@ function testRenderDifferentWeight() { * * @see simpletest_js_alter() */ - function testAlter() { + public function testAlter() { // Add both tableselect.js and simpletest.js. $build['#attached']['library'][] = 'core/drupal.tableselect'; $build['#attached']['library'][] = 'simpletest/drupal.simpletest'; @@ -406,7 +406,7 @@ function testAlter() { * * @see common_test_library_info_alter() */ - function testLibraryAlter() { + public function testLibraryAlter() { // Verify that common_test altered the title of Farbtastic. /** @var \Drupal\Core\Asset\LibraryDiscoveryInterface $library_discovery */ $library_discovery = \Drupal::service('library.discovery'); @@ -425,7 +425,7 @@ function testLibraryAlter() { /** * Dynamically defines an asset library and alters it. */ - function testDynamicLibrary() { + public function testDynamicLibrary() { /** @var \Drupal\Core\Asset\LibraryDiscoveryInterface $library_discovery */ $library_discovery = \Drupal::service('library.discovery'); // Retrieve a dynamic library definition. @@ -449,7 +449,7 @@ function testDynamicLibrary() { * * @see common_test.library.yml */ - function testLibraryNameConflicts() { + public function testLibraryNameConflicts() { /** @var \Drupal\Core\Asset\LibraryDiscoveryInterface $library_discovery */ $library_discovery = \Drupal::service('library.discovery'); $farbtastic = $library_discovery->getLibraryByName('common_test', 'jquery.farbtastic'); @@ -459,7 +459,7 @@ function testLibraryNameConflicts() { /** * Tests JavaScript files that have querystrings attached get added right. */ - function testAddJsFileWithQueryString() { + public function testAddJsFileWithQueryString() { $build['#attached']['library'][] = 'common_test/querystring'; $assets = AttachedAssets::createFromRenderArray($build); diff --git a/core/tests/Drupal/KernelTests/Core/Bootstrap/GetFilenameTest.php b/core/tests/Drupal/KernelTests/Core/Bootstrap/GetFilenameTest.php index 6cdc4b3..167971b 100644 --- a/core/tests/Drupal/KernelTests/Core/Bootstrap/GetFilenameTest.php +++ b/core/tests/Drupal/KernelTests/Core/Bootstrap/GetFilenameTest.php @@ -19,7 +19,7 @@ class GetFilenameTest extends KernelTestBase { /** * Tests that drupal_get_filename() works when the file is not in database. */ - function testDrupalGetFilename() { + public function testDrupalGetFilename() { // drupal_get_profile() is using obtaining the profile from state if the // install_state global is not set. global $install_state; diff --git a/core/tests/Drupal/KernelTests/Core/Bootstrap/ResettableStaticTest.php b/core/tests/Drupal/KernelTests/Core/Bootstrap/ResettableStaticTest.php index 7b90ec1..a2a7a85 100644 --- a/core/tests/Drupal/KernelTests/Core/Bootstrap/ResettableStaticTest.php +++ b/core/tests/Drupal/KernelTests/Core/Bootstrap/ResettableStaticTest.php @@ -17,7 +17,7 @@ class ResettableStaticTest extends KernelTestBase { * Tests that a variable reference returned by drupal_static() gets reset when * drupal_static_reset() is called. */ - function testDrupalStatic() { + public function testDrupalStatic() { $name = __CLASS__ . '_' . __METHOD__; $var = &drupal_static($name, 'foo'); $this->assertEqual($var, 'foo', 'Variable returned by drupal_static() was set to its default.'); diff --git a/core/tests/Drupal/KernelTests/Core/Cache/GenericCacheBackendUnitTestBase.php b/core/tests/Drupal/KernelTests/Core/Cache/GenericCacheBackendUnitTestBase.php index d5f3e95..6425840 100644 --- a/core/tests/Drupal/KernelTests/Core/Cache/GenericCacheBackendUnitTestBase.php +++ b/core/tests/Drupal/KernelTests/Core/Cache/GenericCacheBackendUnitTestBase.php @@ -491,7 +491,7 @@ public function testDeleteAll() { * Test Drupal\Core\Cache\CacheBackendInterface::invalidate() and * Drupal\Core\Cache\CacheBackendInterface::invalidateMultiple(). */ - function testInvalidate() { + public function testInvalidate() { $backend = $this->getCacheBackend(); $backend->set('test1', 1); $backend->set('test2', 2); @@ -523,7 +523,7 @@ function testInvalidate() { /** * Tests Drupal\Core\Cache\CacheBackendInterface::invalidateTags(). */ - function testInvalidateTags() { + public function testInvalidateTags() { $backend = $this->getCacheBackend(); // Create two cache entries with the same tag and tag value. diff --git a/core/tests/Drupal/KernelTests/Core/Common/SizeTest.php b/core/tests/Drupal/KernelTests/Core/Common/SizeTest.php index 2bcee75..ecc52f6 100644 --- a/core/tests/Drupal/KernelTests/Core/Common/SizeTest.php +++ b/core/tests/Drupal/KernelTests/Core/Common/SizeTest.php @@ -41,7 +41,7 @@ protected function setUp() { /** * Checks that format_size() returns the expected string. */ - function testCommonFormatSize() { + public function testCommonFormatSize() { foreach (array($this->exactTestCases, $this->roundedTestCases) as $test_cases) { foreach ($test_cases as $expected => $input) { $this->assertEqual( @@ -56,7 +56,7 @@ function testCommonFormatSize() { /** * Cross-tests Bytes::toInt() and format_size(). */ - function testCommonParseSizeFormatSize() { + public function testCommonParseSizeFormatSize() { foreach ($this->exactTestCases as $size) { $this->assertEqual( $size, diff --git a/core/tests/Drupal/KernelTests/Core/Common/XssUnitTest.php b/core/tests/Drupal/KernelTests/Core/Common/XssUnitTest.php index 7c68ed4..6b3abb8 100644 --- a/core/tests/Drupal/KernelTests/Core/Common/XssUnitTest.php +++ b/core/tests/Drupal/KernelTests/Core/Common/XssUnitTest.php @@ -28,7 +28,7 @@ protected function setUp() { /** * Tests t() functionality. */ - function testT() { + public function testT() { $text = t('Simple text'); $this->assertEqual($text, 'Simple text', 't leaves simple text alone.'); $text = t('Escaped text: @value', array('@value' => '