fix: created btc dust amount constant and renaming

[ws4charlie]

Description

The temporary hotfix for dust btc amount is in zetaclient. The fix needs to be done in zetacore to prevent dust CCTX from registration (#2326). The PR only ported the constant and remaining.

• Renamed BtcDustOffset -> BtcNonceMarkOffset and created BTCWithdrawalDustAmount constant for zetacore usage.

Closes:

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.

• Tested CCTX in localnet
• Tested in development environment
• Go unit tests
• Go integration tests
• Tested via GitHub Actions

Checklist:

• I have added unit tests that prove my fix feature works

[github-actions]

!!!WARNING!!!
nosec detected in the following files: pkg/chains/conversion.go

Be very careful about using #nosec in code. It can be a quick way to suppress security warnings and move forward with development, it should be employed with caution. Suppressing warnings with #nosec can hide potentially serious vulnerabilities. Only use #nosec when you're absolutely certain that the security issue is either a false positive or has been mitigated in another way.

Only suppress a single rule (or a specific set of rules) within a section of code, while continuing to scan for other problems. To do this, you can list the rule(s) to be suppressed within the #nosec annotation, e.g: /* #nosec G401 */ or //#nosec G201 G202 G203
Broad #nosec annotations should be avoided, as they can hide other vulnerabilities. The CI will block you from merging this PR until you remove #nosec annotations that do not target specific rules.

Pay extra attention to the way #nosec is being used in the files listed above.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.06%. Comparing base (14bbf10) to head (c317dfb).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2327   +/-   ##
========================================
  Coverage    68.06%   68.06%           
========================================
  Files          299      299           
  Lines        18971    18971           
========================================
  Hits         12913    12913           
  Misses        5421     5421           
  Partials       637      637           

Files	Coverage Δ
pkg/chains/chains.go	100.00% <100.00%> (ø)
pkg/chains/conversion.go	100.00% <100.00%> (ø)

[lumtis]

It doesn't seem the change modify any logic in ZetaClient did we forget to do

2000+BTCWithdrawalDustAmount

?

[ws4charlie]

It doesn't seem the change modify any logic in ZetaClient did we forget to do

2000+BTCWithdrawalDustAmount

?

The 2nd commit: b40b511
The 2nd commit according to Panruo's comment: 8a34fbe

[lumtis]

Does this fix #2326?