learn-test 3.2.4 → 3.3.0.pre.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78b850c796efd54f383ea05c625ad8d5f6038d9dcf865b19fc21fd6dcad435fb
-  data.tar.gz: 88336d0c6c787c1680b22453220a1046741f7ffaf06d97ce090b7fd4d02d4f25
+  metadata.gz: 8bcb52a7bdd3a25aa1938ba7d13b929bb52e2ecae2107810dd074ad400a04c1c
+  data.tar.gz: f5fbd110ec71a7be53ef29ee44debbfa5febff13d70cdd6633624bae7fd9ccd3
 SHA512:
-  metadata.gz: ed0abed1a972742bdbd28e05ca20218109f3834267ef560bf884b358e237107f6c294ecbda9248d570fe88c8f4014efb7cdc44bd6779b246acc59d639a35e80c
-  data.tar.gz: c36adeff838bf630cc640df29e38f7a82de8e65c5aeff5f20a1c107b93d277c4af07ceb2703306d4306a5295237aec822a1f3105bfebfd7a7d64998be5a69700
+  metadata.gz: 2ae93ca8deccd1c7879ba6b22aee84b64b92d85826ebfcf188c02bdccfb07935df88e661d75d4dfe0453bdbe02fbef4f0e2dcc15b020950191be07e9017c3b25
+  data.tar.gz: 4462b95bd0165cf7693b63aab7faab069e02a99f6c96ac47bea33d2a44d427a6b78b1c834abe5148732c1ff062ddc5e0959e282884dc8487a24d18659776f1c0

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,23 @@
 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',
+  'nodejs' => 'NodeJS',
+  'phantomjs' => 'PhantomJS'
+)
+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,84 @@
+# frozen_string_literal: true
+
+require 'fileutils'
+require 'git'
+require 'logger'
+require 'tempfile'
+
+module LearnTest
+  module Git
+    module Wip
+      class Base < ::Git::Path
+        TEMPFILE = '.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 = Reference.new(base: @base, name: 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)
+
+          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,38 @@
+# 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
+              regex = LearnTest::Git::Wip::NoCommitsError::REGEX
+
+              if raise_no_commits
+                raise e.message.match(regex) ? LearnTest::Git::Wip::NoCommitsError.new(@name) : e
+              end
+
+              raise unless e.message.match(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/git/wip/reference.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module LearnTest
+  module Git
+    module Wip
+      class Reference < Branch
+        attr_accessor :parent
+
+        PREFIX = 'refs/wip/'
+
+        def initialize(base:, name:)
+          dir = File.join(base.repo.path, PREFIX)
+          Dir.mkdir(dir, 0755) unless Dir.exist?(dir)
+
+          super(base: base, name: "#{PREFIX}#{name}")
+        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.5'
 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,23 @@ describe 'Running a RSpec Unit Test' do
     LearnTest::UsernameParser.get_username
   end
 
+  before :each do
+    copy '%/rspec-unit-spec', 'example'
+    cd 'example'
+
+    git_init
+    git_add
+    git_commit '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 +34,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"}