caperoma 0.1.0 → 4.0.1

data/spec/models/task_with_commit_spec.rb

@@ -0,0 +1,105 @@
+# frozen_string_literal: true
+
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+RSpec.describe TaskWithCommit, type: :model do
+  describe 'inheritence' do
+    it { expect(subject).to be_a_kind_of Task }
+  end
+
+  describe 'relations' do
+    it { expect(subject).to belong_to :branch }
+  end
+
+  describe 'methods' do
+    describe '#finish' do
+      let!(:task) { create :task_with_commit }
+
+      it 'should commit with generated message' do
+        expect_any_instance_of(TaskWithCommit).to receive(:commit_message).and_return 'commit msg'
+        expect_any_instance_of(TaskWithCommit).to receive(:commit_rubocop_message).and_return 'rubocop commit msg'
+
+        expect(task).to receive(:git_commit).with('commit msg')
+        expect(task).to receive(:git_commit).with('rubocop commit msg')
+        expect(task).to receive(:git_push)
+
+        task.finish(nil)
+      end
+    end
+
+    describe '#pause' do
+      let!(:task) { create :task_with_commit }
+
+      it 'should commit with generated message' do
+        expect_any_instance_of(TaskWithCommit).to receive(:commit_message).and_return 'commit msg'
+        expect_any_instance_of(TaskWithCommit).to receive(:commit_rubocop_message).and_return 'rubocop commit msg'
+
+        expect(task).to receive(:git_commit).with('commit msg')
+        expect(task).to receive(:git_commit).with('rubocop commit msg')
+        expect(task).to receive(:git_push)
+
+        task.pause(nil)
+      end
+    end
+
+    describe '#abort' do
+      let!(:task) { create :task_with_commit }
+
+      it 'should not commit' do
+        expect_any_instance_of(TaskWithCommit).not_to receive(:commit_message)
+        expect_any_instance_of(TaskWithCommit).not_to receive(:commit_rubocop_message)
+
+        expect(task).not_to receive(:git_commit).with('commit msg')
+        expect(task).not_to receive(:git_push)
+
+        task.abort(nil)
+      end
+    end
+  end
+
+  describe 'private methods' do
+    let!(:task) { create :task_with_commit, title: 'Hello World', jira_key: jira_key, pivotal_id: pivotal_id }
+
+    describe '#commit_message' do
+      context 'neither jira nor pt id' do
+        let(:jira_key) { nil }
+        let(:pivotal_id) { nil }
+
+        it 'should generate only title, no jira or pt ids' do
+          result = task.send(:commit_message)
+          expect(result).to eq 'Hello World'
+        end
+      end
+
+      context 'jira id but no pt id' do
+        let(:jira_key) { 'RUC-123' }
+        let(:pivotal_id) { nil }
+
+        it 'should generate commit msg with jira key only' do
+          result = task.send(:commit_message)
+          expect(result).to eq '[RUC-123] Hello World'
+        end
+      end
+
+      context 'no jira id, but pt id' do
+        let(:jira_key) { nil }
+        let(:pivotal_id) { 12_345_678 }
+
+        it 'should generate commit msg with pt id only' do
+          result = task.send(:commit_message)
+          expect(result).to eq '[#12345678] Hello World'
+        end
+      end
+
+      context 'both jira id and pt id' do
+        let(:jira_key) { 'RUC-123' }
+        let(:pivotal_id) { 12_345_678 }
+
+        it 'should append both jira and pivotal ids' do
+          result = task.send(:commit_message)
+          expect(result).to eq '[RUC-123][#12345678] Hello World'
+        end
+      end
+    end
+  end
+end

data/spec/models/task_with_separate_branch_spec.rb

@@ -0,0 +1,97 @@
+# frozen_string_literal: true
+
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+RSpec.describe TaskWithSeparateBranch, type: :model do
+  describe 'methods' do
+    describe '#finish' do
+      let(:task) { create :task_with_separate_branch, title: 'title' }
+
+      it 'should commit with generated message' do
+        allow_any_instance_of(TaskWithSeparateBranch).to receive(:description_for_pull_request).and_return 'pivotal-url'
+        allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_current_branch).and_return 'parent-branch'
+
+        expect(task).to receive(:git_pull_request).with('parent-branch', 'title', 'pivotal-url')
+        expect(task).to receive(:git_checkout).with('parent-branch')
+
+        task.finish(nil)
+      end
+    end
+
+    describe '#pause' do
+      let(:task) { create :task_with_separate_branch, title: 'title' }
+
+      it 'should commit with generated message' do
+        allow_any_instance_of(TaskWithSeparateBranch).to receive(:description_for_pull_request).and_return 'pivotal-url'
+        expect(task).not_to receive(:git_pull_request).with('parent-branch', 'title', 'pivotal-url')
+        expect(task).not_to receive(:git_checkout).with('parent-branch')
+
+        task.pause(nil)
+      end
+    end
+
+    describe '#abort' do
+      let(:task) { create :task_with_separate_branch, title: 'title' }
+
+      it 'should commit with generated message' do
+        allow_any_instance_of(TaskWithSeparateBranch).to receive(:description_for_pull_request).and_return 'pivotal-url'
+        allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_current_branch).and_return 'parent-branch'
+
+        expect(task).not_to receive(:git_pull_request).with('parent-branch', 'title', 'pivotal-url')
+        expect(task).to receive(:git_checkout).with('parent-branch')
+
+        task.abort(nil)
+      end
+    end
+  end
+
+  describe 'observers' do
+    describe '::update_parent_branch' do
+      let(:task) { build :task_with_separate_branch }
+
+      it 'should get latest version of remote branch before switching' do
+        expect(task).to receive(:git_rebase_to_upstream)
+        task.save!
+      end
+    end
+
+    describe '::remember_parent_branch' do
+      let(:task) { build :task_with_separate_branch }
+
+      it 'should save branch to send pull request to' do
+        expect_any_instance_of(TaskWithSeparateBranch).to receive(:git_current_branch).and_return 'parent-branch'
+        task.save!
+        expect(task.reload.parent_branch).to eq 'parent-branch'
+      end
+    end
+
+    describe '::new_git_branch' do
+      it 'should make a new branch' do
+        expect_any_instance_of(TaskWithSeparateBranch).to receive(:branch_name).and_return 'branch-name'
+        expect_any_instance_of(TaskWithSeparateBranch).to receive(:git_branch).with('branch-name')
+
+        create :task_with_separate_branch
+      end
+    end
+  end
+
+  describe 'private methods' do
+    let!(:task) { create :task_with_separate_branch }
+
+    before { expect(task).to receive(:jira_key).and_return 'RUC-123' }
+
+    describe '#branch_name' do
+      it 'should generate branch name' do
+        expect(task).to receive(:title).and_return 'Hello World'
+        result = task.send(:branch_name)
+        expect(result).to eq 'ruc-123-hello-world'
+      end
+
+      it 'should cut the long titles' do
+        expect(task).to receive(:title).and_return 'Hello World And Welcome To My Really Awesome Project'
+        result = task.send(:branch_name)
+        expect(result).to eq 'ruc-123-hello-world-and-welcome-t'
+      end
+    end
+  end
+end

data/spec/models/three_day_report_spec.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+RSpec.describe ThreeDayReport, type: :model do
+  describe 'relations' do
+    it { expect(subject).to have_many :tasks }
+  end
+
+  describe 'callbacks' do
+    describe '::asskgn_unreported_tasks' do
+      let!(:daily_report1) { create :daily_report }
+      let!(:three_day_report1) { create :three_day_report }
+
+      let!(:task1) { create :task, daily_report: daily_report1, three_day_report: nil, finished_at: Time.now }
+      let!(:task2) { create :task, daily_report: daily_report1, three_day_report: three_day_report1, finished_at: Time.now }
+      let!(:task3) { create :task, daily_report: nil, three_day_report: nil, finished_at: Time.now }
+      let!(:task4) { create :task, daily_report: nil, three_day_report: nil, finished_at: nil }
+
+      specify do
+        new_report = create :three_day_report
+        expect(new_report.tasks).to match_array [task1, task3]
+      end
+    end
+  end
+
+  describe 'message format' do
+    describe 'subject' do
+      let(:report) { create :three_day_report }
+
+      specify do
+        Timecop.freeze(Time.parse('06/02/2015 15:06')) do
+          expect(report.send(:report_subject)).to eq 'Three Day Report (Feb 4 - Feb 6)'
+        end
+      end
+    end
+
+    describe 'message body', :unstub_puts do
+      let(:report) { create :three_day_report }
+      let!(:task1) { create :task, jira_key: 'DOV-2', pivotal_id: 2345678, three_day_report: nil, finished_at: 2.days.ago }
+      let!(:task2) { create :task, jira_key: 'MOB-8', pivotal_id: 2345674, three_day_report: nil, finished_at: 1.day.ago }
+
+      specify do
+        expect(report.send(:report_body)).to match /table/
+      end
+
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,30 +1,40 @@
-require 'simplecov'
+# frozen_string_literal: true
 
-module SimpleCov::Configuration
-  def clean_filters
-    @filters = []
-  end
-end
-
-SimpleCov.configure do
-  clean_filters
-  load_adapter 'test_frameworks'
-end
+ENV['CAPEROMA_TEST'] = 'true'
 
-ENV["COVERAGE"] && SimpleCov.start do
-  add_filter "/.rvm/"
-end
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib', 'models'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 
-require 'rspec'
 require 'caperoma'
+require 'rspec'
+require 'shoulda/matchers'
+require 'factory_girl'
+require 'database_cleaner'
+require 'timecop'
 
 # Requires supporting files with custom matchers and macros, etc,
 # in ./support/ and its subdirectories.
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+Dir["#{File.dirname(__FILE__)}/factories/*.rb"].each { |f| require f }
 
 RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+
   config.color = true
   config.tty = true
+  config.filter_run focus: true
+  config.filter_run_excluding :pending
+  config.run_all_when_everything_filtered = true
+
+  config.include(Shoulda::Matchers::ActiveModel, type: :model)
+end
+
+Shoulda::Matchers.configure do |config|
+  config.integrate do |with|
+    with.test_framework :rspec
+    with.library :active_record
+  end
 end
+
+I18n.enforce_available_locales = false

data/spec/support/capefile_generator.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+require 'yaml'
+
+CAPEFILE_SAMPLE_DATA = {
+  'github_repo' => 'owner/repo',
+  'jira_url' => 'https://owner.atlassian.net/',
+  'jira_project_id' => 10001,
+  'jira_issue_type_ids' => {
+    'feature' => 10101,
+    'bug' => 10103,
+    'chore' => 10100,
+    'fix' => 10101,
+    'meeting' => 10100,
+  },
+  'jira_transition_ids' => {
+    'todo' => 11,
+    'in_progress' => 21,
+    'done' => 31,
+  },
+  'pivotal_tracker_project_id' => 2374972,
+  'create_features_in_pivotal' => true,
+  'create_bugs_in_pivotal' => true,
+  'create_chores_in_pivotal' => true,
+  'create_fixes_in_pivotal_as_chores' => false,
+  'create_meetings_in_pivotal_as_chores' => false
+}
+
+def create_capefile(jira_project_id)
+  CAPEFILE_SAMPLE_DATA['jira_project_id'] = jira_project_id
+  yaml = CAPEFILE_SAMPLE_DATA.to_yaml
+  File.write 'Capefile.test', yaml
+end
+
+def remove_capefile
+  File.delete 'Capefile.test'
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.configure do |config|
+  config.before(:suite) do
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.strategy = if ENV['spec_type'] == 'feature'
+                                 :truncation
+                               else
+                                 :transaction
+                               end
+
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/stubs.rb

@@ -1,9 +1,178 @@
-RSpec.configure do |config|
-  config.before do |example|
-    if example.metadata[:expect_db_write].nil?
-      allow(SDBM).to receive(:open).with('jira_credentials').and_return SDBM.open 'test'
-    else
-      expect(SDBM).to receive(:open).with('jira_credentials').and_return SDBM.open 'test'
-    end
-  end
-end
+# frozen_string_literal: true
+
+PROJECTS_LIST = [
+  {
+    'self' => 'http://www.example.com/jira/rest/api/2/project/EX',
+    'id' => '123',
+    'key' => 'EX',
+    'name' => 'Dummy',
+    'avatarUrls' => {
+      '24x24' => 'http://www.example.com/jira/secure/projectavatar?size=small&pid=10000',
+      '16x16' => 'http://www.example.com/jira/secure/projectavatar?size=xsmall&pid=10000',
+      '32x32' => 'http://www.example.com/jira/secure/projectavatar?size=medium&pid=10000',
+      '48x48' => 'http://www.example.com/jira/secure/projectavatar?size=large&pid=10000'
+    },
+    'projectCategory' => {
+      'self' => 'http://www.example.com/jira/rest/api/2/projectCategory/10000',
+      'id' => '10000',
+      'name' => 'FIRST',
+      'description' => 'First Project Category'
+    }
+  }
+].freeze
+
+JIRA_ISSUE_CREATION = {
+  'update' => {
+    'worklog' => [{
+      'add' => {
+        'started' => '2011-07-05T11:05:00.000+0000',
+        'timeSpent' => '60m'
+      }
+    }]
+  },
+  'fields' => {
+    'project' => { 'id' => '10000' },
+    'summary' => 'awesome issue',
+    'issuetype' => { 'id' => '10000' },
+    'assignee' => { 'name' => 'homer' },
+    'description' => 'description',
+    'components' => [{ 'id' => '10000' }]
+  }
+}.freeze
+
+JIRA_ISSUE_CREATION_RESPONSE = Jbuilder.encode do |j|
+  j.id '10000'
+  j.key 'TST-24'
+  j.self 'http://www.example.com/jira/rest/api/2/issue/10000'
+end
+
+PIVOTAL_ISSUE_CREATION_RESPONSE = Jbuilder.encode do |j|
+  j.kind 'story'
+  j.id '12345678'
+  j.created_at '2019-07-03T18:38:59Z'
+  j.updated_at '2019-07-04T12:31:04Z'
+  j.estimate 2
+  j.story_type 'feature'
+  j.name 'Feature name'
+  j.description 'Feature description'
+  j.current_state 'unstarted'
+  j.requested_by_id '23456789'
+  j.url 'https://www.pivotaltracker.com/story/show/12345678'
+  j.project_id 34_567_890
+  j.owner_ids ['23456789']
+  j.labels []
+  j.owned_by_id '234567890'
+end
+
+RSpec.configure do |config|
+  config.before do |example|
+    # if example.metadata[:unstab_capefile].blank?
+    #   Caperoma::Capefile::JIRA_URL = '123'
+    # end
+
+    # if example.metadata[:unstub_git].blank?
+    #   allow_any_instance_of(Feature).to receive(:git_branch)
+    #   allow_any_instance_of(Feature).to receive(:git_commit)
+    #   allow_any_instance_of(Feature).to receive(:git_push)
+    #   allow_any_instance_of(Feature).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Feature).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Feature).to receive(:git_pull_request)
+    #   allow_any_instance_of(Feature).to receive(:git_checkout)
+
+    #   allow_any_instance_of(Bug).to receive(:git_branch)
+    #   allow_any_instance_of(Bug).to receive(:git_commit)
+    #   allow_any_instance_of(Bug).to receive(:git_push)
+    #   allow_any_instance_of(Bug).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Bug).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Bug).to receive(:git_pull_request)
+    #   allow_any_instance_of(Bug).to receive(:git_checkout)
+
+    #   allow_any_instance_of(Chore).to receive(:git_branch)
+    #   allow_any_instance_of(Chore).to receive(:git_commit)
+    #   allow_any_instance_of(Chore).to receive(:git_push)
+    #   allow_any_instance_of(Chore).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Chore).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Chore).to receive(:git_pull_request)
+    #   allow_any_instance_of(Chore).to receive(:git_checkout)
+
+    #   allow_any_instance_of(Fix).to receive(:git_branch)
+    #   allow_any_instance_of(Fix).to receive(:git_commit)
+    #   allow_any_instance_of(Fix).to receive(:git_push)
+    #   allow_any_instance_of(Fix).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Fix).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Fix).to receive(:git_pull_request)
+    #   allow_any_instance_of(Fix).to receive(:git_checkout)
+
+    #   allow_any_instance_of(Meeting).to receive(:git_branch)
+    #   allow_any_instance_of(Meeting).to receive(:git_commit)
+    #   allow_any_instance_of(Meeting).to receive(:git_push)
+    #   allow_any_instance_of(Meeting).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Meeting).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Meeting).to receive(:git_pull_request)
+    #   allow_any_instance_of(Meeting).to receive(:git_checkout)
+
+    #   allow_any_instance_of(Task).to receive(:git_branch)
+    #   allow_any_instance_of(Task).to receive(:git_commit)
+    #   allow_any_instance_of(Task).to receive(:git_push)
+    #   allow_any_instance_of(Task).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(Task).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(Task).to receive(:git_pull_request)
+    #   allow_any_instance_of(Task).to receive(:git_checkout)
+
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_branch)
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_commit)
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_push)
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_pull_request)
+    #   allow_any_instance_of(TaskWithSeparateBranch).to receive(:git_checkout)
+
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_branch)
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_commit)
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_push)
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_last_commit_name).and_return 'some commit'
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_current_branch).and_return 'parent-branch'
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_pull_request)
+    #   allow_any_instance_of(TaskWithCommit).to receive(:git_checkout)
+    # end
+
+    if example.metadata[:unstub_reports].blank?
+      allow_any_instance_of(Report).to receive(:send_email)
+    end
+
+    if example.metadata[:unstab_api_calls].blank?
+      faraday = spy('faraday')
+      allow(Faraday).to receive(:new).and_return faraday
+      allow(Faraday).to receive(:default_adapter)
+      allow(faraday).to receive(:get)
+      allow(faraday).to receive(:post)
+      allow(faraday).to receive(:put)
+    end
+
+    allow(STDOUT).to receive(:puts) if example.metadata[:unstub_puts].blank?
+
+    if example.metadata[:unstub_time_now].blank?
+      allow(Time).to receive(:now).and_return Time.parse('5 April 2014')
+    end
+
+    if example.metadata[:unstub_jira_creation].blank?
+      allow_any_instance_of(Task).to receive :create_issue_on_jira
+    end
+
+    if example.metadata[:unstub_jira_starting].blank?
+      allow_any_instance_of(Task).to receive :start_issue_on_jira
+    end
+
+    if example.metadata[:unstub_pivotal_creation].blank?
+      allow_any_instance_of(Task).to receive :create_issue_on_pivotal
+    end
+
+    if example.metadata[:unstub_pivotal_starting].blank?
+      allow_any_instance_of(Task).to receive :start_issue_on_pivotal
+    end
+
+    if example.metadata[:unstub_key_output].blank?
+      allow_any_instance_of(Task).to receive :output_jira_key
+    end
+  end
+end