learn-test 3.2.4 → 3.3.0.pre.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78b850c796efd54f383ea05c625ad8d5f6038d9dcf865b19fc21fd6dcad435fb
-  data.tar.gz: 88336d0c6c787c1680b22453220a1046741f7ffaf06d97ce090b7fd4d02d4f25
+  metadata.gz: 8068b9ca6cd44a8bea55fbe7aff5ec5ee08f811e171021726a6e6364c250a210
+  data.tar.gz: 2e6ab893c0f4492cae089cb0c08f836a56486c5d3d3a1140cdf7b4172dc6fbff
 SHA512:
-  metadata.gz: ed0abed1a972742bdbd28e05ca20218109f3834267ef560bf884b358e237107f6c294ecbda9248d570fe88c8f4014efb7cdc44bd6779b246acc59d639a35e80c
-  data.tar.gz: c36adeff838bf630cc640df29e38f7a82de8e65c5aeff5f20a1c107b93d277c4af07ceb2703306d4306a5295237aec822a1f3105bfebfd7a7d64998be5a69700
+  metadata.gz: e409f578b4721d75066dc5430d7c725ca47df693487b1cda69149ba4db5d3ca0e7ae33e61e57150ec43c2aaeb1c3e03e5b66ee1d0ddbf281b3c26b8f423aa087
+  data.tar.gz: 4a05f33e6601662c068f6d10fb9d225c68f0d60b1edb90dcf195271f4594c426bcf80e99fb987fc49cc69d7f8e723be2fc16f16d4b467f10cd0af103dd043260

data/.rubocop.yml

@@ -0,0 +1,27 @@
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+
+Layout/FirstHashElementIndentation:
+  EnforcedStyle: consistent
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
+Lint/RaiseException:
+  Enabled: true
+
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/BlockLength:
+  Enabled: false
+
+Metrics/MethodLength:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+# TODO: Add top-level module documentation
+Style/Documentation:
+  Enabled: false

data/Gemfile

@@ -3,7 +3,9 @@
 source 'https://rubygems.org'
 
 group :test do
+  gem 'aruba', '~> 1.0.3'
   gem 'rspec_junit_formatter', '~> 0.4.1'
+  gem 'simplecov', require: false
 end
 
 gemspec

data/learn-test.gemspec

@@ -29,4 +29,5 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "faraday", "~> 1.0"
   spec.add_runtime_dependency "crack", "~> 0.4.3"
   spec.add_runtime_dependency "colorize", "~> 0.8.1"
+  spec.add_runtime_dependency "zeitwerk", "~> 2.4.0"
 end

data/lib/learn_test.rb

@@ -3,42 +3,21 @@
 require 'fileutils'
 require 'oj'
 require 'colorize'
+require 'zeitwerk'
 
-require_relative 'learn_test/version'
-require_relative 'learn_test/netrc_interactor'
-require_relative 'learn_test/git_wip'
-require_relative 'learn_test/github_interactor'
-require_relative 'learn_test/user_id_parser'
-require_relative 'learn_test/username_parser'
-require_relative 'learn_test/learn_oauth_token_parser'
-require_relative 'learn_test/repo_parser'
-require_relative 'learn_test/file_finder'
-require_relative 'learn_test/runner'
-
-require_relative 'learn_test/dependency'
-
-require_relative 'learn_test/strategy'
-require_relative 'learn_test/js_strategy'
-require_relative 'learn_test/strategies/rspec'
-require_relative 'learn_test/strategies/karma'
-require_relative 'learn_test/strategies/protractor'
-require_relative 'learn_test/strategies/java_junit'
-require_relative 'learn_test/strategies/csharp_nunit'
-require_relative 'learn_test/strategies/mocha'
-require_relative 'learn_test/strategies/pytest'
-require_relative 'learn_test/strategies/none'
+loader = Zeitwerk::Loader.for_gem
+loader.inflector.inflect(
+  'csharp' => 'CSharp',
+  'csharp_nunit' => 'CSharpNunit'
+)
+loader.setup
 
 module LearnTest
-  module Dependencies
-    autoload :NodeJS,         'learn_test/dependencies/nodejs'
-    autoload :PhantomJS,      'learn_test/dependencies/phantomjs'
-    autoload :Karma,          'learn_test/dependencies/karma'
-    autoload :Protractor,     'learn_test/dependencies/protractor'
-    autoload :Java,           'learn_test/dependencies/java'
-    autoload :CSharp,         'learn_test/dependencies/csharp'
-    autoload :Ant,            'learn_test/dependencies/ant'
-    autoload :Imagemagick,    'learn_test/dependencies/imagemagick'
-    autoload :SeleniumServer, 'learn_test/dependencies/selenium_server'
-    autoload :Pytest,         'learn_test/dependencies/pytest'
+  def self.root
+    File.dirname __dir__
+  end
+
+  def self.bin
+    File.join root, 'bin'
   end
 end

data/lib/learn_test/git.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'git'
+
+module LearnTest
+  module Git
+    def self.open(directory: './', options: {})
+      Base.open(directory, options)
+    end
+
+    class Base < ::Git::Base
+      def wip(message:)
+        wip = Wip::Base.new(base: self, message: message)
+        wip.process!
+        wip
+      end
+    end
+  end
+end

data/lib/learn_test/git/wip/base.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+require 'fileutils'
+require 'git'
+require 'logger'
+require 'tempfile'
+
+module LearnTest
+  module Git
+    module Wip
+      class Base < ::Git::Path
+        TEMPFILE = '.wip'
+        PREFIX = 'refs/wip/'
+
+        attr_reader :working_branch, :wip_branch
+
+        def initialize(base:, message:)
+          @base = base
+          @message = message
+          @success = nil
+
+          current_branch = @base.current_branch
+
+          raise NoCommitsError, 'master' if current_branch.nil? # TODO: Swap to `main`?
+
+          @tmp = Tempfile.new(TEMPFILE)
+          @working_branch = Branch.new(base: @base, name: current_branch)
+          @wip_branch = Branch.new(base: @base, name: "#{PREFIX}#{current_branch}")
+        end
+
+        def process!
+          if @wip_branch.last_revision
+            merge = @base.merge_base(@wip_branch.last_revision, @working_branch.last_revision)
+
+            @wip_branch.parent = if merge == @working_branch.last_revision
+                                   @wip_branch.last_revision
+                                 else
+                                   @working_branch.last_revision
+                                 end
+          else
+            @wip_branch.parent = @working_branch.last_revision
+          end
+
+          new_tree = build_new_tree(@wip_branch.parent)
+          @base.diff(new_tree, @wip_branch.parent)
+
+          # tree_diff = @base.diff(new_tree, @wip_branch.parent)
+          # raise NoChangesError, @wip_branch if tree_diff.count.zero?
+
+          commit = @base.commit_tree(new_tree, parent: @wip_branch.parent)
+
+          @base.lib.send(:command, 'update-ref', ['-m', @message, @wip_branch, commit.objectish])
+
+          @success = true
+        ensure
+          cleanup
+        end
+
+        def success?
+          @success
+        end
+
+        private
+
+        def build_new_tree(wip_parent)
+          index = "#{@tmp.path}-index"
+
+          FileUtils.rm(index, force: true)
+          FileUtils.cp("#{@base.dir.path}/.git/index", index)
+
+          @base.read_tree(wip_parent)
+          @base.lib.send(:command, 'add', ['--update', '--', '.'])
+          @base.add(all: true)
+
+          new_tree_obj = @base.write_tree
+
+          FileUtils.rm(index, force: true)
+
+          new_tree_obj
+        end
+
+        def cleanup
+          FileUtils.rm("#{@tmp.path}-*", force: true)
+        end
+      end
+    end
+  end
+end

data/lib/learn_test/git/wip/branch.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module LearnTest
+  module Git
+    module Wip
+      class Branch
+        attr_accessor :parent
+
+        def initialize(base:, name:)
+          @base = base
+          @name = name
+        end
+
+        def last_revision(raise_no_commits: false)
+          @last_revision ||= begin
+            begin
+              @base.revparse(@name)
+            rescue ::Git::GitExecuteError => e
+              if raise_no_commits
+                raise e.message.match(NoCommitsError::REGEX) ? NoCommitsError.new(@name) : e
+              end
+
+              raise unless e.message.match(NoCommitsError::REGEX)
+
+              false
+            end
+          end
+        end
+
+        def to_s
+          @name
+        end
+      end
+    end
+  end
+end

data/lib/learn_test/git/wip/error.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module LearnTest
+  module Git
+    module Wip
+      class Error < StandardError; end
+
+      class NoChangesError < Error
+        def initialize(branch)
+          super "No changes found on `#{branch}`"
+        end
+      end
+
+      class NoCommitsError < Error
+        REGEX = /unknown revision or path not in the working tree/i.freeze
+
+        def initialize(branch)
+          super "Branch `#{branch}` doesn't have any commits. Please commit and try again."
+        end
+      end
+    end
+  end
+end

data/lib/learn_test/repo_parser.rb

@@ -6,7 +6,7 @@ module LearnTest
   class RepoParser
     def self.get_repo
       begin
-        repo = Git.open(FileUtils.pwd)
+        repo = ::Git.open(FileUtils.pwd)
       rescue
         puts "You don't appear to be in a Learn lesson's directory. Please enter 'learn open' or cd to an appropriate directory and try again."
         die

data/lib/learn_test/reporter.rb

@@ -3,9 +3,6 @@
 require 'fileutils'
 require 'json'
 
-require_relative 'client'
-require_relative 'git_wip'
-
 module LearnTest
   class Reporter
     attr_accessor :output_path, :debug
@@ -50,16 +47,30 @@ module LearnTest
       endpoint = strategy.service_endpoint
       augment_results!(results)
 
-      if client.post_results(endpoint, results)
-        if !LearnTest::GitWip.run! && @debug
-          puts 'There was a problem connecting to Github. Not pushing current branch state.'.red
-        end
-      else
-        if @debug
-          puts 'There was a problem connecting to Learn. Not pushing test results.'.red
-        end
+      unless client.post_results(endpoint, results)
+        puts 'There was a problem connecting to Learn. Not pushing test results.'.red if @debug
 
         save_failed_attempt(endpoint, results)
+        return
+      end
+
+      logger = @debug ? Logger.new(STDOUT, level: Logger::DEBUG) : false
+      repo = LearnTest::Git.open(options: { log: logger })
+
+      res = repo.wip(message: 'Automatic test submission')
+
+      unless res.success?
+        puts 'There was a problem creating your WIP branch. Not pushing current branch state.'.red if @debug
+        return
+      end
+
+      begin
+        repo.push('origin', "#{res.wip_branch}:refs/heads/fis-wip", { force: true })
+      rescue ::Git::GitExecuteError => e
+        if @debug
+          puts 'There was a problem connecting to Github. Not pushing current branch state.'.red
+          puts e.message
+        end
       end
     end
 
@@ -89,7 +100,7 @@ module LearnTest
 
     def augment_results!(results)
       if File.exist?("#{FileUtils.pwd}/.learn")
-        dot_learn = YAML.load(File.read("#{FileUtils.pwd}/.learn"))
+        dot_learn = YAML.safe_load(File.read("#{FileUtils.pwd}/.learn"))
 
         unless dot_learn['github'].nil?
           results[:github] = dot_learn['github']

data/lib/learn_test/strategy.rb

@@ -63,7 +63,7 @@ module LearnTest
       if npm_install
         system(command)
       else
-        Open3.popen3(command) do |stdin, stdout, stderr, wait_thr|
+        Open3.popen3(command) do |_, stdout, stderr, wait_thr|
           while out = stdout.gets do
             puts out
           end

data/lib/learn_test/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LearnTest
-  VERSION = '3.2.4'
+  VERSION = '3.3.0.pre.1'
 end

data/spec/features/rspec_unit_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-describe 'Running a RSpec Unit Test' do
-  before(:all) do
+describe 'Running a RSpec Unit Test', type: :aruba do
+  before :all do
     # While it doesn't cause these tests to fail, nasty messages occur (and more)
     # when either a ~/.netrc entry or file itself doesn't exist. This aims to correct that,
     # and will only ever be called once.
@@ -9,9 +9,22 @@ describe 'Running a RSpec Unit Test' do
     LearnTest::UsernameParser.get_username
   end
 
+  before :each do
+    copy '%/rspec-unit-spec', 'example'
+    cd 'example'
+    run_command_and_stop 'git init'
+    run_command_and_stop 'git add .'
+    run_command_and_stop 'git commit -m "Initial Commit"'
+  end
+
+  def run(flags = '')
+    run_command_and_stop("#{File.join(LearnTest.bin, 'learn-test')} #{flags}")
+    last_command_started.output
+  end
+
   context 'a basic rspec unit test' do
     it 'runs the spec with 0 failures' do
-      output = `cd ./spec/fixtures/rspec-unit-spec && ./../../../bin/learn-test --local --test`
+      output = run('--local --test')
 
       expect(output).to include('3 examples, 0 failures')
       expect(output).to_not include('1 failures')
@@ -20,14 +33,14 @@ describe 'Running a RSpec Unit Test' do
 
   context 'with the --example flag' do
     it 'runs only the appropriate tests' do
-      output = `cd ./spec/fixtures/rspec-unit-spec && ./../../../bin/learn-test --local --test --example multiple`
+      output = run('--local --test --example multiple')
 
       expect(output).to include('1 example, 0 failures')
       expect(output).to_not include('2 examples')
     end
 
-     it 'accepts multiple examples' do
-      output = `cd ./spec/fixtures/rspec-unit-spec && ./../../../bin/learn-test --local --test --example multiple --example accepts`
+    it 'accepts multiple examples' do
+      output = run(' --local --test --example multiple --example accepts')
 
       expect(output).to include('2 examples, 0 failures')
       expect(output).to_not include('3 examples')

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