Refactored existing monolith_opcodes tests
ClosedPublic
Actions

Authored by jasonbcox on Mar 29 2018, 03:41.

Details

Reviewers

deadalnix
schancel

Group Reviewers

Restricted Project

Commits

rSTAGING5857e829f528: Refactored existing monolith_opcodes tests
rABC5857e829f528: Refactored existing monolith_opcodes tests

Summary

These changes are needed in order to merge D1212 and D1097

Co-authored-by: Joshua Yabut <yabut.joshua@gmail.com>
Co-authored-by: Marcos Mayorga <mm@mm-studios.com>

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2197
Build 2538: Bitcoin ABC Buildbot (legacy)
Build 2537: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Mar 29 2018, 03:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 29 2018, 03:41

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging started.

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging passed.

Harbormaster completed remote builds in B2197: Diff 3330.Mar 29 2018, 04:04

jasonbcox added a child revision: D1212: Add support for DIV and MOD opcodes.Mar 29 2018, 06:47

deadalnix accepted this revision.Mar 29 2018, 19:35

deadalnix added inline comments.

src/test/monolith_opcodes.cpp
81	This comment seems unnecessary.

This revision is now accepted and ready to land.Mar 29 2018, 19:35

Closed by commit rABC5857e829f528: Refactored existing monolith_opcodes tests (authored by jasonbcox). · Explain WhyMar 29 2018, 19:43

This revision was automatically updated to reflect the committed changes.

jasonbcox marked an inline comment as done.

Revision Contents
Changeset List

Path

Size

src/

test/

monolith_opcodes.cpp

85 lines

Diff 3330

View Options

src/test/monolith_opcodes.cpp

Refactored existing monolith_opcodes testsClosedPublicActions