Issue #3122547 by dww, Kristen Pol, jungle: Fix duplicate word typos …

…(the the, to to, etc) for test assertions
drupal · May 1, 2020 · cc19e5f · cc19e5f
1 parent 411c18e
commit cc19e5f
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 9 deletions.
diff --git a/modules/aggregator/tests/src/Kernel/AggregatorPluginManagerTest.php b/modules/aggregator/tests/src/Kernel/AggregatorPluginManagerTest.php
@@ -35,7 +35,7 @@ public function testParserInfoAlter() {
     $widget_definition = \Drupal::service('plugin.manager.aggregator.parser')->getDefinition('aggregator_test_parser');
 
     // Test if hook_aggregator_parser_info_alter is being called.
-    $this->assertTrue($widget_definition['definition_altered'], "The 'aggregator_test_parser' plugin definition was updated in in `hook_aggregator_parser_info_alter()`");
+    $this->assertTrue($widget_definition['definition_altered'], "The 'aggregator_test_parser' plugin definition was updated in `hook_aggregator_parser_info_alter()`");
   }
 
   /**
@@ -45,7 +45,7 @@ public function testProcessorInfoAlter() {
     $widget_definition = \Drupal::service('plugin.manager.aggregator.processor')->getDefinition('aggregator_test_processor');
 
     // Test if hook_aggregator_processor_info_alter is being called.
-    $this->assertTrue($widget_definition['definition_altered'], "The 'aggregator_test_processor' plugin definition was updated in in `hook_aggregator_processor_info_alter()`");
+    $this->assertTrue($widget_definition['definition_altered'], "The 'aggregator_test_processor' plugin definition was updated in `hook_aggregator_processor_info_alter()`");
   }
 
 }
diff --git a/modules/system/tests/modules/js_message_test/js/js_message_test.es6.js b/modules/system/tests/modules/js_message_test/js/js_message_test.es6.js
@@ -51,7 +51,7 @@
           if (action === 'add') {
             messageObjects[area].indexes[type].push(
               message.add(
-                `This is a message of the type, ${type}. You be the the judge of its importance.`,
+                `This is a message of the type, ${type}. You be the judge of its importance.`,
                 { type },
               ),
             );
@@ -71,7 +71,7 @@
               messageObjects.default.zone.add(
                 `This is message number ${i} of the type, ${
                   testMessages.types[i % testMessages.types.length]
-                }. You be the the judge of its importance.`,
+                }. You be the judge of its importance.`,
                 { type: testMessages.types[i % testMessages.types.length] },
               ),
             );

diff --git a/modules/system/tests/modules/js_message_test/js/js_message_test.js b/modules/system/tests/modules/js_message_test/js/js_message_test.js
@@ -41,14 +41,14 @@
         var action = $target.attr('data-action');
 
         if (action === 'add') {
-          messageObjects[area].indexes[type].push(message.add('This is a message of the type, ' + type + '. You be the the judge of its importance.', { type: type }));
+          messageObjects[area].indexes[type].push(message.add('This is a message of the type, ' + type + '. You be the judge of its importance.', { type: type }));
         } else if (action === 'remove') {
           message.remove(messageObjects[area].indexes[type].pop());
         }
       });
       $('[data-action="add-multiple"]').once('add-multiple').on('click', function () {
         [0, 1, 2, 3, 4, 5].forEach(function (i) {
-          messageObjects.multiple.push(messageObjects.default.zone.add('This is message number ' + i + ' of the type, ' + testMessages.types[i % testMessages.types.length] + '. You be the the judge of its importance.', { type: testMessages.types[i % testMessages.types.length] }));
+          messageObjects.multiple.push(messageObjects.default.zone.add('This is message number ' + i + ' of the type, ' + testMessages.types[i % testMessages.types.length] + '. You be the judge of its importance.', { type: testMessages.types[i % testMessages.types.length] }));
         });
       });
       $('[data-action="remove-multiple"]').once('remove-multiple').on('click', function () {

diff --git a/tests/Drupal/FunctionalJavascriptTests/Core/JsMessageTest.php b/tests/Drupal/FunctionalJavascriptTests/Core/JsMessageTest.php
@@ -50,8 +50,8 @@ public function testAddRemoveMessages() {
         $selector = "$messagesSelector .messages.messages--$type";
         $msg_element = $web_assert->waitForElementVisible('css', $selector);
         $this->assertNotEmpty($msg_element, "Message element visible: $selector");
-        $web_assert->elementContains('css', $selector, "This is a message of the type, $type. You be the the judge of its importance.");
-        $current_messages[$selector] = "This is a message of the type, $type. You be the the judge of its importance.";
+        $web_assert->elementContains('css', $selector, "This is a message of the type, $type. You be the judge of its importance.");
+        $current_messages[$selector] = "This is a message of the type, $type. You be the judge of its importance.";
         $this->assertCurrentMessages($current_messages, $messagesSelector);
       }
       // Remove messages 1 by 1 and confirm the messages are expected.
@@ -69,7 +69,7 @@ public function testAddRemoveMessages() {
     $types = JSMessageTestController::getTypes();
     $nb_messages = count($types) * 2;
     for ($i = 0; $i < $nb_messages; $i++) {
-      $current_messages[] = "This is message number $i of the type, {$types[$i % count($types)]}. You be the the judge of its importance.";
+      $current_messages[] = "This is message number $i of the type, {$types[$i % count($types)]}. You be the judge of its importance.";
     }
     // Test adding multiple messages at once.
     // @see processMessages()