Allow satoshis to be a string on output creation

cryptapus · Jan 28, 2015 · 614a228 · 614a228
1 parent b7ef077
commit 614a228
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/lib/transaction/output.js b/lib/transaction/output.js
@@ -43,6 +43,9 @@ Object.defineProperty(Output.prototype, 'satoshis', {
     if (num instanceof BN) {
       this._satoshisBN = num;
       this._satoshis = num.toNumber();
+    } else if (_.isString(num)) {
+      this._satoshis = parseInt(num);
+      this._satoshisBN = BN.fromNumber(this._satoshis);
     } else {
       this._satoshisBN = BN.fromNumber(num);
       this._satoshis = num;

diff --git a/test/transaction/transaction.js b/test/transaction/transaction.js
@@ -49,6 +49,11 @@ describe('Transaction', function() {
     object.outputs[0].satoshis.should.equal(testAmount - 10000);
   });
 
+  it('can take a string argument as an amount', function() {
+    var stringTx = new Transaction().to('mrU9pEmAx26HcbKVrABvgL7AwA5fjNFoDc', '10000');
+    (stringTx._outputAmount).should.equal(10000);
+  });
+
   it('returns the fee correctly', function() {
     testTransaction.getFee().should.equal(10000);
   });