Remove unused method Group#blocked_owners

Kerri Millerrequested to merge
kerrizor/remove-unused-group-blocked-owners into master

Summary

Removes the unused method Group#blocked_owners from app/models/group.rb.

Related to #542966

Verification

This method was verified as unused by:

  • keela --type methods flagged it as unused
  • Grep search found no callers (only the def line matched anywhere in app/, lib/, ee/, views, and config)
  • No metaprogramming patterns reference it (no send, symbol dispatch, or dynamic invocation)
  • Not a Rails convention method
  • Not a callback

Cascade Removals

None - this was a standalone removal.

Metrics

Source File Complexity (flog)

Metric Before After Improvement
File flog total 1303.1 1299.2 -3.9 (0.3%)
Method flog score 3.9 N/A -3.9 removed

Spec File Complexity (flog)

Metric Before After Improvement
Spec file flog total 13423.1 13385.3 -37.8 (0.28%)
Spec block flog score 33.7 N/A -33.7 removed

Combined Flog Summary

File Flog Removed
app/models/group.rb 3.9
spec/models/group_spec.rb 33.7
Total 3.9 + 33.7 = 37.6

Test Suite Impact

  • Specs removed: 1 example
  • Spec file: spec/models/group_spec.rb
  • Time saved per run: 0.29 seconds

Estimated CI Savings

Estimate Pipelines/Month Time Saved Cost Saved/Month Cost Saved/Year
Pessimistic 15,000 1.19 hrs/month $2.15 $25.80
Moderate 30,000 2.39 hrs/month $4.30 $51.61
Optimistic 50,000 3.98 hrs/month $7.17 $86.02

Based on 0.29 seconds saved per pipeline × $0.03/minute (large runners: 3x cost factor on $10/1000 base minutes)

Files Modified

  • app/models/group.rb (method removed - 4 lines)
  • spec/models/group_spec.rb (specs removed - 17 lines, 1 example)
  • scripts/lint/keela_baseline.yml (entry removed - 1 line)

Related

  • Related to #542966 (Improve unused helper methods workflow)
  • Part of dead code cleanup effort
  • Method was listed in scripts/lint/keela_baseline.yml

Merge request reports