chore: Remove unused ESLint ignore directives

[Gudahtt]

Explanation

References

Relates to #6790

These changes were extracted from the draft PR #7148

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
• I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

---

Note

Removes redundant ESLint ignore comments (e.g., jest/no-conditional-in-test, related enable/disable blocks, and a stray no-var) across numerous test files without functional changes.

• Tests cleanup:
  • Strip unused ESLint disable/enable comments across many test files in packages like account-tree-controller, assets-controllers, bridge-controller, bridge-status-controller, chain-agnostic-permission, controller-utils, earn-controller, eth-json-rpc-*, json-rpc-engine, network-controller, network-enablement-controller, profile-sync-controller, sample-controllers, and shield-controller.
  • Primarily removes /* eslint-disable jest/no-conditional-in-test */ and matching enable blocks; also removes a few other unused directives (e.g., no-var).

^{Written by Cursor Bugbot for commit e38e3e7. This will update automatically on new commits. Configure here.}

[GuillaumeRx]

Nits

[Gudahtt]

Thanks @GuillaumeRx , great catch! I forgot to do a pass for obsolete comments. I've just self-reviewed to double check we got them all as well. They should all be removed now.