HomePhorge

Test deleteOldRevisions.php

This commit no longer exists in the repository. It may have been part of a branch which was deleted.

Description

Test deleteOldRevisions.php

Why:

  • Maintenance scripts in core are mostly untested and testing them will help to avoid regressions.
    • For example, in testing the script a bug is found where the Maintenance::purgeRedundantText method will throw an unexpected exception if no valid rows exist in the content_address table.

What:

  • Create DeleteOldRevisionsTest that fully tests the associated maintenance script.
  • Update Maintenance::purgeRedundantText to not attempt to use an empty list as the value provided to IReadableDatabase::getExpr.
    • Add a regression test for this.

Bug: T371167
Change-Id: Id0c3c5d6fe56bc9fe5d5347c82dab9ab61137f58

Details

Provenance
Dreamy JazzAuthored on Aug 19 2024, 9:43 AM

Event Timeline

Dreamy Jazz <dreamyjazzwikipedia@gmail.com> committed rMW06ccac8c49fa: Test deleteOldRevisions.php (authored by Dreamy Jazz <dreamyjazzwikipedia@gmail.com>).Aug 29 2024, 6:54 PM

Commit No Longer Exists

This commit no longer exists in the repository.