From 77f75777eebc68df534417ecb4c9893b373bf838 Mon Sep 17 00:00:00 2001 From: Alex Pott Date: Fri, 7 Nov 2014 11:39:58 -0800 Subject: [PATCH] Issue #2362205 by Devin Carlson | dawehner: Fixed admin/theme/install|update should point to admin/appearance/update|install. --- .../src/Unit/Menu/UpdateLocalTasksTest.php | 88 +++++++++++++++++++ core/modules/update/update.links.task.yml | 12 +-- 2 files changed, 94 insertions(+), 6 deletions(-) create mode 100644 core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php diff --git a/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php b/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php new file mode 100644 index 00000000000..b82ec79a460 --- /dev/null +++ b/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php @@ -0,0 +1,88 @@ +directoryList = array('update' => 'core/modules/update'); + parent::setUp(); + } + + /** + * Checks update report tasks. + * + * @dataProvider getUpdateReportRoutes + */ + public function testUpdateReportLocalTasks($route) { + $this->assertLocalTasks($route, array( + 0 => array('update.status', 'update.settings', 'update.report_update'), + )); + } + + /** + * Provides a list of report routes to test. + */ + public function getUpdateReportRoutes() { + return array( + array('update.status'), + array('update.settings'), + array('update.report_update'), + ); + } + + /** + * Checks update module tasks. + * + * @dataProvider getUpdateModuleRoutes + */ + public function testUpdateModuleLocalTasks($route) { + $this->assertLocalTasks($route, array( + 0 => array('update.module_update'), + )); + ; + } + + /** + * Provides a list of module routes to test. + */ + public function getUpdateModuleRoutes() { + return array( + array('update.module_update'), + ); + } + + /** + * Checks update theme tasks. + * + * @dataProvider getUpdateThemeRoutes + */ + public function testUpdateThemeLocalTasks($route) { + $this->assertLocalTasks($route, array( + 0 => array('update.theme_update'), + )); + ; + } + + /** + * Provides a list of theme routes to test. + */ + public function getUpdateThemeRoutes() { + return array( + array('update.theme_update'), + ); + } + +} diff --git a/core/modules/update/update.links.task.yml b/core/modules/update/update.links.task.yml index 62469313fee..d831bf01c9e 100644 --- a/core/modules/update/update.links.task.yml +++ b/core/modules/update/update.links.task.yml @@ -7,20 +7,20 @@ update.settings: base_route: system.admin_reports title: Settings weight: 50 -update.report_install: - route_name: update.report_install +update.report_update: + route_name: update.report_update base_route: system.admin_reports title: Update weight: 10 -update.module_install: - route_name: update.module_install +update.module_update: + route_name: update.module_update base_route: system.modules_list title: Update weight: 10 -update.theme_install: - route_name: update.theme_install +update.theme_update: + route_name: update.theme_update base_route: system.themes_page title: Update weight: 10