Problem/Motivation

Failed on unrelated MR, once out of a few runs today
https://git.drupalcode.org/issue/drupal-3395404/-/pipelines/49731/test_r...

Drupal\FunctionalJavascriptTests\Core\Field\TimestampFormatterWithTimeDiffViewsTest::testTimestampFormatterWithTimeDiff
Failed asserting that '1 hour 1 minute ago' matches PCRE pattern "#1 hour ago#".

/builds/issue/drupal-3395404/vendor/phpunit/phpunit/src/Framework/Constraint/Constraint.php:121
/builds/issue/drupal-3395404/vendor/phpunit/phpunit/src/Framework/Constraint/Constraint.php:55
/builds/issue/drupal-3395404/core/tests/Drupal/FunctionalJavascriptTests/Core/Field/TimestampFormatterWithTimeDiffViewsTest.php:64
/builds/issue/drupal-3395404/vendor/phpunit/phpunit/src/Framework/TestResult.php:728

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3401720

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

acbramley created an issue. See original summary.

neclimdul made their first commit to this issue’s fork.

neclimdul’s picture

neclimdul
He/Him
commented
Status: Active » Closed (duplicate)
Related issues: +#3401720: [random test fail] TimestampFormatterWithTimeDiffViewsTest::testTimestampFormatterWithTimeDiff

Duplicates #3401720: [random test fail] TimestampFormatterWithTimeDiffViewsTest::testTimestampFormatterWithTimeDiff

catch’s picture

catch
he/him
Primary language English
commented
Priority: Normal » Major
Status: Closed (duplicate) » Active

I think we could keep #3400150: [random test failure] TimestampFormatterWithTimeDiffTest::testTimestampFormatterWithTimeDiff open to unskip all of these tests, but actually skip it here. Otherwise we'd need to open a new issue either to skip or unskip.

neclimdul opened merge request !5433

neclimdul’s picture

neclimdul
He/Him
commented
Status: Active » Needs review

Gotcha, MR opened

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

haha have to laugh that we get more random failures when skipping already a random.

  • catch committed efba3a64 on 10.2.x 
    Issue #3401720 by neclimdul, acbramley: [random test fail]...

  • catch committed ba666c8d on 11.x 
    Issue #3401720 by neclimdul, acbramley: [random test fail]...
catch’s picture

catch
he/him
Primary language English
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, and cherry-picked to 10.2.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

quietone’s picture

quietone commented
Parent issue: » #2829040: [meta] Known intermittent, random, and environment-specific test failures

Adding to the meta, according to How to deal with a random test failure