startling 0.0.9 → 0.0.10

data/spec/spec_helper.rb

@@ -1,19 +0,0 @@
-# This file was generated by the `rspec --init` command. Conventionally, all
-# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# Require this file using `require "spec_helper"` to ensure that it is only
-# loaded once.
-#
-# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
-RSpec.configure do |config|
-  config.run_all_when_everything_filtered = true
-  config.filter_run :focus
-
-  # 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'
-end
-
-require 'pry'
-Dir['./spec/support/**/*.rb'].map {|f| require f}

data/spec/startling/command_spec.rb

@@ -1,85 +0,0 @@
-require 'spec_helper'
-
-require 'startling/command'
-
-describe Startling::Command do
-  let(:command) { Startling::Command.new }
-
-  describe '#execute' do
-    before do
-      allow(Startling::Commands::CheckForLocalMods).to receive(Startling::Command::RUN)
-
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :before_story_start) { [] }
-
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :after_story_start) { [] }
-
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :before_pull_request) { [] }
-
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :after_pull_request) { [] }
-
-      allow(Startling).to receive(:story_handler) { nil }
-
-      allow(Startling::Commands::CreatePullRequest).to receive(Startling::Command::RUN)
-    end
-
-    it 'should check for local modifications' do
-      expect(Startling::Commands::CheckForLocalMods).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should run before story start commands' do
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :before_story_start) { [:create_branch] }
-
-      expect(Startling::Commands::CreateBranch).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should start the story if a story handler is defined' do
-      allow(Startling).to receive(:story_handler) { Startling::Commands::CreateBranch }
-
-      expect(Startling::Commands::CreateBranch).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should run after story start commands' do
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :after_story_start) { [:create_branch] }
-
-      expect(Startling::Commands::CreateBranch).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should run before pull request commands' do
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :before_pull_request) { [:create_branch] }
-
-      expect(Startling::Commands::CreateBranch).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should create the pull request' do
-      expect(Startling::Commands::CreatePullRequest).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-
-    it 'should run after pull request commands' do
-      allow(Startling)
-        .to receive_message_chain(:hook_commands, :after_pull_request) { [:create_branch] }
-
-      expect(Startling::Commands::CreateBranch).to receive(Startling::Command::RUN)
-
-      command.execute
-    end
-  end
-end

data/spec/startling/commands/base_spec.rb

@@ -1,32 +0,0 @@
-require 'spec_helper'
-
-require 'startling/commands/base'
-
-describe Startling::Commands::Base do
-  it "should assigns key/values passed in as attributes" do
-    base = Startling::Commands::Base.new(foo: 'bar', baz: 'qux')
-    expect(base.foo).to eq 'bar'
-    expect(base.baz).to eq 'qux'
-  end
-
-  describe '#execute' do
-    subject { Startling::Commands::Base.new }
-    it 'should throws NotImplementedError' do
-      expect{ subject.execute }.to raise_error(NotImplementedError)
-    end
-  end
-
-  describe '#run' do
-    subject { Startling::Commands::Base.run(foo: 'bar')}
-
-    it 'should assign attributes and call execute' do
-      base =  double :base_command
-      allow(Startling::Commands::Base).to receive(:new).and_return(base)
-
-      expect(Startling::Commands::Base).to receive(:new).with(foo: 'bar')
-      expect(base).to receive(:execute)
-
-      subject
-    end
-  end
-end

data/spec/startling/commands/create_branch_spec.rb

@@ -1,105 +0,0 @@
-require 'spec_helper'
-
-require 'startling/commands/create_branch'
-
-describe Startling::Commands::CreateBranch do
-  let(:git) { double('GitLocal') }
-  let(:attrs) { { args: [], git: git } }
-  let(:create_branch) { Startling::Commands::CreateBranch.new(attrs) }
-
-  describe '#execute' do
-    let(:branch_name) { 'my-branch' }
-    let(:attrs) { { args: [1, branch_name], git: git } }
-
-    before do
-      allow(create_branch).to receive(:valid_branch_name?) { true }
-      allow(git).to receive(:current_branch) { 'current-branch' }
-      allow(create_branch).to receive(:create_branch)
-    end
-
-    it 'returns an error if the branch name is invalid' do
-      allow(create_branch).to receive(:valid_branch_name?) { false }
-
-      expect { create_branch.execute }.to raise_exception(SystemExit)
-    end
-
-    it 'creates the branch if it is not the current branch' do
-      allow(git).to receive(:current_branch) { 'current-branch' }
-
-      expect(create_branch).to receive(:create_branch)
-      create_branch.execute
-    end
-
-    it 'does not create the branch if it is the current branch' do
-      allow(git).to receive(:current_branch) { branch_name }
-
-      expect(create_branch).not_to receive(:create_branch)
-      create_branch.execute
-    end
-
-    it 'returns the branch name' do
-      expect(create_branch.execute).to eq(branch_name)
-    end
-  end
-
-  describe '#branch_name' do
-    subject { create_branch.branch_name }
-
-    context 'when branch command line option is set' do
-      let(:attrs) { { args: [1, 'test everything'] } }
-
-      it 'returns the command line option' do
-        expect(subject).to eq('test-everything')
-      end
-    end
-
-    context 'when branch command line option is not set' do
-      let(:default_branch) { 'master' }
-
-      before do
-        allow(create_branch).to receive(:default_branch) { default_branch }
-      end
-
-      it 'returns the branch entered through the prompt' do
-        allow_any_instance_of(Startling::Commands::CreateBranch).to receive(:ask) { 'entered branch' }
-
-        expect(subject).to eq('entered-branch')
-      end
-
-      it 'returns the current branch when no branch is entered through the prompt' do
-        allow_any_instance_of(Startling::Commands::CreateBranch).to receive(:ask) { '' }
-        allow(git).to receive(:current_branch) { 'current-branch' }
-
-        expect(subject).to eq('current-branch')
-      end
-    end
-  end
-
-  describe '#valid_branch_name?' do
-    let(:branch_name) { 'validate-me' }
-    let(:attrs) { { args: [1, branch_name] } }
-
-    subject { create_branch.valid_branch_name? }
-
-    before do
-      allow(create_branch).to receive(:default_branch) { 'master' }
-      allow(Startling).to receive(:validate_branch_name) { nil }
-    end
-
-    it 'succeeds if branch passes validation' do
-      expect(subject).to be_truthy
-    end
-
-    it 'fails if the branch is the default branch' do
-      allow(create_branch).to receive(:default_branch) { branch_name }
-
-      expect(subject).to be_falsey
-    end
-
-    it 'fails if the custom validation Proc fails' do
-      allow(Startling).to receive(:validate_branch_name) { -> (branch_name) { /feature\/.*/ =~ branch_name } }
-
-      expect(subject).to be_falsey
-    end
-  end
-end

data/spec/startling/commands/create_pull_request_spec.rb

@@ -1,46 +0,0 @@
-require 'spec_helper'
-
-require 'startling/commands/create_pull_request'
-
-describe Startling::Commands::CreatePullRequest do
-  let(:git) { double('GitLocal') }
-  let(:attrs) { { git: git } }
-  let(:create_pull_request) { Startling::Commands::CreatePullRequest.new(attrs) }
-
-  describe '#open_pull_request' do
-    let(:commit_message) { 'test' }
-    let(:pull_request_handler) do
-      double(:pull_request_handler,
-        commit_message: commit_message,
-        title: 'title',
-        body: 'body'
-      )
-    end
-
-    let(:pull_request) { double(:pull_request, url: 'pull request URL') }
-    let(:repo) { double(:repo, open_pull_request: pull_request) }
-
-    before do
-      allow(create_pull_request).to receive(:pull_request_handler) { pull_request_handler }
-      allow(create_pull_request).to receive(:repo) { repo }
-
-      allow(git).to receive(:current_branch_has_no_commits?) { false }
-      allow(git).to receive(:create_empty_commit)
-      allow(git).to receive(:push_origin_head)
-    end
-
-    it 'should create an empty commit when the branch has no commits' do
-      allow(git).to receive(:current_branch_has_no_commits?) { true }
-
-      expect(git).to receive(:create_empty_commit).with(commit_message)
-      create_pull_request.open_pull_request
-    end
-
-    it 'should not create an empty commit when the branch has commits' do
-      allow(git).to receive(:current_branch_has_no_commits?) { false }
-
-      expect(git).not_to receive(:create_empty_commit)
-      create_pull_request.open_pull_request
-    end
-  end
-end

data/spec/startling/configuration_spec.rb

@@ -1,189 +0,0 @@
-require 'spec_helper'
-require 'startling'
-
-module Startling
-  describe Configuration do
-    let(:configuration) { Configuration.new }
-    let(:current_repo) { 'current' }
-
-    before do
-      allow_any_instance_of(GitLocal).to receive(:repo_name) { current_repo }
-    end
-
-    describe "Default settings" do
-      it "sets the default cache_dir to pwd" do
-        expect(configuration.cache_dir).to eql(Dir.pwd)
-      end
-
-      it "sets the default root_dir to pwd" do
-        expect(configuration.root_dir).to eql(Dir.pwd)
-      end
-
-      it "sets the default wip limit to WIP_LIMIT" do
-        expect(configuration.wip_limit)
-          .to eql(Configuration::DEFAULT_WIP_LIMIT)
-      end
-
-      it "sets the WIP labels to empty" do
-        expect(configuration.wip_labels).to eql([])
-      end
-
-      it "sets the default repos to the current repo" do
-        expect(configuration.repos).to eql([current_repo])
-      end
-
-      it "sets the default story handler to nil" do
-        expect(configuration.story_handler).to eql(nil)
-      end
-
-      it "sets the default branch name validator to nil" do
-        expect(configuration.validate_branch_name).to eql(nil)
-      end
-
-      it "sets the default pull request handler to nil" do
-        expect(configuration.pull_request_handler).to eql(nil)
-      end
-
-      it "sets the default pull request body" do
-        expect(configuration.pull_request_body)
-          .to eql(Configuration::DEFAULT_BODY)
-      end
-
-      it "sets the default pull request commit message" do
-        expect(configuration.pull_request_commit_message)
-          .to eql(Configuration::DEFAULT_COMMIT_MESSAGE)
-      end
-
-      it "sets the default pull request labels" do
-        expect(configuration.pull_request_labels).to eql([])
-      end
-    end
-
-    describe "#cache_dir" do
-      it "can set the value" do
-        configuration.cache_dir = "new dir"
-        expect(configuration.cache_dir).to eql("new dir")
-      end
-    end
-
-    describe "#root_dir" do
-      it "can set the value" do
-        configuration.root_dir = "new dir"
-        expect(configuration.root_dir).to eql("new dir")
-      end
-    end
-
-    describe "#wip_limit" do
-      it "can set the value" do
-        configuration.wip_limit = 6
-        expect(configuration.wip_limit).to eql(6)
-      end
-    end
-
-    describe "#wip_labels" do
-      it "can set the value" do
-        configuration.wip_labels = ['WIP']
-        expect(configuration.wip_labels).to eql(['WIP'])
-      end
-    end
-
-    describe "#repos" do
-      it "can set the value" do
-        configuration.repos << "repo path"
-        expect(configuration.repos).to eql([current_repo, "repo path"])
-      end
-    end
-
-    describe "#story_handler" do
-      it "can set the value" do
-        configuration.story_handler = :pivotal_start
-        expect(configuration.story_handler).to eql(:pivotal_start)
-      end
-    end
-
-    describe "#validate_branch_name" do
-      it "can set the value" do
-        validate_branch_name = -> (branch_name) { /feature\/.*/ =~ branch_name }
-        configuration.validate_branch_name = validate_branch_name
-        expect(configuration.validate_branch_name).to eql(validate_branch_name)
-      end
-    end
-
-    describe "#pull_request_handler" do
-      it "can set the value" do
-        configuration.pull_request_handler = :custom
-        expect(configuration.pull_request_handler).to eql(:custom)
-      end
-    end
-
-    describe "#pull_request_body" do
-      it "can set the value" do
-        configuration.pull_request_body = "Startling Body"
-        expect(configuration.pull_request_body).to eql("Startling Body")
-      end
-    end
-
-    describe "#pull_request_commit_message" do
-      it "can set the value" do
-        configuration.pull_request_commit_message = "The Commit"
-        expect(configuration.pull_request_commit_message).to eql("The Commit")
-      end
-    end
-
-    describe "#pull_request_labels" do
-      it "can set the value" do
-        configuration.pull_request_labels = ["WIP", "REVIEW"]
-        expect(configuration.pull_request_labels).to eql(["WIP", "REVIEW"])
-      end
-    end
-
-    describe ".load_configuration" do
-      let(:git) { double(GitLocal) }
-      let(:config_files) {
-        {
-          caps:  "Startlingfile.rb",
-          lower: "startlingfile.rb"
-        }
-      }
-      let(:caps_config_file) { "Startlingfile.rb" }
-      let(:lower_config_file) { "startlingfile.rb" }
-
-      before do
-        allow(Startling::GitLocal).to receive(:new) { git }
-        allow(git).to receive(:project_root) { Dir.pwd }
-      end
-
-      after do
-        config_files.each do |_, v|
-          delete_config_file(v)
-        end
-      end
-
-      context "when no configuration file exists" do
-        it "uses default configuration" do
-          expect(Configuration.load_configuration).to eql(nil)
-        end
-      end
-
-      context "when a configuration file exists" do
-        it "loads configuration from Startlingfile.rb" do
-          create_config_file(config_files[:caps])
-          expect(Configuration.load_configuration).to eql(config_files[:caps])
-        end
-
-        it "loads configuration from startlingfile.rb" do
-          create_config_file(config_files[:lower])
-          expect(Configuration.load_configuration).to eql(config_files[:lower])
-        end
-      end
-    end
-
-    def create_config_file(config_file)
-      File.open(config_file, "w")
-    end
-
-    def delete_config_file(config_file)
-      File.delete(config_file) if File.exists? config_file
-    end
-  end
-end