Unverified Commit 50a60755 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3126563 by mondrake, daffie, alexpott: Replace usage of...

Issue #3126563 by mondrake, daffie, alexpott: Replace usage of assertAttributeEquals() that is deprecated

(cherry picked from commit a6e402fc)
parent 7dcb6fa2
Loading
Loading
Loading
Loading
+5 −0
Changes for core/modules/block/tests/modules/block_test/src/PluginForm/EmptyBlockForm.php: 5 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -10,6 +10,11 @@
 */
class EmptyBlockForm extends PluginFormBase {

  /**
   * {@inheritdoc}
   */
  public $plugin;

  /**
   * {@inheritdoc}
   */
+3 −1
Changes for core/modules/comment/tests/src/Unit/Plugin/views/field/CommentBulkFormTest.php: 3 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -93,7 +93,9 @@ public function testConstructor() {
    $comment_bulk_form = new CommentBulkForm([], 'comment_bulk_form', $definition, $entity_type_manager, $language_manager, $messenger, $entity_repository);
    $comment_bulk_form->init($executable, $display, $options);

    $this->assertAttributeEquals(array_slice($actions, 0, -1, TRUE), 'actions', $comment_bulk_form);
    $reflected_actions = (new \ReflectionObject($comment_bulk_form))->getProperty('actions');
    $reflected_actions->setAccessible(TRUE);
    $this->assertEquals(array_slice($actions, 0, -1, TRUE), $reflected_actions->getValue($comment_bulk_form));
  }

}
+0 −12
Changes for core/modules/forum/tests/src/Unit/Breadcrumb/ForumBreadcrumbBuilderBaseTest.php: 0 added lines, 12 removed lines.
Original line number Diff line number Diff line
@@ -57,18 +57,6 @@ public function testConstructor() {
      ]
    );

    // Reflect upon our properties, except for config which is a special case.
    $property_names = [
      'entityTypeManager' => $entity_type_manager,
      'forumManager' => $forum_manager,
      'stringTranslation' => $translation_manager,
    ];
    foreach ($property_names as $property_name => $property_value) {
      $this->assertAttributeEquals(
        $property_value, $property_name, $builder
      );
    }

    // Test that the constructor made a config object with our info in it.
    $reflector = new \ReflectionClass($builder);
    $ref_property = $reflector->getProperty('config');
+1 −1
Changes for core/modules/migrate/tests/src/Unit/MigrateSqlIdMapTest.php: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -194,7 +194,7 @@ public function testSetMessage() {
    $message = $this->createMock('Drupal\migrate\MigrateMessageInterface');
    $id_map = $this->getIdMap();
    $id_map->setMessage($message);
    $this->assertAttributeEquals($message, 'message', $id_map);
    $this->assertEquals($message, $id_map->message);
  }

  /**
+5 −0
Changes for core/modules/migrate/tests/src/Unit/TestSqlIdMap.php: 5 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -34,6 +34,11 @@ public function __construct(Connection $database, array $configuration, $plugin_
    parent::__construct($configuration, $plugin_id, $plugin_definition, $migration, $event_dispatcher);
  }

  /**
   * {@inheritdoc}
   */
  public $message;

  /**
   * {@inheritdoc}
   */
Loading