861 hip 745 integration tests

[NicolaMirchev]

Description:
Added more integration tests regarding HIP-745

Fixes #

Notes for reviewer:

Checklist

• Documented (Code comments, README, etc.)
• [x ] Tested (unit, integration, etc.)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (dc676cc) 72.19% compared to head (447ffc3) 72.24%.

❗ Current head 447ffc3 differs from pull request most recent head d22fe0f. Consider uploading reports for the commit d22fe0f to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #862      +/-   ##
===========================================
+ Coverage    72.19%   72.24%   +0.04%     
===========================================
  Files          164      164              
  Lines        26517    26517              
===========================================
+ Hits         19144    19156      +12     
+ Misses        6690     6679      -11     
+ Partials       683      682       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gsstoykov]

I think that the integration tests for example for AccountCreateTransaction should belong to the integration tests of the specific transaction not having a separate serialize_deserialize_test.go. Also do some investigation on FileAppendTransactions there should be a logic for ChunkedTransaction and add the same tests there if not all of them just demonstrate the old flow and then add a Serialize->Deserialize->Modify the transaction fields a bit and then compare.

EDIT: Didn't see that the TopicMessageSubmitTransaction was a chunked one I guess that you could add some more tests there ignore the FileAppend stuff I mentioned.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[gsstoykov]

LGTM.