Skip to content

Commit ae3bfa6

Browse files
committed
Remove route_name from recording if it is the same as the id.
1 parent 7f4f773 commit ae3bfa6

12 files changed

Lines changed: 26 additions & 32 deletions

File tree

modules/controller_attributes/tests/recordings/Kernel/ControllerAttributesSnapshotTest/controller_attributes_test.links.action.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,10 @@ module: controller_attributes_test
22
type: links.action
33
'++ controller_attributes_test.hello_user_overview.add':
44
title: 'Add user'
5-
route_name: controller_attributes_test.hello_user_overview.add
65
appears_on:
76
- controller_attributes_test.hello_user_overview.overview
87
'++ controller_attributes_test.hello_user_overview.invite':
98
title: 'Invite user'
109
weight: 10
11-
route_name: controller_attributes_test.hello_user_overview.invite
1210
appears_on:
1311
- controller_attributes_test.hello_user_overview.overview

modules/controller_attributes/tests/recordings/Kernel/ControllerAttributesSnapshotTest/controller_attributes_test.links.menu.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,28 +2,22 @@ module: controller_attributes_test
22
type: links.menu
33
'++ controller_attributes_test.hello.goodbye':
44
menu_name: tools
5-
route_name: controller_attributes_test.hello.goodbye
65
title: Goodbye
76
description: 'A farewell greeting'
87
'++ controller_attributes_test.hello.hello':
98
menu_name: tools
10-
route_name: controller_attributes_test.hello.hello
119
title: null
1210
'++ controller_attributes_test.hello_user_overview.overview':
1311
menu_name: tools
14-
route_name: controller_attributes_test.hello_user_overview.overview
1512
title: 'List of users'
1613
'++ controller_attributes_test.parent_child.automatic_child':
1714
menu_name: tools
18-
route_name: controller_attributes_test.parent_child.automatic_child
1915
title: Parent
2016
parent: controller_attributes_test.parent_child.parent
2117
'++ controller_attributes_test.parent_child.explicit_child':
2218
menu_name: tools
23-
route_name: controller_attributes_test.parent_child.explicit_child
2419
title: Parent
2520
parent: controller_attributes_test.parent_child.parent
2621
'++ controller_attributes_test.parent_child.parent':
2722
menu_name: tools
28-
route_name: controller_attributes_test.parent_child.parent
2923
title: Parent
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,14 @@
11
module: controller_attributes_test
22
type: links.task
33
'++ controller_attributes_test.hello_user.delete_user_different_base_route':
4-
route_name: controller_attributes_test.hello_user.delete_user_different_base_route
54
title: Edit
65
base_route: entity.user.canonical
76
'++ controller_attributes_test.hello_user.edit':
8-
route_name: controller_attributes_test.hello_user.edit
97
title: Edit
108
base_route: controller_attributes_test.hello_user.view
119
'++ controller_attributes_test.hello_user.invite':
12-
route_name: controller_attributes_test.hello_user.invite
1310
title: Invite
1411
base_route: controller_attributes_test.hello_user.view
1512
'++ controller_attributes_test.hello_user.view':
16-
route_name: controller_attributes_test.hello_user.view
1713
title: View
1814
base_route: controller_attributes_test.hello_user.view

modules/ock/modules/ock_preset/tests/recordings/Kernel/OckPresetSnapshotTest/ock_preset.links.action.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,5 @@ module: ock_preset
22
type: links.action
33
'++ ock_preset.iface_presets.add':
44
title: 'Add preset'
5-
route_name: ock_preset.iface_presets.add
65
appears_on:
76
- ock_preset.iface_presets.index

modules/ock/modules/ock_preset/tests/recordings/Kernel/OckPresetSnapshotTest/ock_preset.links.menu.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,5 @@ module: ock_preset
22
type: links.menu
33
'++ ock_preset.all_presets_overview.index':
44
menu_name: tools
5-
route_name: ock_preset.all_presets_overview.index
65
title: 'ock_preset presets'
76
parent: system.admin_structure
Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,14 @@
11
module: ock_preset
22
type: links.task
33
'++ ock_preset.iface_presets.add':
4-
route_name: ock_preset.iface_presets.add
54
title: 'Add preset'
65
base_route: ock_preset.iface_presets.index
76
'++ ock_preset.iface_presets.index':
8-
route_name: ock_preset.iface_presets.index
97
title: List
108
base_route: ock_preset.iface_presets.index
119
'++ ock_preset.preset.delete':
12-
route_name: ock_preset.preset.delete
1310
title: Delete
1411
base_route: ock_preset.preset.edit
1512
'++ ock_preset.preset.edit':
16-
route_name: ock_preset.preset.edit
1713
title: Edit
1814
base_route: ock_preset.preset.edit

modules/ock/tests/recordings/Kernel/OckSnapshotTest/ock.links.menu.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,11 @@ module: ock
22
type: links.menu
33
'++ ock.devel_ops.clear':
44
menu_name: devel
5-
route_name: ock.devel_ops.clear
65
title: 'Rediscover Composition Plugins'
76
'++ ock.flush_ock.clear':
87
menu_name: admin
9-
route_name: ock.flush_ock.clear
108
title: 'Flush Ock plugins'
119
'++ ock.report_overview.overview':
1210
menu_name: tools
13-
route_name: ock.report_overview.overview
1411
title: 'ock plugins'
1512
parent: system.admin_reports
Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,23 @@
11
module: ock
22
type: links.task
33
'++ ock.report_iface.code':
4-
route_name: ock.report_iface.code
54
title: Code
65
base_route: ock.report_iface.list_of_plugins
76
'++ ock.report_iface.demo':
8-
route_name: ock.report_iface.demo
97
title: Demo
108
base_route: ock.report_iface.list_of_plugins
119
'++ ock.report_iface.list_of_plugins':
12-
route_name: ock.report_iface.list_of_plugins
1310
title: 'List of plugins'
1411
base_route: ock.report_iface.list_of_plugins
1512
'++ ock.report_plugin.code':
16-
route_name: ock.report_plugin.code
1713
title: Code
1814
base_route: ock.report_plugin.plugin
1915
'++ ock.report_plugin.demo':
20-
route_name: ock.report_plugin.demo
2116
title: Demo
2217
base_route: ock.report_plugin.plugin
2318
'++ ock.report_plugin.devel':
24-
route_name: ock.report_plugin.devel
2519
title: Devel
2620
base_route: ock.report_plugin.plugin
2721
'++ ock.report_plugin.plugin':
28-
route_name: ock.report_plugin.plugin
2922
title: Plugin
3023
base_route: ock.report_plugin.plugin

modules/renderkit/tests/recordings/Kernel/RenderkitSnapshotTest/renderkit.links.menu.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,9 @@ module: renderkit
22
type: links.menu
33
'++ renderkit.report.demo':
44
menu_name: tools
5-
route_name: renderkit.report.demo
65
title: 'Build provider demo'
76
parent: renderkit.report.index
87
'++ renderkit.report.index':
98
menu_name: tools
10-
route_name: renderkit.report.index
119
title: Renderkit
1210
parent: system.admin_reports

packages/drupal-testing/src/Snapshotter/Snapshotter_ActionLinks.php

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,17 @@ class Snapshotter_ActionLinks extends SnapshotterBase {
1414
* {@inheritdoc}
1515
*/
1616
protected function getItems(): array {
17-
return DrupalTesting::service(LocalActionManagerInterface::class)
17+
$definitions = DrupalTesting::service(LocalActionManagerInterface::class)
1818
->getDefinitions();
19+
foreach ($definitions as $id => $definition) {
20+
if (!array_key_exists('route_name', $definition)) {
21+
$definitions[$id]['route_name'] = '(missing)';
22+
}
23+
elseif ($definition['route_name'] === $id) {
24+
unset($definitions[$id]['route_name']);
25+
}
26+
}
27+
return $definitions;
1928
}
2029

2130
/**

0 commit comments

Comments
 (0)