ppl 0.3.0 → 0.9.0

data/spec/ppl/command/email_spec.rb

@@ -0,0 +1,56 @@
+
+describe Ppl::Command::Email do
+
+  before(:each) do
+    @input   = Ppl::Application::Input.new
+    @output  = Ppl::Application::Output.new(nil, nil)
+    @contact = Ppl::Entity::Contact.new
+    @command = Ppl::Command::Email.new
+    @storage = double(Ppl::Adapter::Storage)
+
+    @list_format = double(Ppl::Format::Contact)
+    @show_format = double(Ppl::Format::Contact)
+
+    @command.storage     = @storage
+    @command.show_format = @show_format
+    @command.list_format = @list_format
+    @contact.id = "jim"
+  end
+
+  describe "#name" do
+    it "should be 'email'" do
+      @command.name.should eq "email"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should list all email addresses if no contact ID is given" do
+      @storage.should_receive(:load_address_book).and_return(@address_book)
+      @list_format.should_receive(:process).and_return("all the email addresses")
+      @output.should_receive(:line).with("all the email addresses")
+      @input.arguments = []
+      @command.execute(@input, @output)
+    end
+
+    it "should show the current address if no new address is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @show_format.should_receive(:process).and_return("jdoe@example.org")
+      @output.should_receive(:line).with("jdoe@example.org")
+      @input.arguments = ["jim"]
+      @command.execute(@input, @output)
+    end
+
+    it "should change the contact's email address if an address is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @storage.should_receive(:save_contact) do |contact|
+        contact.email_address.should eq "jim@example.org"
+      end
+      @input.arguments = ["jim", "jim@example.org"]
+      @command.execute(@input, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/help_spec.rb

@@ -0,0 +1,79 @@
+
+describe Ppl::Command::Help do
+
+  before(:each) do
+    @command = Ppl::Command::Help.new
+    @output  = double(Ppl::Application::Output)
+
+    @command.command_suite = Ppl::Application::CommandSuite.new
+  end
+
+  describe "#command_suite=" do
+    it "should accept a value" do
+      suite = Object.new
+      @command.command_suite = suite
+      @command.command_suite.should be suite
+    end
+  end
+
+  describe "#name" do
+    it "should be 'help'" do
+      @command.name.should eq "help"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should not list itself" do
+      @command.command_suite.add_command(@command)
+
+      @output.should_receive(:line).with("usage: ppl <command>")
+      @output.should_receive(:line).with(nil)
+      @output.should_receive(:line).with(nil)
+
+      @command.execute(nil, @output)
+    end
+
+    it "should list available commands" do
+      command = double(Ppl::Application::Command)
+      command.should_receive(:name).twice.and_return("one")
+      command.should_receive(:description).and_return("The first command")
+      @command.command_suite.add_command(command)
+
+      @output.should_receive(:line).with("usage: ppl <command>")
+      @output.should_receive(:line).with(nil)
+      @output.should_receive(:line).with("   one   The first command")
+      @output.should_receive(:line).with(nil)
+
+      @command.execute(nil, @output)
+    end
+
+    it "should arrange output into columns" do
+      command_foo = Ppl::Application::Command.new
+      command_foo.name = "shortname"
+      command_foo.description = "This is a command with a short name"
+
+      command_bar = Ppl::Application::Command.new
+      command_bar.name = "veryverylongname"
+      command_bar.description = "This is a command with a longer name"
+
+      @command.command_suite.add_command(command_foo)
+      @command.command_suite.add_command(command_bar)
+
+      @output.should_receive(:line).with("usage: ppl <command>")
+      @output.should_receive(:line).with(nil)
+      [
+        "   shortname          This is a command with a short name",
+        "   veryverylongname   This is a command with a longer name",
+      ].each do |line|
+        @output.should_receive(:line).with(line)
+      end
+      @output.should_receive(:line).with(nil)
+
+      @command.execute(nil, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/init_spec.rb

@@ -0,0 +1,40 @@
+
+describe Ppl::Command::Init do
+
+  before(:each) do
+    @command = Ppl::Command::Init.new
+    @input   = Ppl::Application::Input.new
+    @output  = Ppl::Application::Output.new(nil, nil)
+  end
+
+  describe "#name" do
+    it "should be 'init'" do
+      @command.name.should eq "init"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should pass the given path through to the storage adapter" do
+      Ppl::Adapter::Storage::Git.stub(:create_address_book) do |path|
+        path.should eq "/contacts"
+      end
+      @input.arguments.push "/contacts"
+      @command.execute(@input, @output)
+    end
+
+    it "should pass the cwd to the storage adapter if no path is given" do
+      FakeFS.activate!
+      Dir.mkdir "/current"
+      Dir.chdir "/current"
+      Ppl::Adapter::Storage::Git.stub(:create_address_book) do |path|
+        path.should eq "/current"
+      end
+      @command.execute(@input, @output)
+      FakeFS.deactivate!
+    end
+
+  end
+
+end
+

data/spec/ppl/command/ls_spec.rb

@@ -0,0 +1,32 @@
+
+describe Ppl::Command::Ls do
+
+  before(:each) do
+    @command = Ppl::Command::Ls.new
+    @input   = Ppl::Application::Input.new
+    @output  = double(Ppl::Application::Output)
+    @storage = double(Ppl::Adapter::Storage)
+    @format  = double(Ppl::Format::AddressBook)
+    @address_book = double(Ppl::Entity::AddressBook)
+
+    @command.format   = @format
+    @command.storage  = @storage
+  end
+
+  describe "#name" do
+    it "should be 'ls'" do
+      @command.name.should eq "ls"
+    end
+  end
+
+  describe "#execute" do
+    it "should show the list of contacts in the address book" do
+      @storage.should_receive(:load_address_book).and_return(@address_book)
+      @format.should_receive(:process).and_return("list of contacts")
+      @output.should_receive(:line).with("list of contacts")
+      @command.execute(@input, @output)
+    end
+  end
+
+end
+

data/spec/ppl/command/mv_spec.rb

@@ -0,0 +1,62 @@
+
+describe Ppl::Command::Mv do
+
+  before(:each) do
+    @command = Ppl::Command::Mv.new
+    @input   = Ppl::Application::Input.new
+    @output  = double(Ppl::Application::Output)
+    @contact = Ppl::Entity::Contact.new
+    @storage = double(Ppl::Adapter::Storage)
+
+    @command.storage = @storage
+    @contact.id = "old"
+  end
+
+  describe "#name" do
+    it "should be 'mv'" do
+      @command.name.should eq "mv"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should raise an error if no contact is specified" do
+      @input.arguments = []
+      expect{@command.execute(@input, @output)}.to raise_error(Ppl::Error::IncorrectUsage)
+    end
+
+    it "should raise an error if no new ID is specified" do
+      @input.arguments = ["test", nil]
+      expect{@command.execute(@input, @output)}.to raise_error(Ppl::Error::IncorrectUsage)
+    end
+
+    it "should return false if the new id is taken" do
+      @storage.should_receive(:require_contact).with("old").and_return(@contact)
+      @storage.should_receive(:load_contact).with("new").and_return(@contact)
+      @output.should_receive(:error)
+      @input.arguments = ["old", "new"]
+
+      @command.execute(@input, @output).should eq false
+    end
+
+    it "should rename the given contact" do
+      @storage.should_receive(:require_contact).with("old").and_return(@contact)
+      @storage.should_receive(:load_contact).with("new").and_return(nil)
+
+      @storage.should_receive(:delete_contact).with(@contact) do |contact|
+        contact.id.should eq "old"
+      end
+
+      @storage.should_receive(:save_contact).with(@contact) do |contact|
+        contact.id.should eq "new"
+      end
+
+      @input.arguments = ["old", "new"]
+
+      @command.execute(@input, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/name_spec.rb

@@ -0,0 +1,59 @@
+
+describe Ppl::Command::Name do
+
+  before(:each) do
+    @input   = Ppl::Application::Input.new
+    @output  = Ppl::Application::Output.new(nil, nil)
+    @contact = Ppl::Entity::Contact.new
+    @command = Ppl::Command::Name.new
+
+    @address_book = Ppl::Entity::AddressBook.new
+
+    @storage     = double(Ppl::Adapter::Storage)
+    @show_format = double(Ppl::Format::Contact)
+    @list_format = double(Ppl::Format::Contact)
+
+    @command.storage     = @storage
+    @command.show_format = @show_format
+    @command.list_format = @list_format
+
+    @contact.id = "jim"
+  end
+
+  describe "#name" do
+    it "should be 'name'" do
+      @command.name.should eq "name"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should list all contact names if no contact ID is given" do
+      @storage.should_receive(:load_address_book).and_return(@address_book)
+      @list_format.should_receive(:process).and_return("all the names")
+      @output.should_receive(:line).with("all the names")
+      @input.arguments = []
+      @command.execute(@input, @output)
+    end
+
+    it "should show the specified contact's name if no name is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @show_format.should_receive(:process).and_return("John Doe")
+      @output.should_receive(:line).with("John Doe")
+      @input.arguments = ["jim"]
+      @command.execute(@input, @output)
+    end
+
+    it "should change the contact's name if a name is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @storage.should_receive(:save_contact) do |contact|
+        contact.name.should eq "Jim Jamieson"
+      end
+      @input.arguments = ["jim", "Jim Jamieson"]
+      @command.execute(@input, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/org_spec.rb

@@ -0,0 +1,57 @@
+
+describe Ppl::Command::Org do
+
+  before(:each) do
+    @command = Ppl::Command::Org.new
+    @input   = Ppl::Application::Input.new
+    @output  = Ppl::Application::Output.new(nil, nil)
+    @contact = Ppl::Entity::Contact.new
+    @storage = double(Ppl::Adapter::Storage)
+
+    @show_format = double(Ppl::Format::Contact)
+    @list_format = double(Ppl::Format::Contact)
+
+    @command.storage     = @storage
+    @command.show_format = @show_format
+    @command.list_format = @list_format
+
+    @contact.id = "tim"
+  end
+
+  describe "#name" do
+    it "should be 'org'" do
+      @command.name.should eq "org"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should list all organizations if no contact ID is given" do
+      @storage.should_receive(:load_address_book).and_return(@address_book)
+      @list_format.should_receive(:process).and_return("all the organizations")
+      @output.should_receive(:line).with("all the organizations")
+      @input.arguments = []
+      @command.execute(@input, @output)
+    end
+
+    it "should show the contact's organization if no new organization is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @show_format.should_receive(:process).and_return("Example Ltd")
+      @output.should_receive(:line).with("Example Ltd")
+      @input.arguments = ["jim"]
+      @command.execute(@input, @output)
+    end
+
+    it "should change the contact's organization if one is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @storage.should_receive(:save_contact) do |contact|
+        contact.organization.should eq "Example Ltd"
+      end
+      @input.arguments = ["jim", "Example Ltd"]
+      @command.execute(@input, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/phone_spec.rb

@@ -0,0 +1,55 @@
+
+describe Ppl::Command::Phone do
+
+  before(:each) do
+    @command = Ppl::Command::Phone.new
+    @input   = Ppl::Application::Input.new
+    @output  = Ppl::Application::Output.new(nil, nil)
+    @contact = Ppl::Entity::Contact.new
+    @storage = double(Ppl::Adapter::Storage)
+
+    @show_format = double(Ppl::Format::Contact)
+    @list_format = double(Ppl::Format::Contact)
+
+    @command.storage     = @storage
+    @command.show_format = @show_format
+    @command.list_format = @list_format
+  end
+
+  describe "#name" do
+    it "should be 'phone'" do
+      @command.name.should eq "phone"
+    end
+  end
+
+  describe "#execute" do
+
+    it "should list all phone numbers if no contact ID is given" do
+      @storage.should_receive(:load_address_book).and_return(@address_book)
+      @list_format.should_receive(:process).and_return("all the numbers")
+      @output.should_receive(:line).with("all the numbers")
+      @input.arguments = []
+      @command.execute(@input, @output)
+    end
+
+    it "should show the contact's phone number if no new number is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @show_format.should_receive(:process).and_return("0123456789")
+      @output.should_receive(:line).with("0123456789")
+      @input.arguments = ["jim"]
+      @command.execute(@input, @output)
+    end
+
+    it "should change the contact's phone number if a number is given" do
+      @storage.should_receive(:require_contact).and_return(@contact)
+      @storage.should_receive(:save_contact) do |contact|
+        contact.phone_number.should eq "01234567890"
+      end
+      @input.arguments = ["jim", "01234567890"]
+      @command.execute(@input, @output)
+    end
+
+  end
+
+end
+

data/spec/ppl/command/{contact_delete_spec.rb → rm_spec.rb}

@@ -1,13 +1,14 @@
 
-describe Ppl::Command::ContactDelete do
+describe Ppl::Command::Rm do
 
   before(:each) do
+    @command = Ppl::Command::Rm.new
     @input   = Ppl::Application::Input.new
     @output  = Ppl::Application::Output.new(nil, nil)
     @contact = Ppl::Entity::Contact.new
-    @command = Ppl::Command::ContactDelete.new
     @storage = double(Ppl::Adapter::Storage)
 
+    @input.arguments = ["test"]
     @command.storage = @storage
   end
 
@@ -19,6 +20,11 @@ describe Ppl::Command::ContactDelete do
 
   describe "#execute" do
 
+    it "should raise an error if the contact isn't specified" do
+      @input.arguments = [nil]
+      expect{@command.execute(@input, @output)}.to raise_error(Ppl::Error::IncorrectUsage)
+    end
+
     it "should delete the given contact" do
       @storage.should_receive(:require_contact).and_return(@contact)
       @storage.should_receive(:delete_contact).with(@contact)

data/spec/ppl/command/{contact_show_spec.rb → show_spec.rb}

@@ -1,14 +1,16 @@
 
-describe Ppl::Command::ContactShow do
+describe Ppl::Command::Show do
 
   before(:each) do
-    @command = Ppl::Command::ContactShow.new
+    @command = Ppl::Command::Show.new
     @input   = Ppl::Application::Input.new
     @output  = double(Ppl::Application::Output)
     @contact = Ppl::Entity::Contact.new
     @storage = double(Ppl::Adapter::Storage)
+    @format  = double(Ppl::Format::Contact)
 
     @command.storage = @storage
+    @command.format  = @format
   end
 
   describe "#name" do
@@ -18,15 +20,19 @@ describe Ppl::Command::ContactShow do
   end
 
   describe "#execute" do
+
+    it "should raise an error if no contact is specified" do
+      @input.arguments = []
+      expect{@command.execute(@input, @output)}.to raise_error(Ppl::Error::IncorrectUsage)
+    end
+
     it "should show the contact's name" do
       @storage.should_receive(:require_contact).and_return(@contact)
+      @format.should_receive(:process).and_return("John Doe")
 
-      @contact.name          = "John Doe"
-      @contact.email_address = "johndoe@example.org"
+      @input.arguments = ["john"]
 
       @output.should_receive(:line).with("John Doe")
-      @output.should_receive(:line).with("johndoe@example.org")
-
       @command.execute(@input, @output)
     end
   end

data/spec/ppl/format/address_book/birthdays_spec.rb

@@ -0,0 +1,38 @@
+
+describe Ppl::Format::AddressBook::Birthdays do
+
+  before(:each) do
+    @format       = Ppl::Format::AddressBook::Birthdays.new
+    @address_book = Ppl::Entity::AddressBook.new
+    @contact      = Ppl::Entity::Contact.new
+    @table        = double(Ppl::Format::Table)
+
+    @contact.id = "test"
+    @format.table = @table
+
+    @address_book.add_contact(@contact)
+  end
+
+  describe "#process" do
+
+    it "should at least show the contact's id" do
+      @table.should_receive(:add_row).with({
+        :id       => "test:",
+        :birthday => nil,
+      })
+      @format.process(@address_book)
+    end
+
+    it "should show the birthday if it's available" do
+      @contact.birthday = Date.parse("2001-02-03")
+      @table.should_receive(:add_row).with({
+        :id        => "test:",
+        :birthday  => "2001-02-03",
+      })
+      @format.process(@address_book)
+    end
+
+  end
+
+end
+

data/spec/ppl/format/address_book/email_addresses_spec.rb

@@ -0,0 +1,38 @@
+
+describe Ppl::Format::AddressBook::EmailAddresses do
+
+  before(:each) do
+    @format       = Ppl::Format::AddressBook::EmailAddresses.new
+    @address_book = Ppl::Entity::AddressBook.new
+    @contact      = Ppl::Entity::Contact.new
+    @table        = double(Ppl::Format::Table)
+
+    @contact.id = "test"
+    @format.table = @table
+
+    @address_book.add_contact(@contact)
+  end
+
+  describe "#process" do
+
+    it "should at least show the contact's id" do
+      @table.should_receive(:add_row).with({
+        :id            => "test:",
+        :email_address => nil,
+      })
+      @format.process(@address_book)
+    end
+
+    it "should show the email address if it's available" do
+      @contact.email_address = "jdoe@example.org"
+      @table.should_receive(:add_row).with({
+        :id            => "test:",
+        :email_address => "jdoe@example.org",
+      })
+      @format.process(@address_book)
+    end
+
+  end
+
+end
+

data/spec/ppl/format/address_book/names_spec.rb

@@ -0,0 +1,38 @@
+
+describe Ppl::Format::AddressBook::Names do
+
+  before(:each) do
+    @format       = Ppl::Format::AddressBook::Names.new
+    @address_book = Ppl::Entity::AddressBook.new
+    @contact      = Ppl::Entity::Contact.new
+    @table        = double(Ppl::Format::Table)
+
+    @contact.id = "test"
+    @format.table = @table
+
+    @address_book.add_contact(@contact)
+  end
+
+  describe "#process" do
+
+    it "should at least show the contact's id" do
+      @table.should_receive(:add_row).with({
+        :id   => "test:",
+        :name => nil,
+      })
+      @format.process(@address_book)
+    end
+
+    it "should show the name if it's available" do
+      @contact.name = "John Doe"
+      @table.should_receive(:add_row).with({
+        :id   => "test:",
+        :name => "John Doe",
+      })
+      @format.process(@address_book)
+    end
+
+  end
+
+end
+

data/spec/ppl/format/address_book/one_line_spec.rb

@@ -0,0 +1,41 @@
+
+describe Ppl::Format::AddressBook::OneLine do
+
+  before(:each) do
+    @format       = Ppl::Format::AddressBook::OneLine.new
+    @address_book = Ppl::Entity::AddressBook.new
+    @contact      = Ppl::Entity::Contact.new
+    @table        = double(Ppl::Format::Table)
+
+    @contact.id = "test"
+    @format.table = @table
+
+    @address_book.add_contact(@contact)
+  end
+
+  describe "#process" do
+
+    it "should at least show the contact's id" do
+      @table.should_receive(:add_row).with({
+        :id    => "test:",
+        :name  => nil,
+        :email => nil,
+      })
+      @format.process(@address_book)
+    end
+
+    it "should show all the info if it's available" do
+      @contact.name = "John Doe"
+      @contact.email_address = "jdoe@example.org"
+      @table.should_receive(:add_row).with({
+        :id    => "test:",
+        :name  => "John Doe",
+        :email => "<jdoe@example.org>",
+      })
+      @format.process(@address_book)
+    end
+
+  end
+
+end
+