Skip to content
Snippets Groups Projects

Issue #3275991: Write test coverage for the bug fix in #3275311

2 unresolved threads

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Ted BowmanTed Bowman 2 years ago (Nov 15, 2022 5:59pm UTC)

Merge details

  • Changes merged into 8.x-2.x with 5c6b2e4d (commits were squashed).
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ted Bowman
  • Ted Bowman
  • Ted Bowman
  • Yash Rode added 8 commits

    added 8 commits

    • 18d1ca72 - 1 commit from branch project:8.x-2.x
    • a0f8c032 - Added test coverage to check the package manager help link is clickable
    • 4af53c09 - Fix the markup handling in the validator
    • c2aae33d - Removed debug
    • 1c1a2e22 - cspell error
    • 4b106a9e - Repalced readiness check with status check.
    • 126dbda5 - modified test coverage
    • 3940754a - nit picks

    Compare with previous version

  • Yash Rode added 1 commit

    added 1 commit

    Compare with previous version

  • Wim Leers
  • Wim Leers
  • Wim Leers
  • 5 /**
    6 * Tests package manager help link is clickable.
    7 *
    8 * @group automatic_updates
    9 */
    10 class ClickableHelpTest extends AutomaticUpdatesFunctionalTestBase {
    11
    12 /**
    13 * {@inheritdoc}
    14 */
    15 protected static $modules = ['automatic_updates', 'help'];
    16
    17 /**
    18 * {@inheritdoc}
    19 */
    20 protected $defaultTheme = 'stark';
  • Wim Leers
  • Yash Rode added 15 commits

    added 15 commits

    Compare with previous version

  • 25 protected $defaultTheme = 'starterkit_theme';
    26
    27 /**
    28 * {@inheritdoc}
    29 */
    30 protected function setUp(): void {
    31 unset($this->disableValidators[array_search('package_manager.validator.composer_executable', $this->disableValidators)]);
    32 parent::setUp();
    33 $this->setReleaseMetadata(__DIR__ . '/../../../package_manager/tests/fixtures/release-history/drupal.9.8.1-security.xml');
    34 $this->checkerRunnerUser = $this->createUser([
    35 'administer site configuration',
    36 ]);
    37 }
    38
    39 /**
    40 * Tests if composer executable is not present then the help link in error is clickable.
  • Ted Bowman added 1 commit

    added 1 commit

    • 1d68b244 - Comment longer than 80 chars

    Compare with previous version

  • Ted Bowman added 11 commits

    added 11 commits

    Compare with previous version

  • merged

  • Please register or sign in to reply
    Loading