bitcoin-ruby 0.0.5 → 0.0.6

data/spec/bitcoin/storage/reorg_spec.rb

@@ -5,9 +5,7 @@ require_relative '../spec_helper'
 include Bitcoin::Builder
 
 Bitcoin.network = :testnet
-
-Bitcoin::Validation::Block::RETARGET = 10
-
+    
 [
  [:utxo, :sqlite, index_all_addrs: true],
  [:sequel, :sqlite], # [:sequel, :postgres],
@@ -28,10 +26,15 @@ Bitcoin::Validation::Block::RETARGET = 10
     @store = storage
     @store.reset
     def @store.in_sync?; true; end
+    
+    Bitcoin.network = :testnet
+    Bitcoin.network[:retarget_interval] = 10
     Bitcoin.network[:proof_of_work_limit] = Bitcoin.encode_compact_bits("ff"*32)
+    
     @key = Bitcoin::Key.generate
     @block0 = create_block "00"*32, false, [], @key
     Bitcoin.network[:genesis_hash] = @block0.hash
+    
     @store.store_block(@block0)
     @store.get_head.should == @block0
   end
@@ -99,6 +102,39 @@ Bitcoin::Validation::Block::RETARGET = 10
     @store.get_head.hash.should == block_b.hash
   end
 
+  it "should validate duplicate tx in a side chain" do
+    block1 = create_block @block0.hash, true, [], @key
+
+    block_2_0 = create_block block1.hash, false, [ ->(t) {
+        t.input {|i| i.prev_out block1.tx[0], 0; i.signature_key @key }
+        t.output {|o| o.value 50e8; o.script {|s| s.recipient @key.addr } }
+      }], @key
+
+    @store.store_block(block_2_0).should == [2, 0]
+
+    block_3_0 = create_block block_2_0.hash, false, [->(t) {
+        t.input {|i| i.prev_out block_2_0.tx[1], 0; i.signature_key @key }
+        t.output {|o| o.value 50e8; o.script {|s| s.recipient @key.addr } }
+       }], @key
+    @store.store_block(block_3_0).should == [3, 0]
+
+    block_2_1 = create_block block1.hash, false
+    block_2_1.tx << block_2_0.tx[1]
+    block_2_1.recalc_mrkl_root
+    block_2_1.recalc_block_hash
+    @store.store_block(block_2_1).should == [2, 1]
+
+    block_3_1 = create_block block_2_1.hash, false
+    block_3_1.tx << block_3_0.tx[1]
+    block_3_1.recalc_mrkl_root
+    block_3_1.recalc_block_hash
+
+    @store.store_block(block_3_1).should == [3, 1]
+
+    block_4 = create_block block_3_1.hash, false
+    @store.store_block(block_4).should == [4, 0]
+  end
+
   it "should reorg a single side block" do
     @store.get_head.should == @block0
 
@@ -163,7 +199,6 @@ Bitcoin::Validation::Block::RETARGET = 10
   end
 
   it "should handle existing blocks" do
-    Bitcoin.network = :testnet
     blocks = [@block0]
     3.times { blocks << create_block(blocks.last.hash, false) }
     blocks[1..-1].each.with_index {|b, idx| @store.store_block(b).should == [idx+1, 0] }
@@ -173,8 +208,9 @@ Bitcoin::Validation::Block::RETARGET = 10
 
   # see https://bitcointalk.org/index.php?topic=46370.0
   it "should pass reorg unit tests" do
-    class Bitcoin::Validation::Block; def difficulty; true; end; end
     Bitcoin.network = :bitcoin
+    # Disable difficulty check
+    Bitcoin.network[:no_difficulty] = true
     @store.import "./spec/bitcoin/fixtures/reorg/blk_0_to_4.dat"
     @store.get_depth.should == 4
     @store.get_head.hash.should =~ /000000002f264d65040/
@@ -192,13 +228,8 @@ Bitcoin::Validation::Block::RETARGET = 10
     balance("1NiEGXeURREqqMjCvjCeZn6SwEBZ9AdVet").should == 1000000000
     balance("1KXFNhNtrRMfgbdiQeuJqnfD7dR4PhniyJ").should == 0
     balance("1JyMKvPHkrCQd8jQrqTR1rBsAd1VpRhTiE").should == 14000000000
+    Bitcoin.network.delete :no_difficulty
     Bitcoin.network = :testnet
-    class Bitcoin::Validation::Block
-      def difficulty
-        return true  if Bitcoin.network_name == :testnet3
-        block.bits == next_bits_required || [block.bits, next_bits_required]
-      end
-    end
   end
 
 end

data/spec/bitcoin/storage/storage_spec.rb

@@ -24,7 +24,7 @@ Bitcoin::network = :testnet
   describe "Storage::Backends::#{options[0].to_s.capitalize}Store (#{options[1]})" do
 
     before do
-      class Bitcoin::Validation::Block; def difficulty; true; end; end
+      Bitcoin.network[:no_difficulty] = true
       Bitcoin.network[:proof_of_work_limit] = Bitcoin.encode_compact_bits("ff"*32)
 
       @store = storage
@@ -46,12 +46,7 @@ Bitcoin::network = :testnet
     end
 
     after do
-      class Bitcoin::Validation::Block
-        def difficulty
-          return true  if Bitcoin.network_name == :testnet3
-          block.bits == next_bits_required || [block.bits, next_bits_required]
-        end
-      end
+      Bitcoin.network.delete :no_difficulty
     end
 
     it "should get backend name" do
@@ -154,14 +149,19 @@ Bitcoin::network = :testnet
       @store.get_block_by_tx(@blk.tx[0].hash).should == @blk
     end
 
-    # TODO calling .is_a? on @store breaks it (?!?)
-    if @store.class.name =~ /SequelStore/
+    it "should get block id for tx id" do
+      @store.store_block(@blk)
+      tx = @store.get_tx(@blk.tx[0].hash)
+      @store.get_block_id_for_tx_id(tx.id).should == @store.get_block(@blk.hash).id
+    end
+
+    unless @store.backend_name == 'utxo'
       describe :transactions do
 
         it "should store tx" do
           @store.store_tx(@tx, false).should != false
         end
-        
+
         it "should not store tx if already stored and return existing id" do
           id = @store.store_tx(@tx, false)
           @store.store_tx(@tx, false).should == id
@@ -221,13 +221,14 @@ Bitcoin::network = :testnet
         end
 
         it "should store multisig tx and index hash160's" do
-          *keys = Bitcoin::Key.generate, Bitcoin::Key.generate
+          keys = Array.new(2) { Bitcoin::Key.generate }
           pk_script = Bitcoin::Script.to_multisig_script(1, keys[0].pub, keys[1].pub)
           txout = P::TxOut.new(1000, pk_script)
-          @store.store_txout(0, txout, 0)
+          @tx.out[0] = txout
+          @store.store_tx(@tx, false)
           keys.each do |key|
             hash160 = Bitcoin.hash160(key.pub)
-            txouts = @store.get_txouts_for_hash160(hash160, true)
+            txouts = @store.get_txouts_for_hash160(hash160, :hash160, true)
             txouts.size.should == 1
             txouts[0].pk_script.should == txout.pk_script
           end
@@ -264,7 +265,7 @@ Bitcoin::network = :testnet
       end
 
       it "should get txouts for hash160" do
-        @store.get_txouts_for_hash160(@key2.hash160, true)
+        @store.get_txouts_for_hash160(@key2.hash160, :hash160, true)
           .should == [@block.tx[1].out[0]]
       end
 
@@ -273,11 +274,53 @@ Bitcoin::network = :testnet
           .should == [@block.tx[1].out[0]]
       end
 
+      it "should get txouts for txin" do
+        prev_tx = @block.tx[0]
+        tx = build_tx { |t| create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, Bitcoin::Key.generate]], @key) }
+        @store.get_txout_for_txin(tx.in[0]).should == prev_tx.out[0]
+      end
+
+      it "should get unspent txouts for address" do
+        @store.get_unspent_txouts_for_address(@key2.addr, true)
+          .should == [@block.tx[1].out[0]]
+        @block2 = create_block @block.hash, true, [->(t) {
+           create_tx(t, @block.tx[1], 0, [[20, @key2]], @key2) }], @key
+        @store.get_unspent_txouts_for_address(@key2.addr, true)
+          .should == [@block2.tx[1].out[0]]
+      end
+
       it "should get balance for address" do
-        @store.get_balance(@key.addr).should == 0
+        @store.get_balance(@key2.addr).should == 50
         @store.get_balance(@key2.hash160).should == 50
       end
 
+      it "should get txouts for p2sh address (and not confuse regular and p2sh-type hash160" do
+        block = create_block(@block.hash, false, [], @key)
+
+        tx = build_tx do |t|
+          t.input {|i| i.prev_out(@block.tx[1], 0); i.signature_key(@key2) }
+          t.output do |o|
+            o.value 10
+            o.to @key2.hash160, :p2sh
+          end
+          t.output do |o|
+            o.value 10
+            o.to @key2.addr
+          end
+        end
+        block.tx << tx; block.recalc_mrkl_root; block.recalc_block_hash
+
+        @store.store_block(block)
+
+        p2sh_address = Bitcoin.hash160_to_p2sh_address(@key2.hash160)
+        o1 = @store.get_unspent_txouts_for_address(@key2.addr)
+        o2 = @store.get_unspent_txouts_for_address(p2sh_address)
+
+        o1.size.should == 1
+        o2.size.should == 1
+        o1.should != o2
+      end
+
     end
 
     describe "validation" do

data/spec/bitcoin/storage/validation_spec.rb

@@ -67,14 +67,16 @@ Bitcoin.network = :spec
   it "4. Block hash must satisfy claimed nBits proof of work" do
     @block.bits = Bitcoin.encode_compact_bits("0000#{"ff" * 30}")
     @block.recalc_block_hash
-    target = Bitcoin.decode_compact_bits(block.bits).to_i(16)
+    target = Bitcoin.decode_compact_bits(@block.bits).to_i(16)
     check_block(@block, [:bits, [@block.hash.to_i(16), target]])
   end
 
   it "5. Block timestamp must not be more than two hours in the future" do
-    fake_time = (Time.now + 3 * 60 * 60).to_i
-    check_block(@block, [:max_timestamp, [fake_time, Time.now.to_i + 7200]]) {|b|
-      b.time = fake_time }
+    Time.freeze do
+      fake_time = (Time.now + 3 * 60 * 60).to_i
+      check_block(@block, [:max_timestamp, [fake_time, Time.now.to_i + 7200]]) {|b|
+        b.time = fake_time }
+    end
   end
 
   it "6. First transaction must be coinbase (i.e. only 1 input, with hash=0, n=-1), the rest must not be" do
@@ -108,20 +110,27 @@ Bitcoin.network = :spec
     prev_block = @block1
     12.times do |i|
       prev_block = create_block(prev_block.hash, false, [])
-      prev_block.time = Time.now.to_i - (12-i)
+      prev_block.time = i
       prev_block.recalc_block_hash
       @store.store_block(prev_block).should == [i+2, 0]
     end
+
     block = create_block(prev_block.hash, false, [], @key)
 
-    fake_time = Time.now.to_i - 100
+    fake_time = @store.get_block_by_depth(8).time - 1
     times = @store.db[:blk].where("depth > 2").map{|b|b[:time]}.sort
     m, r = times.size.divmod(2)
     min_time = (r == 0 ? times[m-1, 2].inject(:+) / 2.0 : times[m])
+
+    # reject before median time
     check_block(block, [:min_timestamp, [fake_time, min_time]]) {|b| b.time = fake_time }
 
+    # reject at exactly median time
     fake_time = @store.get_block_by_depth(8).time
     check_block(block, [:min_timestamp, [fake_time, fake_time]]) {|b| b.time = fake_time }
+
+    # accept after median time
+    block.time = @store.get_block_by_depth(8).time + 1; block.recalc_block_hash
     @store.store_block(block).should == [14, 0]
   end
 
@@ -145,13 +154,13 @@ Bitcoin.network = :spec
     block3 = create_block(block2.hash, false, [], @key, 60e8)
     -> { @store.store_block(block3) }.should.raise(ValidationError)
 
-    Bitcoin::Validation::REWARD_DROP = 2
+    Bitcoin::REWARD_DROP = 2
     block4 = create_block(block2.hash, false, [], @key, 50e8)
     -> { @store.store_block(block4) }.should.raise(ValidationError)
 
     block5 = create_block(block2.hash, false, [], @key, 25e8)
     @store.store_block(block5).should == [3, 0]
-    Bitcoin::Validation::REWARD_DROP = 210_000
+    Bitcoin::REWARD_DROP = 210_000
   end
 
 end
@@ -207,10 +216,10 @@ describe "transaction rules (#{options[0]} - #{options[1]})" do
   end
 
   it "3. Size in bytes < MAX_BLOCK_SIZE" do
-    max = Bitcoin::Validation::MAX_BLOCK_SIZE; Bitcoin::Validation::MAX_BLOCK_SIZE = 1000
+    max = Bitcoin::MAX_BLOCK_SIZE; Bitcoin::MAX_BLOCK_SIZE = 1000
     check_tx(@tx, [:max_size, [@tx.payload.bytesize+978, 1000]]) {|tx|
       tx.out[0].pk_script = "\x00" * 1001 }
-    Bitcoin::Validation::MAX_BLOCK_SIZE = max
+    Bitcoin::MAX_BLOCK_SIZE = max
   end
 
   it "4. Each output value, as well as the total, must be in legal money range" do
@@ -225,8 +234,8 @@ describe "transaction rules (#{options[0]} - #{options[1]})" do
     end
   end
 
-  it "6. Check that nLockTime <= INT_MAX, size in bytes >= 100, and sig opcount <= 2" do
-    check_tx(@tx, [:lock_time, [INT_MAX + 1, INT_MAX]]) {|tx| tx.lock_time = INT_MAX + 1 }
+  it "6. Check that nLockTime <= UINT32_MAX, size in bytes >= 100, and sig opcount <= 2" do
+    check_tx(@tx, [:lock_time, [Bitcoin::UINT32_MAX + 1, Bitcoin::UINT32_MAX]]) {|tx| tx.lock_time = Bitcoin::UINT32_MAX + 1 }
     # TODO: validate sig opcount
   end
 
@@ -242,7 +251,7 @@ describe "transaction rules (#{options[0]} - #{options[1]})" do
   end
 
   it "13. Verify crypto signatures for each input; reject if any are bad" do
-    check_tx(@tx, [:signatures, [0]]) {|tx| @tx.in[0].script_sig[-1] = "\x00" }
+    check_tx(@tx, [:signatures, [0]]) {|tx| @tx.in[0].script_sig = "bad sig" }
   end
 
   it "14. For each input, if the referenced output has already been spent by a transaction in the main branch, reject this transaction" do
@@ -251,7 +260,7 @@ describe "transaction rules (#{options[0]} - #{options[1]})" do
     if @store.class.name =~ /Utxo/
       check_tx(@tx, [:prev_out, [[@tx.in[0].prev_out.reverse_hth, 0]]])
     else
-      check_tx(@tx, [:spent, [0]])
+      check_tx(@tx, [:not_spent, [0]])
     end
   end
   
@@ -265,6 +274,27 @@ describe "transaction rules (#{options[0]} - #{options[1]})" do
     check_tx(tx, [:output_sum, [100e8, 50e8]])
   end
 
+
+  it "should not allow double spend within the same block" do
+    # double-spend output from previous block
+    prev_tx = @block1.tx[0]
+    block = create_block @block1.hash, false, [
+     ->(t) { create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, @key]]) },
+     ->(t) { create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, @key]]) }
+    ]
+    -> { @store.store_block(block) }.should.raise(Bitcoin::Validation::ValidationError)
+
+    # double-spend output from current block
+    block = create_block @block1.hash, false, [
+      ->(t) { create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, @key]]) }
+    ]
+    prev_tx = block.tx[1]
+    block.tx << build_tx {|t| create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, @key]]) }
+    block.tx << build_tx {|t| create_tx(t, prev_tx, 0, [[prev_tx.out[0].value, @key]]) }
+    block.recalc_mrkl_root; block.recalc_block_hash
+    -> { @store.store_block(block) }.should.raise(Bitcoin::Validation::ValidationError)
+  end
+
 end
 
 end

data/spec/bitcoin/wallet/keygenerator_spec.rb

@@ -28,7 +28,10 @@ describe "Bitcoin::KeyGenerator" do
   it "should use given nonce" do
     g = KeyGenerator.new("foo", 2116)
     g.nonce.should == 2116
-    g.get_key(0).addr.should == '1GjyUrY3XcR4BvfgL8HqoAJbNDEgxSJdm1'
+    key = g.get_key(0)
+    key.addr.should == '1JvRdnShvscPtoP44VxPk5VaFBAo7ozRPb'
+    key.instance_eval { @pubkey_compressed = false }
+    key.addr.should == '1GjyUrY3XcR4BvfgL8HqoAJbNDEgxSJdm1'
   end
 
   it "should check nonce if given" do
@@ -38,10 +41,10 @@ describe "Bitcoin::KeyGenerator" do
   it "should use different target if given" do
     g = KeyGenerator.new("foo", nil, @target)
     g.nonce.should == 127
-    g.get_key(0).addr.should == "13E68pPJyGycgQ4ZmV45xV9r9XEeyWqZdp"
+    g.get_key(0).addr.should == "1KLBACvBnz9BTdBnuJmNuQpKQrsi55sstj"
     g = KeyGenerator.new("bar", nil, @target)
     g.nonce.should == 40
-    g.get_key(0).addr.should == "12iQpWRRQBmWcHYkTZkpDFrykzc9xn5kAU"
+    g.get_key(0).addr.should == "14T4deW5BGVA7wXpR3eoU9U8xprUJepxcy"
   end
 
   it "should find keys" do

data/spec/bitcoin/wallet/keystore_spec.rb

@@ -173,7 +173,7 @@ describe "Bitcoin::Wallet::DeterministicKeyStore" do
   end
 
   it "should get key" do
-    @ks.key('1KDUUSjPJkKwVEJsfpxEzBAf7iEbmqUwUu').priv.should ==
+    @ks.key('1GKjKQemNRhxL1ChTRFJNLZCXeCDxut2d7').priv.should ==
       '7f27bb0ca02e558c4b4b4e267417437adac01403e0d0bb9b07797d1dbb1adfd1'
   end
 
@@ -183,8 +183,8 @@ describe "Bitcoin::Wallet::DeterministicKeyStore" do
   end
 
   it "should export key" do
-    @ks.export('1KDUUSjPJkKwVEJsfpxEzBAf7iEbmqUwUu').should ==
-      '5JnHbCHicVj2Wgd2KgNPU7dQ6te55GzHjc4PH9cQDFUjeepYSHX'
+    @ks.export('1GKjKQemNRhxL1ChTRFJNLZCXeCDxut2d7').should ==
+      'L1UtDvpnffnVg1szqSmQAgFexzvcysZrs3jwLH1FT4uREpZqcXaR'
   end
 
 end

data/spec/bitcoin/wallet/wallet_spec.rb

@@ -46,14 +46,12 @@ describe Bitcoin::Wallet::Wallet do
     @storage = Mock.new
     @key = Key.from_base58('5J2hn1E8KEXmQn5gqykzKotyCcHbKrVbe8fjegsfVXRdv6wwon8')
     @addr = '1M89ZeWtmZmATzE3b6PHTBi8c7tGsg5xpo'
-    @key2 = Key.from_base58('5KK9Lw8gtNd4kcaXQJmkwcmNy8Y5rLGm49RqhcYAb7qRhWxaWMJ')
-    @addr2 = '134A4Bi8jN5V2KjkwmXUHjokDqdyqZ778J'
-    @key3 = Key.from_base58('5JFcJByQvwYnWjQ2RHTTu6LLGiBj9oPQYsHqKWuKLDVAvv4cQ7E')
-    @addr3 = '1EnrPVaRiRgrs1D7pujYZNN1N6iD9unZV6'
+    #@key2 = Key.from_base58('5KK9Lw8gtNd4kcaXQJmkwcmNy8Y5rLGm49RqhcYAb7qRhWxaWMJ')
+    #@addr2 = '134A4Bi8jN5V2KjkwmXUHjokDqdyqZ778J'
+    #@key3 = Key.from_base58('5JFcJByQvwYnWjQ2RHTTu6LLGiBj9oPQYsHqKWuKLDVAvv4cQ7E')
+    #@addr3 = '1EnrPVaRiRgrs1D7pujYZNN1N6iD9unZV6'
 
     @storage.expect(:add_watched_address, [], [@addr])
-    @storage.expect(:add_watched_address, [], [@addr2])
-    @storage.expect(:add_watched_address, [], [@addr3])
 
     keystore_data = [{:addr => @key.addr, :priv => @key.priv, :pub => @key.pub}]
     file_stub = StringIO.new
@@ -100,110 +98,110 @@ describe Bitcoin::Wallet::Wallet do
     list[1].should == 5000
   end
 
-  # it "should create new addr" do
-  #   @wallet.addrs.size.should == 1
+  it "should create new addr" do
+    @wallet.addrs.size.should == 1
 
-  #   key = Key.generate
-  #   a = @wallet.get_new_addr
-  #   @wallet.addrs.size.should == 2
-  #   @wallet.addrs[1].should == a
-  # end
+    @storage.expect(:add_watched_address, [], [String])
+    a = @wallet.get_new_addr
+    @wallet.addrs.size.should == 2
+    @wallet.addrs[1].should == a
+  end
 
-  describe "Bitcoin::Wallet::Wallet#tx" do
-
-    before do
-      txout = txout_mock(5000, nil)
-      tx = Mock.new
-      2.times { tx.expect(:binary_hash, "foo") }
-      8.times { tx.expect(:out, [txout]) }
-      3.times { tx.expect(:get_block, true) }
-      5.times { txout.expect(:get_tx, tx) }
-      6.times { txout.expect(:get_address, @addr) }
-      8.times { txout.expect(:pk_script, Script.to_address_script(@addr)) }
-      2.times { @storage.expect(:get_txouts_for_address, [txout], [@addr]) }
-      2.times { @storage.expect(:class, Bitcoin::Storage::Backends::SequelStore, []) }
-      selector = Bitcoin::Wallet::SimpleCoinSelector.new([txout])
-      2.times { @selector.expect(:new, selector, [[txout]]) }
-      @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]])
-    end
+  # describe "Bitcoin::Wallet::Wallet#tx" do
 
+  #   before do
+  #     txout = txout_mock(5000, nil)
+  #     tx = Mock.new
+  #     2.times { tx.expect(:binary_hash, "foo") }
+  #     8.times { tx.expect(:out, [txout]) }
+  #     3.times { tx.expect(:get_block, true) }
+  #     5.times { txout.expect(:get_tx, tx) }
+  #     6.times { txout.expect(:get_address, @addr) }
+  #     8.times { txout.expect(:pk_script, Script.to_address_script(@addr)) }
+  #     2.times { @storage.expect(:get_txouts_for_address, [txout], [@addr]) }
+  #     2.times { @storage.expect(:class, Bitcoin::Storage::Backends::SequelStore, []) }
+  #     selector = Bitcoin::Wallet::SimpleCoinSelector.new([txout])
+  #     2.times { @selector.expect(:new, selector, [[txout]]) }
+  #     @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]])
+  #   end
 
-    it "should have hash" do
-      @tx.hash.size.should == 64
-    end
 
-    it "should have correct inputs" do
-      @tx.in.size.should == 1
-      @tx.in.first.prev_out.should == ("foo" + "\x00"*29)
-      @tx.in.first.prev_out_index.should == 0
-    end
+  #   it "should have hash" do
+  #     @tx.hash.size.should == 64
+  #   end
 
-    it "should have correct outputs" do
-      @tx.out.size.should == 2
-      @tx.out.first.value.should == 1000
-      s = Script.new(@tx.out.first.pk_script)
-      s.get_address.should == '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7'
-    end
+  #   it "should have correct inputs" do
+  #     @tx.in.size.should == 1
+  #     @tx.in.first.prev_out.should == ("foo" + "\x00"*29)
+  #     @tx.in.first.prev_out_index.should == 0
+  #   end
 
-    it "should have change output" do
-      @tx.out.last.value.should == 4000
-      s = Script.new(@tx.out.last.pk_script)
-      s.get_address.should == @addr
-    end
+  #   it "should have correct outputs" do
+  #     @tx.out.size.should == 2
+  #     @tx.out.first.value.should == 1000
+  #     s = Script.new(@tx.out.first.pk_script)
+  #     s.get_address.should == '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7'
+  #   end
 
-    it "should leave tx fee" do
-      @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50)
-      @tx.out.last.value.should == 3950
-    end
+  #   it "should have change output" do
+  #     @tx.out.last.value.should == 4000
+  #     s = Script.new(@tx.out.last.pk_script)
+  #     s.get_address.should == @addr
+  #   end
 
-    it "should send change to specified address" do
-      @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50,
-        '1EAntvSjkNeaJJTBQeQcN1ieU2mYf4wU9p')
-      Script.new(@tx.out.last.pk_script).get_address.should ==
-        '1EAntvSjkNeaJJTBQeQcN1ieU2mYf4wU9p'
-    end
+  #   it "should leave tx fee" do
+  #     @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50)
+  #     @tx.out.last.value.should == 3950
+  #   end
 
-    it "should send change to new address" do
-      @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50, :new)
-      @wallet.addrs.size.should == 2
-      Script.new(@tx.out.last.pk_script).get_address.should == @wallet.addrs.last
-    end
+  #   it "should send change to specified address" do
+  #     @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50,
+  #       '1EAntvSjkNeaJJTBQeQcN1ieU2mYf4wU9p')
+  #     Script.new(@tx.out.last.pk_script).get_address.should ==
+  #       '1EAntvSjkNeaJJTBQeQcN1ieU2mYf4wU9p'
+  #   end
 
-    it "should raise exception if insufficient balance" do
-      -> {@tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 7000]])}
-      .should.raise(RuntimeError).message.should == "Insufficient funds."
-    end
+  #   it "should send change to new address" do
+  #     @tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 1000]], 50, :new)
+  #     @wallet.addrs.size.should == 2
+  #     Script.new(@tx.out.last.pk_script).get_address.should == @wallet.addrs.last
+  #   end
 
+  #   it "should raise exception if insufficient balance" do
+  #     -> {@tx = @wallet.new_tx([[:address, '1M2JjkX7KAgwMyyF5xc2sPSfE7mL1jqkE7', 7000]])}
+  #     .should.raise(RuntimeError).message.should == "Insufficient funds."
+  #   end
 
-    it "should create unsigned tx" do
-      Bitcoin.network = :spec
-      @key = Bitcoin::Key.generate
-      @key2 = Bitcoin::Key.generate
-      @store = Storage.sequel(db: "sqlite:/")
-      @store.log.level = :debug
 
-      @keystore = SimpleKeyStore.new(file: StringIO.new("[]"))
-      @wallet = Wallet.new(@store, @keystore, SimpleCoinSelector)
+  #   it "should create unsigned tx" do
+  #     Bitcoin.network = :spec
+  #     @key = Bitcoin::Key.generate
+  #     @key2 = Bitcoin::Key.generate
+  #     @store = Storage.sequel(db: "sqlite:/")
+  #     @store.log.level = :debug
 
-      @wallet.keystore.add_key(addr: @key.addr)
+  #     @keystore = SimpleKeyStore.new(file: StringIO.new("[]"))
+  #     @wallet = Wallet.new(@store, @keystore, SimpleCoinSelector)
 
-      @genesis = Bitcoin::P::Block.new("0100000000000000000000000000000000000000000000000000000000000000000000003ba3edfd7a7b12b27ac72c3e67768f617fc81bc3888a51323a9fb8aa4b1e5e4adae5494dffff001d1aa4ae180101000000010000000000000000000000000000000000000000000000000000000000000000ffffffff4d04ffff001d0104455468652054696d65732030332f4a616e2f32303039204368616e63656c6c6f72206f6e206272696e6b206f66207365636f6e64206261696c6f757420666f722062616e6b73ffffffff0100f2052a01000000434104678afdb0fe5548271967f1a67130b7105cd6a828e03909a67962e0ea1f61deb649f6bc3f4cef38c4f35504e51ec112de5c384df7ba0b8d578a4c702b6bf11d5fac00000000".htb)
+  #     @wallet.keystore.add_key(addr: @key.addr)
 
-      @store.new_block @genesis
-      create_block(@genesis.hash, true, [], @key, 50e8)
+  #     @genesis = Bitcoin::P::Block.new("0100000000000000000000000000000000000000000000000000000000000000000000003ba3edfd7a7b12b27ac72c3e67768f617fc81bc3888a51323a9fb8aa4b1e5e4adae5494dffff001d1aa4ae180101000000010000000000000000000000000000000000000000000000000000000000000000ffffffff4d04ffff001d0104455468652054696d65732030332f4a616e2f32303039204368616e63656c6c6f72206f6e206272696e6b206f66207365636f6e64206261696c6f757420666f722062616e6b73ffffffff0100f2052a01000000434104678afdb0fe5548271967f1a67130b7105cd6a828e03909a67962e0ea1f61deb649f6bc3f4cef38c4f35504e51ec112de5c384df7ba0b8d578a4c702b6bf11d5fac00000000".htb)
 
-      list = @wallet.list
-      list.size.should == 1
-      list[0][0].should == {addr: @key.addr}
-      list[0][1].should == 50e8
+  #     @store.new_block @genesis
+  #     create_block(@genesis.hash, true, [], @key, 50e8)
 
-      tx = @wallet.new_tx([[:address, @key2.addr, 10e8]])
-      tx.in[0].sig_hash.should != nil
-    end
+  #     list = @wallet.list
+  #     list.size.should == 1
+  #     list[0][0].should == {addr: @key.addr}
+  #     list[0][1].should == 50e8
 
-  end
+  #     tx = @wallet.new_tx([[:address, @key2.addr, 10e8]])
+  #     tx.in[0].sig_hash.should != nil
+  #   end
+
+  # end
 
-  # TODO
+  # # TODO
   # describe "Bitcoin::Wallet::Wallet#tx (multisig)" do