learn-test 2.5.4 → 2.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: fe6bf93302286117f667a4a13e13b30091dd6276
-  data.tar.gz: ec99bebcb1a15014eac0dfad7c78138231af0c95
+SHA256:
+  metadata.gz: a98ee1823d7eaa0784005b131da8261b48f009c5479536157d2061ef1e9ec29f
+  data.tar.gz: 21dfb4ce2145c6c662f0ef0c0c686ad80a325920cd506c8629e730dcbdbd6430
 SHA512:
-  metadata.gz: 2ea80e1ab517dc67cd11e1fe420e6179b26ea9638847c94d53dd651227fdf18d84755bbe4cad697ab02b410a3caf27d82d1627ae528ca0c47b704642e7f0b246
-  data.tar.gz: f86b526d6d7959913c02bb1ef0629b68f4dfe01fb23069bf4c13c44f277a6b5e6e82ba2465c0bf01e844ef8a8dca8c6fdb64d95bfafde285b423de9e6874a72a
+  metadata.gz: f11dbf50918100c382e61b8457ca1edda76685766045afbc546bac3d08eada57c08b3e6a5272793ff0c4a5b6ac9f1dc2cf8b03558a0a7197c3271fe17b6c0d71
+  data.tar.gz: 4341c6a7b0fd636b3302ec4dd5c49f0b9a50ee3b261542a2304167596dd3d7c2a301f1289e747a3d2e5e912eb08edea7ac99ef34cf6dad80a5a5c6a582074566

data/.circleci/config.yml

@@ -0,0 +1,59 @@
+version: 2.1
+
+jobs:
+  build:
+    docker:
+       - image: circleci/ruby:2.5.8
+
+    working_directory: ~/repo
+
+    steps:
+      - checkout
+
+      - restore_cache:
+          keys:
+            - gems-v3-{{ checksum "Gemfile.lock" }}
+            - gems-v3-
+
+      - run:
+          name: Force Bundler Version
+          command: |
+            sudo gem update --system
+            echo 'export BUNDLER_VERSION=$(cat Gemfile.lock | tail -1 | tr -d " ")' >> $BASH_ENV
+            source $BASH_ENV
+
+      - run:
+          name: Install Dependencies
+          command: |
+            bundle config --local set vendor/bundle
+            bundle install
+
+      - save_cache:
+          key: gems-v3-{{ checksum "Gemfile.lock" }}
+          paths:
+            - vendor/bundle
+
+      - run:
+          name: Create ~/.netrc
+          command: | 
+            cp spec/fixtures/.netrc ~/
+            chmod 0600 ~/.netrc
+
+      - run:
+          name: Run tests
+          command: |
+            mkdir /tmp/test-results
+            TEST_FILES="$(circleci tests glob "spec/**/*_spec.rb" | circleci tests split --split-by=timings)"
+            
+            bundle exec rspec --format progress \
+                            --format RspecJunitFormatter \
+                            --out /tmp/test-results/rspec.xml \
+                            --format progress \
+                            $TEST_FILES
+
+      - store_test_results:
+          path: /tmp/test-results
+
+      - store_artifacts:
+          path: /tmp/test-results
+          destination: test-results

data/.gitignore

@@ -1,6 +1,5 @@
 /.bundle/
 /.yardoc
-/Gemfile.lock
 /_yardoc/
 /coverage/
 /doc/

data/Gemfile

@@ -1,4 +1,7 @@
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in learn.gemspec
 gemspec
+
+group :test do
+  gem 'rspec_junit_formatter', '~> 0.4.1'
+end

data/Gemfile.lock

@@ -0,0 +1,83 @@
+PATH
+  remote: .
+  specs:
+    learn-test (2.6.1)
+      colorize (~> 0.8.1)
+      crack (~> 0.4.3)
+      faraday (~> 0.9)
+      git (~> 1.2)
+      jasmine (~> 2.6.0, >= 2.6.0)
+      jasmine-core (< 2.99.1)
+      netrc (~> 0.11.0)
+      oj (~> 2.9)
+      rainbow (= 1.99.2)
+      rspec (~> 3.0)
+      selenium-webdriver (~> 2.52.0, >= 2.52.0)
+      webrick (~> 1.3.1, >= 1.3.1)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    childprocess (0.9.0)
+      ffi (~> 1.0, >= 1.0.11)
+    colorize (0.8.1)
+    crack (0.4.3)
+      safe_yaml (~> 1.0.0)
+    diff-lcs (1.4.4)
+    faraday (0.17.3)
+      multipart-post (>= 1.2, < 3)
+    ffi (1.13.1)
+    git (1.7.0)
+      rchardet (~> 1.8)
+    jasmine (2.6.0)
+      jasmine-core (>= 2.6.0, < 3.0.0)
+      phantomjs
+      rack (>= 1.2.1)
+      rake
+    jasmine-core (2.99.0)
+    multi_json (1.15.0)
+    multipart-post (2.1.1)
+    netrc (0.11.0)
+    oj (2.18.5)
+    phantomjs (2.1.1.0)
+    rack (2.2.3)
+    rainbow (1.99.2)
+    rake (13.0.1)
+    rchardet (1.8.0)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.2)
+      rspec-support (~> 3.9.3)
+    rspec-expectations (3.9.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.3)
+    rspec_junit_formatter (0.4.1)
+      rspec-core (>= 2, < 4, != 2.12.0)
+    rubyzip (1.3.0)
+    safe_yaml (1.0.5)
+    selenium-webdriver (2.52.0)
+      childprocess (~> 0.5)
+      multi_json (~> 1.0)
+      rubyzip (~> 1.0)
+      websocket (~> 1.0)
+    webrick (1.3.1)
+    websocket (1.2.8)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 2.1.4)
+  learn-test!
+  rake (~> 13.0.1)
+  rspec (~> 3.8)
+  rspec_junit_formatter (~> 0.4.1)
+
+BUNDLED WITH
+   2.1.4

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2014-2015 Flatiron School
+Copyright (c) 2014-2018 Flatiron School
 
 MIT License
 

data/README.md

@@ -1,6 +1,6 @@
 # learn-test
 
-Runs RSpec, Jasmine, and Python Unit Test test suites and uploads the results to Learn.
+Runs RSpec, Jasmine, and Python Pytest test suites and uploads the results to Learn.
 
 ## Installation
 
@@ -22,7 +22,7 @@ And then execute:
 
 ## Usage
 
-From within a directory with an RSpec, Jasmine, Karma, Mocha, or Python Unit Test test suite, run:
+From within a directory with an RSpec, Jasmine, Karma, Mocha, or Python Pytest test suite, run:
 
 ```
 $ learn-test
@@ -35,4 +35,3 @@ $ learn-test
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create a new Pull Request
-

data/bin/learn-test

@@ -8,6 +8,10 @@ options = {}
 OptionParser.new do |opts|
   opts.banner = "Usage: learn [options]"
 
+  opts.on("--sync", "Report test results synchronously") do |s|
+    options[:sync] = s
+  end
+
   opts.on("-d", "--debug", "Output errors during results synchronization") do |d|
     options[:debug] = d
   end
@@ -53,6 +57,10 @@ OptionParser.new do |opts|
     options[:fail_fast] = f
   end
 
+  opts.on("-e", "--example STRING", "Run examples whose full nested names include STRING (may be used more than once)") do |s|
+    (options[:example] ||= []) << s
+  end
+
   if ARGV.any? { |arg| arg == "init" }
     options[:init] = true
   end

data/learn-test.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   spec.version       = LearnTest::VERSION
   spec.authors       = ["Flatiron School"]
   spec.email         = ["learn@flatironschool.com"]
-  spec.summary       = %q{Runs RSpec, Jasmine, Karma, Mocha, and Python Unit Test builds and pushes JSON output to Learn.}
+  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"
 
@@ -18,8 +18,9 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib", "bin"]
   spec.required_ruby_version = '>= 2.3.0'
 
-  spec.add_development_dependency "bundler", "~> 1.7"
-  spec.add_development_dependency "rake", "~> 10.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_runtime_dependency "rspec", "~> 3.0"
   spec.add_runtime_dependency "netrc", "~> 0.11.0"
   spec.add_runtime_dependency "git", "~> 1.2"

data/lib/learn_test.rb

@@ -17,7 +17,6 @@ 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/python_unittest'
 require_relative 'learn_test/strategies/rspec'
 require_relative 'learn_test/strategies/karma'
 require_relative 'learn_test/strategies/protractor'
@@ -25,6 +24,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/green_onion'
+require_relative 'learn_test/strategies/pytest'
 
 module LearnTest
   module Dependencies
@@ -39,5 +39,6 @@ module LearnTest
     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

@@ -3,7 +3,7 @@ require 'json'
 
 module LearnTest
   class Client
-    SERVICE_URL = 'http://ironbroker-v2.flatironschool.com'.freeze
+    SERVICE_URL = ENV.fetch('IRONBROKER_URL', 'http://ironbroker-v2.flatironschool.com').freeze
 
     def initialize(service_url = SERVICE_URL)
       @conn = Faraday.new(url: service_url) do |faraday|

data/lib/learn_test/js_strategy.rb

@@ -8,11 +8,15 @@ module LearnTest
       [:dependencies, :devDependencies].any? { |key| js_package[key] && js_package[key][dep] }
     end
 
+    def modules_missing?(module_names)
+      module_names.any? { |name| !File.exist?("node_modules/#{name}") }
+    end
+
     def missing_dependencies?
       return true if !File.exist?("node_modules")
-
-      [:dependencies, :devDependencies, :peerDependencies].any? do |d|
-        (js_package[d] || {}).any? { |p, v| !File.exist?("node_modules/#{p}") }
+      [:dependencies, :devDependencies, :peerDependencies].any? do |dep_group|
+        modules = js_package[dep_group] || {}
+        modules_missing?(modules.keys)
       end
     end
 

data/lib/learn_test/repo_parser.rb

@@ -6,14 +6,14 @@ module LearnTest
       begin
         repo = Git.open(FileUtils.pwd)
       rescue
-        puts "You don't appear to be in a Learn lesson's directory. Please cd to an appropriate directory and try again."
+        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
       end
 
       if url = repo.remote.url
         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."
+        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
       end
     end
@@ -23,4 +23,3 @@ module LearnTest
     end
   end
 end
-

data/lib/learn_test/runner.rb

@@ -14,7 +14,7 @@ module LearnTest
       strategy.check_dependencies
       strategy.configure
       strategy.run
-      if options[:debug]
+      if options[:debug] || options[:sync]
         report_and_clean
       else
         Process.detach(Process.fork do
@@ -57,7 +57,7 @@ module LearnTest
         LearnTest::Strategies::Protractor,
         LearnTest::Strategies::JavaJunit,
         LearnTest::Strategies::Mocha,
-        LearnTest::Strategies::PythonUnittest,
+        LearnTest::Strategies::Pytest,
       ]
     end
 

data/lib/learn_test/strategies/pytest.rb

@@ -0,0 +1,85 @@
+require 'crack'
+require_relative 'pytest/requirements_checker'
+
+module LearnTest
+  module Strategies
+    class Pytest < LearnTest::Strategy
+
+      def service_endpoint
+        '/e/flatiron_pytest'
+      end
+
+      def detect
+        test_files.count > 0
+      end
+
+      def check_dependencies
+        LearnTest::Pytest::RequirementsChecker.check_installation
+      end
+
+      def run
+        system("python -m pytest #{options[:argv].join(' ')} --junitxml='./.results.xml'")
+      end
+
+      def output
+        @output ||= Crack::XML.parse(File.read('.results.xml'))["testsuite"]
+      end
+
+      def test_files
+        # pytest will run all files of the form test_*.py or *_test.py
+        @test_files ||= Dir.glob("**/test_*.py") + Dir.glob("**/*_test.py")
+      end
+
+      def results
+        failed_count = output["failures"].to_i + output["errors"].to_i
+        skipped_count = output["skips"].to_i
+        passed_count = output["tests"].to_i - failed_count - skipped_count
+        {
+          username: username,
+          github_user_id: user_id,
+          learn_oauth_token: learn_oauth_token,
+          repo_name: runner.repo,
+          build: {
+            test_suite: [{
+              framework: 'pytest',
+              formatted_output: output.to_json,
+              duration: output["time"]
+            }]
+          },
+          examples: output["tests"].to_i,
+          passing_count: passed_count,
+          pending_count: skipped_count,
+          failure_count: failed_count,
+          failure_descriptions: concat_failure_descriptions
+        }
+      end
+
+      def cleanup
+        FileUtils.rm('.results.xml')
+      end
+
+      private
+
+      def calculate_duration
+        output[:results].map do |example|
+          example[:time]
+        end.inject(:+)
+      end
+
+      def concat_failure_descriptions
+        # if there is a single test the `testcase` xml parse turns out a hash
+        #   instead of an array with a single hash. this will make sure single
+        #   tests have the same output structure (Array) as multiple tests
+        output["testcase"] = [output["testcase"]].flatten
+
+        output["testcase"].reduce([]) do |errors, example|
+          if example.has_key?("failure")
+            errors << example.map{|k, v| "#{k}: #{v}"}
+          end
+          errors
+        end
+      end
+
+    end
+  end
+end

data/lib/learn_test/strategies/{python_unittest → pytest}/requirements_checker.rb

@@ -1,35 +1,28 @@
 require 'open3'
 
 module LearnTest
-  module PythonUnittest
+  module Pytest
     class RequirementsChecker
       def self.check_installation
-        new.check_installation
-      end
-
-      def check_installation
         PythonChecker.check
         PipChecker.check
+        PytestChecker.check
       end
     end
 
     class PythonChecker
       def self.check
-        new.check
-      end
-
-      def check
-        if !python_installed? || !correct_python_version?
+        if !self.python_installed? || !self.correct_python_version?
           puts "Please install python 2.7.x or 3.x.x"
           exit
         end
       end
 
-      def python_installed?
+      def self.python_installed?
         !`which python`.empty?
       end
 
-      def correct_python_version?
+      def self.correct_python_version?
         output = Open3.popen2e('python', '--version')
         version = output[1].read.strip
         !!version.match(/ 2.7.*| 3.*/)
@@ -38,20 +31,29 @@ module LearnTest
 
     class PipChecker
       def self.check
-        new.check
-      end
-
-      def check
-        if !pip_installed?
+        if !self.pip_installed?
           puts "Please ensure pip is installed"
           exit
         end
       end
 
-      def pip_installed?
+      def self.pip_installed?
         !`which pip`.empty?
       end
     end
+
+    class PytestChecker
+      def self.check
+        if !self.pytest_installed?
+          puts "Please ensure pytest is installed"
+          exit
+        end
+      end
+
+      def self.pytest_installed?
+        !`which pytest`.empty?
+      end
+    end
+
   end
 end
-

data/lib/learn_test/strategies/rspec.rb

@@ -25,6 +25,10 @@ module LearnTest
           argv << '--fail-fast'
         end
 
+        if example_option_present?
+          argv << options[:example].map{|e| "--example #{e}"}.join(" ")
+        end
+
         # Don't pass the test/local flag from learn binary to rspec runner.
         argv.delete("--test")
         argv.delete("-t")
@@ -83,6 +87,10 @@ module LearnTest
         options[:fail_fast]
       end
 
+      def example_option_present?
+        options[:example]
+      end
+
       def dot_rspec
         @dot_rspec ||= File.readlines('.rspec').map(&:strip) if File.exist?('.rspec')
       end

data/lib/learn_test/username_parser.rb

@@ -7,7 +7,7 @@ module LearnTest
 
       if !LearnTest::LearnOauthTokenParser.get_learn_oauth_token && (!username || user_id == 'none')
         print "Enter your github username: "
-        username = gets.strip
+        username = $stdin.gets.strip
         user_id = LearnTest::GithubInteractor.get_user_id_for(username)
         parser.write(username, user_id)
       end

data/lib/learn_test/version.rb

@@ -1,3 +1,3 @@
 module LearnTest
-  VERSION = '2.5.4'
+  VERSION = '2.7.0'
 end

data/spec/features/rspec_unit_spec.rb

@@ -1,10 +1,35 @@
 describe "Running a RSpec Unit Test" 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.
+
+    LearnTest::UsernameParser.get_username
+  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`
 
-      expect(output).to include('1 example, 0 failures')
+      expect(output).to include('3 examples, 0 failures')
       expect(output).to_not include('1 failures')
     end
   end
+
+  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`
+
+      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`
+
+      expect(output).to include('2 examples, 0 failures')
+      expect(output).to_not include('3 examples')
+      expect(output).to_not include('1 example')
+    end 
+  end
 end

data/spec/fixtures/.netrc

@@ -0,0 +1,3 @@
+machine flatiron-push
+  login tmilewski
+  password 26691

data/spec/fixtures/rspec-unit-spec/spec/dog_spec.rb

@@ -4,4 +4,12 @@ describe Dog do
   it 'runs the spec spec correctly' do
     expect(subject).to be_an_instance_of(Dog)
   end
+
+  it 'has multiple specs' do
+    expect(subject).not_to be_an_instance_of(String)
+  end
+
+  it 'accepts more than one example' do
+    expect(subject).not_to be_an_instance_of(String)
+  end  
 end

data/spec/learn_test/username_parser_spec.rb

@@ -0,0 +1,61 @@
+describe LearnTest::UsernameParser do
+  subject { LearnTest::UsernameParser }
+
+  let(:dbl) { instance_double(LearnTest::NetrcInteractor) }
+
+  let!(:username) { 'learn-co' }
+  let!(:user_id) { '12345' }
+  let!(:token) { 'abc123' }
+
+  context 'user details are stored in ~/.netrc' do
+    context 'username/password' do
+      it 'should return a username' do
+        expect(LearnTest::LearnOauthTokenParser).to receive(:get_learn_oauth_token).and_return(token)
+
+        expect(LearnTest::NetrcInteractor).to receive(:new).and_return(dbl)
+        expect(dbl).to receive(:username).and_return(username)
+        expect(dbl).to receive(:user_id).and_return(user_id)
+
+        expect(dbl).to_not receive(:write)
+
+        expect(subject.get_username).to eq(username)
+      end
+    end
+
+    context 'oauth token' do
+      it 'should return a username' do
+        expect(LearnTest::LearnOauthTokenParser).to receive(:get_learn_oauth_token).and_return(token)
+
+        expect(LearnTest::NetrcInteractor).to receive(:new).and_return(dbl)
+        expect(dbl).to receive(:username).and_return(nil)
+        expect(dbl).to receive(:user_id).and_return(nil)
+
+        expect(dbl).to_not receive(:write)
+
+        expect(subject.get_username).to eq(nil)
+      end
+    end
+  end
+
+  context 'user details are not stored in ~/.netrc' do
+    it 'should ask for and store a username' do
+      @original_stdout = $stdout
+      $stdout = File.open(File::NULL, 'w')
+
+      expect(LearnTest::LearnOauthTokenParser).to receive(:get_learn_oauth_token).and_return(nil)
+
+      expect(LearnTest::NetrcInteractor).to receive(:new).and_return(dbl)
+      expect(dbl).to receive(:username).and_return(nil)
+      expect(dbl).to receive(:user_id).and_return(nil)
+
+      expect($stdin).to receive(:gets).and_return(username)
+      expect(LearnTest::GithubInteractor).to receive(:get_user_id_for).with(username).and_return(user_id)
+      expect(dbl).to receive(:write).with(username, user_id)
+
+      expect(subject.get_username).to eq(username)
+
+      $stdout = @original_stdout
+      @original_stdout = nil
+    end
+  end
+end

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

@@ -16,44 +16,47 @@ describe LearnTest::Strategies::Mocha do
         }
       }
     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)
+    context "node_modules/ does not exist" do
+      it "returns true" do
+        allow(File).to receive(:exist?).with("node_modules").and_return(false)
+        expect(strategy.missing_dependencies?).to eq(true)
+      end
     end
 
-    context "node_modules exists" do
+
+    context "node_modules/ exists" do
       before(:each) do
-        allow(File).to receive(:exists?) { true }
+        allow(File).to receive(:exist?).and_return(true)
+        allow(strategy).to receive(:js_package).and_return(package)
       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)
+        allow(File).to receive(:exist?).with("node_modules/dep2").and_return(false)
+        expect(strategy.missing_dependencies?).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)
+        allow(File).to receive(:exist?).with("node_modules/devDep2").and_return(false)
+        expect(strategy.missing_dependencies?).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)
+        allow(File).to receive(:exist?).with("node_modules/peerDep2").and_return(false)
+        expect(strategy.missing_dependencies?).to eq(true)
       end
 
       it "returns false if missing no dependencies" do
-        expect(strategy.missing_dependencies?(package)).to eq(false)
+        allow(File).to receive(:exist?).and_return(true)
+        expect(strategy.missing_dependencies?).to eq(false)
       end
 
       it "returns false if there are no dependencies" do
-        expect(strategy.missing_dependencies?({})).to eq(false)
+        allow(strategy).to receive(:js_package).and_return({})
+        expect(strategy.missing_dependencies?).to eq(false)
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: learn-test
 version: !ruby/object:Gem::Version
-  version: 2.5.4
+  version: 2.7.0
 platform: ruby
 authors:
 - Flatiron School
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-08 00:00:00.000000000 Z
+date: 2020-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,28 +16,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: 2.1.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: 2.1.4
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 13.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 13.0.1
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.8'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -232,9 +246,11 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -289,9 +305,8 @@ files:
 - lib/learn_test/strategies/karma/karma.conf.js
 - lib/learn_test/strategies/mocha.rb
 - lib/learn_test/strategies/protractor.rb
-- lib/learn_test/strategies/python_unittest.rb
-- lib/learn_test/strategies/python_unittest/nose_installer.rb
-- lib/learn_test/strategies/python_unittest/requirements_checker.rb
+- lib/learn_test/strategies/pytest.rb
+- lib/learn_test/strategies/pytest/requirements_checker.rb
 - lib/learn_test/strategies/rspec.rb
 - lib/learn_test/strategy.rb
 - lib/learn_test/user_id_parser.rb
@@ -304,6 +319,7 @@ files:
 - node_modules/karma-json-reporter/package.json
 - spec/features/jasmine_jquery_fixtures_spec.rb
 - spec/features/rspec_unit_spec.rb
+- spec/fixtures/.netrc
 - spec/fixtures/jasmine-jquery-fixtures/index.html
 - spec/fixtures/jasmine-jquery-fixtures/requires.yml
 - spec/fixtures/jasmine-jquery-fixtures/spec/jasmine-jquery-fixtures-spec.js
@@ -312,6 +328,7 @@ files:
 - spec/fixtures/rspec-unit-spec/spec/dog_spec.rb
 - spec/fixtures/rspec-unit-spec/spec/spec_helper.rb
 - spec/learn_test/reporter_spec.rb
+- spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb
@@ -335,15 +352,15 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
-summary: Runs RSpec, Jasmine, Karma, Mocha, and Python Unit Test builds and pushes
+summary: Runs RSpec, Jasmine, Karma, Mocha, and Python Pytest Test builds and pushes
   JSON output to Learn.
 test_files:
 - spec/features/jasmine_jquery_fixtures_spec.rb
 - spec/features/rspec_unit_spec.rb
+- spec/fixtures/.netrc
 - spec/fixtures/jasmine-jquery-fixtures/index.html
 - spec/fixtures/jasmine-jquery-fixtures/requires.yml
 - spec/fixtures/jasmine-jquery-fixtures/spec/jasmine-jquery-fixtures-spec.js
@@ -352,6 +369,7 @@ test_files:
 - spec/fixtures/rspec-unit-spec/spec/dog_spec.rb
 - spec/fixtures/rspec-unit-spec/spec/spec_helper.rb
 - spec/learn_test/reporter_spec.rb
+- spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb
 - spec/repo_parser_spec.rb
 - spec/spec_helper.rb

data/lib/learn_test/strategies/python_unittest.rb

@@ -1,72 +0,0 @@
-require_relative 'python_unittest/requirements_checker'
-require_relative 'python_unittest/nose_installer'
-
-module LearnTest
-  module Strategies
-    class PythonUnittest < LearnTest::Strategy
-      def service_endpoint
-        '/e/flatiron_unittest'
-      end
-
-      def detect
-        test_files.any? {|f| f.match(/.*\.py$/) }
-      end
-
-      def check_dependencies
-        LearnTest::PythonUnittest::RequirementsChecker.check_installation
-        LearnTest::PythonUnittest::NoseInstaller.install
-      end
-
-      def run
-        system("nosetests #{options[:argv].join(' ')} --verbose --with-json --json-file='./.results.json'")
-      end
-
-      def output
-        @output ||= Oj.load(File.read('.results.json'), symbol_keys: true)
-      end
-
-      def test_files
-        @test_files ||= Dir.glob("**/*_test.py")
-      end
-
-      def results
-        {
-          username: username,
-          github_user_id: user_id,
-          learn_oauth_token: learn_oauth_token,
-          repo_name: runner.repo,
-          build: {
-            test_suite: [{
-              framework: 'unittest',
-              formatted_output: output,
-              duration: calculate_duration
-            }]
-          },
-          examples: output[:stats][:total],
-          passing_count: output[:stats][:passes],
-          pending_count: output[:stats][:skipped],
-          failure_count: output[:stats][:errors],
-          failure_descriptions: concat_failure_descriptions
-        }
-      end
-
-      def cleanup
-        FileUtils.rm('.results.json')
-      end
-
-      private
-
-      def calculate_duration
-        output[:results].map do |example|
-          example[:time]
-        end.inject(:+)
-      end
-
-      def concat_failure_descriptions
-        output[:results].select do |example|
-          example[:type] == 'failure'
-        end.map { |ex| ex[:tb] }.join(';')
-      end
-    end
-  end
-end

data/lib/learn_test/strategies/python_unittest/nose_installer.rb

@@ -1,35 +0,0 @@
-module LearnTest
-  module PythonUnittest
-    class NoseInstaller
-      def self.install
-        new.install
-      end
-
-      def install
-        install_nose
-        install_nose_json
-      end
-
-      def install_nose
-        if !nose_installed?
-          `easy_install nose`
-        end
-      end
-
-      def nose_installed?
-        !`which nosetests`.empty?
-      end
-
-      def install_nose_json
-        if !nose_json_installed?
-          `pip install nose-json`
-        end
-      end
-
-      def nose_json_installed?
-        !`pip show nose-json`.empty?
-      end
-    end
-  end
-end
-