Problem/Motivation

The flood_control module currently has ~25% test coverage, focusing mainly on basic installation, login blocking, and settings form happy paths. Critical functionality lacks any test coverage:

  • FloodWhiteList service (IP whitelist decorator) - security-critical, 0% coverage
  • FloodUnblockManagerDatabase (database operations) - 0% coverage
  • FloodUnblockManagerBase (isBlocked(), fetchIdentifiers()) - 0% coverage
  • FloodUnblockAdminForm (filtering, item removal) - 0% coverage
  • FloodControlSettingsForm (IP validation logic) - 0% coverage
  • FloodUnblockCommands (Drush commands) - 0% coverage

Proposed Resolution

Add comprehensive PHPUnit tests to achieve more coverage.

Issue fork flood_control-3564198

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

batigolix created an issue. See original summary.

batigolix opened merge request !61

batigolix’s picture

batigolix
Primary language Dutch
Location Utrecht
commented
Issue summary: View changes
batigolix’s picture

batigolix
Primary language Dutch
Location Utrecht
commented
Issue summary: View changes
batigolix’s picture

batigolix
Primary language Dutch
Location Utrecht
commented
Priority: Normal » Major