learn-test 2.5.6 → 3.1.0

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.6'
+  VERSION = '3.1.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.6
+  version: 3.1.0
 platform: ruby
 authors:
 - Flatiron School
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-21 00:00:00.000000000 Z
+date: 2020-07-30 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
@@ -72,42 +86,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.9'
+        version: '3.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.9'
+        version: '3.10'
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: crack
   requirement: !ruby/object:Gem::Requirement
@@ -122,40 +136,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.4.3
-- !ruby/object:Gem::Dependency
-  name: jasmine
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.6.0
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.6.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.6.0
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.6.0
-- !ruby/object:Gem::Dependency
-  name: jasmine-core
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: 2.99.1
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: 2.99.1
 - !ruby/object:Gem::Dependency
   name: colorize
   requirement: !ruby/object:Gem::Requirement
@@ -170,60 +150,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.1
-- !ruby/object:Gem::Dependency
-  name: webrick
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.1
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.1
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.1
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.1
-- !ruby/object:Gem::Dependency
-  name: rainbow
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 1.99.2
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 1.99.2
-- !ruby/object:Gem::Dependency
-  name: selenium-webdriver
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.52.0
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.52.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.52.0
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.52.0
 description: 
 email:
 - learn@flatironschool.com
@@ -232,9 +158,11 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -244,8 +172,6 @@ files:
 - lib/learn_test/client.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
 - lib/learn_test/dependencies/java.rb
 - lib/learn_test/dependencies/karma.rb
@@ -263,35 +189,13 @@ files:
 - lib/learn_test/reporter.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
-- lib/learn_test/strategies/jasmine/console.js
-- lib/learn_test/strategies/jasmine/formatters/jasmine2-junit.js
-- lib/learn_test/strategies/jasmine/helpers/ConsoleHelper.js
-- lib/learn_test/strategies/jasmine/helpers/ConsoleHelperNoColor.js
-- lib/learn_test/strategies/jasmine/initializer.rb
-- lib/learn_test/strategies/jasmine/jasmine-html.js
-- lib/learn_test/strategies/jasmine/jasmine-jquery.js
-- lib/learn_test/strategies/jasmine/jasmine.css
-- lib/learn_test/strategies/jasmine/jasmine.js
-- lib/learn_test/strategies/jasmine/jasmine_favicon.png
-- lib/learn_test/strategies/jasmine/jquery-1.8.0.min.js
-- lib/learn_test/strategies/jasmine/jquery-ui-1.8.23.custom.min.js
-- lib/learn_test/strategies/jasmine/runners/SpecRunner.html
-- lib/learn_test/strategies/jasmine/runners/run-jasmine.js
-- lib/learn_test/strategies/jasmine/templates/SpecRunnerTemplate.html.erb
-- lib/learn_test/strategies/jasmine/templates/SpecRunnerTemplateNoColor.html.erb
-- lib/learn_test/strategies/jasmine/templates/requires.yml.example
-- lib/learn_test/strategies/jasmine/vendor/require.js
 - lib/learn_test/strategies/java_junit.rb
 - lib/learn_test/strategies/karma.rb
 - 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
@@ -302,16 +206,14 @@ files:
 - node_modules/karma-json-reporter/README.md
 - node_modules/karma-json-reporter/index.js
 - node_modules/karma-json-reporter/package.json
-- spec/features/jasmine_jquery_fixtures_spec.rb
 - spec/features/rspec_unit_spec.rb
-- 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
+- spec/fixtures/.netrc
 - spec/fixtures/rspec-unit-spec/.rspec
 - 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/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
@@ -328,30 +230,27 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !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
-  JSON output to Learn.
+summary: Runs RSpec, 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/jasmine-jquery-fixtures/index.html
-- spec/fixtures/jasmine-jquery-fixtures/requires.yml
-- spec/fixtures/jasmine-jquery-fixtures/spec/jasmine-jquery-fixtures-spec.js
+- spec/fixtures/.netrc
 - spec/fixtures/rspec-unit-spec/.rspec
 - 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/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