Page MenuHomePhabricator

BookmarkBloomFilterCacheTest::testGetBloomFilterStatus_returnsUnusableStateWhenCachedPayloadHasNoState: Failed asserting that false is an instance of class "StatusValue"
Closed, ResolvedPublic

Description

Seen this failure twice in CirrusSearch CI in one day (https://integration.wikimedia.org/ci/job/quibble-with-gated-extensions-vendor-mysql-php83/56271/ & https://integration.wikimedia.org/ci/job/quibble-with-gated-extensions-vendor-mysql-php83/56341/)

11:02:09 Report from `split_group3`:
11:02:09 
11:02:09 There was 1 failure:
11:02:09 
11:02:09 1) MediaWiki\Extension\ReadingLists\Tests\Unit\Service\BookmarkBloomFilterCacheTest::testGetBloomFilterStatus_returnsUnusableStateWhenCachedPayloadHasNoState
11:02:09 Failed asserting that false is an instance of class "StatusValue".
11:02:09 
11:02:09 /workspace/src/extensions/ReadingLists/tests/phpunit/unit/Service/BookmarkBloomFilterCacheTest.php:208

Event Timeline

siebrand subscribed.

Also seen multiple times in Translate tests. Appears to be intermittent. Example: https://integration.wikimedia.org/ci/job/quibble-with-gated-extensions-vendor-mysql-php83/56679/console

13:22:10 Some split_groups failed - returning failure status
13:22:10 Summarizing parallel error logs for databaseless group...
13:22:10 
13:22:10 
13:22:10 Report from `split_group3`:
13:22:10 
13:22:10 There was 1 failure:
13:22:10 
13:22:10 1) MediaWiki\Extension\ReadingLists\Tests\Unit\Service\BookmarkBloomFilterCacheTest::testGetBloomFilterStatus_returnsUnusableStateWhenCachedPayloadHasNoState
13:22:10 Failed asserting that false is an instance of class "StatusValue".
13:22:10 
13:22:10 /workspace/src/extensions/ReadingLists/tests/phpunit/unit/Service/BookmarkBloomFilterCacheTest.php:208

The test in question is reading from a cache, so the flakiness is clearly due to sometimes having cache hits (the test passes), sometimes having cache misses (it fails).

A wild guess: maybe different test cases use the same cache, and depending on the timing, one test case may erase the cache entry that the other one is expecting to see?

Change #1341321 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/ReadingLists@master] tests: Fix flaky BookmarkBloomFilterCacheTest

https://gerrit.wikimedia.org/r/1341321

Change #1341321 merged by jenkins-bot:

[mediawiki/extensions/ReadingLists@master] tests: Fix flaky BookmarkBloomFilterCacheTest

https://gerrit.wikimedia.org/r/1341321