Skip to content
Snippets Groups Projects

Resolve #3395901 "Fix tests"

Merged Adam Shepherd requested to merge issue/simplenews-3395901:3395901-fix-tests into 4.x
3 unresolved threads

Closes #3395901

Merge request reports

Merge request pipeline passed with warnings for 1ce08a8c

Code Quality is loading
Test summary results are being parsed

Merged by Adam ShepherdAdam Shepherd 8 months ago (Jun 11, 2024 1:00pm UTC)

Loading

Pipeline #211074 failed

Pipeline failed for fa6bbe68 on 4.x

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
122 122 'language_configuration[content_translation]' => TRUE,
123 123 ];
124 124 $this->drupalGet('admin/structure/types/manage/simplenews_issue');
125 $this->submitForm($edit, 'Save content type');
125 $this->submitForm($edit, (string) t('Save'));
  • 150 150
    151 151 // Send newsletter.
    152 152 $this->clickLink(t('Newsletter'));
    153 $this->submitForm([], t('Send now'));
    153 $this->submitForm([], (string) t('Send now'));
  • 321 321 * Extract a confirmation link from a mail body.
    322 322 */
    323 323 protected function extractConfirmationLink($body, $action = 'confirm') {
    324 $pattern = "@simplenews/$action/.+/.+/.{20,}@";
    324 $pattern = "@simplenews/$action/.+/.+/(?:(?!…).)*$@m";
  • vlad.dancer added 1 commit

    added 1 commit

    • 1ce08a8c - Resolve MR40 comments (removed t and added regex comment)

    Compare with previous version

  • Adam Shepherd marked this merge request as ready

    marked this merge request as ready

  • Please register or sign in to reply
    Loading