[tests] Add uacomment tests
ClosedPublic
Actions

Authored by Fabien on Jan 9 2019, 11:09.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGING44a2098ca3f5: [tests] Add uacomment tests
rABC44a2098ca3f5: [tests] Add uacomment tests

Summary

Checks for setting the value, max length and reserved characters.
The backported test has been modified to match our method, not throwing
exception but truncating and sanitizing the comment.

Backport of core PR11486

Depends on D2275

Test Plan

./test/functional/test_runner.py uacomment

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR11486

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4453
Build 6970: Bitcoin ABC Buildbot (legacy)
Build 6969: arc lint + arc unit

Event Timeline

Fabien created this revision.Jan 9 2019, 11:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 9 2019, 11:09

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4453: Diff 6567.Jan 9 2019, 11:17

jasonbcox requested changes to this revision.Jan 9 2019, 17:26

jasonbcox added a subscriber: jasonbcox.

jasonbcox added inline comments.

test/functional/timing.json
3	This file isn't part of the backport. Please remove.

This revision now requires changes to proceed.Jan 9 2019, 17:26

Fabien requested review of this revision.Jan 10 2019, 12:32

Fabien added inline comments.

test/functional/timing.json
3	Even if this is not part of the original PR, I think it makes sense to have the timings file here: This file does not exist in core. You can compare to a CMakeFile.txt ; even if this is not in the original PR you need to update it. It doesn't really make the diff harder to review It is tightly coupled to the added test. If the timing file is in another diff, this diff will have no value without the test. It will avoid to forget updating the timing file :)