Bumps [rubocop-govuk](https://github.com/alphagov/rubocop-govuk) from 4.10.0 to 4.11.0.
- [Changelog](https://github.com/alphagov/rubocop-govuk/blob/main/CHANGELOG.md)
- [Commits](alphagov/rubocop-govuk@v4.10.0...v4.11.0)

---
updated-dependencies:
- dependency-name: rubocop-govuk
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

…rect

…utocorrect

Update factories/hmrc/responses in line with guidance

Address all infringing test harnesses

FactoryBot/FactoryAssociationWithStrategy, exclude spec/factories/legal_aid_applications.rb
This is because there are too many traits that create objects that loop back

RSpec/IndexedLet, set the max number of spec variables to 2

This will allow up to 2 number let variables e.g.
```ruby
    let!(:transaction1) { create(:bank_transaction, :benefits, happened_at: 10.days.ago) }
    let!(:transaction2) { create(:bank_transaction, :benefits, happened_at: 40.days.ago) }
```
But trying to create `transaction3` will raise an error

The aim is to slowly reduce the number over time to 1 as we phase out the use of large numbered blocks

We also exclude spec/services/cfe/* as the CFE classic integration is due to be deleted soon