Skip to content

Commit

Permalink
Merge pull request bitcoin#28 from mh001/master
Browse files Browse the repository at this point in the history
POS output and coin locking
  • Loading branch information
JordanLeePeershares committed Apr 29, 2014
2 parents a5c0f27 + e1fa94d commit a9b7d4a
Showing 1 changed file with 16 additions and 7 deletions.
23 changes: 16 additions & 7 deletions testing/coincontrol.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
Test Report: Coin Control
===================================================

This report covers test results of the coin control feature of Peerunity v0.4.0 released [here](http://www.peercointalk.org/index.php?topic=2648.msg23426#msg23426). The test method used is basically clicking/typing on the GUI of Peerunity running on Peercoin testnet, and comparing the results with those specified in [_Yet another Coin Control Release_ by cozz](https://bitcointalk.org/index.php?topic=144331.0). The tests are done on windows 7 professional 32bit. Peerunity distribution peerunity-rc.zip is downloaded and unzip'ped in D:\bin
This report covers test results of the coin control feature of Peerunity v0.4.0 released [here](http://www.peercointalk.org/index.php?topic=2648.msg23426#msg23426). The tests are done on windows 7 professional 32bit. Peerunity distribution peerunity-rc.zip is downloaded and unzip'ped in D:\bin

## Basic features

The test method used is basically clicking/typing on the GUI of Peerunity running on Peercoin testnet, and comparing the results with those specified in [_Yet another Coin Control Release_ by cozz](https://bitcointalk.org/index.php?topic=144331.0).

There are three addresses with initial balance used in the test,
* test1 mkLh7wYGYQiKVm8tLwCs9LusKF2y1gffm9 8.01PPC
Expand Down Expand Up @@ -71,7 +75,7 @@ daemon=1
Note for the last item: the tooltip not only shows in list mode but also in tree mode.

* This is verified **except I do not see "lock" and "unlock"**
* This item is verified except that [coin locking is not supported](https://bitcointalk.org/index.php?topic=276606.msg2958814#msg2958814) as of v0.4.

> * Context menu
> Copy to clipboard (amount,label,address,transaction id,lock,unlock)
Expand Down Expand Up @@ -109,7 +113,7 @@ Low Output is yes when the amount is less than 0.01 PPC.
> You always must select enough outputs, so that those outputs can pay the fee.

* **I do not know how to independently calculate transaction size so test transaction size calculation is not tested.**
* **I do not know how to independently calculate transaction size so transaction size calculation is not tested.**

> **Calculation of fee and transaction size**
> The fee is calculated according to the fee set in the Settings menu.
Expand All @@ -132,14 +136,19 @@ Low Output is yes when the amount is less than 0.01 PPC.
> Also remember that violating one of the first 2 rules means 0.0005 PER kilobyte min-fee,
> while violating one of the last 2 means 0.0005 min-fee only.
## Features related to Proof-of-Stake

* Mint some POS coins and test features [added](https://bitcointalk.org/index.php?topic=276948.msg2980120#msg2980120) by YAC for POS output. Pass.

> graying-out (checkbox can't be ticked) and setting a red background of immature PoS coins in the Coin Control view.
### Summary
The coin control implmented in peerunity_coin-control passed tests of all main function items [specified by cozz](https://bitcointalk.org/index.php?topic=144331.0). Two minor items that did not pass, and two minor untested items, are marked in bold above.
## Summary
The coin control implmented in peerunity-rc passed tests of all main function items [specified by cozz](https://bitcointalk.org/index.php?topic=144331.0). One minor item that did not pass, and two minor untested items, are marked in bold above.

### History
## History
Test reprt for an early build of peerunity_coin-control is [here](http://www.peercointalk.org/index.php?topic=2699.msg23386#msg23386).

mhps
2014-04-22
2014-04-29


0 comments on commit a9b7d4a

Please sign in to comment.