Fix Drupal.Array.Array.LongLineDeclaration coding standard for instances of the drupalCreateUser() test method

In #3132745-7: Fix Drupal.Array.Array.LongLineDeclaration coding standard for instances of the $modules test property said @xjm the following:

This is also one of the sorts of patches we do as beta targets since they go stale quickly, so tagging for that. It's an RC-eligible piece of that work, although the rule itself (once enabled) will need to wait for 9.1.x probably.But we can and should clean up the backport branches first and then add the rule to 9.1 when ready.

Therefor adding the tags: "beta target", "rc target".

The code changes that where make look good, only you forgot a lot of changes, like in:
- core/modules/node/tests/src/Functional/NodeQueryAlterTest.php;
- core/modules/block/tests/src/Functional/BlockLanguageTest.php;
- core/modules/contextual/tests/src/Functional/ContextualDynamicContextTest.php.

@kapilkumar0324: Could you also post an interdiff file. It makes reviewing you patch a lot easier. See: https://www.drupal.org/documentation/git/interdiff

+++ b/core/modules/block/tests/src/Functional/BlockLanguageTest.php
@@ -30,10 +30,16 @@ class BlockLanguageTest extends BrowserTestBase {
+  /**
+   *
+   */

@@ -149,9 +155,18 @@ public function testMultipleLanguageTypes() {
-    $this->drupalGet('node', ['query' => ['language' => 'en']]);
+    $this->drupalGet('node', [
+      'query' => [
+        'language' => 'en
+    ',
+      ],
+    ]);
     $this->assertNoText('Powered by Drupal', 'The body of the block does not appear on the page.');
-    $this->drupalGet('node', ['query' => ['language' => 'fr']]);
+    $this->drupalGet('node', [
+      'query' => [
+        'language' => 'fr',
+      ],
+    ]);

@@ -174,9 +189,17 @@ public function testMultipleLanguageTypes() {
-    $this->drupalGet('node', ['query' => ['language' => 'en']]);
+    $this->drupalGet('node', [
+      'query' => [
+        'language' => 'en',
+      ],
+    ]);
     $this->assertNoText('Powered by Drupal', 'The body of the block does not appear on the page.');
-    $this->drupalGet('node', ['query' => ['language' => 'fr']]);
+    $this->drupalGet('node', [
+      'query' => [
+        'language' => 'fr',
+      ],
+    ]);

These changes are not what this issue is about. In this issue only changes the method calls of drupalCreateUser() where it goes over the 80 characters on 1 line limit. These changes should go in an other issue.

@Hardik_Patel_12: This patch needs to land first in 9.1. For that branch does the current patch not apply. Therefore does the patch need a reroll. I was wrong about this.

My mistake. The patch for 9.1 has still un-addressed remarks from comment #8 and #12.

The patch for 9.1 has still un-addressed remarks from comment #8 and #12.

@munish.kamur: Please keep the status to "needs work" until the remarks from comment #8 and #12 are addressed.

The points from comment #8 and #12 have been addressed.
All changes in the patch look good.

I still found a couple of cases that need to be addressed:
core/modules/layout_builder/tests/src/Functional/LayoutBuilderAccessTest.php
core/modules/node/tests/src/Functional/NodeFormSaveChangedTimeTest.php
core/modules/user/tests/src/Functional/UserRolesAssignmentTest.php
core/profiles/minimal/tests/src/Functional/MinimalTest.php
core/tests/Drupal/FunctionalTests/Entity/DeleteMultipleFormTest.php
core/tests/Drupal/FunctionalTests/Routing/CaseInsensitivePathTest.php
core/tests/Drupal/FunctionalTests/Routing/RouteCachingNonPathLanguageNegotiationTest.php

@munish.kumar: Adding the comma to the last permission is a good find. I missed it.

I cannot find any more occurrences of$this->drupalCreateUser() where the line goes over the 80 character limit.
There are no coding standard violations.
All changes look good to me.
For me it is RTBC.

The reroll looks good to me.
Back to RTBC.