hyperledger / caliper

A blockchain benchmark framework to measure performance of multiple blockchain solutions https://wiki.hyperledger.org/display/caliper
https://hyperledger.github.io/caliper/
Apache License 2.0
642 stars 403 forks source link

`caliper-core`: add tests for `Message` classes #1561

Closed EshaanAgg closed 1 month ago

EshaanAgg commented 2 months ago

This PR introduces the tests for the base Message class in caliper-core, and then introduces three similar test suites for the constructors of three types of messages. When approved, I can work on adding similar tests for all the message classes.

Checklist

Issue/User story

Fixes part of #1557

Steps to Reproduce

1. 2. 3. 4.

Existing issues

Design of the fix

Validation of the fix

Automated Tests

What documentation has been provided for this pull request

davidkel commented 2 months ago

@aklenik Can you take a look

davidkel commented 2 months ago

On a quick glance, these seem ok to me, but really want @aklenik to take a look as he is more familiar with this part of the code I think