[Fix #9622] Fixed Style/BisectedAttrAccessor autocorrection to handle multiple bisected attrs in the same macro

[dvandersluis]

This ended up being a pretty heavy refactor of the Style/BisectedAttrAccessor cop.

Previously we were doing multiple passes at determining what macros are relevant for the cop, and which names within them. Despite that, if a macro had multiple names and was rewritten, we'd lose context of what other names there were and only the first one would survive the autocorrection.

Instead, I have now extracted a Macro value object that encapsulates all the data for the macro, including visibility (since macros of different visibilities are not combined), bisected and non-bisected attribute names, etc. When investigating a class, any macros found have objects created, and they are retained to be used to correct the class as a whole after wards (but we still register offenses on individual macro attributes as before).

I have also added some more edge case tests that I encountered as I was working on this.

Fixes #9622.

---

Before submitting the PR make sure the following are checked:

• The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
• Wrote good commit messages.
• Commit message starts with [Fix #issue-number] (if the related issue exists).
• Feature branch is up-to-date with master (if not - rebase it).
• Squashed related commits together.
• Added tests.
• Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
• Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

[koic]

Great work! Can you add a changelog entry?

[dvandersluis]

@koic 🤦🏻‍♂️ I knew I forgot something. Thanks!

[bbatsov]

Great work! 🏅