pgit 0.0.3 → 0.0.4

data/spec/pgit/story_branch/application_spec.rb

@@ -1,4 +1,4 @@
-require 'pgit'
+require 'spec_helper'
 
 describe 'PGit::StoryBranch::Application' do
   describe '#new' do

data/spec/pgit/story_branch_spec.rb

@@ -1,4 +1,4 @@
-require 'pgit'
+require 'spec_helper'
 
 describe 'PGit::StoryBranch' do
   describe '#id' do

data/spec/pgit/story_spec.rb

@@ -1,4 +1,4 @@
-require 'pgit'
+require 'spec_helper'
 
 describe 'PGit::Story' do
   describe '#get' do
@@ -31,24 +31,5 @@ describe 'PGit::Story' do
       expect(story.name).to eq "Bring me the passengers"
       expect(story.description).to eq "ignore the droids"
     end
-
-    describe 'if there is an error' do
-      it 'should raise an error' 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_json_str_with_error = <<-ERROR_JSON
-          {
-            "code": "unfound_resource",
-                    "kind": "error",
-                    "error": "The object you tried to access could not be found.  It may have been removed by another user, you may be using the ID of another object type, or you may be trying to access a sub-resource at the wrong point in a tree."
-          }
-        ERROR_JSON
-
-        allow(PGit::Story).to receive(:`).with(get_request).and_return(fake_json_str_with_error)
-
-        expect{ PGit::Story.get(story_id, current_project) }.to raise_error(fake_json_str_with_error)
-      end
-    end
   end
 end

data/spec/spec_helper.rb

@@ -0,0 +1,101 @@
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause this
+# file to always be loaded, without a need to explicitly require it in any files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need it.
+#
+# The `.rspec` file also contains a few flags that are not defaults but that
+# users commonly want.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+require 'simplecov'
+require 'coveralls'
+
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter
+]
+
+SimpleCov.start
+
+require 'pgit'
+
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    # be_bigger_than(2).and_smaller_than(4).description
+    #   # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #   # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+# The settings below are suggested to provide a good initial experience
+# with RSpec, but feel free to customize to your heart's content.
+=begin
+  # These two settings work together to allow you to limit a spec run
+  # to individual examples or groups you care about by tagging them with
+  # `:focus` metadata. When nothing is tagged with `:focus`, all examples
+  # get run.
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+
+  # Limits the available syntax to the non-monkey patched syntax that is recommended.
+  # For more details, see:
+  #   - http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax
+  #   - http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://myronmars.to/n/dev-blog/2014/05/notable-changes-in-rspec-3#new__config_option_to_disable_rspeccore_monkey_patching
+  config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = 'doc'
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+=end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pgit
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Edderic Ugaddan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-13 00:00:00.000000000 Z
+date: 2014-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -38,6 +38,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: gli
   requirement: !ruby/object:Gem::Requirement
@@ -60,6 +88,7 @@ extensions: []
 extra_rdoc_files:
 - README.markdown
 files:
+- ".coveralls.yml"
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
@@ -70,10 +99,20 @@ files:
 - bin/pgit
 - lib/pgit.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/installer.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/pivotal_request_validator.rb
 - lib/pgit/story.rb
 - lib/pgit/story_branch.rb
 - lib/pgit/story_branch/application.rb
@@ -83,16 +122,24 @@ files:
 - man/pgit.1.ronn
 - pgit.gemspec
 - pgit.rdoc
-- spec/pgit/application_spec.rb
+- 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/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/installer_spec.rb
+- spec/pgit/external_error_spec.rb
+- spec/pgit/installer/configuration_spec.rb
 - spec/pgit/pgit_spec.rb
+- spec/pgit/pivotal_request_validator_spec.rb
 - spec/pgit/story_branch/application_spec.rb
 - spec/pgit/story_branch/name_parser_spec.rb
 - spec/pgit/story_branch_spec.rb
 - spec/pgit/story_spec.rb
+- spec/spec_helper.rb
 homepage: https://github.com/Edderic/pgit
 licenses:
 - MIT

data/lib/pgit/installer.rb

@@ -1,32 +0,0 @@
-module PGit
-  class Installer
-    def initialize(glob_opts, opts, args)
-      file_path = "~/.pgit.rc.yml"
-      @expanded_path = File.expand_path(file_path)
-
-      if File.exists? @expanded_path
-        raise "Error: #{file_path} already exists"
-      else
-        ask_continue
-      end
-    end
-
-    def ask_continue
-      puts  "*** Installing example pgit configuration file under ~/.pgit.rc.yml. Continue? [Y/n]"
-      if STDIN.gets.chomp.match(/y/i)
-        puts "Saving example pgit config in ~/.pgit.rc.yml..."
-        write_example_pgit_rc_file
-      else
-        puts "Aborting installation..."
-      end
-    end
-
-    def write_example_pgit_rc_file
-      File.open(@expanded_path, 'w') do |f|
-        YAML.dump(PGit::Configuration.default_options, f)
-      end
-
-      puts "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
-    end
-  end
-end

data/spec/pgit/application_spec.rb

@@ -1,16 +0,0 @@
-# require_relative '../../../lib/pgit'
-#
-# describe 'PGit::Application' do
-  # describe 'no arguments' do
-    # it 'should bring up the usage' do
-      # argv = []
-      # usage_message = "Usage: pgit\n"\
-                      # "  cob [pivotal-story-id]"
-      # error_message = "POOP"
-#
-      # expect do
-        # PGit::Application.new(argv)
-      # end.to raise_error(error_message)
-    # end
-  # end
-# end

data/spec/pgit/installer_spec.rb

@@ -1,158 +0,0 @@
-require 'pgit'
-
-describe 'PGit::Installer' do
-  describe '~/pgit.rc.yml exists' do
-    it 'should raise an error' do
-      global_opts = {}
-      opts = { }
-      args = {}
-      message = "Error: ~/.pgit.rc.yml already exists"
-      default_file_path = "~/.pgit.rc.yml"
-      file_expanded_path = "/home/edderic/.pgit.rc.yml"
-
-      allow(File).to receive(:expand_path).
-        with(".")
-      allow(File).to receive(:expand_path).
-        with(default_file_path).and_return(file_expanded_path)
-      allow(File).to receive(:exists?).
-        with(file_expanded_path).and_return(true)
-
-      expect do
-        PGit::Installer.new(global_opts, opts, args)
-      end.to raise_error message
-    end
-  end
-
-  describe '~/pgit.rc.yml does not exist' do
-    it 'should ask to continue or not' do
-      global_opts = {}
-      opts = { }
-      args = {}
-
-      message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
-        "Continue? [Y/n]"
-      confirmation_message = "Saving example pgit config in ~/.pgit.rc.yml..."
-      edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
-      answer = instance_double("String", chomp: 'y')
-      expanded_path = "/home/edderic/.pgit.rc.yml"
-      fake_writable_file = double('File')
-      allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
-      allow(File).to receive(:exists?).with(expanded_path).and_return(false)
-      allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
-      allow(STDIN).to receive(:gets).and_return(answer)
-      allow_any_instance_of(PGit::Installer).to receive(:puts).with(message)
-      allow_any_instance_of(PGit::Installer).to receive(:puts).with(confirmation_message)
-      allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
-      installer = PGit::Installer.new(global_opts, opts, args)
-
-      expect(installer).to have_received(:puts).with(message)
-    end
-
-    describe 'user answers "y"' do
-      it "should show the save message" do
-        global_opts = {}
-        opts = { }
-        args = {}
-
-        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
-          "Continue? [Y/n]"
-        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
-        expanded_path = "/home/edderic/.pgit.rc.yml"
-        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
-        fake_writable_file = double('File')
-        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
-        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
-        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
-
-        answer = instance_double("String", chomp: 'Y')
-        allow(STDIN).to receive(:gets).and_return(answer)
-
-        installer = PGit::Installer.new(global_opts, opts, args)
-
-        expect(installer).to have_received(:puts).with(save_message)
-      end
-
-      it 'should save the file under ~/.pgit.rc.yml' do
-        global_opts = {}
-        opts = { }
-        args = {}
-
-        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
-          "Continue? [Y/n]"
-        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
-        expanded_path = "/home/edderic/.pgit.rc.yml"
-        fake_writable_file = double('File')
-        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
-        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
-        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
-        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
-
-        answer = instance_double("String", chomp: 'Y')
-        allow(STDIN).to receive(:gets).and_return(answer)
-
-        installer = PGit::Installer.new(global_opts, opts, args)
-
-        expect(File).to have_received(:open).with(expanded_path, 'w')
-      end
-
-      it 'should ask the user to edit the config file' do
-        global_opts = {}
-        opts = { }
-        args = {}
-
-        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
-          "Continue? [Y/n]"
-        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
-        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
-        expanded_path = "/home/edderic/.pgit.rc.yml"
-        fake_writable_file = double('File')
-        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
-        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
-        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
-
-        answer = instance_double("String", chomp: 'Y')
-        allow(STDIN).to receive(:gets).and_return(answer)
-
-        installer = PGit::Installer.new(global_opts, opts, args)
-
-        expect(installer).to have_received(:puts).with(edit_message)
-      end
-    end
-
-    describe 'user answers with "n"' do
-      it "should show the abort message" do
-        message = "Aborting installation..."
-        global_opts = {}
-        opts = { }
-        args = {}
-
-        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
-          "Continue? [Y/n]"
-        expanded_path = "/home/edderic/.pgit.rc.yml"
-        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
-        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
-        allow_any_instance_of(PGit::Installer).to receive(:puts).with(message)
-
-        answer = instance_double("String", chomp: 'n')
-        allow(STDIN).to receive(:gets).and_return(answer)
-
-        installer = PGit::Installer.new(global_opts, opts, args)
-
-        expect(installer).to have_received(:puts).with(message)
-      end
-    end
-
-    describe 'user answers with something els' do
-    end
-  end
-end