This repository has been archived by the owner on Nov 5, 2023. It is now read-only.
Add tests to decode error results #366
Merged
Commits on Jan 4, 2023
-
Add getOperationResults to clients to make it easier to decode operat…
…ion errors. Change existing contract test case to use getOperationResults. Add typemoq to allow typed mocking. Bump clients patch version.
-
Remove Typemoq Fix encoding error for test Remove number of events check in getOperationResults Remove other event test case
-
-
Finish test coverage of OperationResults
Add html output to nyc test coverage to more easily find gaps.
-
Address additional PR feedback
Remove unnecessary unknown casts. Split test error encoding functions into one for strings and one for abi encoded messages. Remove magic numbers/slicing to extract error action data and properly generate instead.
-
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.