put n shelley/byron key witnesses arguments in correct order

[brendanrbrown]

Changelog

- description: |
    Puts the `nShelleyKeyWitnesses`, `nByronKeyWitnesses` arguments to `runCalculateMinFeeCmd` in the correct order.
  type:
    - bugfix  

Context

The runCalculateMinFeeCmd swaps the number of Byron and Shelley witnesses used to calculate fees, in its call to estimateTransactionFee (see here in the function's body) of cardano-api-8.30.0.0.

The witness numbers get multiplied by a factor that differs for the Byron and Shelley eras, and the Byron multiplier is larger than the Shelley number as defined here.

So the min fee printed by runCalculateMinFeeCmd will be larger than it should be if the number of Shelley witnesses exceeds the number Byron witnesses provided.

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]

Good catch! LGTM 👍

[brendanrbrown]

Great, thanks. FYI I don't have merge privileges, so one of you will have to push the button.

[smelc]

FYI @Jimbo4350