Improve test coverage of the flood memory backend test and convert it to a unit test

Looks good, but I think testMemoryBackendThreshold should be moved into the new class MemoryBackendTest from FloodTest.

Thanks!

Done

Thanks!

Committed/pushed to 10.1.x, cherry-picked back through to 9.5.x, thanks!

This broke HEAD on 10.0.x and 9.5.x with:

----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
 6 | WARNING | [x] Unused use statement
   |         |     (Drupal.Classes.UnusedUseStatement.UnusedUse)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

The class is used in 10.1.x, but not 10.0.x or 9.5.x somehow. I think maybe due to changes in ✨ Add a clearByPrefix() method to the flood API Fixed that were not backported.

Marking Needs work, but if it turns out this is test coverage for something in 10.1.x only, then it would be okay not to backport this and just mark it fixed against 10.1.x.

Thanks!

Combining with @longwave's patch from 🐛 Fix unused use statement in FloodTest Closed: duplicate which should give us a clean test run against 10.0.x.

catch → credited longwave → .

RTBC, the PHPCS checks have passed on both branches.

Since I didn't actually write anything in #31, just combined two patches together, I think I'm good to commit this again, so did so to 10.0.x and 9.5.x. Thanks all.

Thanks everyone!

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