learn-test 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: befbe9633e983c7302c3aa5955778b8fc38970dd
-  data.tar.gz: 5b84c70e6b27fc2c2cd5c2584328a613b1d5564e
+  metadata.gz: e45c494858f664acb5872e93ee1b961b5bf11338
+  data.tar.gz: 0e9baa5e5ae7cbeeddca89f031d8e2bf44d361ca
 SHA512:
-  metadata.gz: b820e01ce5acd8d3cc8371c6937418931a6443a891ff79c18f6fc031d41c3258d9132e9f58e7d669b33f91a6dd0f257806af53c29c4824002a5af060bbfd7b19
-  data.tar.gz: 658d425794585202a49bd4a7dd42282ae4e0b93b3cc80f5380e4671eb1aa551694788133cd7a0931bcff4788c5ad2a167204118ba573aba28e3356a9435d8446
+  metadata.gz: ea657a5625131a18b24a6a5a602926f650b14198428b46630ae2d371cdca8c245d7c81e9cd35bde271caff13adc716ffd78b4a3472bc0f35442b774e6810fe25
+  data.tar.gz: 9af6b3551aa357358c360861f66990e50710df8c7f620700144b41368525faf54df50e1caba592acd325849f44ea6bb59c6687f8410771b7027ea23ff72a3384

data/bin/learn-test

@@ -16,7 +16,7 @@ OptionParser.new do |opts|
     options[:format] = f
   end
 
-  opts.on("-l", "--local", "Don't push results to Flatiron LMS") do |l|
+  opts.on("-l", "--local", "Don't push results to Learn") do |l|
     options[:local] = l
   end
 

data/lib/learn_test.rb

@@ -19,6 +19,7 @@ require_relative 'learn_test/dependencies/phantomjs'
 require_relative 'learn_test/dependencies/karma'
 require_relative 'learn_test/dependencies/protractor'
 require_relative 'learn_test/dependencies/java'
+require_relative 'learn_test/dependencies/csharp'
 require_relative 'learn_test/dependencies/ant'
 require_relative 'learn_test/dependencies/imagemagick'
 require_relative 'learn_test/dependencies/selenium_server'
@@ -32,6 +33,7 @@ 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'
 

data/lib/learn_test/dependencies/csharp.rb

@@ -0,0 +1,22 @@
+module LearnTest
+  module Dependencies
+    class CSharp < LearnTest::Dependency
+      def missing?
+        if win?
+          `where dotnet`.empty?
+        else
+          `which dotnet`.empty?
+        end
+      end
+
+      def install
+        die("Please install the .NET core from https://www.microsoft.com/net/core")
+      end
+
+      def die(message)
+        puts message
+        exit
+      end
+    end
+  end
+end

data/lib/learn_test/repo_parser.rb

@@ -11,7 +11,7 @@ module LearnTest
       end
 
       if url = repo.remote.url
-        url.match(/(?:https:\/\/|git@).*\/(.+?)(?:\.git)?$/)[1]
+        url.match(/(?:https?:\/\/|git@).*\/(.+?)(?:\.git)?$/)[1]
       else
         puts "You don't appear to be in a Learn lesson's directory. Please cd to an appropriate directory and try again."
         die

data/lib/learn_test/runner.rb

@@ -15,7 +15,7 @@ module LearnTest
       strategy.check_dependencies
       strategy.configure
       strategy.run
-      if !help_option_present? && strategy.push_results?
+      if !help_option_present? && strategy.push_results? && !local_test_run?
         push_results(strategy)
       end
       strategy.cleanup unless keep_results?
@@ -53,6 +53,7 @@ module LearnTest
 
     def strategies
       [
+        LearnTest::Strategies::CSharpNunit,
         LearnTest::Strategies::Jasmine,
         LearnTest::Strategies::GreenOnion,
         LearnTest::Strategies::Rspec,
@@ -83,6 +84,10 @@ module LearnTest
       options.include?('-h') || options.include?('--help')
     end
 
+    def local_test_run?
+      options.include?('-h') || options.include?('--local')
+    end
+
     def die
       puts "This directory doesn't appear to have any specs in it."
       exit

data/lib/learn_test/strategies/csharp_nunit.rb

@@ -0,0 +1,53 @@
+require 'crack'
+require 'json'
+
+module LearnTest
+  module Strategies
+    class CSharpNunit < LearnTest::Strategy
+      def service_endpoint
+        '/e/flatiron_csharp_nunit'
+      end
+
+      def detect
+        runner.files.any? {|f| f.match(/project.json/)}
+      end
+
+      def check_dependencies
+        Dependencies::CSharp.new.execute
+      end
+
+      def run
+        system("dotnet test")
+        output
+        cleanup
+      end
+
+      def output
+        @output ||= Crack::XML.parse(File.read('TestResult.xml'))
+      end
+
+      def results
+        {
+          username: username,
+          github_user_id: user_id,
+          repo_name: runner.repo,
+          build: {
+            test_suite: [{
+              framework: 'nunit',
+              formatted_output: output.to_json,
+              duration: output['test_run']['duration']
+            }]
+          },
+          examples: output['test_run']['total'],
+          passing_count: output['test_run']['passed'],
+          pending_count: output['test_run']['skipped'],
+          failure_count: output['test_run']['failed'],
+        }
+      end
+      
+      def cleanup
+        FileUtils.rm("TestResult.xml") if File.exist?("TestResult.xml")
+      end
+    end
+  end
+end

data/lib/learn_test/strategies/mocha.rb

@@ -59,12 +59,20 @@ module LearnTest
         FileUtils.rm('.results.json') if File.exist?('.results.json')
       end
 
+      def missing_dependencies?(package)
+        return true if !File.exists?("node_modules")
+
+        [:dependencies, :devDependencies, :peerDependencies].any? do |d|
+          (package[d] || {}).any? { |p, v| !File.exists?("node_modules/#{p}") }
+        end
+      end
+
       private
 
       def run_mocha
         package = Oj.load(File.read('package.json'), symbol_keys: true)
 
-        npm_install
+        npm_install(package)
 
         command = if (package[:scripts] && package[:scripts][:test] || "").include?(".results.json")
           "npm test"
@@ -78,14 +86,12 @@ module LearnTest
 
       def install_mocha_multi
         if !File.exists?('node_modules/mocha-multi')
-          run_install('npm install mocha-multi')
+          run_install('npm install mocha-multi', npm_install: true)
         end
       end
 
-      def npm_install
-        if !File.exists?('node_modules')
-          run_install('npm install')
-        end
+      def npm_install(package)
+        run_install('npm install', npm_install: true) if missing_dependencies?(package)
       end
     end
   end

data/lib/learn_test/strategy.rb

@@ -57,18 +57,25 @@ module LearnTest
       exit
     end
 
-    def run_install(command)
-      Open3.popen3(command) do |stdin, stdout, stderr, wait_thr|
-        while out = stdout.gets do
-          puts out
-        end
-
-        while err = stderr.gets do
-          puts err
-        end
-
-        if wait_thr.value.exitstatus != 0
-          die("There was an error running #{command}")
+    ##
+    # npm_install option added to fix the proxying of the npm install progress
+    # bar output.
+    def run_install(command, npm_install: false)
+      if npm_install
+        system(command)
+      else
+        Open3.popen3(command) do |stdin, stdout, stderr, wait_thr|
+          while out = stdout.gets do
+            puts out
+          end
+
+          while err = stderr.gets do
+            puts err
+          end
+
+          if wait_thr.value.exitstatus != 0
+            die("There was an error running #{command}")
+          end
         end
       end
     end

data/lib/learn_test/version.rb

@@ -1,3 +1,3 @@
 module LearnTest
-  VERSION = '2.4.0'
+  VERSION = '2.4.1'
 end

data/spec/lib/learn_test/strategies/mocha_spec.rb

@@ -0,0 +1,60 @@
+describe LearnTest::Strategies::Mocha do
+  describe '#missing_dependencies?' do
+    let(:package) do
+      {
+        dependencies: {
+          dep1: "",
+          dep2: ""
+        },
+        devDependencies: {
+          devDep1: "",
+          devDep2: ""
+        },
+        peerDependencies: {
+          peerDep1: "",
+          peerDep2: ""
+        }
+      }
+    end
+    let(:runner) { double("Runner", options: {}) }
+    let(:strategy) { LearnTest::Strategies::Mocha.new(runner) }
+
+    it "returns true if no node_modules directory" do
+      expect(File).to receive(:exists?).with("node_modules") { false }
+
+      expect(strategy.missing_dependencies?(package)).to eq(true)
+    end
+
+    context "node_modules exists" do
+      before(:each) do
+        allow(File).to receive(:exists?) { true }
+      end
+
+      it "returns true if missing a dependency" do
+        expect(File).to receive(:exists?).with("node_modules/dep1") { true }
+        expect(File).to receive(:exists?).with("node_modules/dep2") { false }
+        expect(strategy.missing_dependencies?(package)).to eq(true)
+      end
+
+      it "returns true if missing a devDependency" do
+        expect(File).to receive(:exists?).with("node_modules/devDep1") { true }
+        expect(File).to receive(:exists?).with("node_modules/devDep2") { false }
+        expect(strategy.missing_dependencies?(package)).to eq(true)
+      end
+
+      it "returns true if missing a peerDependency" do
+        expect(File).to receive(:exists?).with("node_modules/peerDep1") { true }
+        expect(File).to receive(:exists?).with("node_modules/peerDep2") { false }
+        expect(strategy.missing_dependencies?(package)).to eq(true)
+      end
+
+      it "returns false if missing no dependencies" do
+        expect(strategy.missing_dependencies?(package)).to eq(false)
+      end
+
+      it "returns false if there are no dependencies" do
+        expect(strategy.missing_dependencies?({})).to eq(false)
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: learn-test
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - Flatiron School
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-14 00:00:00.000000000 Z
+date: 2017-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -210,6 +210,7 @@ files:
 - learn-test.gemspec
 - lib/learn_test.rb
 - lib/learn_test/dependencies/ant.rb
+- lib/learn_test/dependencies/csharp.rb
 - lib/learn_test/dependencies/firefox.rb
 - lib/learn_test/dependencies/green_onion.rb
 - lib/learn_test/dependencies/imagemagick.rb
@@ -226,6 +227,7 @@ files:
 - lib/learn_test/netrc_interactor.rb
 - lib/learn_test/repo_parser.rb
 - lib/learn_test/runner.rb
+- lib/learn_test/strategies/csharp_nunit.rb
 - lib/learn_test/strategies/green_onion.rb
 - lib/learn_test/strategies/jasmine.rb
 - lib/learn_test/strategies/jasmine/boot.js
@@ -274,6 +276,7 @@ files:
 - spec/fixtures/rspec-unit-spec/lib/dog.rb
 - spec/fixtures/rspec-unit-spec/spec/dog_spec.rb
 - spec/fixtures/rspec-unit-spec/spec/spec_helper.rb
+- spec/lib/learn_test/strategies/mocha_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/learn-co/learn-test
@@ -297,7 +300,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.6
 signing_key: 
 specification_version: 4
 summary: Runs RSpec, Jasmine, and Python Unit Test builds and pushes JSON output to
@@ -312,5 +315,6 @@ test_files:
 - spec/fixtures/rspec-unit-spec/lib/dog.rb
 - spec/fixtures/rspec-unit-spec/spec/dog_spec.rb
 - spec/fixtures/rspec-unit-spec/spec/spec_helper.rb
+- spec/lib/learn_test/strategies/mocha_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb