pgit 0.0.4 → 1.0.0

data/spec/pgit/status_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe PGit::Status do
+  describe 'when there is no associated story' do
+    it 'should raise an error' do
+      story_id = nil
+      options = double('options')
+      global_options = double('global_options')
+      args = double('args')
+      current_branch = instance_double('PGit::CurrentBranch', story_id: story_id)
+      allow(PGit::CurrentBranch).to receive(:new).and_return(current_branch)
+      status = PGit::Status.new(global_options, options, args)
+
+      expect{status.execute!}.to raise_error(PGit::Error::User, 'The current branch is not associated with a story. Does not have a story id.')
+    end
+  end
+
+  describe 'when there is an associated story' do
+    it 'should puts info about the story' do
+      story_id = 12345
+      options = double('options')
+      global_options = double('global_options')
+      args = double('args')
+      current_branch = instance_double('PGit::CurrentBranch', story_id: story_id)
+      allow(PGit::CurrentBranch).to receive(:new).and_return(current_branch)
+      story_hash = {'kind' => 'story'}
+      story = double('PGit::Story', get!: story_hash)
+      allow(PGit::Pivotal::Story).to receive(:new).with(current_branch.story_id).and_return(story)
+      table = double('table')
+      allow(table).to receive(:rows=).with(story_hash.to_a)
+      allow(Terminal::Table).to receive(:new).and_yield(table).and_return(table)
+      status = PGit::Status.new(global_options, options, args)
+      allow(status).to receive(:puts).with(table)
+
+      status.execute!
+      expect(table).to have_received(:rows=).with(story_hash.to_a)
+      expect(status).to have_received(:puts).with(table)
+    end
+  end
+end

data/spec/pgit/story_branch/application_spec.rb

@@ -9,19 +9,16 @@ describe 'PGit::StoryBranch::Application' do
         opts = { start: 1234 }
         args = {}
         fake_story_branch = instance_double('PGit::StoryBranch')
-        fake_config_yaml = double('config_yaml')
-        fake_configuration = instance_double('PGit::Configuration', to_yaml: fake_config_yaml)
+        fake_configuration = instance_double('PGit::Configuration')
         allow(fake_story_branch).to receive(:start)
-        fake_get_request_results = double('json_response')
         fake_story_id = double('fake_story_id')
 
-        fake_story = double('PGit::Story', id: fake_story_id)
-        fake_story_class = class_double('PGit::Story', get: fake_get_request_results)
-        fake_current_project = double('current_project')
+        fake_story = double('PGit::Pivotal::Story', id: fake_story_id, get!: fake_story)
+        fake_current_project = double('current_project', id: 1, api_token: 'someapitoken')
         fake_name_parser = instance_double(PGit::StoryBranch::NameParser)
-        allow(PGit::CurrentProject).to receive(:new).with(fake_config_yaml).and_return(fake_current_project)
+        allow(PGit::CurrentProject).to receive(:new).with(fake_configuration).and_return(fake_current_project)
         allow(PGit::StoryBranch::NameParser).to receive(:new).with(fake_story).and_return(fake_name_parser)
-        allow(PGit::Story).to receive(:get).with(1234, fake_current_project).and_return(fake_story)
+        allow(PGit::Pivotal::Story).to receive(:new).with(1234).and_return(fake_story)
         allow(PGit::StoryBranch).to receive(:new).with(fake_name_parser).and_return(fake_story_branch)
         allow(PGit::Configuration).to receive(:new).and_return(fake_configuration)
 

data/spec/pgit/story_branch/name_parser_spec.rb

@@ -7,7 +7,7 @@ describe 'PGit::StoryBranch::NameParser' do
 
       unparsed_name = "Deemphasize the Clue on the stimulus screen of multi-screen exercises."
       story_id = '12345'
-      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+      fake_story = double('PGit::Pivotal::Story', id: story_id, name: unparsed_name)
 
       name_parser = PGit::StoryBranch::NameParser.new(fake_story)
       parsed = name_parser.name
@@ -19,7 +19,7 @@ describe 'PGit::StoryBranch::NameParser' do
       unparsed_name = "Some don't like putin's dictatorship"
       story_id = '29292'
 
-      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+      fake_story = double('PGit::Pivotal::Story', id: story_id, name: unparsed_name)
 
       name_parser = PGit::StoryBranch::NameParser.new(fake_story)
       parsed = name_parser.name
@@ -31,7 +31,7 @@ describe 'PGit::StoryBranch::NameParser' do
       unparsed_name = "Some *@#   don't like ,putin's dictator-ship"
       story_id = '29292'
 
-      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+      fake_story = double('PGit::Pivotal::Story', id: story_id, name: unparsed_name)
 
       name_parser = PGit::StoryBranch::NameParser.new(fake_story)
       parsed = name_parser.name

data/spec/pgit/story_branch/story_id_parser_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe PGit::StoryBranch::StoryIdParser do
+  describe 'if it exists' do
+    it 'parse should give us the id' do
+      parser = PGit::StoryBranch::StoryIdParser.new("pgit-status-shows-information-about-current-story-if-there-is-one-97997060")
+      expect(parser.parse).to eq "97997060"
+    end
+  end
+
+  describe 'if does not exist' do
+    it 'should raise an error' do
+      parser = PGit::StoryBranch::StoryIdParser.new("pgit-status-shows-information-about-current-story-if-there-is-one")
+      expect{parser.parse}.to raise_error(PGit::Error::User, 'The current branch is not associated with a story. Does not have a story id.')
+    end
+  end
+end

data/spec/pgit/validators/project_validator_spec.rb

@@ -0,0 +1,39 @@
+require 'pgit'
+
+describe 'PGit::Validators::ProjectValidator' do
+  describe '#valid?' do
+    it 'should not be valid when the request for stories fails' do
+      class SomeFakeProjectWithKindError
+        include ActiveModel::Validations
+        validates_with PGit::Validators::ProjectValidator
+        def get!
+        end
+
+        def kind
+          'error'
+        end
+      end
+      project = SomeFakeProjectWithKindError.new
+      validator = PGit::Validators::ProjectValidator.new
+      validator.validate(project)
+
+      expect(project).not_to be_valid
+    end
+
+    it 'should not be valid if the project does not have a kind' do
+      class SomeFakeProjectWithNoKind
+        include ActiveModel::Validations
+        validates_with PGit::Validators::ProjectValidator
+
+        def get!
+        end
+      end
+
+      project = SomeFakeProjectWithNoKind.new
+      validator = PGit::Validators::ProjectValidator.new
+      validator.validate(project)
+
+      expect(project).not_to be_valid
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pgit
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 1.0.0
 platform: ruby
 authors:
 - Edderic Ugaddan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-18 00:00:00.000000000 Z
+date: 2015-06-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -66,6 +66,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.0.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.0.1
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.5.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.5.1
 - !ruby/object:Gem::Dependency
   name: gli
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +108,48 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 2.12.2
+- !ruby/object:Gem::Dependency
+  name: activemodel
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.2.0
+- !ruby/object:Gem::Dependency
+  name: rainbow
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+- !ruby/object:Gem::Dependency
+  name: interactive
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.6.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.6.0
 description: 
 email: edderic@gmail.com
 executables:
@@ -88,6 +158,7 @@ extensions: []
 extra_rdoc_files:
 - README.markdown
 files:
+- ".agignore"
 - ".coveralls.yml"
 - ".gitignore"
 - ".rspec"
@@ -98,47 +169,101 @@ files:
 - Rakefile
 - bin/pgit
 - lib/pgit.rb
+- lib/pgit/bilateral/handle_back.rb
+- lib/pgit/bilateral/handle_choose_story.rb
+- lib/pgit/bilateral/story.rb
+- lib/pgit/command.rb
+- lib/pgit/command/add.rb
+- lib/pgit/command/application.rb
+- lib/pgit/command/edit.rb
+- lib/pgit/command/remove.rb
+- lib/pgit/command/run.rb
+- lib/pgit/command/show.rb
 - lib/pgit/configuration.rb
-- lib/pgit/configuration/layout_error.rb
-- lib/pgit/configuration/missing_attributes_error.rb
-- lib/pgit/configuration/not_found_error.rb
-- lib/pgit/configuration/project_missing_error.rb
-- lib/pgit/configuration/validator.rb
 - lib/pgit/current_branch.rb
 - lib/pgit/current_project.rb
-- lib/pgit/current_project/no_paths_match_working_dir_error.rb
 - lib/pgit/current_project/validator.rb
 - lib/pgit/error.rb
-- lib/pgit/external_error.rb
-- lib/pgit/installer/configuration.rb
-- lib/pgit/name_parser.rb
+- lib/pgit/error/external.rb
+- lib/pgit/error/user.rb
+- lib/pgit/helpers/heredoc.rb
+- lib/pgit/helpers/query_methods.rb
+- lib/pgit/helpers/string_extensions.rb
+- lib/pgit/installer/bash_auto_completion.rb
+- lib/pgit/pivotal/collection_request.rb
+- lib/pgit/pivotal/individual_request.rb
+- lib/pgit/pivotal/iteration.rb
+- lib/pgit/pivotal/iterations.rb
+- lib/pgit/pivotal/project.rb
+- lib/pgit/pivotal/projects.rb
+- lib/pgit/pivotal/query.rb
+- lib/pgit/pivotal/request.rb
+- lib/pgit/pivotal/request/query.rb
+- lib/pgit/pivotal/story.rb
 - lib/pgit/pivotal_request_validator.rb
-- lib/pgit/story.rb
+- lib/pgit/project.rb
+- lib/pgit/project/add.rb
+- lib/pgit/project/application.rb
+- lib/pgit/project/interactive_adder.rb
+- lib/pgit/project/remove.rb
+- lib/pgit/project/reuse_api_token_adder.rb
+- lib/pgit/response_handler.rb
+- lib/pgit/root.rb
+- lib/pgit/status.rb
 - lib/pgit/story_branch.rb
 - lib/pgit/story_branch/application.rb
+- lib/pgit/story_branch/name_parser.rb
+- lib/pgit/story_branch/story_id_parser.rb
+- lib/pgit/validators/project_validator.rb
 - lib/pgit/version.rb
+- lib/pivotal
 - man/pgit.1
 - man/pgit.1.html
 - man/pgit.1.ronn
 - pgit.gemspec
 - pgit.rdoc
-- spec/pgit/configuration/missing_attributes_error_spec.rb
-- spec/pgit/configuration/not_found_error_spec.rb
-- spec/pgit/configuration/project_missing_error_spec.rb
-- spec/pgit/configuration/validator_spec.rb
+- spec/fixtures/iterations
+- spec/pgit/bilateral/handle_back_spec.rb
+- spec/pgit/bilateral/handle_choose_story_spec.rb
+- spec/pgit/bilateral/story_spec.rb
+- spec/pgit/command/add_spec.rb
+- spec/pgit/command/application_spec.rb
+- spec/pgit/command/edit_spec.rb
+- spec/pgit/command/remove_spec.rb
+- spec/pgit/command/run_spec.rb
+- spec/pgit/command/show_spec.rb
+- spec/pgit/command_spec.rb
 - spec/pgit/configuration_spec.rb
 - spec/pgit/current_branch_spec.rb
-- spec/pgit/current_project/no_paths_match_working_dir_error_spec.rb
 - spec/pgit/current_project/validator_spec.rb
 - spec/pgit/current_project_spec.rb
-- spec/pgit/external_error_spec.rb
-- spec/pgit/installer/configuration_spec.rb
+- spec/pgit/error/external_spec.rb
+- spec/pgit/error/user_spec.rb
+- spec/pgit/helpers/heredoc_spec.rb
+- spec/pgit/helpers/query_methods_spec.rb
+- spec/pgit/helpers/string_extensions_spec.rb
+- spec/pgit/installer/bash_auto_completion_spec.rb
 - spec/pgit/pgit_spec.rb
+- spec/pgit/pivotal/individual_request_spec.rb
+- spec/pgit/pivotal/iteration_spec.rb
+- spec/pgit/pivotal/iterations_spec.rb
+- spec/pgit/pivotal/project_spec.rb
+- spec/pgit/pivotal/projects_spec.rb
+- spec/pgit/pivotal/request/query_spec.rb
+- spec/pgit/pivotal/story_spec.rb
 - spec/pgit/pivotal_request_validator_spec.rb
+- spec/pgit/project/add_spec.rb
+- spec/pgit/project/application_spec.rb
+- spec/pgit/project/interactive_adder_spec.rb
+- spec/pgit/project/remove_spec.rb
+- spec/pgit/project/reuse_api_token_adder_spec.rb
+- spec/pgit/project_spec.rb
+- spec/pgit/status_spec.rb
 - spec/pgit/story_branch/application_spec.rb
 - spec/pgit/story_branch/name_parser_spec.rb
+- spec/pgit/story_branch/story_id_parser_spec.rb
 - spec/pgit/story_branch_spec.rb
-- spec/pgit/story_spec.rb
+- spec/pgit/validators/project_validator_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/Edderic/pgit
 licenses:
@@ -177,7 +302,7 @@ requirements:
 - At least one project that uses Pivotal Tracker and Git
 - cURL, at least 7.35.0
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Optimize your Pivotal Tracker and Github workflow

data/lib/pgit/configuration/layout_error.rb

@@ -1,9 +0,0 @@
-module PGit
-  class Configuration
-    class LayoutError < PGit::Error
-      def prepend_general_message(message)
-        "#{message}\nPlease have the following layout:\n" + YAML.dump(PGit::Configuration.default_options)
-      end
-    end
-  end
-end

data/lib/pgit/configuration/missing_attributes_error.rb

@@ -1,10 +0,0 @@
-module PGit
-  class Configuration
-    class MissingAttributesError < PGit::Configuration::LayoutError
-      def initialize(path)
-        @message = prepend_general_message "#{path} must have a path, id, and api_token for each project."
-        super(@message)
-      end
-    end
-  end
-end

data/lib/pgit/configuration/not_found_error.rb

@@ -1,10 +0,0 @@
-module PGit
-  class Configuration
-    class NotFoundError < PGit::Error
-      def initialize(config_path)
-        @message = "#{config_path} configuration file does not exist. Please run `pgit install`"
-        super(@message)
-      end
-    end
-  end
-end

data/lib/pgit/configuration/project_missing_error.rb

@@ -1,10 +0,0 @@
-module PGit
-  class Configuration
-    class ProjectMissingError < PGit::Configuration::LayoutError
-      def initialize(path)
-        @message = prepend_general_message "#{path} needs at least one project."
-        super(@message)
-      end
-    end
-  end
-end

data/lib/pgit/configuration/validator.rb

@@ -1,41 +0,0 @@
-module PGit
-  class Configuration
-    class Validator
-      attr_reader :yaml
-
-      def initialize(config_path)
-        @expanded_path = File.expand_path(config_path)
-        if File.exists? @expanded_path
-          config_file = File.open(@expanded_path, 'r')
-          @yaml = YAML.load(config_file)
-
-          validate_existence_of_at_least_one_project
-          validate_presence_of_items_in_each_project
-        else
-          raise PGit::Configuration::NotFoundError.new(@expanded_path)
-        end
-      end
-
-      private
-
-      def validate_presence_of_items_in_each_project
-        projects = @yaml["projects"]
-        all_present = projects.all? do |project|
-          project["api_token"] &&
-            project["path"] &&
-            project["id"]
-        end
-
-        unless all_present
-          raise PGit::Configuration::MissingAttributesError.new(@expanded_path)
-        end
-      end
-
-      def validate_existence_of_at_least_one_project
-        unless @yaml["projects"]
-          raise PGit::Configuration::ProjectMissingError.new(@expanded_path)
-        end
-      end
-    end
-  end
-end