pgit 0.0.4 → 1.0.0

data/spec/pgit/pivotal/iterations_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe PGit::Pivotal::Iterations do
+  describe '#get!' do
+    it 'makes the correct sublink' do
+      project = instance_double('PGit::Project', id: 123, api_token: 'someapitoken')
+      allow(PGit::CurrentProject).to receive(:new).and_return(project)
+      query = instance_double('PGit::Pivotal::Request::Query', to_s: "?scope=current_backlog")
+      hash_query = {scope: :current_backlog}
+      allow(PGit::Pivotal::Request::Query).to receive(:new).with(hash_query).and_return(query)
+      iterations = PGit::Pivotal::Iterations.new(hash_query)
+
+      expect(iterations.sublink).to eq "projects/123/iterations/?scope=current_backlog"
+    end
+
+    it 'instantiates iterations' do
+      iterations_file = File.read(File.join(PGit.root, 'spec', 'fixtures', 'iterations'))
+      iterations_json = JSON.parse(iterations_file)
+      first_iteration_json = iterations_json.first
+      last_iteration_json = iterations_json.last
+      first_iteration = instance_double('PGit::Pivotal::Iteration')
+      last_iteration = instance_double('PGit::Pivotal::Iteration')
+      fake_iterations = [first_iteration, last_iteration]
+      iteration_config = double('config', :hash= => nil)
+      allow(PGit::Pivotal::Iteration).to receive(:new).and_yield(iteration_config).and_return(first_iteration, last_iteration)
+      project = instance_double('PGit::Project', api_token: double('api_token'), id: double('id'))
+      allow(PGit::CurrentProject).to receive(:new).and_return(project)
+      iterations_instance = PGit::Pivotal::Iterations.new(project)
+      allow(iterations_instance).to receive(:get_request).and_return(iterations_file)
+
+      iterations = iterations_instance.get!
+
+      expect(iterations.first).to eq first_iteration
+      expect(iterations.last).to eq last_iteration
+    end
+  end
+end

data/spec/pgit/pivotal/project_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+describe PGit::Pivotal::Project do
+  describe '.ancestors' do
+    it 'should include PGit::Pivotal::IndividualRequest' do
+      expect(PGit::Pivotal::Project.ancestors).to include(PGit::Pivotal::IndividualRequest)
+    end
+  end
+end

data/spec/pgit/pivotal/projects_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+
+describe PGit::Pivotal::Projects do
+  it 'should be a CollectionRequest' do
+    expect(PGit::Pivotal::Projects.ancestors).to include(PGit::Pivotal::CollectionRequest)
+  end
+
+  describe 'get!' do
+    it 'should instantiate a bunch of PGit::Pivotal::Project instances' do
+      project = double('PGit::Pivotal::Project')
+      allow(project).to receive(:hash=)
+      get_request = double('get_request')
+      api_token = 'some_api_token_123'
+      hashes_of_items = [{'name' => 'PGit', 'kind' => 'project' }]
+      allow(PGit::Pivotal::Project).to receive(:new).and_yield(project).and_return(project)
+      allow_any_instance_of(PGit::Pivotal::CollectionRequest).to receive(:api_token).and_return(api_token)
+      allow(JSON).to receive(:parse).with(get_request).and_return(hashes_of_items)
+
+      projects = PGit::Pivotal::Projects.new
+      allow(projects).to receive(:get_request).and_return(get_request)
+      instantiated_projects = projects.get!
+      expect(instantiated_projects).to include(project)
+      expect(projects.api_token).to eq api_token
+    end
+  end
+
+  describe 'with specific api_token' do
+    it 'should instantiate a bunch of PGit::Pivotal::Project instances only associated with the token' do
+      project = double('PGit::Pivotal::Project')
+      allow(project).to receive(:hash=)
+      get_request = double('get_request')
+      hashes_of_items = [{'name' => 'PGit', 'kind' => 'project' }]
+      allow(PGit::Pivotal::Project).to receive(:new).and_yield(project).and_return(project)
+      allow(JSON).to receive(:parse).with(get_request).and_return(hashes_of_items)
+
+      api_token = 'some_api_token_123'
+      projects = PGit::Pivotal::Projects.new(api_token: api_token)
+      expect(projects.api_token).to eq api_token
+    end
+  end
+
+  describe 'sublink' do
+    it 'should be "projects"' do
+      projects = PGit::Pivotal::Projects.new
+      expect(projects.sublink).to eq 'projects'
+    end
+  end
+end

data/spec/pgit/pivotal/request/query_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe PGit::Pivotal::Request::Query do
+  describe '#new(scope: :current_backlog)' do
+    it 'should generate the right query string ' do
+      query = PGit::Pivotal::Request::Query.new
+      expect("#{query}").to eq ''
+    end
+  end
+
+  describe '#new(scope: :current_backlog)' do
+    it 'should generate the right query string ' do
+      query = PGit::Pivotal::Request::Query.new(scope: :current_backlog)
+      expect("#{query}").to eq '?scope=current_backlog'
+    end
+  end
+
+  describe '#new(scope: :current_backlog, offset: 1)' do
+    it 'should generate the right query string ' do
+      query = PGit::Pivotal::Request::Query.new(scope: :current_backlog, offset: 1)
+      expect("#{query}").to eq '?scope=current_backlog&offset=1'
+    end
+  end
+end

data/spec/pgit/pivotal/story_spec.rb

@@ -0,0 +1,113 @@
+require 'spec_helper'
+
+describe 'PGit::Pivotal::Story' do
+  describe '#get!' do
+    it 'should generate the story' do
+      story_id = '123'
+      current_project = double('current_project', id: '321', api_token: 'abc10xyz')
+      get_request = "curl -X GET -H 'X-TrackerToken: abc10xyz' https://www.pivotaltracker.com/services/v5/projects/321/stories/123"
+      fake_good_json = <<-GOOD_JSON
+      {
+         "created_at": "2014-11-25T12:00:00Z",
+         "current_state": "unstarted",
+         "description": "ignore the droids",
+         "estimate": 2,
+         "id": 555,
+         "kind": "story",
+         "labels": [],
+         "name": "Bring me the passengers",
+         "owner_ids": [],
+         "project_id": 99,
+         "requested_by_id": 101,
+         "story_type": "feature",
+         "updated_at": "2014-11-25T12:00:00Z",
+         "url": "http://localhost/story/show/555"
+      }
+      GOOD_JSON
+
+      allow(PGit::CurrentProject).to receive(:new).and_return(current_project)
+      story = PGit::Pivotal::Story.new(story_id)
+      allow(story).to receive(:`).with(get_request).and_return(fake_good_json)
+      allow_any_instance_of(PGit::Pivotal::IndividualRequest).to receive(:`).with(get_request).and_return(fake_good_json)
+      story.get!
+
+      expect(story.name).to eq "Bring me the passengers"
+      expect(story.description).to eq "ignore the droids"
+    end
+  end
+
+  describe '#put' do
+    it 'updates the story' do
+      story_id = '123'
+      current_project = double('current_project', id: '321', api_token: 'abc10xyz')
+      put_request = "curl -X PUT -H 'X-TrackerToken: abc10xyz' -H 'Content-Type: application/json' -d '{\"estimate\":3}' https://www.pivotaltracker.com/services/v5/projects/321/stories/123"
+      fake_good_json = <<-GOOD_JSON
+      {
+         "created_at": "2014-11-25T12:00:00Z",
+         "current_state": "unstarted",
+         "description": "ignore the droids",
+         "estimate": 2,
+         "id": 555,
+         "kind": "story",
+         "labels": [],
+         "name": "Bring me the passengers",
+         "owner_ids": [],
+         "project_id": 99,
+         "requested_by_id": 101,
+         "story_type": "feature",
+         "updated_at": "2014-11-25T12:00:00Z",
+         "url": "http://localhost/story/show/555"
+      }
+      GOOD_JSON
+
+
+      allow(PGit::CurrentProject).to receive(:new).and_return(current_project)
+      story = PGit::Pivotal::Story.new(story_id)
+      allow(story).to receive(:`)
+      story.estimate = 3
+      story.put!
+
+      expect(story).to have_received(:`).with(put_request)
+    end
+
+    describe 'with a story instantiated with a block' do
+      it 'should still update the story' do
+        story_id = '123'
+        current_project = double('current_project', id: '321', api_token: 'abc10xyz')
+        get_request = "curl -X GET -H 'X-TrackerToken: abc10xyz' https://www.pivotaltracker.com/services/v5/projects/321/stories/123"
+        put_request = "curl -X PUT -H 'X-TrackerToken: abc10xyz' -H 'Content-Type: application/json' -d '{\"estimate\":3,\"requested_by_id\":102}' https://www.pivotaltracker.com/services/v5/projects/321/stories/123"
+        fake_good_json = <<-GOOD_JSON
+      {
+         "created_at": "2014-11-25T12:00:00Z",
+         "current_state": "unstarted",
+         "description": "ignore the droids",
+         "estimate": 2,
+         "id": 123,
+         "kind": "story",
+         "labels": [],
+         "name": "Bring me the passengers",
+         "owner_ids": [],
+         "project_id": 321,
+         "requested_by_id": 101,
+         "story_type": "feature",
+         "updated_at": "2014-11-25T12:00:00Z",
+         "url": "http://localhost/story/show/555"
+      }
+        GOOD_JSON
+
+
+        allow(PGit::CurrentProject).to receive(:new).and_return(current_project)
+        story = PGit::Pivotal::Story.new(story_id)
+        allow(story).to receive(:`).with(get_request).and_return(fake_good_json)
+        allow(story).to receive(:`).with(put_request)
+        allow_any_instance_of(PGit::Pivotal::IndividualRequest).to receive(:`).with(get_request).and_return(fake_good_json)
+        story.get!
+        story.estimate = 3
+        story.requested_by_id = 102
+        story.put!
+
+        expect(story).to have_received(:`).with(put_request)
+      end
+    end
+  end
+end

data/spec/pgit/pivotal_request_validator_spec.rb

@@ -40,19 +40,19 @@ describe 'PGit::PivotalRequestValidator' do
 
       expect do
         PGit::PivotalRequestValidator.new(request)
-      end.to raise_error(PGit::ExternalError)
+      end.to raise_error(PGit::Error::External)
     end
   end
 
   describe 'when the request has no "kind"' do
-    it 'should raise a PGit::ExternalError' do
+    it 'should raise a PGit::Error::External' do
       request = <<-REQUEST
         some unrecognizable request
       REQUEST
 
       expect do
         PGit::PivotalRequestValidator.new(request)
-      end.to raise_error(PGit::ExternalError)
+      end.to raise_error(PGit::Error::External)
     end
   end
 

data/spec/pgit/project/add_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe 'PGit::Project::Add' do
+  describe 'user is on a project path that already exists' do
+    it 'raises an error' do
+      app = instance_double('PGit::Project::Application',
+                            exists?: true)
+      expect{PGit::Project::Add.new(app)}.to raise_error(PGit::Error::User, 'Project path already exists. See `pgit proj update --help.`')
+    end
+  end
+
+  describe '#execute!' do
+    it 'instantiates a project and saves it' do
+      class SomeProject
+        include ActiveModel::Validations
+        validates_with PGit::Validators::ProjectValidator
+        def get!
+        end
+
+        def kind
+          'project'
+        end
+      end
+
+      project = SomeProject.new
+      config_project = instance_double('PGit::Project')
+      projects = [config_project]
+      app = instance_double('PGit::Project::Application',
+                            exists?: false,
+                            project: project,
+                            projects: projects)
+      adder = instance_double('PGit::Project::InteractiveAdder',
+                              execute!: nil,
+                              project: project,
+                              save!: nil)
+      reuse_adder = instance_double('PGit::Project::ReuseApiTokenAdder',
+                                    execute!: nil,
+                                    project: project)
+      allow(PGit::Project::InteractiveAdder).to receive(:new).with(project).and_return(adder)
+      allow(PGit::Project::ReuseApiTokenAdder).to receive(:new).with(project, projects).and_return(reuse_adder)
+      add = PGit::Project::Add.new(app)
+      allow(add).to receive(:puts).with("Successfully added the project!")
+
+      add.execute!
+
+      expect(reuse_adder).to have_received(:execute!)
+      expect(adder).to have_received(:execute!)
+      expect(adder).to have_received(:save!)
+      expect(add).to have_received(:puts).with("Successfully added the project!")
+    end
+  end
+end

data/spec/pgit/project/application_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+describe 'PGit::Project::Application' do
+  describe '#exists?' do
+    it 'delegates to @project' do
+      global_opts = {}
+      opts = { 'path' => '/some/path',
+               'api_token' => 'someapitoken1234',
+               'id' => 12345}
+      args = []
+      configuration = instance_double('PGit::Configuration')
+      allow(PGit::Configuration).to receive(:new).and_return(configuration)
+      project_exists = double('response')
+      allow_any_instance_of(PGit::Project).to receive(:exists?).and_return(project_exists)
+      app = PGit::Project::Application.new(global_opts, opts, args)
+
+      expect(app.exists?).to eq project_exists
+    end
+
+    it 'delegates to @project' do
+      global_opts = {}
+      opts = { 'path' => '/some/path',
+               'api_token' => 'someapitoken1234',
+               'id' => 12345}
+      args = []
+      configuration = instance_double('PGit::Configuration')
+      allow(PGit::Configuration).to receive(:new).and_return(configuration)
+      project_save = double('response')
+      allow_any_instance_of(PGit::Project).to receive(:save!).and_return(project_save)
+      app = PGit::Project::Application.new(global_opts, opts, args)
+
+      expect(app.save!).to eq project_save
+    end
+  end
+
+  describe '#project' do
+    it 'returns the project' do
+      global_opts = {}
+      opts = { 'path' => '/some/path',
+               'api_token' => 'someapitoken1234',
+               'id' => 12345}
+      args = []
+      proj = instance_double('PGit::Project')
+      allow(PGit::Project).to receive(:new).and_return(proj)
+
+      app = PGit::Project::Application.new(global_opts, opts, args)
+      expect(app.project).to eq proj
+    end
+  end
+
+  describe '#projects' do
+    it 'returns the projects' do
+      global_opts = {}
+      opts = { 'path' => '/some/path',
+               'api_token' => 'someapitoken1234',
+               'id' => 12345}
+      args = []
+      proj = instance_double('PGit::Project')
+      projects = instance_double('Array')
+      config = instance_double('PGit::Configuration', projects: projects)
+      allow(PGit::Configuration).to receive(:new).and_return(config)
+      allow(PGit::Project).to receive(:new).and_return(proj)
+
+      app = PGit::Project::Application.new(global_opts, opts, args)
+      expect(app.projects).to eq projects
+    end
+  end
+end
+

data/spec/pgit/project/interactive_adder_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+require 'byebug'
+
+describe 'PGit::Project::InteractiveAdder.new(proj)' do
+  describe 'api_token does not exist' do
+    it 'asks what the api token associated for the project is' do
+      question = "What's the project api_token?"
+      api_token = 'SOMEAPITOKEN'
+      id_1 = 123
+      id_2 = 127
+      project = instance_double('PGit::Project',
+                                defaulted_attrs: [:api_token],
+                               :id= => true)
+
+      def project.api_token
+        @api_token
+      end
+
+      def project.api_token=(some_api_token)
+        @api_token = some_api_token
+      end
+
+      project.api_token = :no_api_token_given
+
+      allow_any_instance_of(PGit::Project::InteractiveAdder).to receive(:puts).with(question)
+      response = instance_double('String', chomp: api_token)
+      allow(STDIN).to receive(:gets).and_return(response, '0')
+      pivotal_project_1 = double('PGit::Pivotal::Project', name: 'Some Project Name 1',
+                                id: id_1)
+      pivotal_project_2 = double('PGit::Pivotal::Project', name: 'Some Project Name 2',
+                                id: id_2)
+      instantiated_projects = [pivotal_project_1, pivotal_project_2]
+      projects = double('PGit::Pivotal::Projects')
+      allow(projects).to receive(:get!).and_return(instantiated_projects)
+      allow(PGit::Pivotal::Projects).to receive(:new).with(api_token: api_token).
+        and_return(projects)
+      adder = PGit::Project::InteractiveAdder.new(project)
+      adder.execute!
+
+      expect(adder).to have_received(:puts).with(question)
+      expect(project.api_token).to eq api_token
+      expect(project).to have_received(:id=).with(id_1)
+    end
+  end
+end

data/spec/pgit/project/remove_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+describe 'PGit::Project::Remove' do
+  describe '#execute' do
+    describe 'project path matches a project path in the configuration' do
+      describe 'user answers yes' do
+        it 'should remove the project' do
+          path = 'some/path'
+          common_path = 'some/path'
+          project = instance_double('PGit::Project', path: common_path, exists?: true)
+          config_project_1 = instance_double('PGit::Project', path: common_path)
+          config_project_2 = instance_double('PGit::Project', path: 'some/other/path')
+          projects = [config_project_1, config_project_2]
+          app = instance_double('PGit::Project::Application', project: project, projects: projects)
+          rm = PGit::Project::Remove.new(app)
+          question = "Are you sure you want to remove #{path} from the configuration file?"
+          config = double('config', :question= => nil, :options= => nil)
+          confirmation_question = instance_double('Interactive::Question')
+          options = [:yes, :no]
+          response = double('response', yes?: true)
+          allow(Interactive::Question).to receive(:new).and_yield(config).and_return(confirmation_question)
+          allow(config).to receive(:question).with(question)
+          allow(config).to receive(:options).with(options)
+          allow(confirmation_question).to receive(:ask_and_wait_for_valid_response).and_yield(response)
+          allow(rm).to receive(:puts)
+          allow(project).to receive(:remove!)
+
+          rm.execute!
+          expect(Interactive::Question).to have_received(:new)
+          expect(config).to have_received(:question=).with(question)
+          expect(config).to have_received(:options=).with(options)
+          expect(confirmation_question).to have_received(:ask_and_wait_for_valid_response)
+          expect(rm).to have_received(:puts).with("Removing #{path} from the configuration file...")
+          expect(rm).to have_received(:puts).with("Removed.")
+          expect(project).to have_received(:remove!)
+        end
+      end
+
+      describe 'user answers no' do
+        it 'should respond with "Canceling..."' do
+          path = 'some/path'
+          common_path = 'some/path'
+          project = instance_double('PGit::Project', path: common_path, exists?: true)
+          config_project_1 = instance_double('PGit::Project', path: common_path)
+          config_project_2 = instance_double('PGit::Project', path: 'some/other/path')
+          projects = [config_project_1, config_project_2]
+          app = instance_double('PGit::Project::Application', project: project, projects: projects)
+          rm = PGit::Project::Remove.new(app)
+          question = "Are you sure you want to remove #{path} from the configuration file?"
+          config = double('config', :question= => nil, :options= => nil)
+          confirmation_question = instance_double('Interactive::Question')
+          options = [:yes, :no]
+          response = double('response', yes?: false, no?: true)
+          allow(Interactive::Question).to receive(:new).and_yield(config).and_return(confirmation_question)
+          allow(config).to receive(:question).with(question)
+          allow(config).to receive(:options).with(options)
+          allow(confirmation_question).to receive(:ask_and_wait_for_valid_response).and_yield(response)
+          allow(rm).to receive(:puts)
+          allow(project).to receive(:remove!)
+
+          rm.execute!
+          expect(Interactive::Question).to have_received(:new)
+          expect(config).to have_received(:question=).with(question)
+          expect(config).to have_received(:options=).with(options)
+          expect(confirmation_question).to have_received(:ask_and_wait_for_valid_response)
+          expect(rm).to have_received(:puts).with("Cancelling...")
+          expect(rm).not_to have_received(:puts).with("Removing #{path} from the configuration file...")
+          expect(project).not_to have_received(:remove!)
+        end
+      end
+    end
+  end
+
+  describe 'project path does not match a project path in the configuration' do
+    it 'raises an error' do
+      path = 'some/path'
+      some_path_not_in_config = 'some/path/not/in/config'
+      project = instance_double('PGit::Project', path: some_path_not_in_config, exists?: false)
+      config_project_1 = instance_double('PGit::Project', path: path)
+      config_project_2 = instance_double('PGit::Project', path: 'some/other/path')
+      projects = [config_project_1, config_project_2]
+      app = instance_double('PGit::Project::Application', project: project, projects: projects)
+      expect{PGit::Project::Remove.new(app)}.to raise_error(PGit::Error::User, "#{some_path_not_in_config} is not in the configuration file.")
+    end
+  end
+end