Merge pull request #157 from Bitcoin-com/stage

New Release

Author: christroutner

.prettierrc

@@ -0,0 +1,3 @@
+{
+    "semi": false
+}

lib/Blockchain.ts

@@ -153,9 +153,7 @@ export class Blockchain {
 
     try {
       const response: AxiosResponse = await axios.get(
-        `${
-          this.restURL
-        }blockchain/getMempoolAncestors/${txid}?verbose=${verbose}`
+        `${this.restURL}blockchain/getMempoolAncestors/${txid}?verbose=${verbose}`
       )
       return response.data
     } catch (error) {
@@ -173,9 +171,7 @@ export class Blockchain {
 
     try {
       const response: AxiosResponse = await axios.get(
-        `${
-          this.restURL
-        }blockchain/getMempoolDescendants/${txid}?verbose=${verbose}`
+        `${this.restURL}blockchain/getMempoolDescendants/${txid}?verbose=${verbose}`
       )
       return response.data
     } catch (error) {
@@ -239,18 +235,27 @@ export class Blockchain {
     }
   }
 
+  // Returns details about an unspent transaction output.
   public async getTxOut(
     txid: string,
     n: any,
     include_mempool: boolean = true
   ): Promise<TxOutResult | null> {
-    // TODO confirm this works
+    // Input validation
+    if (typeof txid !== "string" || txid.length !== 64)
+      throw new Error(`txid needs to be a proper transaction ID`)
+
+    if (isNaN(n)) throw new Error(`n must be an integer`)
+
+    if (typeof include_mempool !== "boolean")
+      throw new Error(`include_mempool input must be of type boolean`)
+
     try {
-      const response: AxiosResponse = await axios.get(
-        `${
-          this.restURL
-        }blockchain/getTxOut/${txid}/n?include_mempool=${include_mempool}`
-      )
+      const path: string = `${this.restURL}blockchain/getTxOut/${txid}/${n}?include_mempool=${include_mempool}`
+      // console.log(`path: ${path}`)
+
+      const response: AxiosResponse = await axios.get(path)
+
       return response.data
     } catch (error) {
       if (error.response && error.response.data) throw error.response.data
@@ -322,9 +327,7 @@ export class Blockchain {
   ): Promise<boolean> {
     try {
       const response: AxiosResponse = await axios.get(
-        `${
-          this.restURL
-        }blockchain/verifyChain?checklevel=${checklevel}&nblocks=${nblocks}`
+        `${this.restURL}blockchain/verifyChain?checklevel=${checklevel}&nblocks=${nblocks}`
       )
       return response.data
     } catch (error) {

package.json

@@ -69,7 +69,7 @@
     "@types/wif": "^2.0.1",
     "chai": "^4.1.2",
     "coveralls": "^3.0.2",
-    "eslint": "^5.5.0",
+    "eslint": "^5.16.0",
     "eslint-config-prettier": "^3.0.1",
     "eslint-plugin-node": "7.0.1",
     "eslint-plugin-prettier": "^2.6.2",
@@ -78,7 +78,7 @@
     "node-mocks-http": "^1.7.0",
     "nyc": "^14.1.1",
     "prettier": "^1.14.2",
-    "semantic-release": "^15.13.3",
+    "semantic-release": "^15.13.31",
     "sinon": "^4.5.0",
     "source-map-support": "^0.5.12",
     "ts-node": "^8.1.0",

test/integration/address.js

@@ -130,7 +130,8 @@ describe(`#address`, () => {
         "legacyAddress",
         "cashAddress",
         "scriptPubKey",
-        "slpAddress"
+        "slpAddress",
+        "asm"
       ])
       assert.isArray(result.utxos)
       assert.hasAnyKeys(result.utxos[0], [
@@ -158,7 +159,8 @@ describe(`#address`, () => {
         "legacyAddress",
         "cashAddress",
         "scriptPubKey",
-        "slpAddress"
+        "slpAddress",
+        "asm"
       ])
       assert.isArray(result[0].utxos)
       assert.hasAnyKeys(result[0].utxos[0], [
@@ -215,7 +217,8 @@ describe(`#address`, () => {
         "legacyAddress",
         "cashAddress",
         "scriptPubKey",
-        "slpAddress"
+        "slpAddress",
+        "asm"
       ])
       assert.isArray(result.utxos)
     })
@@ -235,7 +238,8 @@ describe(`#address`, () => {
         "legacyAddress",
         "cashAddress",
         "scriptPubKey",
-        "slpAddress"
+        "slpAddress",
+        "asm"
       ])
       assert.isArray(result[0].utxos)
     })

test/integration/blockchain.js

@@ -205,13 +205,46 @@ describe(`#blockchain`, () => {
     })
   })
 
+  describe(`#getTxOut`, () => {
+    it(`should get information on valid utxo`, async () => {
+      const txid = `91874bf385a36d54f06c2154b34bce887a03b99540bfddaa17ac78ebc65202d0`
+
+      const result = await bitbox.Blockchain.getTxOut(txid, 0, true)
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
+
+      assert.hasAllKeys(result, [
+        "bestblock",
+        "confirmations",
+        "value",
+        "scriptPubKey",
+        "coinbase"
+      ])
+      assert.hasAllKeys(result.scriptPubKey, [
+        "asm",
+        "hex",
+        "reqSigs",
+        "type",
+        "addresses"
+      ])
+    })
+
+    it(`should return null for a spent utxo`, async () => {
+      const txid = `8db6dd4f8a5bb1308541d4a6d4ecdae6c65426679e79f783638ce32b2fb0725b`
+
+      const result = await bitbox.Blockchain.getTxOut(txid, 0, true)
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
+
+      assert.equal(result, null)
+    })
+  })
+
   describe(`#getTxOutProof`, () => {
     it(`should get single tx out proof`, async () => {
       const txid =
         "03f69502ca32e7927fd4f38c1d3f950bff650c1eea3d09a70e9df5a9d7f989f7"
 
       const result = await bitbox.Blockchain.getTxOutProof(txid)
-      //console.log(`result: ${JSON.stringify(result, null, 2)}`)
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
 
       assert.isString(result)
     })
@@ -223,7 +256,7 @@ describe(`#blockchain`, () => {
       ]
 
       const result = await bitbox.Blockchain.getTxOutProof(txid)
-      //console.log(`result: ${JSON.stringify(result, null, 2)}`)
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
 
       assert.isArray(result)
       assert.isString(result[0])

test/integration/other/README.md

@@ -0,0 +1,8 @@
+# Other Integration Tests
+
+This directory holds stand-alone integration tests that should not be part of
+the integration test suite.
+
+An example of a test that fits this criteria are rate limit tests. Rate limit
+tests are complex, require a solid internet connection, and can easily disrupt
+other tests. For those reasons, it is better to run them on their own. 

test/integration/transaction.js

@@ -87,7 +87,7 @@ describe(`#Transaction`, () => {
         // console.log(`result: ${util.inspect(result)}`)
         assert.equal(false, false, "Unexpected result!")
       } catch (err) {
-        console.log(`err: ${util.inspect(err)}`)
+        // console.log(`err: ${util.inspect(err)}`)
 
         assert.hasAnyKeys(err, ["error"])
         assert.include(err.error, "Array too large")

test/integration/util.js

@@ -44,7 +44,7 @@ describe(`#util`, () => {
       assert.equal(result.isvalid, false)
     })
 
-    it(`should return validate valid address`, async () => {
+    it(`should return a valid address`, async () => {
       const address = `bitcoincash:qp4k8fjtgunhdr7yq30ha4peuwupzan2vcnwrmpy0z`
 
       const result = await bitbox.Util.validateAddress(address)
@@ -54,8 +54,6 @@ describe(`#util`, () => {
         "isvalid",
         "address",
         "scriptPubKey",
-        "ismine",
-        "iswatchonly",
         "isscript"
       ])
       assert.equal(result.isvalid, true)
@@ -75,8 +73,6 @@ describe(`#util`, () => {
         "isvalid",
         "address",
         "scriptPubKey",
-        "ismine",
-        "iswatchonly",
         "isscript"
       ])
     })

test/integration/z9-rate-limits.js

@@ -8,6 +8,8 @@ import { REST_URL } from "../../lib/BITBOX"
 import * as util from "util"
 import { BlockHeaderResult } from "bitcoin-com-rest"
 
+const mockData = require("./mocks/blockchain-mock")
+
 // consts
 const bitbox: BITBOX = new BITBOX()
 const assert: Chai.AssertStatic = chai.assert
@@ -347,27 +349,61 @@ describe("#Blockchain", (): void => {
   })
 
   describe("#getTxOut", (): void => {
-    // TODO finish this test
     let sandbox: any
     beforeEach(() => (sandbox = sinon.sandbox.create()))
     afterEach(() => sandbox.restore())
     const data = {
       result: {}
     }
 
-    it("should get TODO", done => {
-      const resolved = new Promise(r => r({ data: data }))
-      sandbox.stub(axios, "get").returns(resolved)
+    it("should throw an error for improper txid.", async () => {
+      try {
+        await bitbox.Blockchain.getTxOut("badtxid", 0)
+      } catch (err) {
+        assert.include(err.message, "txid needs to be a proper transaction ID")
+      }
+    })
 
-      bitbox.Blockchain.getTxOut(
-        "daf58932cb91619304dd4cbd03c7202e89ad7d6cbd6e2209e5f64ce3b6ed7c88",
+    it("should throw an error if vout is not an integer.", async () => {
+      try {
+        await bitbox.Blockchain.getTxOut(
+          "daf58932cb91619304dd4cbd03c7202e89ad7d6cbd6e2209e5f64ce3b6ed7c88", 'a'
+        )
+      } catch (err) {
+        assert.include(err.message, "n must be an integer")
+      }
+    })
+
+    it("should get information on an unspent tx", async () => {
+      sandbox.stub(axios, "get").resolves({ data: mockData.txOutUnspent })
+
+      const result = await bitbox.Blockchain.getTxOut(
+        "62a3ea958a463a372bc0caf2c374a7f60be9c624be63a0db8db78f05809df6d8",
         0,
         true
       )
-        .then((result: any) => {
-          assert.deepEqual(data, result)
-        })
-        .then(done, done)
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
+
+      assert.hasAllKeys(result, [
+        "bestblock",
+        "confirmations",
+        "value",
+        "scriptPubKey",
+        "coinbase"
+      ])
+    })
+
+    it("should get information on a spent tx", async () => {
+      sandbox.stub(axios, "get").resolves({ data: null })
+
+      const result = await bitbox.Blockchain.getTxOut(
+        "87380e52d151856b23173d6d8a3db01b984c6b50f77ea045a5a1cf4f54497871",
+        0,
+        true
+      )
+      // console.log(`result: ${JSON.stringify(result, null, 2)}`)
+
+      assert.equal(result, null)
     })
   })