Throw an error on identical action-ids in votes of one voter

[smelc]

Changelog

- description: |
    transaction build{,-raw}: throw an error on identical action-ids in votes files
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - improvement    # QoL changes e.g. refactoring
  - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

Fixes #662

How to trust this PR

• Run the new test before the main commit: cabal test cardano-cli-test --test-options '-p "/conway transaction build one voter many votes/"'
• Witness the test fails because the CLI returns ExitSuccess
• Run the same test after the main commit, witness the test passes: the CLI now fails.

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

[smelc]

Question to reviewers:

• Should I move mergeVotingProcedures to API and remove cardano-api's unsafeMergeVotingProcedures?

[carbolymer]

Question to reviewers:

• Should I move mergeVotingProcedures to API and remove cardano-api's unsafeMergeVotingProcedures?

Yes. Can be done in a separate PR.

[smelc]

PR to move mergeVotingProcedures to cardano-api is at IntersectMBO/cardano-api#498

[Jimbo4350]

👍

[Jimbo4350]

We could avoid the existential by setting the crypto to StandardCrypto

[Jimbo4350]

We should be able to us differenceWith here.