bitcoin 0.1.2 → 0.2.0

data/spec/bitcoin/commands/satoshi_wallet/add_address_command_spec.rb

@@ -1,149 +0,0 @@
-# -*- encoding: utf-8 -*-
-
-require 'spec_helper'
-
-require 'bitcoin/commands/satoshi_wallet/add_address_command'
-
-# Dependencies required only in the spec
-require 'bitcoin/console/capturing_stream_bundle'
-require 'bitcoin/data_access/satoshi/satoshi_wallet'
-
-module Bitcoin
-  module Commands
-    module SatoshiWallet
-      describe AddAddressCommand do
-        let(:stream_bundle) { Console::CapturingStreamBundle.test_bundle }
-        let(:wallet) { mock(DataAccess::Satoshi::SatoshiWallet, add_address: nil) }
-
-        let(:subject) { AddAddressCommand.new(stream_bundle) }
-        
-        context "the input args are valid" do
-          def run_command
-            subject.run(wallet, args: [ "14Z1mazY4HfysZyMaKudFr63EwHqQT2njz", "Bitcoin Monitor" ])
-          end
-
-          it "adds the address" do
-            wallet.should_receive(:add_address).with(
-              Domain::BitcoinAddress.new("14Z1mazY4HfysZyMaKudFr63EwHqQT2njz"),
-              name: "Bitcoin Monitor"
-            )
-            run_command
-          end
-
-          it "informs the user that the address was added" do
-            run_command
-            stream_bundle.captured_output.should eq "Address added successfully\n"
-          end
-        end
-
-        context "the input args are invalid" do
-          context "missing arguments" do
-            invalid_arg_sets = [
-              %w[ ], # Pressed enter too soon
-              %w[ 14Z1mazY4HfysZyMaKudFr63EwHqQT2njz ] # Missing name
-            ]
-
-            invalid_arg_sets.each do |invalid_args|
-              it "prints an error message" do
-                expect {
-                  subject.run(wallet, args: invalid_args)
-                }.to raise_error
-
-                stream_bundle.captured_error.should include("You must provide both a Bitcoin address and a name to add to a wallet")
-                stream_bundle.captured_error.should =~ /No changes were made to the wallet\n\Z/
-              end
-
-              it "raises an error" do
-                expect {
-                  subject.run(wallet, args: invalid_args)
-                }.to raise_error(CLI::CommandError)
-              end
-            end
-          end
-
-          context "too many args" do
-            def run_command
-              subject.run(wallet, args: %w[ 14Z1mazY4HfysZyMaKudFr63EwHqQT2njz foo bar baz ])
-            end
-
-            it "prints a helpful error message" do
-              expect { run_command }.to raise_error
-
-              stream_bundle.captured_error.should include(
-                -%{
-                  Received too many arguments
-
-                  To make sure the name is formatted correctly we ask you to use use quotes
-                  around the name, e.g.: "foo bar baz"
-
-                  No changes were made to the wallet
-                }
-              )
-              stream_bundle.captured_error.should =~ /\n\Z/
-            end
-
-            it "raises an error" do
-              expect { run_command }.to raise_error(CLI::CommandError)
-            end
-          end
-
-          context "invalid Bitcoin address" do
-            def run_command
-              subject.run(wallet, args: [ "Invalid Bitcoin address", "Bitcoin Monitor" ])
-            end
-
-            it "prints an error message" do
-              expect {
-                run_command
-              }.to raise_error
-
-              # Ugh! There's got to be a better way of getting the error message than this,
-              # hopefully without having to stub out the creating of a value object...
-              expected_error =
-                begin
-                  Domain::BitcoinAddress.new("Invalid Bitcoin address")
-                rescue Domain::BitcoinAddress::InvalidBitcoinAddressError => error
-                  error.message
-                end
-
-              stream_bundle.captured_error.should include(error.message)
-              stream_bundle.captured_error.should =~ /The address was not added to the wallet\n\Z/
-            end
-
-            it "raises an error" do
-              expect {
-                run_command
-              }.to raise_error(CLI::CommandError)
-            end
-          end
-
-          # This is paranoid, but currently we don't handle string length properly in the Berkeley DB code
-          # (This error handling is not publically documented)
-          context "excessively long names" do
-            string_253_bytes_long = "#{"☃" * 84}."
-
-            define_method :run_command do
-              subject.run(wallet, args: [ "14Z1mazY4HfysZyMaKudFr63EwHqQT2njz", string_253_bytes_long ])
-            end
-
-            it "prints an error message" do
-              expect {
-                run_command
-              }.to raise_error
-
-              stream_bundle.captured_error.should include(%'The address name "#{string_253_bytes_long}" is too long\n')
-              stream_bundle.captured_error.should include("Currently we do not allow address names over 252 bytes long\n")
-              stream_bundle.captured_error.should =~ /The address was not added to the wallet\n\Z/
-            end
-
-            it "raises an error" do
-              expect {
-                run_command
-              }.to raise_error(CLI::CommandError)
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/spec/bitcoin/commands/satoshi_wallet/show_addresses_command_spec.rb

@@ -1,26 +0,0 @@
-require 'spec_helper'
-
-require 'bitcoin/commands/satoshi_wallet/show_addresses_command'
-
-# Dependencies required only in the spec
-require 'bitcoin/console/capturing_stream_bundle'
-require 'bitcoin/data_access/satoshi/satoshi_wallet'
-
-module Bitcoin
-  module Commands
-    module SatoshiWallet
-      describe ShowAddressesCommand do
-        let(:stream_bundle) { Console::CapturingStreamBundle.test_bundle }
-        let(:address_book) { mock(Domain::AddressBook, to_s: "addresses") }
-        let(:wallet) { mock(DataAccess::Satoshi::SatoshiWallet, addresses: address_book) }
-
-        let(:subject) { ShowAddressesCommand.new(stream_bundle) }
-
-        it "prints the addresses to the output stream" do
-          subject.run(wallet, wallet_filename: "wallet.dat")
-          stream_bundle.captured_output.should eq "# All addresses in wallet.dat\naddresses\n"
-        end
-      end
-    end
-  end
-end

data/spec/bitcoin/commands/satoshi_wallet/show_version_command_spec.rb

@@ -1,26 +0,0 @@
-require 'spec_helper'
-
-require 'bitcoin/commands/satoshi_wallet/show_version_command'
-
-# Dependencies required only in the spec
-require 'bitcoin/console/capturing_stream_bundle'
-require 'bitcoin/data_access/satoshi/satoshi_version'
-require 'bitcoin/data_access/satoshi/satoshi_wallet'
-
-module Bitcoin
-  module Commands
-    module SatoshiWallet
-      describe ShowVersionCommand do
-        let(:stream_bundle) { Console::CapturingStreamBundle.test_bundle }
-        let(:wallet) { mock(DataAccess::Satoshi::SatoshiWallet, version: DataAccess::Satoshi::SatoshiVersion.new(0, 3, 24)) }
-
-        let(:subject) { ShowVersionCommand.new(stream_bundle) }
-
-        it "prints the wallet's version" do
-          subject.run(wallet)
-          stream_bundle.captured_output.should eq "0.3.24\n"
-        end
-      end
-    end
-  end
-end

data/spec/bitcoin/commands/satoshi_wallet_command_environment_spec.rb

@@ -1,76 +0,0 @@
-require 'spec_helper'
-
-require 'bitcoin/commands/satoshi_wallet_command_environment'
-
-# Dependencies required only in the spec
-require 'bitcoin/data_access/satoshi/satoshi_wallet'
-require 'bitcoin/console/capturing_stream_bundle'
-
-module Bitcoin
-  module Commands
-    describe SatoshiWalletCommandEnvironment do
-      include FakeFS::SpecHelpers
-
-      let(:empty_temp_dir) { mock(FileSystem::EmptyTempDir, absolute_path: "/path/to/temp/dir") }
-      let(:satoshi_wallet) { mock(DataAccess::Satoshi::SatoshiWallet) }
-
-      before(:each) do
-        FileSystem::EmptyTempDir.stub(:open).and_yield(empty_temp_dir)
-        DataAccess::Satoshi::SatoshiWallet.stub(:open).and_yield(satoshi_wallet)
-      end
-
-      let(:command) { mock(Commands::SatoshiWallet, run: nil) }
-      let(:stream_bundle) { Console::CapturingStreamBundle.test_bundle }
-
-      subject { SatoshiWalletCommandEnvironment.new(stream_bundle) }
-
-      context "the wallet file exists" do
-        before(:each) { File.open("wallet.dat", "w") { |file| file << "unused wallet data" } }
-
-        it "create temp dir" do
-          FileSystem::EmptyTempDir.should_receive(:open).with(with_name: "bitcoinrb_db_tmp", parallel_to: PROJECT_ROOT + "/wallet.dat")
-          subject.run_command(command, %w[ wallet.dat foo bar ])
-        end
-
-        it "opens the wallet" do
-          DataAccess::Satoshi::SatoshiWallet.should_receive(:open).with(
-            wallet_filename: File.join(PROJECT_ROOT, "/wallet.dat"), # because this is the directory the specs run in
-            db_dirname: "/path/to/temp/dir"
-          )
-          subject.run_command(command, %w[ wallet.dat foo bar ])
-        end
-
-        it "runs the command" do
-          command.should_receive(:run).with(satoshi_wallet, wallet_filename: "wallet.dat", args: %w[ foo bar ])
-          subject.run_command(command, %w[ wallet.dat foo bar ])
-        end
-      end
-
-      context "the wallet file does not exist" do
-        it "prints an error" do
-          expect {
-            subject.run_command(command, %w[ missing_wallet.dat foo bar ])
-          }.to raise_error
-
-          stream_bundle.captured_error.should eq "Couldn't find wallet file: missing_wallet.dat\n"
-        end
-
-        it "raises a CLI::CommandError" do
-          expect {
-            subject.run_command(command, %w[ missing_wallet.dat foo bar ])
-          }.to raise_error(CLI::CommandError)
-        end
-
-        it "does not attempt the rest of the code" do
-          # Although this does involve a lot of knowledge about the code, as
-          # it's not immediately obvious why this proves nothing else is run
-          FileSystem::EmptyTempDir.should_not_receive(:open)
-
-          expect {
-            subject.run_command(command, %w[ missing_wallet.dat foo bar ])
-          }.to raise_error
-        end
-      end
-    end
-  end
-end

data/spec/bitcoin/commands/satoshi_wallet_command_spec.rb

@@ -1,73 +0,0 @@
-require 'spec_helper'
-
-require 'bitcoin/commands/satoshi_wallet_command'
-
-# Dependencies only used in the spec
-require 'bitcoin/domain/address_book'
-require 'bitcoin/console/capturing_stream_bundle'
-require 'bitcoin/commands/satoshi_wallet_command_environment'
-
-module Bitcoin
-  module Commands
-    describe SatoshiWalletCommand do
-      let(:stream_bundle) { Console::CapturingStreamBundle.test_bundle }
-      let(:command_environment) { mock(SatoshiWalletCommandEnvironment, run_command: nil) }
-
-      subject { SatoshiWalletCommand.new(stream_bundle, command_environment) }
-
-      describe "#run" do
-        shared_examples_for "SatoshiWalletCommand#run" do |options|
-          let(:command) { mock(options[:command_class]) }
-
-          before(:each) do
-            options[:command_class].stub(new: command)
-          end
-
-          it "makes a command" do
-            options[:command_class].should_receive(:new).with(stream_bundle)
-            subject.run([ options[:command_name], "wallet.dat", "foo", "bar" ])
-          end
-
-          it "runs the command" do
-            command_environment.should_receive(:run_command).with(command, %w[ wallet.dat foo bar ])
-            subject.run([ options[:command_name], "wallet.dat", "foo", "bar" ])
-          end
-        end
-
-        context "show-version" do
-          it_behaves_like "SatoshiWalletCommand#run",
-            command_name: "show-version",
-            command_class: Commands::SatoshiWallet::ShowVersionCommand
-        end
-
-        context "show-addresses" do
-          it_behaves_like "SatoshiWalletCommand#run",
-            command_name: "show-addresses",
-            command_class: Commands::SatoshiWallet::ShowAddressesCommand
-        end
-
-        context "add-address" do
-          it_behaves_like "SatoshiWalletCommand#run",
-            command_name: "add-address",
-            command_class: Commands::SatoshiWallet::AddAddressCommand
-        end
-
-        context "<unknown subcommand>" do
-          it "prints an error" do
-            expect {
-              subject.run(%w[ unknown-subcommand wallet.dat foo bar ])
-            }.to raise_error
-
-            stream_bundle.captured_error.should eq %'Unknown satoshi-wallet subcommand: "unknown-subcommand"\n'
-          end
-
-          it "raises an error" do
-            expect {
-              subject.run(%w[ unknown-subcommand wallet.dat foo bar ])
-            }.to raise_error(CLI::CommandError)
-          end
-        end
-      end
-    end
-  end
-end

data/spec/bitcoin/console/_contracts/stream_bundle_contract.rb

@@ -1,29 +0,0 @@
-contract "StreamBundle" do
-  let(:input)   { StringIO.new }
-  let(:output)  { StringIO.new }
-  let(:error)   { StringIO.new }
-
-  let(:subject) { described_class.new(input, output, error) }
-
-  describe "#puts_output" do
-    it "writes a line to the output" do
-      subject.puts_output("foo")
-      subject.puts_output("bar")
-      subject.puts_output("baz")
-
-      output.rewind
-      output.read.should eq "foo\nbar\nbaz\n"
-    end
-  end
-
-  describe "#puts_error" do
-    it "writes a line to the output" do
-      subject.puts_error("foo")
-      subject.puts_error("bar")
-      subject.puts_error("baz")
-
-      error.rewind
-      error.read.should eq "foo\nbar\nbaz\n"
-    end
-  end
-end

data/spec/bitcoin/console/capturing_stream_bundle_spec.rb

@@ -1,74 +0,0 @@
-require 'spec_helper'
-
-require 'bitcoin/console/capturing_stream_bundle'
-
-require_relative '_contracts/stream_bundle_contract'
-
-module Bitcoin
-  module Console
-    describe CapturingStreamBundle do
-      it_satisfies_contract "StreamBundle"
-
-      let(:input)   { StringIO.new }
-      let(:output)  { StringIO.new }
-      let(:error)   { StringIO.new }
-
-      let(:subject) { CapturingStreamBundle.new(input, output, error) }
-
-      describe ".test_bundle" do
-        it "creates a CapturingStreamBundle with three StringIO streams", because: "we only care about captured output" do
-          # Really should be a separate factory object, but not for the sake of one method
-          CapturingStreamBundle.should_receive(:new).with(
-            an_instance_of(StringIO), an_instance_of(StringIO), an_instance_of(StringIO)
-          ).and_return(:stream_bundle)
-          CapturingStreamBundle.test_bundle.should eq :stream_bundle
-        end
-      end
-
-      # The output and error streams are effectively the same with different names,
-      # but the duplication is only textual and I don't see a neat way to remove it yet
-
-      describe "#captured_output" do
-        before(:each) do
-          subject.puts_output("foo")
-          subject.puts_output("bar")
-          subject.puts_output("baz")
-        end
-
-        its(:captured_output) { should eq "foo\nbar\nbaz\n" }
-
-        it "doesn't manipulate the output stream" do
-          output.stub(:rewind).and_raise(RuntimeError.new)
-          expect { subject.captured_output }.to_not raise_error
-        end
-
-        it "accumulates output across reads" do
-          subject.captured_output
-          subject.puts_output("quux")
-          subject.captured_output.should eq "foo\nbar\nbaz\nquux\n"
-        end
-      end
-
-      describe "#captured_error" do
-        before(:each) do
-          subject.puts_error("foo")
-          subject.puts_error("bar")
-          subject.puts_error("baz")
-        end
-
-        its(:captured_error) { should eq "foo\nbar\nbaz\n" }
-
-        it "doesn't manipulate the error stream" do
-          error.stub(:rewind).and_raise(RuntimeError.new)
-          expect { subject.captured_error }.to_not raise_error
-        end
-
-        it "accumulates output across reads" do
-          subject.captured_error
-          subject.puts_error("quux")
-          subject.captured_error.should eq "foo\nbar\nbaz\nquux\n"
-        end
-      end
-    end
-  end
-end