chore(test): consolidate hlint snake_case exceptions across codebase

[newhoggy]

Changelog

- description: |
    Consolidates hlint configuration by moving per-file snake_case exceptions
    into a centralized configuration rule, improving maintainability and
    reducing code duplication across test modules.
  type:
  - maintenance    # not directly related to the code
  projects:
  - cardano-api
  - cardano-api-gen

Context

This PR addresses code maintainability by consolidating duplicate hlint pragma comments that were scattered across 12 test files. Previously, each test file contained its own {- HLINT ignore "Use camelCase" -} pragma to allow snake_case naming conventions commonly used in Haskell test properties.

The change centralizes this configuration into the root .hlint.yaml file, applying the exception globally to all test modules matching the patterns Test.Cardano.Api.** and Test.Golden.Cardano.Api.**. This approach:

• Reduces code duplication and maintenance burden
• Ensures consistent linting rules across all test modules
• Makes it easier to manage hlint exceptions in one place
• Follows established Haskell testing conventions where snake_case is commonly used for property names

This allows #515 to be closed

How to trust this PR

This is a purely mechanical change that affects only hlint configuration and comments. The actual test code remains unchanged.

Verification steps:

1. Run hlint . to verify no new warnings are introduced
2. Confirm all tests still pass: cabal test all
3. Review the diff to see that only hlint pragmas were removed from test files
4. Check that the new .hlint.yaml rule correctly applies to the affected modules

Files changed:

• .hlint.yaml: Added global rule to ignore "Use camelCase" warnings for test modules and "Use + directly" warning
• 12 test files: Removed duplicate {- HLINT ignore "Use camelCase" -} pragmas from:
  • cardano-api/gen/Test/Hedgehog/Roundtrip/CBOR.hs
  • cardano-api/test/cardano-api-golden/Test/Golden/Cardano/Api/Script.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Address.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/CBOR.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Cip129.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Envelope.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Json.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/KeysByron.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Ord.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/RawBytes.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/Transaction/Autobalance.hs
  • cardano-api/test/cardano-api-test/Test/Cardano/Api/TxBody.hs

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• New tests are added if needed and existing tests are updated. See Running tests for more details
• Self-reviewed the diff

[carbolymer]

Thanks. This only addresses HLINT ignore "Use camelCase" rule. Do you think we could also remove other ones as well?

[newhoggy]

I wasn't sure with the other lint warnings if they occur frequently enough to warrant global suppression.

I'll get a list and the affected code and let you decide.

[newhoggy]

Actually, I had a look at the pragmas and they seemed sensible to suppress so I did that.