codefumes 0.1.10 → 0.2.0

data/spec/codefumes/api/payload_spec.rb

@@ -0,0 +1,148 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+describe "API::Payload" do
+  include CodeFumesServiceHelpers::PayloadHelpers
+
+  after(:all) do
+    FakeWeb.allow_net_connect = false
+    FakeWeb.clean_registry
+  end
+
+  before(:each) do
+    setup_fixture_base
+    @payload = Payload.new(@project, @commit_data)
+    @payload_query = {:payload => {:commits => @commit_data}}
+    @basic_auth_params = {:username => @pub_key, :password => @priv_key}
+  end
+
+  describe "#save" do
+    it "sets basic auth with the public and private key" do
+      register_create_uri(["401", "Unauthorized"], "")
+      API.should_receive(:post).with("/projects/#{@project.public_key}/payloads", :query => @payload_query, :basic_auth => @basic_auth_params).and_return(mock("response", :code => 401))
+      @payload.save
+    end
+
+    context "with valid parameters" do
+      before(:each) do
+        register_create_uri(["201", "Created"])
+      end
+
+      it "sets a value for 'created_at'" do
+        @payload.created_at.should == nil
+        @payload.save.should == true
+        @payload.created_at.should_not == nil
+      end
+    end
+
+    context "with Unauthorized response" do
+      before(:each) do
+        register_create_uri(["401", "Unauthorized"], "")
+      end
+
+      it "does not set 'created_at'" do
+        @payload.created_at.should == nil
+        @payload.save.should == false
+        @payload.created_at.should == nil
+      end
+    end
+
+    context "when the payload does not have any commit information" do
+      before(:each) do
+        @payload = Payload.new(@project, "")
+      end
+
+      it "returns true without attempting to save to the site" do
+        Payload.should_not_receive(:post)
+        @payload.save.should == true
+      end
+
+      it "does not set the value of created_at" do
+        @payload.save
+        @payload.created_at.should == nil
+      end
+    end
+
+    context "with invalid parameters" do
+      before(:each) do
+        register_create_uri
+      end
+
+      it "does not set a value for 'created_at'" do
+        @payload.save.should == false
+        @payload.created_at.should == nil
+      end
+    end
+  end
+
+  describe "#prepare" do
+    it "is compatible with the entity returned from SourceControl#payload_between" do
+      git_repo_path = File.expand_path(File.dirname(__FILE__) + '/../../fixtures/sample_project_dirs/git_repository')
+      repository = SourceControl.new(git_repo_path)
+      project = Project.new('public_key', 'private_key')
+      lambda {
+        pl = Payload.prepare(project, repository.payload_between('HEAD^', 'HEAD'))
+        pl.length.should == 1
+      }.should_not raise_error
+    end
+
+    context "when supplying nil commit information" do
+      it "returns an empty array" do
+        Payload.prepare(@project, nil).should == []
+      end
+    end
+
+    context "when supplying an empty Hash" do
+      it "returns an empty array" do
+        Payload.prepare(@project, {}).should == []
+      end
+    end
+
+    context "when supplying a hash with less than 4,000 characters" do
+      before(:each) do
+        single_commit_ex =  {:identifier => "92dd08477f0ca144ee0f12ba083760dd810760a2_000"}
+        commit_count = 4000 / single_commit_ex.to_s.length
+        commits = commit_count.times.map do |index|
+          {:identifier => "92dd08477f0ca144ee0f12ba083760dd810760a2_#{index}"}
+        end
+        @prepared = Payload.prepare(@project, {:commits => commits})
+      end
+
+      it "returns an Array with a single payload element" do
+        @prepared.should be_instance_of(Array)
+        @prepared.size.should == 1
+        @prepared.first.should be_instance_of(Payload)
+      end
+
+      it "associated the specified project with all generated payloads" do
+        @prepared.each do |payload|
+          payload.project.should == @project
+        end
+      end
+    end
+
+    context "when supplying a hash with approximately 15,000 characters" do
+      before(:each) do
+        single_commit_ex =  {:identifier => "92dd08477f0ca144ee0f12ba083760dd810760a2_000"}
+        commit_count = 15000 / single_commit_ex.to_s.length + 1
+        commits = commit_count.times.map do |index|
+          {:identifier => "92dd08477f0ca144ee0f12ba083760dd810760a2_#{index}"}
+        end
+        raw_payload = {:commits => commits}
+        @prepared = Payload.prepare(@project, raw_payload)
+      end
+
+      it "returns an Array with a four payload elements" do
+        @prepared.should be_instance_of(Array)
+        @prepared.size.should == 4
+        all_are_payloads = @prepared.all? {|chunk| chunk.instance_of?(Payload)}
+        all_are_payloads.should == true
+      end
+
+      it "associated the specified project with all generated payloads" do
+        @prepared.each do |payload|
+          payload.project.should == @project
+        end
+      end
+    end
+  end
+end

data/spec/codefumes/api/project_spec.rb

@@ -0,0 +1,286 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+describe "API::Project" do
+  include CodeFumesServiceHelpers::ProjectHelpers
+
+  before(:each) do
+    setup_fixture_base
+  end
+
+  after(:all) do
+    FakeWeb.allow_net_connect = false
+    FakeWeb.clean_registry
+  end
+
+  describe "#create" do
+    context "when successful" do
+      before(:each) do
+        register_create_uri
+      end
+
+      [:public_key,
+       :private_key,
+       :short_uri,
+       :community_uri,
+       :api_uri].each do |method_name|
+         it "sets the '#{method_name.to_s}'" do
+           project = Project.create
+           project.send(method_name).should_not be_nil
+         end
+       end
+    end
+
+    context "when unsuccessful" do
+      before(:each) do
+        register_create_uri(["404", "Not found"])
+      end
+
+      specify {Project.create.should be_false}
+    end
+  end
+
+  describe "#save" do
+    context "with valid parameters" do
+      it "sets basic auth with the public and private key" do
+        register_update_uri
+        API.should_receive(:put).with("/projects/#{@project.public_key}", :query => anything(), :basic_auth => {:username => @project.public_key, :password => @project.private_key}).and_return(mock("response", :code => 401))
+        @project.save
+      end
+
+      context "and the response is '200 Ok'" do
+        before(:each) do
+          register_update_uri
+          @project.name = @updated_name
+          @project.save.should be_true
+        end
+
+        [ :public_key,
+          :private_key,
+          :short_uri,
+          :community_uri,
+          :name,
+          :api_uri].each do |method_name|
+            it "sets the '#{method_name.to_s}'" do
+              @project.send(method_name).should_not be_nil
+            end
+        end
+      end
+
+      context "and the response is '401 Unauthorized'" do
+        it "returns false" do
+          @project.name = @updated_name
+          register_update_uri(["401", "Unauthorized"])
+          @project.save.should be_false
+        end
+      end
+    end
+
+    context "with invalid parameters" do
+      before(:each) do
+        register_show_uri(["404", "Not found"], "")
+        register_create_uri(["422", "Unprocessable Entity"], "")
+      end
+
+      it "returns false" do
+        Project.create.should be_false
+      end
+    end
+  end
+
+  context "#delete" do
+    before(:each) do
+      FakeWeb.register_uri(:delete, @authd_project_api_uri, :status => ["200", "Successful"], :body   => "")
+    end
+
+    it "sets basic auth with the public and private key" do
+      API.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 "without a private key specified" do
+      it "raises an InsufficientCredentials error" do
+        lambda {Project.new('public_key').delete}.should raise_error(Errors::InsufficientCredentials)
+      end
+    end
+
+    context "without a public key specified" do
+      it "raises an InsufficientCredentials error" do
+        lambda {Project.new(nil, 'private_key').delete}.should raise_error(Errors::InsufficientCredentials)
+      end
+    end
+
+    context "with Sucessful response" do
+      it "returns true" do
+        @project.delete.should be_true
+      end
+    end
+
+    context "with Unauthorized response" do
+      it "returns false when invalid Unauthorized response is received" do
+        register_delete_uri(["401", "Unauthorized"], "")
+        @project.delete.should be_false
+      end
+    end
+  end
+
+  describe "#to_config" do
+    before(:each) do
+      register_show_uri(["404", "Not Found"], "")
+      register_create_uri
+      @project = Project.create
+    end
+
+    it "returns an object keyed by the project's public_key as a symbol" do
+      @project.to_config.should include(@project.public_key.to_sym)
+    end
+
+    context "the content under the project's public_key element" do
+      [:private_key, :api_uri, :short_uri].each do |project_attribute|
+        it "includes a key-value pair of ':#{project_attribute.to_s} => [project's #{project_attribute.to_s}]'" do
+          value = @project.send(project_attribute)
+          @project.to_config[@project.public_key.to_sym].should include(project_attribute => value)
+        end
+      end
+    end
+
+    it "doesn't include the private_key in the hash if it is nil" do
+      public_key = "key_to_happiness"
+      Project.new(public_key).to_config[public_key.to_sym].should_not have_key(:private_key)
+    end
+  end
+
+  describe "#claim" do
+    let(:project) {Project.new('public_key_value', 'private_key_value')}
+    let(:api_key) {"USERS_API_KEY"}
+
+    before(:each) do
+      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 "#release" do
+    let(:project) {Project.new('public_key_value', 'private_key_value')}
+    let(:api_key) {"USERS_API_KEY"}
+
+    before(:each) do
+      ConfigFile.stub!(:credentials).and_return({:api_key => api_key})
+    end
+
+    it "delegates the request to the Claim class" do
+      Claim.should_receive(:destroy).with(project, api_key)
+      project.release
+    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
+        p = Project.new(attribute_name => Time.now.to_s)
+        p.send(attribute_name).should be_nil
+      end
+
+      it "calling '#{attribute_name.to_s}= [value]' is not supported" do
+        p = Project.new
+        lambda {p.send("#{attribute_name}=", "my_value")}.should raise_error
+      end
+    end
+  end
+
+  describe "accessible attributes" do
+    let(:key_specified) {"public_key"}
+    let(:name_specified) {"project_name"}
+    let(:project) {Project.new(key_specified, '', :name => name_specified)}
+
+    specify {project.public_key.should == key_specified}
+    specify {project.name.should == name_specified}
+  end
+
+  describe "#find" do
+    context "specifying a public_key which is already associated to a project on the site" do
+      it "returns an initialized instance of the Project class" do
+        register_show_uri
+        expected_config = {
+                            @pub_key.to_sym =>
+                              {
+                                :private_key=>"private_key_value",
+                                :api_uri=>"http://codefumes.com/api/v1/xml/projects/#{@pub_key}.xml",
+                                :short_uri=>"http://codefumes.com/p/#{@pub_key}"
+                              }
+                          }
+        Project.find(@pub_key).to_config.should == expected_config
+      end
+    end
+
+    # TODO: clean up duplication w/ non-Symbol test
+    context "specifying a public_key which is already associated to a project on the site as a Symbol" do
+      it "returns an initialized instance of the Project class" do
+        register_show_uri
+        expected_config = {
+                            @pub_key.to_sym =>
+                              {
+                                :private_key=>"private_key_value",
+                                :api_uri=>"http://codefumes.com/api/v1/xml/projects/#{@pub_key}.xml",
+                                :short_uri=>"http://codefumes.com/p/#{@pub_key}"
+                              }
+                          }
+        Project.find(@pub_key.to_sym).to_config.should == expected_config
+      end
+    end
+
+    context "specifying a public_key which is not associated to any project on the site yet" do
+      before(:each) do
+        register_show_uri(["404", "Not Found"], "")
+      end
+
+      it "returns nil" do
+        Project.find(@pub_key).should == nil
+      end
+    end
+
+    context "supplying a nil public key" do
+      it "does not hit the CodeFumes server" do
+        API.should_not_receive(:get)
+        Project.find(nil).should == nil
+      end
+      it "returns nil" do
+        Project.find(nil).should == nil
+      end
+    end
+
+    context "supplying an empty string as a public key" do
+      it "does not hit the CodeFumes server" do
+        API.should_not_receive(:get)
+        Project.find('').should == nil
+      end
+
+      it "returns nil" do
+        Project.find('').should == nil
+      end
+    end
+  end
+
+  describe "#reinitialize_from_hash!" do
+    let(:option_keys) {[:name, :public_key, :private_key, :short_uri, :community_uri, :api_uri, :build_status]}
+
+    it "supports a Hash with Strings as keys" do
+      params = option_keys.inject({}) {|option_params,key| option_params.merge(key.to_s => key.to_s)}
+      project = Project.new.reinitialize_from_hash!(params)
+      option_keys.each do |attr_name|
+        project.send(attr_name).should == attr_name.to_s
+      end
+    end
+
+    it "supports a Hash with Symbols as keys" do
+      params = option_keys.inject({}) {|option_params,key| option_params.merge(key.to_sym => key.to_s)}
+      project = Project.new.reinitialize_from_hash!(params)
+      option_keys.each do |attr_name|
+        project.send(attr_name.to_sym).should == attr_name.to_s
+      end
+    end
+  end
+end

data/spec/codefumes/api_spec.rb

@@ -1,40 +1,63 @@
 require File.dirname(__FILE__) + '/../spec_helper.rb'
 
-class APIClass < CodeFumes::API
-end
-
 describe "API" do
   after(:all) do
-    CodeFumes::API.mode(:production)
+    CodeFumes::API.mode = :production
   end
 
   it "defaults the base uri to the production site" do
-    APIClass.base_uri.should == 'http://codefumes.com/api/v1/xml'
+    API.base_uri.should == 'http://codefumes.com/api/v1/xml'
   end
 
   context "switching modes" do
     before(:each) do
-      APIClass.mode(:test)
+      API.mode = :test
     end
 
     it "changes the base uri to the test site when switched to test mode" do
-      APIClass.base_uri.should == 'http://test.codefumes.com/api/v1/xml'
+      API.base_uri.should == API::BASE_URIS[:test]
     end
 
     it "changes the base uri to the production site when switched to production mode" do
-      APIClass.mode(:production)
-      APIClass.base_uri.should == 'http://codefumes.com/api/v1/xml'
+      API.mode = :production
+      API.base_uri.should == API::BASE_URIS[:production]
     end
 
     it "ignores unrecognized modes" do
-      APIClass.mode(:incomprehensible)
-      APIClass.base_uri.should == 'http://test.codefumes.com/api/v1/xml'
+      API.mode = :incomprehensible
+      API.base_uri.should == API::BASE_URIS[:test]
+    end
+
+    it "treats empty Strings as an unrecognized mode" do
+      API.mode = ''
+      API.base_uri.should == API::BASE_URIS[:test]
+    end
+
+    it "treats nil as an unrecognized mode" do
+      API.mode = nil
+      API.base_uri.should == API::BASE_URIS[:test]
+    end
+
+    it "supports String versions of the supported modes" do
+      API.base_uri.should == API::BASE_URIS[:test]
+      API.mode = 'production'
+      API.base_uri.should == API::BASE_URIS[:production]
     end
 
-    it "changes the base uri to 'localhost:3000' when switched to local mode (for developer testing)" do
-      APIClass.mode(:local)
-      APIClass.base_uri.should == 'http://codefumes.com.local/api/v1/xml'
+    it "changes the base uri to 'codefumes.com.local' when switched to local mode (for developer testing)" do
+      API.mode = :local
+      API.base_uri.should == API::BASE_URIS[:local]
+    end
+  end
+  context "#mode?" do
+    specify {API.mode?(:production).should == true}
+    specify {API.mode?('production').should == true}
+    specify {API.mode?(:test).should == false}
+    specify {API.mode?(:anything).should == false}
+    specify {API.mode?(nil).should == false}
+
+    it "does not modify the existing mode" do
+      lambda {API.mode?(:test)}.should_not change(API, :base_uri)
     end
   end
-  
 end