codefumes 0.1.2 → 0.1.3

data/spec/codefumes/commit_spec.rb

@@ -1,49 +1,6 @@
 require File.dirname(__FILE__) + '/../spec_helper.rb'
 
-def single_commit(options = {})
-  commit_xml = <<-END_OF_COMMIT
-  <commit>
-    <identifier>f3badd5624dfbcf5176f0471261731e1b92ce957</identifier>
-    <author_name>John Doe</author_name>
-    <author_email>jdoe@example.com</author_email>
-    <committer_name>John Doe</committer_name>
-    <committer_email>jdoe@example.com</committer_email>
-    <short_message>Made command-line option for 'name' actually work</short_message>
-    <message>
-      Made command-line option for 'name' actually work
-      - Commentd out hard-coded 'require' line used for testing
-    </message>
-    <parent_identifiers>9ddj48423jdsjds5176f0471261731e1b92ce957,3ewdjok23jdsjds5176f0471261731e1b92ce957,284djsksjfjsjds5176f0471261731e1b92ce957</parent_identifiers>
-    <committed_at>Wed May 20 09:09:06 -0500 2009</committed_at>
-    <authored_at>Wed May 20 09:09:06 -0500 2009</authored_at>
-    <uploaded_at>2009-06-04 02:43:20 UTC</uploaded_at>
-    <api_uri>http://localhost:3000/api/v1/commits/f3badd5624dfbcf5176f0471261731e1b92ce957.xml</api_uri>
-    <line_additions>20</line_additions>
-    <line_deletions>10</line_deletions>
-    <line_total>30</line_total>
-    <affected_file_count>2</affected_file_count>
-  END_OF_COMMIT
-
-  if options[:include_custom_attributes]
-    commit_xml <<
-    <<-END_OF_COMMIT
-      <custom_attributes>
-        <coverage>83</coverage>
-        <random_attribute>1</random_attribute>
-      </custom_attributes>
-    END_OF_COMMIT
-  end
-
-  commit_xml << "\n</commit>"
-end
-
-
-def register_index_uri
-  FakeWeb.register_uri(
-    :get, "http://www.codefumes.com:80/api/v1/xml/projects/apk/commits",
-    :status => ["200", "Ok"],
-    :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<commits>\n#{single_commit}\n#{single_commit}\n#{single_commit}\n</commits>\n")
-end
+include CodeFumesServiceStubs
 
 describe "Commit" do
   before(:all) do
@@ -58,10 +15,7 @@ describe "Commit" do
   describe "find" do
     context "with a valid commit identifier" do
       before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/commits/f3badd5624dfbcf5176f0471261731e1b92ce957",
-          :status => ["200", "Ok"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
+        stub_codefumes_uri("commits/#{@identifier}", ["200", "Ok"], single_commit)
         @commit = Commit.find(@identifier)
       end
 
@@ -91,8 +45,7 @@ describe "Commit" do
     context "with a non-existant commit identifier" do
       before(:each) do
         @identifier = "non_existant_commit_identifier"
-        FakeWeb.register_uri( :get, "http://www.codefumes.com:80/api/v1/xml/commits/#{@identifier}",
-                              :status => ["404", "Not Found"])
+        stub_codefumes_uri("commits/#{@identifier}", ["404", "Not Found"], "")
       end
 
       it "returns nil" do
@@ -104,14 +57,12 @@ describe "Commit" do
   describe "calling 'latest'" do
     before(:each) do
       @project_public_key = "apk"
+      @request_uri = "projects/#{@project_public_key}/commits/latest"
     end
 
     context "with valid parameters" do
       before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-          :status => ["200", "Ok"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
+        stub_codefumes_uri(@request_uri, ["200", "Ok"], single_commit)
       end
 
       it "returns a commit object for the latest commit" do
@@ -121,10 +72,7 @@ describe "Commit" do
 
     context "with invalid parameters" do
       before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-          :status => ["404", "Not Found"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
+        stub_codefumes_uri(@request_uri, ["404", "Not Found"], single_commit)
       end
 
       it "returns nil" do
@@ -136,45 +84,28 @@ describe "Commit" do
   describe "calling 'latest_identifier'" do
     before(:each) do
       @project_public_key = "apk"
+      @request_uri = "projects/#{@project_public_key}/commits/latest"
     end
 
     context "with valid parameters" do
       context "when the specified project has commits stored" do
-        before(:each) do
-          FakeWeb.register_uri(
-            :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-            :status => ["200", "Ok"],
-            :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
-        end
-
         it "returns the commit identifier of the latest commit" do
+          stub_codefumes_uri(@request_uri, ["200", "Ok"], single_commit)
           Commit.latest_identifier(@project_public_key).should == @identifier
         end
       end
 
       context "when the specified project does not have any commits stored" do
-        before(:each) do
-          FakeWeb.register_uri(
-            :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-            :status => ["404", "Not Found"],
-            :string =>  "")
-        end
-
         it "returns nil" do
+          stub_codefumes_uri(@request_uri, ["404", "Not Found"], single_commit)
           Commit.latest_identifier(@project_public_key).should == nil
         end
       end
     end
 
     context "with invalid parameters" do
-      before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-          :status => ["404", "Not Found"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
-      end
-
       it "returns nil" do
+        stub_codefumes_uri(@request_uri, ["404", "Not Found"], single_commit)
         Commit.latest(@project_public_key).should == nil
       end
     end
@@ -182,25 +113,19 @@ describe "Commit" do
 
   describe "calling 'all'" do
     before(:each) do
-      register_index_uri
       @project_public_key = "apk"
     end
 
     context "with valid parameters" do
       it "returns an array of commits" do
+        register_index_uri
         Commit.all(@project_public_key).should have(3).items
       end
     end
 
     context "with invalid parameters" do
-      before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/apk/commits",
-          :status => ["404", "Not Found"],
-          :string =>  "")
-      end
-
       it "returns nil" do
+        stub_codefumes_uri("projects/apk/commits", ["404", "Not Found"], single_commit)
         Commit.all(@project_public_key).should == nil
       end
     end
@@ -208,12 +133,9 @@ describe "Commit" do
 
   describe "the convenience method" do
     before(:each) do
+      stub_codefumes_uri("commits/f3badd5624dfbcf5176f0471261731e1b92ce957", ["200", "Ok"], single_commit)
       @email = "jdoe@example.com"
       @name  = "John Doe"
-      FakeWeb.register_uri(
-        :get, "http://www.codefumes.com:80/api/v1/xml/commits/f3badd5624dfbcf5176f0471261731e1b92ce957",
-        :status => ["200", "Ok"],
-        :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
       @commit = Commit.find(@identifier)
     end
 
@@ -239,10 +161,7 @@ describe "Commit" do
 
     context "when the commit does not have any custom attributes" do
       before(:each) do
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-          :status => ["200", "Ok"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{single_commit}")
+        stub_codefumes_uri("projects/#{@project_public_key}/commits/latest", ["200", "Ok"], single_commit)
       end
 
       it "returns an empty Hash" do
@@ -253,10 +172,7 @@ describe "Commit" do
     context "when the commit has defined custom attributes" do
       before(:each) do
         commit_content = single_commit(:include_custom_attributes => true)
-        FakeWeb.register_uri(
-          :get, "http://www.codefumes.com:80/api/v1/xml/projects/#{@project_public_key}/commits/latest",
-          :status => ["200", "Ok"],
-          :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{commit_content}")
+        stub_codefumes_uri("projects/#{@project_public_key}/commits/latest", ["200", "Ok"], commit_content)
       end
 
       it "returns a Hash of key-value pairs (attribute_name -> attribute_value)" do

data/spec/codefumes/config_file_spec.rb

@@ -1,14 +1,24 @@
 require File.dirname(__FILE__) + '/../spec_helper.rb'
 
+def raise_if_users_config_file
+  if ConfigFile.path == File.expand_path('~/.codefumes_config')
+    raise "Set a custom config file path"
+  end
+end
+
+def delete_config_file
+  unless ConfigFile.path == File.expand_path('~/.codefumes_config')
+    File.delete(ConfigFile.path) if File.exist?(ConfigFile.path)
+  end
+end
+
 describe "ConfigFile" do
   before(:each) do
     @project = Project.new(:public_key => 'public_key_value', :private_key => 'private_key_value')
   end
 
   after(:all) do
-    unless ConfigFile.path == File.expand_path('~/.codefumes_config')
-      File.delete(ConfigFile.path) if File.exist?(ConfigFile.path)
-    end
+    delete_config_file
   end
 
   describe "calling 'path'" do
@@ -27,10 +37,6 @@ describe "ConfigFile" do
   end
 
   describe "calling 'save_project'" do
-    after(:all) do
-      File.delete(ConfigFile.path) if File.exist?(ConfigFile.path)
-    end
-
     context "when passed a new project" do
       it "creates the config file if it did not exist already" do
         File.exist?(ConfigFile.path).should be_false
@@ -131,12 +137,93 @@ describe "ConfigFile" do
 
     context "via path=" do
       it "updates the value returned from 'path'" do
-        new_path = File.expand_path('./tmp/new_config')
+        new_path = File.expand_path(File.dirname(__FILE__) + '/../new_config')
         ConfigFile.path.should == File.expand_path('~/.codefumes_config')
         ConfigFile.path = new_path
         ConfigFile.path.should == new_path
       end
     end
   end
+  
+  describe "calling 'save_credentials'" do
+    before(:each) do
+      @api_key  = "API_KEY"
+    end
+
+    context "when no credentials exist" do
+      before(:each) do
+        raise_if_users_config_file
+      end
+
+      it "adds a :credentials key" do
+        ConfigFile.save_credentials(@api_key)
+        ConfigFile.serialized.keys.should include(:credentials)
+      end
+
+      it "adds the supplied api_key under the credentials key" do
+        ConfigFile.save_credentials(@api_key)
+        ConfigFile.serialized[:credentials][:api_key].should == @api_key
+      end
+    end
+
+    context "when credentials already exist" do
+      before(:each) do
+        @existing_api_key  = "EXISTING_API_KEY"
+        ConfigFile.save_credentials(@existing_api_key)
+      end
 
+      it "replaces the existing credentials with the supplied api_key" do
+        # sanity check
+        ConfigFile.serialized[:credentials][:api_key].should == @existing_api_key
+        ConfigFile.save_credentials(@api_key)
+        ConfigFile.serialized[:credentials][:api_key].should == @api_key
+      end
+    end
+
+    it "does not affect existing project content" do
+      public_key = "pub_value"
+      project = Project.new(:public_key => public_key, :private_key => "private_key")
+      ConfigFile.save_project(project)
+      ConfigFile.save_credentials(@api_key)
+      ConfigFile.serialized[:projects][public_key.to_sym].should_not be_nil
+    end
+  end
+
+  describe "calling 'credentials'" do
+    context "when credentials exist in the file" do
+      it "returns a Hash containing the credentials section of the config file" do
+        api_key_value  = "API_KEY"
+        ConfigFile.save_credentials(api_key_value)
+        ConfigFile.credentials.should == {:api_key => api_key_value}
+      end
+    end
+
+    context "when no credentials exist in the file" do
+      it "returns an empty Hash" do
+        ConfigFile.credentials.should == {}
+      end
+    end
+  end
+
+  describe "calling 'public_keys'" do
+    context "when no projects exist in the file" do
+      it "returns an empty array" do
+        delete_config_file
+        ConfigFile.public_keys.should == []
+      end
+    end
+
+    context "when projects exist in the file" do
+      before(:each) do
+        create_uniq_project = lambda {|index| Project.new(:public_key => "pub_key_#{index}", :private_key => 'pk')}
+        @projects = 5.times.map {|i| create_uniq_project.call(i)}
+        @projects.each {|project| ConfigFile.save_project(project)}
+      end
+
+      it "returns an array of the keys" do
+        stringified_keys = ConfigFile.public_keys.map {|key| key.to_s}
+        stringified_keys.sort.should == @projects.map {|p| p.public_key}.sort
+      end
+    end
+  end
 end

data/spec/codefumes/payload_spec.rb

@@ -17,11 +17,13 @@ describe "Payload" do
                              :status => ["201", "Created"],
                              :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<payload>\n <created_at>Creation Date</created_at>\n  </payload>\n")
       end
+
       it "sets basic auth with the public and private key" do
         payload = Payload.new(:public_key => @project.public_key, :private_key => @project.private_key, :content => {:commits => "data_to_send_up"})
         Payload.should_receive(:post).with("/projects/#{@project.public_key}/payloads", :query => {:payload => {:commits => "data_to_send_up"}}, :basic_auth => {:username => @project.public_key, :password => @project.private_key}).and_return(mock("response", :code => 401))
         payload.save
       end
+
       context "with Created response" do
         [:created_at].each do |method_name|
           it "sets the '#{method_name.to_s}'" do
@@ -32,12 +34,14 @@ describe "Payload" do
           end
         end
       end
+
       context "with Unauthorized response" do
         before(:each) do
           @project = Project.new(:public_key => "apk")
           FakeWeb.register_uri( :post, "http://www.codefumes.com:80/api/v1/xml/projects/apk/payloads?payload[commits]=data_to_send_up",
                               :status => ["401", "Unauthorized"])
         end
+
         [:created_at].each do |method_name|
           it "sets the '#{method_name.to_s}'" do
             payload = Payload.new(:public_key => @project.public_key, :content => {:commits => "data_to_send_up"})
@@ -122,6 +126,7 @@ describe "Payload" do
           @prepared.size.should == 1
           @prepared.first.should be_instance_of(Payload)
         end
+
         it "sets the private_key on all payloads" do
           @prepared.each do |payload|
             payload.project_private_key.should == 'something_super_secret'
@@ -146,15 +151,12 @@ describe "Payload" do
           all_are_payloads = @prepared.all? {|chunk| chunk.instance_of?(Payload)}
           all_are_payloads.should == true
         end
-        
+
         it "sets the private_key on all payloads" do
           @prepared.each do |payload|
             payload.project_private_key.should == 'something_super_secret'
           end
         end
-
-        it "the first payload contains approximately 10,000 characters"
-        it "the second payload contains approximately 5,000 characters"
       end
     end
   end

data/spec/codefumes/project_spec.rb

@@ -19,6 +19,7 @@ def register_show_uri(public_key = "public_key_value", project_name = "The Proje
 
 end
 
+
 describe "Project" do
   before(:each) do
   end
@@ -119,15 +120,18 @@ describe "Project" do
                             :status => ["200", "Successful"],
                             :string =>  "")
     end
+
     it "sets basic auth with the public and private key" do
       Project.should_receive(:delete).with("/projects/#{@project.public_key}", :basic_auth => {:username => @project.public_key, :password => @project.private_key}).and_return(mock("response", :code => 401))
       @project.delete
     end
+
     context "with Sucessful response" do
       it "returns true" do
         @project.delete.should be_true
       end
     end
+
     context "with Unauthorized response" do
       before(:each) do 
         @project = Project.new(:public_key => 'public_key_value')
@@ -135,6 +139,7 @@ describe "Project" do
                               :status => ["401", "Unauthorized"],
                               :string =>  "")
       end
+
       it "returns false when invalid Unauthorized response is received" do
         @project.delete.should be_false
       end
@@ -211,6 +216,19 @@ describe "Project" do
     end
   end
 
+  describe "claim" do
+    before(:each) do
+      @project = Project.new(:public_key => 'public_key_value', :private_key => 'private_key_value')
+      @api_key = "USERS_API_KEY"
+      ConfigFile.stub!(:credentials).and_return({:api_key => @api_key})
+    end
+
+    it "delegates the request to the Claim class" do
+      Claim.should_receive(:create).with(@project, @api_key)
+      @project.claim
+    end
+  end
+
   describe "protected attributes" do
     [:api_uri, :community_uri, :short_uri].each do |attribute_name|
       it "values passed in during initiazation for '#{attribute_name.to_s}' are silently ignored" do

data/spec/codefumes_service_stubs.rb

@@ -0,0 +1,54 @@
+module CodeFumesServiceStubs
+  def single_commit(options = {})
+    commit_xml = "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n"
+    commit_xml += <<-END_OF_COMMIT
+    <commit>
+      <identifier>f3badd5624dfbcf5176f0471261731e1b92ce957</identifier>
+      <author_name>John Doe</author_name>
+      <author_email>jdoe@example.com</author_email>
+      <committer_name>John Doe</committer_name>
+      <committer_email>jdoe@example.com</committer_email>
+      <short_message>Made command-line option for 'name' actually work</short_message>
+      <message>
+        Made command-line option for 'name' actually work
+        - Commentd out hard-coded 'require' line used for testing
+      </message>
+      <parent_identifiers>9ddj48423jdsjds5176f0471261731e1b92ce957,3ewdjok23jdsjds5176f0471261731e1b92ce957,284djsksjfjsjds5176f0471261731e1b92ce957</parent_identifiers>
+      <committed_at>Wed May 20 09:09:06 -0500 2009</committed_at>
+      <authored_at>Wed May 20 09:09:06 -0500 2009</authored_at>
+      <uploaded_at>2009-06-04 02:43:20 UTC</uploaded_at>
+      <api_uri>http://localhost:3000/api/v1/commits/f3badd5624dfbcf5176f0471261731e1b92ce957.xml</api_uri>
+      <line_additions>20</line_additions>
+      <line_deletions>10</line_deletions>
+      <line_total>30</line_total>
+      <affected_file_count>2</affected_file_count>
+    END_OF_COMMIT
+
+    if options[:include_custom_attributes]
+      commit_xml <<
+      <<-END_OF_COMMIT
+        <custom_attributes>
+          <coverage>83</coverage>
+          <random_attribute>1</random_attribute>
+        </custom_attributes>
+      END_OF_COMMIT
+    end
+
+    commit_xml << "\n</commit>"
+  end
+
+  def register_index_uri
+    FakeWeb.register_uri(
+      :get, "http://www.codefumes.com:80/api/v1/xml/projects/apk/commits",
+      :status => ["200", "Ok"],
+      :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<commits>\n#{single_commit}\n#{single_commit}\n#{single_commit}\n</commits>\n")
+  end
+
+  def stub_codefumes_uri(api_uri, status, response_string)
+    FakeWeb.register_uri(
+      :get, "http://www.codefumes.com:80/api/v1/xml/#{api_uri}",
+      :status => status,
+      :string =>  "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n#{response_string}")
+  end
+end
+

data/spec/spec_helper.rb

@@ -1,18 +1,16 @@
-require 'rubygems'
+require 'ruby-debug'
+require 'fakeweb'
 
 begin
   require 'spec/autorun'
 rescue LoadError
-  gem 'rspec'
   require 'spec'
 end
 
-gem 'ruby-debug'
-require 'ruby-debug'
-
-$:.unshift(File.dirname(__FILE__) + '/../lib')
 require 'codefumes'
-require 'fakeweb'
+
+# CodeFumes service 'fixtures'
+require 'spec/codefumes_service_stubs'
 
 include CodeFumes
 

data/spec/store_codefumes_credentials/cli_spec.rb

@@ -0,0 +1,22 @@
+require 'spec/spec_helper'
+require 'lib/store_codefumes_credentials/cli'
+
+describe StoreCodefumesCredentials::CLI, "execute" do
+  after(:all) do
+    unless ConfigFile.path == File.expand_path('~/.codefumes_config')
+      File.delete(ConfigFile.path) if File.exist?(ConfigFile.path)
+    end
+  end
+
+  before(:each) do
+    @api_key_value = "API_KEY"
+    @stdout_io = StringIO.new
+    StoreCodefumesCredentials::CLI.execute(@stdout_io, [@api_key_value])
+    @stdout_io.rewind
+    @stdout = @stdout_io.read
+  end
+
+  it "store the value supplied as an argument in the config file" do
+    ConfigFile.credentials.keys.should include(:api_key)
+  end
+end

data/tasks/rspec.rake

@@ -18,4 +18,5 @@ desc "Run the specs under spec"
 Spec::Rake::SpecTask.new do |t|
   t.spec_opts = ['--options', "spec/spec.opts"]
   t.spec_files = FileList['spec/**/*_spec.rb']
+  t.libs << 'lib'
 end