acquia / df

src/DFExtension/Context/ModerationSidebar.behat.inc

<?php

namespace Acquia\DFExtension\Context;

use Acquia\LightningExtension\AwaitTrait;

use Drupal\DrupalExtension\Context\DrupalSubContextBase;

/**
 * A context with miscellaneous helpers.
 */
class ModerationSidebarContext extends DrupalSubContextBase {

  use AwaitTrait;

  /**
   * Closes the moderation sidebar.
   *
   * @When I close the moderation sidebar
   */
  public function closeModerationSidebar() {
    $assert = $this->assertSession();

    // Assert that the tab exists...
    $link = $assert->elementExists(
      'named',
      ['link', 'Tasks'],
      $assert->elementExists('css', '.moderation-sidebar-toolbar-tab')
    );

    // ...but only open it if already active.
    if ($link->hasClass('sidebar-open') == TRUE) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

      $link->click();
      $this->awaitAjax();
    }
  }

  /**
   * Selects a moderation sidebar item.
   *
   * @param string $item
   *   The moderation sidebar item to select.
   *
   * @Given I select the :item moderation sidebar button
   */
  public function selectModerationSidebarButton($item) {
    $assert = $this->assertSession();

    // Assert that the tab exists...
    $link = $assert->elementExists(
      'named',
      ['button', $item],
      $assert->elementExists('css', '.moderation-sidebar-container')
    );

    $link->click();
    $this->awaitAjax();
  }

  /**
   * Selects a moderation sidebar item.
   *
   * @param string $item
   *   The moderation sidebar item to select.
   *
   * @Given I select the :item moderation sidebar link
   */
  public function selectModerationSidebarLink($item) {
    $assert = $this->assertSession();

    // Assert that the tab exists...
    $link = $assert->elementExists(
      'named',
      ['link', $item],
      $assert->elementExists('css', '.moderation-sidebar-container')
    );

    $link->click();
    $this->awaitAjax();
  }

}