learn-test 3.2.1.pre.6 → 3.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8794c9f5f79754792e07d3d25d4394cb893f1156c2f90ddf49fa791e3295ed5b
-  data.tar.gz: 106bb411827dd6cc527f5f2703f1720a7cb5abd4f950fae6f9429da46beef4c9
+  metadata.gz: 78b850c796efd54f383ea05c625ad8d5f6038d9dcf865b19fc21fd6dcad435fb
+  data.tar.gz: 88336d0c6c787c1680b22453220a1046741f7ffaf06d97ce090b7fd4d02d4f25
 SHA512:
-  metadata.gz: b0499f0e0986bc8e1a2e8425bfdf364c701540b8f7e799055b7929868fecec7a5c4812a0df10deaf310f5efff8d22d7caa5e3b046f96b719757d32c81f2269e8
-  data.tar.gz: 51369aeb558ad730d61618dbcf426c99250d4368c7b3baec7e0d858e39a29cf18d378cabad353bb2ea13164a3adaa60731ca5ec8de1d3ca6b22cfdacb4eca005
+  metadata.gz: ed0abed1a972742bdbd28e05ca20218109f3834267ef560bf884b358e237107f6c294ecbda9248d570fe88c8f4014efb7cdc44bd6779b246acc59d639a35e80c
+  data.tar.gz: c36adeff838bf630cc640df29e38f7a82de8e65c5aeff5f20a1c107b93d277c4af07ceb2703306d4306a5295237aec822a1f3105bfebfd7a7d64998be5a69700

data/bin/learn-test-wip

@@ -30,7 +30,7 @@ Options for save:
                                     'commit.gpgSign = true'
 "
 
-source "$(git --exec-path)/git-sh-setup"
+. "$(git --exec-path)/git-sh-setup"
 
 require_work_tree
 

data/lib/learn_test.rb

@@ -26,6 +26,7 @@ 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'
 
 module LearnTest
   module Dependencies

data/lib/learn_test/git_wip.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'git'
+require 'logger'
 
 module LearnTest
   module GitWip
@@ -9,25 +10,23 @@ module LearnTest
         git = Git.open('./', log: log)
         working_branch = git.current_branch
 
-        Open3.popen3('learn-test-wip save "Automatic test submission" --editor') do |_stdin, stdout, stderr, wait_thr|
-          while out = stdout.gets do
-            puts out
-          end
-  
-          while err = stderr.gets do
-            puts err
-          end
+        commands = [
+          'learn-test-wip save "Automatic test submission" --editor',
+          "git push origin wip/#{working_branch}:refs/heads/wip"
+        ].join(';')
+
+        Open3.popen3(commands) 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.zero?
-            git.push('origin', "wip/#{working_branch}:refs/heads/wip")
             git.config['remote.origin.url'].gsub('.git', '/tree/wip')
           else
-            puts 'There was an error running learn-test-wip'
+            #  puts 'There was an error running learn-test-wip'
             false
           end
         end
       rescue StandardError => e
-        puts e
         false
       end
     end

data/lib/learn_test/runner.rb

@@ -9,7 +9,6 @@ module LearnTest
     def initialize(repo, options = {})
       @repo = repo
       @options = options
-      die unless strategy
     end
 
     def run
@@ -34,7 +33,11 @@ module LearnTest
     end
 
     def strategy
-      @strategy ||= strategies.map { |s| s.new(self) }.detect(&:detect)
+      return @strategy if @strategy
+
+      detected = strategies.map { |s| s.new(self) }.detect(&:detect)
+
+      @strategy = detected || LearnTest::Strategies::None.new(self)
     end
 
     private
@@ -68,10 +71,5 @@ module LearnTest
     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
-    end
   end
 end

data/lib/learn_test/strategies/none.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module LearnTest
+  module Strategies
+    class None < LearnTest::Strategy
+      def service_endpoint
+        '/e/flatiron_none'
+      end
+
+      def run
+        puts <<~MSG
+          This directory doesn't appear to have any specs in it, so there’s no test to run.
+
+          If you are working on Canvas, this assignment has been submitted. You can resubmit by running `learn test` again.
+        MSG
+      end
+
+      def results
+        {
+          username: username,
+          github_user_id: user_id,
+          learn_oauth_token: learn_oauth_token,
+          repo_name: runner.repo,
+          build: {
+            test_suite: [{ framework: 'none' }]
+          },
+          examples: 0,
+          passing_count: 0,
+          pending_count: 0,
+          failure_count: 0,
+          failure_descriptions: ''
+        }
+      end
+    end
+  end
+end

data/lib/learn_test/version.rb

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

data/spec/learn_test/git_spec.rb

@@ -7,28 +7,35 @@ describe LearnTest::GitWip do
   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)
 
-      expect(subject).to receive(:`).with(/learn-test-wip save ".+" -u &> \/dev\/null/ )
-      expect($?).to receive(:success?).and_return(true)
+      expect(wait_thr).to receive(:value).and_return(wait_thr_value)
+      expect(wait_thr_value).to receive(:exitstatus).and_return(0)
 
-      expect(git_base).to receive(:push).with('origin', "wip/#{working_branch}:refs/heads/wip")
-      expect(git_base).to receive_message_chain(:config, :[]).with('remote.origin.url').and_return("#{git_url}.git")
+      expect(Open3).to receive(:popen3).and_yield(nil, nil, nil, wait_thr)
 
+      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")
     end
   end
 
   context 'failure' do
     it 'should return false on process error' do
-      allow(Git::Base).to receive(:open).and_return(git_base)
-      allow(git_base).to receive(:current_branch)
-      allow(subject).to receive(:`)
+      expect(Git::Base).to receive(:open).with('./', { log: false }).and_return(git_base)
+      expect(git_base).to receive(:current_branch).and_return(working_branch)
+
+      expect(wait_thr).to receive(:value).and_return(wait_thr_value)
+      expect(wait_thr_value).to receive(:exitstatus).and_return(1)
+
+      expect(Open3).to receive(:popen3).and_yield(nil, nil, nil, wait_thr)
 
-      expect($?).to receive(:success?).and_return(false)
       expect(subject.run!).to eq(false)
     end
 

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

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+describe LearnTest::Strategies::None do
+  describe '#run' do
+    it 'prints a message' do
+      strategy = LearnTest::Strategies::None.new(double(:runner, options: {}))
+
+      msg = <<~MSG
+        This directory doesn't appear to have any specs in it, so there’s no test to run.
+
+        If you are working on Canvas, this assignment has been submitted. You can resubmit by running `learn test` again.
+      MSG
+
+      expect { strategy.run }.to output(msg).to_stdout
+    end
+  end
+
+  describe '#results' do
+    it 'contains the appropriate attributes' do
+      user_id = rand(1000..9999)
+      username = "test-username-#{user_id}"
+      oauth_token = "test-token-#{user_id}"
+      repo = double(:repo)
+
+      runner = LearnTest::Runner.new(repo, {})
+      strategy = LearnTest::Strategies::None.new(runner)
+
+      expect(LearnTest::UsernameParser).to receive(:get_username)
+        .and_return(username)
+
+      expect(LearnTest::UserIdParser).to receive(:get_user_id)
+        .and_return(user_id)
+
+      expect(LearnTest::LearnOauthTokenParser).to receive(:get_learn_oauth_token)
+        .and_return(oauth_token)
+
+
+      expect(strategy.results).to eq(
+        username: username,
+        github_user_id: user_id,
+        learn_oauth_token: oauth_token,
+        repo_name: repo,
+        build: {
+          test_suite: [{ framework: 'none' }]
+        },
+        examples: 0,
+        passing_count: 0,
+        pending_count: 0,
+        failure_count: 0,
+        failure_descriptions: ''
+      )
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: learn-test
 version: !ruby/object:Gem::Version
-  version: 3.2.1.pre.6
+  version: 3.2.4
 platform: ruby
 authors:
 - Flatiron School
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-25 00:00:00.000000000 Z
+date: 2020-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -136,7 +136,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.1
-description: 
+description:
 email:
 - learn@flatironschool.com
 executables:
@@ -181,6 +181,7 @@ files:
 - lib/learn_test/strategies/karma.rb
 - lib/learn_test/strategies/karma/karma.conf.js
 - lib/learn_test/strategies/mocha.rb
+- lib/learn_test/strategies/none.rb
 - lib/learn_test/strategies/protractor.rb
 - lib/learn_test/strategies/pytest.rb
 - lib/learn_test/strategies/pytest/requirements_checker.rb
@@ -204,13 +205,14 @@ files:
 - spec/learn_test/reporter_spec.rb
 - spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb
+- spec/lib/learn_test/strategies/none_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/learn-co/learn-test
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -222,12 +224,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.0.6
+signing_key:
 specification_version: 4
 summary: Runs RSpec, Karma, Mocha, and Python Pytest Test builds and pushes JSON output
   to Learn.
@@ -242,5 +244,6 @@ test_files:
 - spec/learn_test/reporter_spec.rb
 - spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb
+- spec/lib/learn_test/strategies/none_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb