Port menu_example to D8

Going to work on this during DrupalCon lab.

I have worked on it on my sandbox and later in the evening, I will upload the patch for what I have done so far.

work in progress here - https://drupal.org/sandbox/kg/2097629

Care to offer a patch please?

Change notice: https://drupal.org/node/2184797

A straight port of menu_example's tests from 7.x-1.x.

These should fail. :-)

Working on this issue got me working on SimpleTest: #2234295: drupalLogin() crashes where it should fail. #1452896: PHP notice in clickLink if link does not exist

New failing tests from #2277667: Port D7 Tests To D8 for menu_example. Consider these the 'official' ones, and dev your heart out with them.

I've got some time in the coming week to try to make some progress on this. I'm going to attempt to contact kgoel directly before assigning this issue to myself. kgoel, if you see this first, have you made any progress beyond what is in the link in #5?

mrjmd- I worked on this during DrupalCon Prgaue so what I have in my sandbox is probably stale anyway.

Okay, I am going to assign this to myself for now and will try my hand at an upgrade. This will be my first D8 module upgrade so it's going to be a learning experience.... I'll either submit a patch or explain that I failed miserably and take the issue out of my name by next week.

Keep in mind that there are a bunch of changes to the menu system, the most important being hook_menu() replaced with *.menu_link.yml: https://drupal.org/node/2223291

Also check the PHPUnit Example module for a template for testing menu links.

Thanks!

Please note that this patch is old and there has been change in routing system including adoption of PSR-4.

Just wanted to post a quick update on my progress, which can currently be followed here: https://github.com/mrjmd/menu_example

Overall, I think most of this is functionally complete and working. I gutted the old module and previous patches because pretty much everything has changed. The only thing that remains at this point from the D7 version is the implementation of hook_permission()!

I'm going to continue working on this but here's what I know is still incomplete:

• Custom Parameter Upcasting - The D7 menu_example module included a 'magic placeholder processing' example, which doesn't really have an equivalent in D8 as far as I've found. The closest equivalent I've seen is custom parameter upcasting, but even that is deemed rarely necessary in current documentation (entities are upcast automatically), so I'm not sure if an example of it is in scope or not. Thoughts? Either way, I haven't been able to get it to work yet.
• Tests - I have not gotten to writing tests at all. Going to try to tackle this next week.
• Documentation - If you look at my repo, you'll notice pretty much all code comments are gone right now. Partially that's because almost all of them need to be completely re-written, which I haven't done yet. But also, I'm not sure where the best place will be to put the new introductory comments. In the D7 version, there is pretty much just one big .module file, so the introduction starts at the top of that file and continues inline through the file. Since the new .module file is almost empty, and our file structure is much more complex, I'm not sure of the best place for our initial overview comments... should we include a README.txt instead or just assume people will still look at the .module file first?
• Another thing worth mentioning / getting feedback on is the way the D7 version of this module was using page arguments to pass in the main content for many of the menu links. That approach is still possible in D8 routing files (and I included an example of it), but I've mostly abandoned that approach and instead implemented custom methods for each link. It means more methods in our controller but less markup in our routing file, which seems like better practice to me. Thoughts?

Once I get tests and documentation included I'll roll a patch, unless someone would like one now.

Also I'm keeping an eye on https://www.drupal.org/node/2256521; I think it's going to break a good bit of my work.

• Upcasting is definitely something to show off in this example.
• As far as tests are concerned, there are a bunch from D7 in the patch at #11. You say the implementations have changed, but have the behaviors? Maybe roll those tests in and see what passes.
• We want the main landing point for documentation to be the .module file. This is for consistency with all the other examples, including D7 and D6. The docblock @defgroup gets parsed into api.drupal.org, where the whole thing can be browsed. Any docblocks describing how to use the module once its installed, what each page/link/etc demonstrates, and pointers to show where to look for the things being demonstrated would be much appreciated. :-)
• Putting methods on the controller is the right thing to do.

Also, wow... #2256521: [META] New plan, Phase 2: Implement menu links as plugins, including static admin links and views, and custom links with menu_link_content entity, all managed via menu_ui module is pretty major. Nothing like a last-minute re-architecting. :-)

Thanks.

Un-assigning. Please re-assign if you're still working on this.

Basically the previous patches don't work because they don't use routes. We're also very close on #2277667: Port D7 Tests To D8 for menu_example which should be the baseline for behavior in this issue.

We have working failing tests now from #2277667: Port D7 Tests To D8 for menu_example

I've closed that issue and the patch here is the patch from that issue.

Proper git attribution for that issue:

git commit -m 'Issue #2277667 by mimran, snehi: Port D7 Tests To D8 for menu_example' --author="imran4uall <imran4uall@2712535.no-reply.drupal.org>"

I am creating fresh patch for this

Ported few files.

Well, I've ported some of the examples, But I am still facing some issues which are
1) Creating the primary and Secondary tab.
2)Processed Placeholder Arguments.
3)Title Callback, instead of username I was only able to show userid.
4)Ports for the callback only , custom access menu item are not made.

Well, It is a working patch of Menu example. @navneet0693 and @vaibhavjain thank you for the guidance.

Thanks, @AbhishekLal :-)

There are probably other coding standards issues, but let's start here:

1. +++ b/menu_example/menu_example.links.menu.yml
   @@ -0,0 +1,77 @@
   +  ¶
   ...
   +  ¶
   
   +++ b/menu_example/menu_example.permissions.yml
   @@ -0,0 +1,7 @@
   +
   +
   
   +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +
   
   +++ b/menu_example/tests/menu_example/functional/MenuExampleTest.php
   @@ -0,0 +1,106 @@
   +
   +
   

   Bunch of white space errors.

2. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +    return [
   +      '#markup' => $this->t('This page is displayed by the simplest (and base)
   +       menu example. Note that the title of the page is the same as the link
   +       title. You can also @link. There are a number of
   +       examples here, from the most basic (like this one) to extravagant
   +        mappings of loaded placeholder arguments. Enjoy!', ['@link' => $link]),
   +    ];
   

   This should use a translatable template. See BlockExampleController (and block_example/templates/description.html.twig)

3. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   ...
   +  /**
   +   *
   +   */
   

   These docblocks should say something. :-) https://www.drupal.org/docs/develop/coding-standards/api-documentation-a... Explain what the controller method is demonstrating.

4. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +/**
   +   *
   +   */
   

   Extraneous docblock.

5. +++ b/menu_example/tests/menu_example/functional/MenuExampleTest.php
   @@ -0,0 +1,106 @@
   +namespace Drupal\tests\menu_example\functional;
   

   These tests aren't run because they use the wrong PSR-4 file mapping. This test file should be in /menu_example/tests/src/Functional/MenuExampleTest.php

   That should map to the namespace Drupal\Tests\menu_example\Functional

6. +++ b/menu_example/tests/menu_example/functional/MenuExampleTest.php
   @@ -0,0 +1,106 @@
   +  /**
   +   * test whether the module was installed.
   +   */
   +  public function testInstalled() {
   +    $this->assertTrue(\Drupal::moduleHandler()->moduleExists('menu_example'));
   +  }
   

   Remove this method. The other tests will fail if the module is not installed.

Hey @AbhishekLal thats a great work and a lot of work :) thanks for you efforts !!

Adding to what @Mile23 pointed to, I am adding few points here :)

1. +++ b/menu_example/menu_example.links.menu.yml
   @@ -0,0 +1,77 @@
   +  ¶
   ...
   +  ¶
   

   Extra white space.

2. +++ b/menu_example/menu_example.permissions.yml
   @@ -0,0 +1,7 @@
   +
   

   Extra line.

3. +++ b/menu_example/menu_example.routing.yml
   @@ -0,0 +1,184 @@
   +  path: '/examples/menu_example/permissioned'
   ...
   +  path: '/examples/menu_example/permissioned/controlled'
   ...
   +  path: '/examples/menu_example/custom-access'
   ...
   +  path: '/examples/menu_example/custom-access/page'
   ...
   +  path: '/examples/menu_example/path-only'
   ...
   +  path: '/examples/menu_example/path-only/callback'
   ...
   +  path: 'examples/menu_example/tabs'
   ...
   +  path: 'examples/menu_example/tabs/default'
   ...
   +  path: 'examples/menu_example/tabs/second'
   ...
   +  path: 'examples/menu_example/tabs/third'
   ...
   +  path: 'examples/menu_example/tabs/forth'
   ...
   +  path: 'examples/menu_example/tabs/default/Second'
   ...
   +  path: 'examples/menu_example/tabs/default/third'
   ...
   +  path: 'examples/menu_example/use-url-arguments/{arg1}'
   ...
   +  path: 'examples/menu_example/use-url-arguments/{arg1}/{arg2}'
   ...
   +  path: 'examples/menu_example/title-callbacks'
   ...
   +  path: 'examples/menu_example/placeholder-argument'
   ...
   +  path: 'examples/menu_example/placeholder-argument/{node}/display'
   ...
   +  path: 'examples/menu_example/default_arg/{no}'
   ...
   +  path: 'examples/menu_example/menu_original_path'
   

   //s /examples/menu_example/permission to /examples/menu-example/permission to main consistency :) and also Drupal 8 use - instead of _ in paths.

4. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +      '#markup' => t('This will be in the Main menu instead of the default Tools menu'),
   

   Use $this->t() instead of t()

5. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +    $urle = Url::fromUri('internal:/examples/menu_example/use-url-arguments/one/two');
   +    $linker = Link::fromTextAndUrl($this->t('examples/menu_example/use-url-arguments/one/two'), $urle)->toString();
   

   More meaningful variable names will look good :)

6. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +      '#markup' => $this->t('Argument1 = @arg1 , Argument2 = @arg2', ['@arg1' => $arg1, '@arg2' => $arg2]),
   

   Use % for replacement here.

   Read more about placeholders here: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Rend...

7. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +    // $title = $account->getUsername()->toString();
   +    // $account = \Drupal::currentUser()->id();
   +    // $title=$account->get('name')->value;
   

   Commented code ???

8. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +      '#markup' => $this->t('This menu item was created strictly to allow the hook_menu_alter() function to have something to operate on.hook_menu defined the path as examples/menu_example/menu_original_path. The hook_menu_alter() changes it to examples/menu_example/menu_altered_path. You can try navigating to both paths and see what happens!'),
   

   //s on.hook_menu to on hook_menu

   We might need to update description ?

9. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,284 @@
   +
   

   Extra line, this can be utilized to leave description.

10. +++ b/menu_example/tests/menu_example/functional/MenuExampleTest.php
    @@ -0,0 +1,106 @@
    +
    +
    +  }
    

    One line before function ending bracket and one after it will look good.

@Mile23 and @navneet0693 I was able to solve some issues you guys have mentioned, I had a tough time with 2nd one in #37 and some are yet to be completed.. This is a working patch..

Setting needs review to run the tests.

Adding few things, I will upload a patch soon.

The patch applied properly and working correctly, thank you so much.

If you check the console output of the last test run, you'll see that no menu_example tests ran: https://dispatcher.drupalci.org/job/drupal8_contrib_patches/8470/console

Mile23 is right, I must have missed something :-( .

+++ b/menu_example/tests/src/Functional/MenuExampleTest.php
@@ -0,0 +1,90 @@
+  protected $profile = 'minimal';
...
+    $this->assertText('Menu Example');

I think the problem is that the minimal profile doesn't show the tools menu block. Though I could be remembering incorrectly.

@Mile23 I am not sure if that is the problem. Quoting the comment from queue_example and also procedure is same as in other examples.

* We need the 'minimal' profile in order to make sure the Tool block is
* available.

We also have http://cgit.drupalcode.org/examples/tree/tests/src/Functional/ExamplesBr... to deal with this very issue. :-)

@Mile23, and now we know why the tests were failing, I was able to produce this when i installed drupal core dev (8.4.x) and ran the test. Somehow title were being read from .links.menu.yml in 8.3.x which were without quotes ( see interdiff.txt ) but not on 8.4.x !

Thanks your hint on IRC, i was able to solve this.

This patch is ready for review.

Nice.... Getting closer. :-)

1. +++ b/menu_example/menu_example.info.yml
   --- /dev/null
   +++ b/menu_example/menu_example.links.menu.yml
   
   +++ b/menu_example/menu_example.links.menu.yml
   --- /dev/null
   +++ b/menu_example/menu_example.links.task.yml
   
   +++ b/menu_example/menu_example.module
   --- /dev/null
   +++ b/menu_example/menu_example.permissions.yml
   
   +++ b/menu_example/menu_example.permissions.yml
   --- /dev/null
   +++ b/menu_example/menu_example.routing.yml
   

   All of these YAML files need a ton of comments to explain the relationships between the different items.

   For instance, the tab examples need to explain how they're set up as tabs.

2. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,255 @@
   +  public function defaultSecond() {
   ...
   +  public function defaultThird() {
   

   There has to be some way to do this with introversion, so that all the secondary tabs can be one method that knows which tab it should represent.

3. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,255 @@
   +  /**
   +   * Demonstrate generation of dynamic creation of page title.
   +   */
   +  public function titleCallback() {
   ...
   +  /**
   +   * Generates title dynamically.
   +   */
   +  public function backTitle() {
   

   These methods need to @see each other. They also need to explain that they're connected by the yaml specification.

4. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,255 @@
   +      '#markup' => $this->t('The title of this page is dynamically changed by the title callback for this route. Also, the title of the menu link is dynamically changed by implementing hook_menu_link_presave() and hook_translated_menu_link_alter().'),
   

   This statement isn't accurate. We specify a title callback in the yaml.

5. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,255 @@
   +  /**
   +   * Displays placeholder argument supplied in URL.
   +   *
   +   * @param int $arg
   +   *   URL argument.
   +   *
   +   * @return array
   +   *   URL argument.
   +   */
   +  public function placeholderArgsDisplay($arg) {
   +    return [
   +      '#markup' => $arg,
   +    ];
   

   This kind of thing creeps me out. :-) Are we sure that sanitization is applied somewhere else?

6. +++ b/menu_example/src/Routing/RouteSubscriber.php
   @@ -0,0 +1,24 @@
   +/**
   + * Listens to the dynamic route events.
   + */
   +class RouteSubscriber extends RouteSubscriberBase {
   

   Should explain what the route subscriber is listening for, and what it will accomplish.

7. +++ b/menu_example/templates/description.html.twig
   @@ -0,0 +1,14 @@
   +<p>This page is displayed by the simplest (and base) menu example. Note that the title of the page is the same as the link title. You can also <a href="{{ link }}">visit a similar page with no menu link</a>.
   +There are a number of examples here, from the most basic (like this one) to extravagant mappings of loaded placeholder arguments. Enjoy!</p>
   

   Wrap to 80.

8. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,90 @@
   + * @ingroup menu-example
   ...
   + * @group menu-example
   

   menu_example

9. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,90 @@
   +  public function testMenuExample() {
   

   It seems like we've defined more routes than are tested here.

Definitely want to include this information: #2384383: "Content Entity Example: Contact Listing" in Tools Menu

@Mile23 For point 5: We are using following, which I think should do the work.

requirements:
    arg: \d+

I am sorry, I forgot to update tests, that's why previous tests got failed.

Patch applied properly and working great!!

Looking good... Just a few things:

1. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,250 @@
   +  /**
   +   * Demonstrate how one can alter the existing routes.
   +   */
   +  public function pathOverride() {
   

   This isn't linked or mentioned in the description page or the module docblock.

2. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,97 @@
   +    // Now start testing other menu entries.
   +    $this->drupalGet('examples/menu-example');
   +    $this->clickLink(t('Custom Access Example'));
   +    $this->assertSession()->pageTextContains(t('Custom Access Example'));
   +    $this->drupalGet('examples/menu-example/custom-access/page');
   +    $this->assertSession()->statusCodeEquals(200);
   

   Needs a test that shows denial of access if you don't have the right permissions.

3. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,97 @@
   +    // Check 'Permission Example' menu entry.
   +    $this->drupalGet('examples/menu-example/permissioned');
   +    $this->assertSession()->pageTextContains(t('Permissioned Example'));
   +    $this->clickLink('examples/menu-example/permissioned/controlled');
   +    $this->assertSession()->statusCodeEquals(403);
   

   Needs an assertion that you can see the page if you have the right permissions.

Thanks @Mile23 for review, I have added the suggested changes, and it's getting better ;-)

Patch applied properly

Thanks. Getting closer....

1. +++ b/menu_example/menu_example.routing.yml
   @@ -0,0 +1,143 @@
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   ...
   +  requirements:
   ...
   +    _access: 'TRUE'
   ...
   +  requirements:
   +    _access: 'TRUE'
   

   These should all be
   requirements:
   _permission: 'access content'

2. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,250 @@
   +    $url = Url::fromUri('internal:/examples/menu-example/permissioned/controlled');
   +    $link = Link::fromTextAndUrl($this->t('examples/menu-example/permissioned/controlled'), $url)->toString();
   ...
   +    $url = Url::fromUri('internal:/examples/menu-example/custom-access/page');
   +    $link = Link::fromTextAndUrl($this->t('examples/menu-example/custom-access/page'), $url)->toString();
   ...
   +    $url = Url::fromUri('internal:/examples/menu-example/route-only/callback');
   +    $link = Link::fromTextAndUrl($this->t('examples/menu-example/route-only/callback'), $url)->toString();
   ...
   +    $url = Url::fromUri('internal:/examples/menu-example/use-url-arguments/one');
   +    $link = Link::fromTextAndUrl($this->t('examples/menu-example/use-url-arguments/one'), $url)->toString();
   ...
   +    $urltwo = Url::fromUri('internal:/examples/menu-example/use-url-arguments/one/two');
   +    $linktwo = Link::fromTextAndUrl($this->t('examples/menu-example/use-url-arguments/one/two'), $urltwo)->toString();
   ...
   +    $url = Url::fromUri('internal:/examples/menu-example/placeholder-argument/3343/display');
   +    $link = Link::fromTextAndUrl($this->t('examples/menu-example/placeholder-argument/3343/display'), $url)->toString();
   

   We want to generate these links using Url::fromRoute() and/or Link::createFromRoute().

   You can see how to do it here: http://cgit.drupalcode.org/examples/tree/ajax_example/src/Form/Dependent...

3. +++ b/menu_example/src/Controller/MenuExampleController.php
   @@ -0,0 +1,250 @@
   +  public function backTitle() {
   +    return \Drupal::currentUser()->getDisplayName();
   +  }
   

   Inject the service. We should also change the title so it says something like: "The new title is your username: user".

4. +++ b/menu_example/templates/description.html.twig
   @@ -0,0 +1,17 @@
   +Description text for the Block Example.
   

   Mmm. Copypasta. :-)

5. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,108 @@
   +    $this->drupalGet('examples/menu-example');
   

   We should use routes in the test. That way the test still works if we change the path.

   $this->drupalGet(Url::fromRoute(route.name));

6. +++ b/menu_example/tests/src/Functional/MenuExampleTest.php
   @@ -0,0 +1,108 @@
   +    $this->assertSession()->linkByHrefExists('/examples/menu-example/tabs');
   

   href assertions like this:

   $assertion->linkByHrefExists(Url::fromRoute($route)->getInternalPath());

Also PHPCS says:

$ ../../vendor/bin/phpcs -ps menu_example
..E..E


FILE: ...s/examples/menu_example/src/Controller/MenuExampleController.php
----------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
----------------------------------------------------------------------
  87 | ERROR | [x] Doc comment star missing
     |       |     (Drupal.Commenting.DocCommentStar.StarMissing)
 145 | ERROR | [x] Short array syntax must be used to define arrays
     |       |     (Drupal.Array.DisallowLongArraySyntax.Found)
 175 | ERROR | [x] Short array syntax must be used to define arrays
     |       |     (Drupal.Array.DisallowLongArraySyntax.Found)
 178 | ERROR | [x] Short array syntax must be used to define arrays
     |       |     (Drupal.Array.DisallowLongArraySyntax.Found)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: ...s/examples/menu_example/tests/src/Functional/MenuExampleTest.php
----------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
----------------------------------------------------------------------
  7 | WARNING | [x] Unused use statement
    |         |     (Drupal.Classes.UnusedUseStatement.UnusedUse)
 24 | ERROR   | [x] Short array syntax must be used to define arrays
    |         |     (Drupal.Array.DisallowLongArraySyntax.Found)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 509ms; Memory: 4Mb

issue no 1,2,3,4 is done and made few changes in the test... Hope it works

+++ b/menu_example/tests/src/Functional/MenuExampleTest.php
@@ -46,30 +46,30 @@
+    $this->drupalGet(Url::fromRoute('examples.menu_example'));
...
+    $this->drupalGet(Url::fromRoute('examples.menu_example'));

Wrong routes here.

@navneet0693 thanks for the hints and please review

Thanks, looks great. Just needs that final push...

+++ b/menu_example/tests/src/Functional/MenuExampleTest.php
@@ -0,0 +1,110 @@
+  public function testMenuExampleRoutes() {

Managed to lose the test of the route that requires an authenticated user.

Added test case for examples.menu_example.custom_access_page.

Looks fixed to me. :-)

Thanks, everyone!