Remove withConsecutive() in menu tests

Seems to have a failure.

Akhil Babu → made their first commit to this issue’s fork.

I have removed the commented code in core/tests/Drupal/Tests/Core/Menu/LocalTaskManagerTest.php. Also fixed coding standard errors.

Reverted the out of scope changes, fixed the other issues.

Thanks @longwave, reviewing MR I see green and don't see any coverage is lost.

This is nearly as big a diff as all the other MRs combined, who knew menu tests were relying on it so much, but mostly shifting from expects/withConsecutive to ::shouldBeCalled() feels like a small net improvement much like the other issues. Committed/pushed to 11.x, thanks!

Automatically closed - issue fixed for 2 weeks with no activity.