Fix SimpleAdderVM test cases after #17 #19
Merged
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.
Apparently, in #17 the state was change from 4 registers to 3 registers, and the omitted register was added as a
runStep()
parameter. This updates the test cases to reflect this.In addition, test case program "instructions" were fixed to be of type
bytes32
instead ofbyte
, as single-byte values were being assigned to the MSB, yielding registers encoded in big-endian format. (See description of third commit.)After these changes, running
truffle test
produces: