learn-test 2.7.0 → 3.2.0

data/learn-test.gemspec

@@ -1,36 +1,32 @@
-# coding: utf-8
+# frozen_string_literal: true
+
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'learn_test/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "learn-test"
+  spec.name          = 'learn-test'
   spec.version       = LearnTest::VERSION
-  spec.authors       = ["Flatiron School"]
-  spec.email         = ["learn@flatironschool.com"]
-  spec.summary       = %q{Runs RSpec, Jasmine, Karma, Mocha, and Python Pytest Test builds and pushes JSON output to Learn.}
-  spec.homepage      = "https://github.com/learn-co/learn-test"
-  spec.license       = "MIT"
+  spec.authors       = ['Flatiron School']
+  spec.email         = ['learn@flatironschool.com']
+  spec.summary       = 'Runs RSpec, Karma, Mocha, and Python Pytest Test builds and pushes JSON output to Learn.'
+  spec.homepage      = 'https://github.com/learn-co/learn-test'
+  spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib", "bin"]
-  spec.required_ruby_version = '>= 2.3.0'
+  spec.require_paths = ['lib', 'bin']
+  spec.required_ruby_version = '>= 2.5.0'
 
-  spec.add_development_dependency "bundler", "~> 2.1.4"
   spec.add_development_dependency "rake", "~> 13.0.1"
-  spec.add_development_dependency "rspec", "~> 3.8"
+  spec.add_development_dependency "rspec", "~> 3.7"
+
   spec.add_runtime_dependency "rspec", "~> 3.0"
   spec.add_runtime_dependency "netrc", "~> 0.11.0"
-  spec.add_runtime_dependency "git", "~> 1.2"
-  spec.add_runtime_dependency "oj", "~> 2.9"
-  spec.add_runtime_dependency "faraday", "~> 0.9"
+  spec.add_runtime_dependency "git", "~> 1.7"
+  spec.add_runtime_dependency "oj", "~> 3.10"
+  spec.add_runtime_dependency "faraday", "~> 1.0"
   spec.add_runtime_dependency "crack", "~> 0.4.3"
-  spec.add_runtime_dependency "jasmine", "~> 2.6.0", ">= 2.6.0"
-  spec.add_runtime_dependency "jasmine-core", "< 2.99.1"
   spec.add_runtime_dependency "colorize", "~> 0.8.1"
-  spec.add_runtime_dependency "webrick", "~> 1.3.1", ">= 1.3.1"
-  spec.add_runtime_dependency "rainbow", "= 1.99.2"
-  spec.add_runtime_dependency "selenium-webdriver", "~> 2.52.0", '>= 2.52.0'
 end

data/lib/learn_test.rb

@@ -1,9 +1,12 @@
+# frozen_string_literal: true
+
 require 'fileutils'
 require 'oj'
 require 'colorize'
 
 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'
@@ -16,14 +19,12 @@ require_relative 'learn_test/dependency'
 
 require_relative 'learn_test/strategy'
 require_relative 'learn_test/js_strategy'
-require_relative 'learn_test/strategies/jasmine'
 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/green_onion'
 require_relative 'learn_test/strategies/pytest'
 
 module LearnTest
@@ -37,8 +38,6 @@ module LearnTest
     autoload :Ant,            'learn_test/dependencies/ant'
     autoload :Imagemagick,    'learn_test/dependencies/imagemagick'
     autoload :SeleniumServer, 'learn_test/dependencies/selenium_server'
-    autoload :Firefox,        'learn_test/dependencies/firefox'
-    autoload :GreenOnion,     'learn_test/dependencies/green_onion'
     autoload :Pytest,         'learn_test/dependencies/pytest'
   end
 end

data/lib/learn_test/client.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'faraday'
 require 'json'
 

data/lib/learn_test/dependencies/ant.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class Ant < LearnTest::Dependency

data/lib/learn_test/dependencies/csharp.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class CSharp < LearnTest::Dependency
@@ -10,7 +12,7 @@ module LearnTest
       end
 
       def install
-        die("Please install the .NET core from https://www.microsoft.com/net/core")
+        die('Please install the .NET core from https://www.microsoft.com/net/core')
       end
 
       def die(message)

data/lib/learn_test/dependencies/imagemagick.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class Imagemagick < LearnTest::Dependency

data/lib/learn_test/dependencies/java.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class Java < LearnTest::Dependency

data/lib/learn_test/dependencies/karma.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class Karma < LearnTest::Dependency

data/lib/learn_test/dependencies/nodejs.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class NodeJS < LearnTest::Dependency

data/lib/learn_test/dependencies/phantomjs.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class PhantomJS < LearnTest::Dependency
@@ -7,8 +9,8 @@ module LearnTest
           return !phantom_installed_on_mac?
         end
 
-        if !phantom_installed_on_linux?
-          die("You must have PhantomJS installed: http://phantomjs.org/download.html")
+        unless phantom_installed_on_linux?
+          die('You must have PhantomJS installed: http://phantomjs.org/download.html')
         end
 
         super
@@ -31,12 +33,12 @@ module LearnTest
       def phantom_installed_on_linux?
         phantom_installed?
       end
+
       def self.check_installation
         new.check_installation
       end
 
-      def check_installation
-      end
+      def check_installation; end
 
       def phantom_installed_by_brew?
         !`brew ls --versions phantomjs`.empty?

data/lib/learn_test/dependencies/protractor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class Protractor < LearnTest::Dependency

data/lib/learn_test/dependencies/selenium_server.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Dependencies
     class SeleniumServer < LearnTest::Dependency

data/lib/learn_test/dependency.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   class Dependency
     attr_reader :options
@@ -14,11 +16,9 @@ module LearnTest
       false
     end
 
-    def install
-    end
+    def install; end
 
-    def die(message)
-    end
+    def die(message); end
 
     def mac?
       !!RUBY_PLATFORM.match(/darwin/)

data/lib/learn_test/file_finder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   class FileFinder
     def self.location_to_dir(dir_name)
@@ -9,4 +11,3 @@ module LearnTest
     end
   end
 end
-

data/lib/learn_test/git_wip.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'git'
+
+module LearnTest
+  module GitWip
+    class << self
+      def run!(log: false)
+        git = Git.open('./', log: log)
+        working_branch = git.current_branch
+
+        `learn-test-wip save "Automatic test submission" -u &> /dev/null`
+
+        return false unless $?.success? # rubocop:disable Style/SpecialGlobalVars
+
+        git.push('origin', "wip/#{working_branch}:refs/heads/wip")
+        git.config['remote.origin.url'].gsub('.git', '/tree/wip')
+      rescue StandardError
+        false
+      end
+    end
+  end
+end

data/lib/learn_test/github_interactor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'open-uri'
 
 module LearnTest
@@ -20,4 +22,3 @@ module LearnTest
     end
   end
 end
-

data/lib/learn_test/js_strategy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module JsStrategy
     def js_package
@@ -13,7 +15,7 @@ module LearnTest
     end
 
     def missing_dependencies?
-      return true if !File.exist?("node_modules")
+      return true unless File.exist?('node_modules')
       [:dependencies, :devDependencies, :peerDependencies].any? do |dep_group|
         modules = js_package[dep_group] || {}
         modules_missing?(modules.keys)

data/lib/learn_test/learn_oauth_token_parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   class LearnOauthTokenParser
     def self.get_learn_oauth_token

data/lib/learn_test/netrc_interactor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'netrc'
 
 module LearnTest
@@ -16,4 +18,3 @@ module LearnTest
     end
   end
 end
-

data/lib/learn_test/repo_parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'git'
 
 module LearnTest

data/lib/learn_test/reporter.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 require 'fileutils'
 require 'json'
 
 require_relative 'client'
+require_relative 'git_wip'
 
 module LearnTest
   class Reporter
-
     attr_accessor :output_path, :debug
 
     def self.report(strategy, options = {})
@@ -37,21 +39,26 @@ module LearnTest
       if previous_reports.empty?
         FileUtils.rm_f(output_path)
       else
-        File.open(output_path, "w") do |file|
+        File.open(output_path, 'w') do |file|
           file.write("#{JSON.dump(previous_reports)}\n")
         end
       end
     end
 
-    def report(out: STDOUT)
+    def report
       results = strategy.results
       endpoint = strategy.service_endpoint
       augment_results!(results)
 
-      unless client.post_results(endpoint, 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
-          out.puts 'There was a problem connecting to Learn. Not pushing test results.'.red
+          puts 'There was a problem connecting to Learn. Not pushing test results.'.red
         end
+
         save_failed_attempt(endpoint, results)
       end
     end
@@ -61,7 +68,7 @@ module LearnTest
     attr_reader :strategy, :client
 
     def save_failed_attempt(endpoint, results)
-      File.open(output_path, File::RDWR|File::CREAT, 0644) do |f|
+      File.open(output_path, File::RDWR | File::CREAT, 0644) do |f|
         if f.flock(File::LOCK_EX)
           begin
             reports = JSON.load(f)
@@ -84,7 +91,7 @@ module LearnTest
       if File.exist?("#{FileUtils.pwd}/.learn")
         dot_learn = YAML.load(File.read("#{FileUtils.pwd}/.learn"))
 
-        if !dot_learn['github'].nil?
+        unless dot_learn['github'].nil?
           results[:github] = dot_learn['github']
         end
       end

data/lib/learn_test/runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 
 module LearnTest
@@ -7,7 +9,7 @@ module LearnTest
     def initialize(repo, options = {})
       @repo = repo
       @options = options
-      die if !strategy
+      die unless strategy
     end
 
     def run
@@ -32,7 +34,7 @@ module LearnTest
     end
 
     def strategy
-      @strategy ||= strategies.map{ |s| s.new(self) }.detect(&:detect)
+      @strategy ||= strategies.map { |s| s.new(self) }.detect(&:detect)
     end
 
     private
@@ -50,14 +52,12 @@ module LearnTest
     def strategies
       [
         LearnTest::Strategies::CSharpNunit,
-        LearnTest::Strategies::Jasmine,
-        LearnTest::Strategies::GreenOnion,
         LearnTest::Strategies::Rspec,
         LearnTest::Strategies::Karma,
         LearnTest::Strategies::Protractor,
         LearnTest::Strategies::JavaJunit,
         LearnTest::Strategies::Mocha,
-        LearnTest::Strategies::Pytest,
+        LearnTest::Strategies::Pytest
       ]
     end
 

data/lib/learn_test/strategies/csharp_nunit.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Strategies
     class CSharpNunit < LearnTest::Strategy
@@ -6,7 +8,7 @@ module LearnTest
       end
 
       def detect
-        runner.files.any? {|f| f.match(/project.json/)}
+        runner.files.any? { |f| f.match(/project.json/) }
       end
 
       def check_dependencies
@@ -17,7 +19,7 @@ module LearnTest
         require 'crack'
         require 'json'
 
-        system("dotnet test")
+        system('dotnet test')
         output
         cleanup
       end
@@ -46,7 +48,7 @@ module LearnTest
       end
 
       def cleanup
-        FileUtils.rm("TestResult.xml") if File.exist?("TestResult.xml")
+        FileUtils.rm('TestResult.xml') if File.exist?('TestResult.xml')
       end
     end
   end

data/lib/learn_test/strategies/java_junit.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module LearnTest
   module Strategies
     class JavaJunit < LearnTest::Strategy
@@ -49,9 +51,8 @@ module LearnTest
       private
 
       def clean_old_results
-        if Dir.entries(lab_dir).any? { |f| f == 'junit' }
-          system("rm -rf #{lab_dir}/junit")
-        end
+        return unless Dir.entries(lab_dir).any? { |f| f == 'junit' }
+        system("rm -rf #{lab_dir}/junit")
       end
 
       def run_ant
@@ -72,7 +73,7 @@ module LearnTest
         if File.exists?(test_path)
           test_xml_files.each do |f|
             parsed = JSON.parse(Crack::XML.parse(File.read(f)).to_json)['testsuite']
-            next if !parsed
+            next unless parsed
 
             if parsed['testcase']
               parsed['testcase'].each do |test_case|
@@ -97,10 +98,10 @@ module LearnTest
           results[:passing_count] = 0
         end
 
-        if runner.keep_results?
-          output_file = '.results.json'
-          write_json_output(output_file: output_file)
-        end
+        return unless runner.keep_results?
+
+        output_file = '.results.json'
+        write_json_output(output_file: output_file)
       end
 
       def write_json_output(output_file:)