startling 0.0.2

data/lib/startling/github/pull_request.rb

@@ -0,0 +1,54 @@
+module Startling
+  module Github
+    class PullRequest
+      attr_reader :attributes
+      attr_accessor :labels
+
+      def initialize(attributes, prefetch_data: true)
+        @attributes = attributes
+        prefetch_data if prefetch_data
+      end
+
+      def id
+        attributes.number
+      end
+
+      def title
+        attributes.title
+      end
+
+      def branch
+        attributes.head.ref
+      end
+
+      def in_progress?
+        label_names.any? { |label| label.match(/(WIP|REVIEW)/) }
+      end
+
+      def label_names
+        labels.map(&:name)
+      end
+
+      def url
+        attributes.rels[:html].href
+      end
+
+      def created_at
+        attributes.created_at
+      end
+
+      def updated_at
+        attributes.updated_at
+      end
+
+      def author
+        @author ||= attributes.user.rels[:self].get.data.name
+      end
+
+      private
+      def prefetch_data
+        author
+      end
+    end
+  end
+end

data/lib/startling/github/repo.rb

@@ -0,0 +1,44 @@
+module Startling
+  module Github
+    class Repo
+      attr_reader :api, :name
+
+      def initialize(name, api)
+        @name = name
+        @api = api
+      end
+
+      def attributes
+        @attributes ||= api.repository_attributes(name)
+      end
+
+      def default_branch
+        attributes.default_branch
+      end
+
+      def pull_requests
+        @pull_requests ||= api.pull_requests(name)
+      end
+
+      def pull_request(branch)
+        pull_requests.find { |pr| pr.branch == branch }
+      end
+
+      def set_labels_for_issue(issue_id:, labels:)
+        labels = Array(labels)
+        api.set_labels_for_issue(repo_name: name, issue_id: issue_id, labels: labels)
+      end
+
+      def open_pull_request(title: nil, body: nil, branch: nil)
+        pull_request = api.open_pull_request(repo_name: name, destination_branch: default_branch,
+          branch: branch, title: title, body: body)
+        return pull_request if pull_request
+        pull_request(branch) # In case this is already open
+      end
+
+      def self.all
+        @all ||= Startling::REPOS.map { |name| new(name) }
+      end
+    end
+  end
+end

data/lib/startling/handlers/default_pull_request_handler.rb

@@ -0,0 +1,21 @@
+require_relative "pull_request_handler_base"
+
+module Startling
+  module Handlers
+    class DefaultPullRequestHandler < PullRequestHandlerBase
+      def title
+        title = ask("Please input a pull request title: ")
+        abort "Title must be specified." if title.empty?
+        title
+      end
+
+      def body
+        Startling.pull_request_body
+      end
+
+      def commit_message
+        Startling.pull_request_commit_message
+      end
+    end
+  end
+end

data/lib/startling/handlers/pull_request_handler_base.rb

@@ -0,0 +1,21 @@
+module Startling
+  module Handlers
+    class PullRequestHandlerBase
+      def initialize(args)
+        @args = args
+      end
+
+      def title
+        raise NotImplementedError
+      end
+
+      def body
+        raise NotImplementedError
+      end
+
+      def commit_message
+        raise NotImplementedError
+      end
+    end
+  end
+end

data/lib/startling/markdown.rb

@@ -0,0 +1,7 @@
+module Startling
+  class Markdown
+    def self.escape(text)
+      text.gsub('[', '\[').gsub(']', '\]')
+    end
+  end
+end

data/lib/startling/shell.rb

@@ -0,0 +1,11 @@
+module Startling
+  class Shell
+    def self.run(command)
+      result = `#{command}`
+      unless $?.success?
+        exit 1
+      end
+      result
+    end
+  end
+end

data/lib/startling/version.rb

@@ -0,0 +1,3 @@
+module Startling
+  VERSION = "0.0.2"
+end

data/spec/spec_helper.rb

@@ -0,0 +1,19 @@
+# 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/commands/base_spec.rb

@@ -0,0 +1,32 @@
+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/configuration_spec.rb

@@ -0,0 +1,127 @@
+require 'spec_helper'
+require 'startling'
+
+module Startling
+  describe Configuration do
+    let(:configuration) { Configuration.new }
+
+    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 default repos to empty" do
+        expect(configuration.repos).to eql([])
+      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 "#repos" do
+      it "can set the value" do
+        configuration.repos << "repo path"
+        expect(configuration.repos).to eql(["repo path"])
+      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

data/spec/startling/git_local_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+require 'startling/git_local'
+
+module Startling
+  describe GitLocal do
+    subject { GitLocal.new }
+
+    it "parses the repo name out of an https url" do
+      url = 'https://github.com/TeachingChannel/startling.git'
+      allow(subject).to receive(:remote_url) { url }
+
+      expect(subject.repo_name).to eq('TeachingChannel/startling')
+    end
+
+    it "parses the repo name out of an ssh url" do
+      url = 'git@github.com:TeachingChannel/startling.git'
+      allow(subject).to receive(:remote_url) { url }
+
+      expect(subject.repo_name).to eq('TeachingChannel/startling')
+    end
+  end
+end

data/spec/startling/github/pull_request_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+require 'startling/github/pull_request'
+
+module Startling
+  module Github
+    describe PullRequest, "#in_progress?" do
+
+      it "should be true for WIP label" do
+        pull_request = PullRequest.new(double(:attributes))
+        allow(pull_request).to receive(:label_names) { ["WIP"] }
+
+        expect(pull_request.in_progress?).to be_truthy
+      end
+
+      it "should be true for REVIEW label" do
+        pull_request = PullRequest.new(double(:attributes))
+        allow(pull_request).to receive(:label_names) { ["REVIEW"] }
+
+        expect(pull_request.in_progress?).to be_truthy
+      end
+
+     it "should be false if for other labels" do
+        pull_request = PullRequest.new(double(:attributes))
+        allow(pull_request).to receive(:label_names) { ["some other thing", "bug"] }
+
+        expect(pull_request.in_progress?).to be_falsey
+     end
+
+     it "should be false if no labels" do
+        pull_request = PullRequest.new(double(:attributes))
+        allow(pull_request).to receive(:label_names) { [] }
+
+        expect(pull_request.in_progress?).to be_falsey
+     end
+    end
+  end
+end

data/spec/startling_configuration_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+require 'startling'
+
+module Startling
+  describe ".configure" do
+    before do
+      Startling.configure do |config|
+        config.cache_dir = "value"
+      end
+    end
+
+    it "sets the configuration attributes" do
+      expect(Startling.cache_dir).to eql("value")
+    end
+  end
+end

data/spec/startling_spec.rb

@@ -0,0 +1,122 @@
+require 'spec_helper'
+require 'fileutils'
+require 'startling'
+require 'startling/git_local'
+require 'startling/github'
+
+describe "bin/start" do
+  let(:feature_name) { 'bin_start_starts_stories' }
+  let(:feature_branch) { "feature/#{feature_name}" }
+  let(:repo_default_branch) { 'master' }
+  let(:pull_request_body) { "This is a test body" }
+  let(:git) { Startling::GitLocal.new }
+  let(:repo) { Startling::Github.repo(git.repo_name) }
+  let(:pull_request) { repo.pull_request(feature_branch) }
+
+  before do
+    local_configuration = <<CONFIG
+Startling.configure do |config|
+  # WIP Limit
+  # config.wip_limit = 4
+
+  # Repos to check against for WIP limit
+  # config.repos << 'substantial/startling-dev'
+
+  # Valid story estimations
+  # config.valid_estimates = [1, 2, 4, 8, 16, 32, 64, 128]
+
+  # Commands to be run before a story is stared
+  # config.hook_commands.before_story_start = [:check_wip]
+
+  # Command to be run after a story has started
+  # config.hook_commands.after_story_start = []
+
+  # Commands to be run before a pull request is created
+  # config.hook_commands.before_pull_request = []
+
+  # Commands to be run after a pull request is created
+  # config.hook_commands.after_pull_request = []
+
+  # Handler used to start a provider specific story related to the pull request
+  # config.story_handler = :pivotal_start
+
+  # Message for pull request commit
+  # config.pull_request_commit_message = "Startling"
+
+  # Message for pull request body
+  config.pull_request_body = "#{pull_request_body}"
+
+  # Labels for a pull request
+  # config.pull_request_labels = [WIP, REVIEW, HOLD]
+
+  # Handler used for setting the title and body of a pull request
+  config.pull_request_handler = :default_pull_request_handler
+end
+CONFIG
+
+    File.open('startlingfile.rb', 'w') { |file| file.write(local_configuration) }
+
+    Startling::Commands::Base.load_configuration
+
+    test_repo_path = "tmp/test_repo"
+    FileUtils.mkdir_p "tmp"
+
+    unless File.exists? "tmp/test_repo/.git"
+      `git clone git@github.com:substantial/startling-testing.git #{test_repo_path}`
+    end
+
+    File.write File.join(test_repo_path, ".github_access_token"), Tokens.github
+
+    FileUtils.cd test_repo_path
+
+    Startling.root_dir = Startling.cache_dir = "."
+
+    git.checkout_branch repo_default_branch
+    git.destroy_branch feature_branch
+  end
+
+  after do
+    FileUtils.cd "#{__dir__}/.."
+    FileUtils.rm "startlingfile.rb"
+  end
+
+  # VCR Preconditions:
+  # * There should be no open pull requests:
+  #   https://github.com/substantial/startling-testing
+
+  it "starts stories from origin/master",
+    vcr: { cassette_name: "bin_start_starts_stories" } do
+
+    allow_any_instance_of(Startling::Handlers::DefaultPullRequestHandler)
+      .to receive(:ask)
+      .with("Please input a pull request title: ") { "The Title" }
+
+    allow_any_instance_of(Startling::Commands::CreateBranch)
+      .to receive(:ask)
+      .with("Enter branch name (enter for current branch): ") { feature_name }
+
+    command = Startling::Command.new(args: [])
+    command.execute
+
+    expect(git.remote_branches).to include feature_branch
+    expect(git.current_branch).to eq feature_branch
+    expect(repo.default_branch).to eq repo_default_branch
+  end
+
+  it "The pull request body is the same as the config",
+    vcr: { cassette_name: "bin_start_starts_stories_pr_body" } do
+
+    allow_any_instance_of(Startling::Handlers::DefaultPullRequestHandler)
+      .to receive(:ask)
+      .with("Please input a pull request title: ") { "The Title" }
+
+    allow_any_instance_of(Startling::Commands::CreateBranch)
+      .to receive(:ask)
+      .with("Enter branch name (enter for current branch): ") { feature_name }
+
+    command = Startling::Command.new(args: [])
+    command.execute
+
+    expect(pull_request.attributes.body).to eq pull_request_body
+  end
+end