learn-test 3.2.3 → 3.3.0.pre.4

data/spec/fixtures/rspec-unit-spec/.results.json

@@ -0,0 +1 @@
+{"version":"3.9.2","examples":[{"id":"./spec/dog_spec.rb[1:1]","description":"runs the spec spec correctly","full_description":"Dog runs the spec spec correctly","status":"passed","file_path":"./spec/dog_spec.rb","line_number":6,"run_time":0.000423,"pending_message":null},{"id":"./spec/dog_spec.rb[1:2]","description":"has multiple specs","full_description":"Dog has multiple specs","status":"passed","file_path":"./spec/dog_spec.rb","line_number":10,"run_time":4.8e-05,"pending_message":null},{"id":"./spec/dog_spec.rb[1:3]","description":"accepts more than one example","full_description":"Dog accepts more than one example","status":"passed","file_path":"./spec/dog_spec.rb","line_number":14,"run_time":3.6e-05,"pending_message":null}],"summary":{"duration":0.001252,"example_count":3,"failure_count":0,"pending_count":0,"errors_outside_of_examples_count":0},"summary_line":"3 examples, 0 failures"}

data/spec/fixtures/rspec-unit-spec/spec/spec_helper.rb

@@ -1,95 +1,13 @@
 # frozen_string_literal: true
 
-# 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_relative '../lib/dog'
 
 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

data/spec/learn_test/git/wip/base_spec.rb

@@ -0,0 +1,89 @@
+# frozen_string_literal: true
+
+describe LearnTest::Git::Wip::Base do
+  describe 'attr_reader' do
+    let(:base) { LearnTest::Git::Base.open('./') }
+    let(:message) { 'foobar' }
+    let(:instance) do
+      described_class.new(
+        base: base,
+        message: message
+      )
+    end
+
+    it 'should have :working_branch, :wip_branch' do
+      expect(instance).to respond_to(:working_branch, :wip_branch)
+    end
+  end
+
+  describe '.process!', type: :aruba do
+    let!(:path) { File.join Aruba.config.home_directory, 'example' }
+    let(:base) { LearnTest::Git::Base.open(path) }
+    let(:message) { 'foobar' }
+    let(:instance) do
+      described_class.new(
+        base: base,
+        message: message
+      )
+    end
+
+    def initialize_repo(commit: true)
+      create_directory 'example'
+      cd 'example'
+      write_file 'README.md', 'Hello World'
+      git_init
+
+      if commit # rubocop:disable Style/GuardClause
+        git_add
+        git_commit 'Initial Commit'
+      end
+    end
+
+    context 'no commits' do
+      before(:each) { initialize_repo(commit: false) }
+
+      it 'should raise' do
+        expect { instance.process! }.to raise_error(LearnTest::Git::Wip::NoCommitsError)
+      end
+    end
+
+    xcontext 'no changes' do
+      before(:each) { initialize_repo }
+
+      it 'should raise' do
+        expect { instance.process! }.to raise_error(LearnTest::Git::Wip::NoChangesError)
+      end
+    end
+
+    context 'changes' do
+      context 'staged' do
+        before :each do
+          initialize_repo
+          write_file 'test.rb', ''
+        end
+
+        it 'should process successfully' do
+          instance.process!
+
+          expect(instance.success?).to eq(true)
+        end
+      end
+
+      context 'committed' do
+        before :each do
+          initialize_repo
+          write_file 'test.rb', ''
+
+          git_add
+          git_commit 'foo'
+        end
+
+        it 'should process successfully' do
+          instance.process!
+
+          expect(instance.success?).to eq(true)
+        end
+      end
+    end
+  end
+end

data/spec/learn_test/git/wip/branch_spec.rb

@@ -0,0 +1,121 @@
+# frozen_string_literal: true
+
+describe LearnTest::Git::Wip::Branch do
+  let(:base) { LearnTest::Git::Base.open('./') }
+  let(:name) { base.current_branch }
+  let(:branch) do
+    described_class.new(
+      base: base,
+      name: name
+    )
+  end
+
+  let(:sha1) { Digest::SHA1.new.hexdigest }
+  let(:nothing_to_commit_error_msg) { 'nothing to commit, working directory clean' }
+  let(:no_commits_error_msg) do
+    "fatal: ambiguous argument '#{name}': unknown revision or path not in the working tree."
+  end
+
+  describe 'accessors' do
+    it 'should have :parent, :parent=' do
+      expect(branch).to respond_to(:parent, :parent=)
+    end
+  end
+
+  describe '.new' do
+    it 'should require :base' do
+      expect { described_class.new(base: {}) }.to raise_error(ArgumentError, /name/)
+    end
+
+    it 'should require :name' do
+      expect { described_class.new(name: 'foo') }.to raise_error(ArgumentError, /base/)
+    end
+  end
+
+  describe '.to_s' do
+    it 'should return @name' do
+      expect(branch.to_s).to eq(name)
+    end
+  end
+
+  describe '.last_revision' do
+    context 'success' do
+      it 'should return a SHA' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_return(sha1)
+
+        expect(branch.last_revision).to eq(sha1)
+      end
+
+      it 'should only run revparse once' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_return(sha1)
+          .once
+
+        expect(branch.last_revision).to eq(sha1)
+        expect(branch.last_revision).to eq(sha1)
+      end
+    end
+
+    context 'hide expected Git errors' do
+      it 'should hide no commits error' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_raise(::Git::GitExecuteError, no_commits_error_msg)
+
+        expect { branch.last_revision }.to_not raise_error
+      end
+
+      it 'should return false' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_raise(::Git::GitExecuteError, no_commits_error_msg)
+
+        expect(branch.last_revision).to eq(false)
+      end
+    end
+
+    context 'raise expected Git errors' do
+      it 'should raise no commits error' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_raise(::Git::GitExecuteError, no_commits_error_msg)
+
+        expect do
+          branch.last_revision(raise_no_commits: true)
+        end.to raise_error(
+          LearnTest::Git::Wip::NoCommitsError, "Branch `#{name}` doesn't have any commits. Please commit and try again."
+        )
+      end
+    end
+
+    context 'raise unexpected Git errors' do
+      it 'should raise no commits error' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_raise(::Git::GitExecuteError, nothing_to_commit_error_msg)
+
+        expect { branch.last_revision }.to raise_error(::Git::GitExecuteError, nothing_to_commit_error_msg)
+      end
+    end
+
+    context 'raise unexpected errors' do
+      it 'should raise' do
+        expect(base)
+          .to receive(:revparse)
+          .with(name)
+          .and_raise(StandardError)
+
+        expect { branch.last_revision }.to raise_error(StandardError)
+      end
+    end
+  end
+end

data/spec/learn_test/git/wip/error_spec.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+describe LearnTest::Git::Wip do
+  let(:branch) { rand(0..999_999).to_s }
+
+  describe LearnTest::Git::Wip::Error do
+    it 'should inherit from StandardError' do
+      expect(described_class).to be < StandardError
+    end
+  end
+
+  describe LearnTest::Git::Wip::NoChangesError do
+    it 'should inherit from Error' do
+      expect(described_class).to be < LearnTest::Git::Wip::Error
+    end
+
+    it 'should require a branch' do
+      expect { described_class.new }.to raise_error(ArgumentError)
+    end
+
+    it 'should have the correct messaging' do
+      error = described_class.new(branch)
+      expect(error.message).to eq "No changes found on `#{branch}`"
+    end
+  end
+
+  describe LearnTest::Git::Wip::NoCommitsError do
+    it 'should inherit from Error' do
+      expect(described_class).to be < LearnTest::Git::Wip::Error
+    end
+
+    it 'should require a branch' do
+      expect { described_class.new }.to raise_error(ArgumentError)
+    end
+
+    it 'should have the correct messaging' do
+      error = described_class.new(branch)
+      expect(error.message).to eq "Branch `#{branch}` doesn't have any commits. Please commit and try again."
+    end
+  end
+end

data/spec/learn_test/git_spec.rb

@@ -1,47 +1,53 @@
 # frozen_string_literal: true
 
-describe LearnTest::GitWip do
-  subject { described_class }
-
-  let!(:working_branch) { 'develop' }
-  let!(:git_url) { 'https://github.com/learn-co/learn-test' }
-  let!(:git_base) { instance_double(Git::Base) }
-
-  let(:wait_thr) { double }
-  let(:wait_thr_value) { double }
-  let(:stdout_and_stderr) { double }
-
-  context 'success' do
-    it 'should return the git url' do
-      expect(Git::Base).to receive(:open).with('./', { log: false }).and_return(git_base)
-      expect(git_base).to receive(:current_branch).and_return(working_branch)
+describe LearnTest::Git do
+  describe '.open' do
+    context 'defaults' do
+      it 'should instantiate LearnTest::Git::Base' do
+        expect(LearnTest::Git::Base).to receive(:open).with('./', {})
+        LearnTest::Git.open
+      end
+    end
 
-      expect(wait_thr).to receive(:value).and_return(wait_thr_value)
-      expect(wait_thr_value).to receive(:exitstatus).and_return(0)
+    context 'with options' do
+      it 'should instantiate LearnTest::Git::Base' do
+        directory = './foo'
+        options = { logger: false }
 
-      expect(Open3).to receive(:popen3).and_yield(nil, nil, nil, wait_thr)
+        expect(LearnTest::Git::Base).to receive(:open).with(directory, options)
 
-      expect(git_base).to receive_message_chain(:config, :[]).with('remote.origin.url').and_return("#{git_url}.git")
-      expect(subject.run!).to eq("#{git_url}/tree/wip")
+        LearnTest::Git.open(
+          directory: directory,
+          options: options
+        )
+      end
     end
   end
 
-  context 'failure' do
-    it 'should return false on process error' do
-      expect(Git::Base).to receive(:open).with('./', { log: false }).and_return(git_base)
-      expect(git_base).to receive(:current_branch).and_return(working_branch)
+  describe LearnTest::Git::Base do
+    it 'should inherit from ::Git::Base' do
+      expect(LearnTest::Git::Base).to be < ::Git::Base
+    end
 
-      expect(wait_thr).to receive(:value).and_return(wait_thr_value)
-      expect(wait_thr_value).to receive(:exitstatus).and_return(1)
+    describe '#wip' do
+      let!(:repo) { described_class.open('./') }
+      let!(:message) { 'FooBar' }
+      let!(:wip) { double(LearnTest::Git::Wip::Base) }
 
-      expect(Open3).to receive(:popen3).and_yield(nil, nil, nil, wait_thr)
+      it 'should require a :message' do
+        expect { repo.wip }.to raise_error(ArgumentError)
+      end
 
-      expect(subject.run!).to eq(false)
-    end
+      it 'should instantiate and run .process!' do
+        expect(LearnTest::Git::Wip::Base)
+          .to receive(:new)
+          .with(base: repo, message: message)
+          .and_return(wip)
+
+        expect(wip).to receive(:process!)
 
-    it 'should return false on StandardError' do
-      expect(Git::Base).to receive(:open).and_raise(StandardError)
-      expect(subject.run!).to eq(false)
+        expect(repo.wip(message: message)).to eq(wip)
+      end
     end
   end
 end

data/spec/learn_test/reporter_spec.rb

@@ -1,15 +1,12 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
-
-require_relative '../../lib/learn_test/reporter'
-
 require 'tempfile'
 require 'json'
 
 describe LearnTest::Reporter do
   let!(:client) { instance_spy(LearnTest::Client) }
-  let!(:git_tree) { 'https://github.com/learn-co/learn-test/tree/wip' }
+  let!(:git_base) { double(LearnTest::Git::Base) }
+  let!(:git_wip) { double(LearnTest::Git::Wip::Base) }
   let!(:strategy) do
     instance_spy(
       LearnTest::Strategy,
@@ -45,42 +42,75 @@ describe LearnTest::Reporter do
     end
 
     context 'sync with Github' do
-      it 'should not run LearnTest::GitWip, if :post_results fails' do
+      it 'should not run LearnTest::Git::Wip, if :post_results fails' do
         expect(client).to receive(:post_results).and_return(false)
-        expect(LearnTest::GitWip).to_not receive(:run!)
+        expect(LearnTest::Git).to_not receive(:open)
 
         reporter.report
       end
 
-      it 'should run LearnTest::GitWip.run!' do
+      it 'should run LearnTest::Git::Wip' do
+        branch = double(LearnTest::Git::Wip::Branch)
+        branch_name = 'foo'
+
         expect(client).to receive(:post_results).and_return(true)
-        expect(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+        expect(LearnTest::Git).to receive(:open).and_return(git_base)
+        expect(git_base).to receive(:wip).and_return(git_wip)
+        expect(git_wip).to receive(:success?).and_return(true)
+
+        expect(branch).to receive(:to_s).and_return(branch_name)
+        expect(git_wip).to receive(:wip_branch).and_return(branch)
+
+        expect(git_base)
+          .to receive(:push)
+          .with('origin', "#{branch_name}:refs/heads/fis-wip", { force: true })
 
         reporter.report
       end
 
       it 'should not output an error message without debug' do
         expect(client).to receive(:post_results).and_return(true)
-        expect(LearnTest::GitWip).to receive(:run!).and_return(false)
+        expect(LearnTest::Git).to receive(:open).and_return(git_base)
+        expect(git_base).to receive(:wip).and_return(git_wip)
+        expect(git_wip).to receive(:success?).and_return(false)
+
+        expect(git_base).to_not receive(:push)
 
         expect { reporter.report }.to_not output.to_stdout
       end
 
       it 'should output an error message with debug' do
         expect(client).to receive(:post_results).and_return(true)
-        expect(LearnTest::GitWip).to receive(:run!).and_return(false)
+
+        expect(LearnTest::Git).to receive(:open).and_return(git_base)
+        expect(git_base).to receive(:wip).and_return(git_wip)
+        expect(git_wip).to receive(:success?).and_return(false)
+
+        expect(git_base).to_not receive(:push)
 
         expect do
           reporter.debug = true
           reporter.report
-        end.to output(/Github/i).to_stdout
+        end.to output(/There was a problem creating your WIP branch/i).to_stdout
       end
     end
 
     it 'posts results to the service endpoint' do
-      reporter.report
+      branch = double(LearnTest::Git::Wip::Branch)
+      branch_name = 'bar'
+
+      expect(LearnTest::Git).to receive(:open).and_return(git_base)
+      expect(git_base).to receive(:wip).and_return(git_wip)
+      expect(git_wip).to receive(:success?).and_return(true)
 
-      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+      expect(branch).to receive(:to_s).and_return(branch_name)
+      expect(git_wip).to receive(:wip_branch).and_return(branch)
+
+      expect(git_base)
+        .to receive(:push)
+        .with('origin', "#{branch_name}:refs/heads/fis-wip", { force: true })
+
+      reporter.report
 
       expect(client).to have_received(:post_results)
         .with(strategy.service_endpoint, strategy.results)
@@ -131,7 +161,6 @@ describe LearnTest::Reporter do
 
     it 'deletes the output file when all reports are sent successfully' do
       allow(client).to receive(:post_results).and_return(true)
-      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
 
       reports = { hello: ['world'] }
 
@@ -145,7 +174,6 @@ describe LearnTest::Reporter do
 
     it 'writes the remaining reports from the output file' do
       allow(client).to receive(:post_results).and_return(true, false)
-      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
 
       reports = { success: ['world'], failure: ['hello'] }