learn-test 2.7.0 → 3.2.0

data/spec/learn_test/reporter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 require_relative '../../lib/learn_test/reporter'
@@ -6,14 +8,15 @@ require 'tempfile'
 require 'json'
 
 describe LearnTest::Reporter do
-  let(:strategy) {
+  let!(:client) { instance_spy(LearnTest::Client) }
+  let!(:git_tree) { 'https://github.com/learn-co/learn-test/tree/wip' }
+  let!(:strategy) do
     instance_spy(
       LearnTest::Strategy,
-      service_endpoint: "test_endpoint",
+      service_endpoint: 'test_endpoint',
       results: { test: 'result' }
     )
-  }
-  let(:client) { instance_spy(LearnTest::Client) }
+  end
 
   def with_file(name, content = nil, &block)
     file = Tempfile.new(name)
@@ -30,36 +33,62 @@ describe LearnTest::Reporter do
   describe '#report' do
     let(:reporter) { described_class.new(strategy, client: client) }
 
-    context "with debug flag and failed attempt posting results" do
-      it "outputs an error message" do
+    context 'with debug flag and failed attempt posting results' do
+      it 'outputs an error message' do
         allow(client).to receive(:post_results).and_return(false)
-        reporter.debug = true
-        io = StringIO.new
 
-        reporter.report(out: io)
+        expect do
+          reporter.debug = true
+          reporter.report
+        end.to output(/Learn/i).to_stdout
+      end
+    end
+
+    context 'sync with Github' do
+      it 'should not run LearnTest::GitWip, if :post_results fails' do
+        expect(client).to receive(:post_results).and_return(false)
+        expect(LearnTest::GitWip).to_not receive(:run!)
+
+        reporter.report
+      end
+
+      it 'should run LearnTest::GitWip.run!' do
+        expect(client).to receive(:post_results).and_return(true)
+        expect(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+
+        reporter.report
+      end
+
+      it 'should not output an error message without debug' do
+        expect(client).to receive(:post_results).and_return(true)
+        expect(LearnTest::GitWip).to receive(:run!).and_return(false)
+
+        expect { reporter.report }.to_not output.to_stdout
+      end
+
+      it 'should output an error message with debug' do
+        expect(client).to receive(:post_results).and_return(true)
+        expect(LearnTest::GitWip).to receive(:run!).and_return(false)
 
-        expect(io.string).to_not be_empty
+        expect do
+          reporter.debug = true
+          reporter.report
+        end.to output(/Github/i).to_stdout
       end
     end
 
     it 'posts results to the service endpoint' do
       reporter.report
 
+      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+
       expect(client).to have_received(:post_results)
         .with(strategy.service_endpoint, strategy.results)
     end
 
     it 'does not output an error message without debug' do
-      allow(client).to receive(:post_results).and_return(false)
-      io = StringIO.new
-
-      with_file('test_debug') do |tmp_file|
-        reporter.output_path = tmp_file.path
-
-        reporter.report(out: io)
-
-        expect(io.string).to be_empty
-      end
+      expect(client).to receive(:post_results).and_return(false)
+      expect { reporter.report }.to_not output.to_stdout
     end
 
     it 'saves the failed attempt when the post results call fails' do
@@ -70,9 +99,9 @@ describe LearnTest::Reporter do
         reporter.report
 
         report = JSON.dump(strategy.results)
-        expect(File.exists?(tmp_file.path)).to be(true)
+        expect(File.exist?(tmp_file.path)).to be(true)
         expect(reporter.failed_reports).to eq({
-          strategy.service_endpoint => [JSON.load(report)]
+          strategy.service_endpoint => [JSON.parse(report)]
         })
       end
     end
@@ -87,8 +116,9 @@ describe LearnTest::Reporter do
         reporter.report
 
         report = JSON.dump(strategy.results)
-        json_report = JSON.load(report)
-        expect(File.exists?(tmp_file.path)).to be(true)
+        json_report = JSON.parse(report)
+
+        expect(File.exist?(tmp_file.path)).to be(true)
         expect(reporter.failed_reports).to eq({
           strategy.service_endpoint => [json_report, json_report]
         })
@@ -101,23 +131,30 @@ describe LearnTest::Reporter do
 
     it 'deletes the output file when all reports are sent successfully' do
       allow(client).to receive(:post_results).and_return(true)
-      reports = {hello: ["world"]}
+      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+
+      reports = { hello: ['world'] }
+
       with_file('retry_failed_reports', reports) do |file|
         reporter.output_path = file.path
         reporter.retry_failed_reports
 
-        expect(File.exists?(file.path)).to be(false)
+        expect(File.exist?(file.path)).to be(false)
       end
     end
 
     it 'writes the remaining reports from the output file' do
       allow(client).to receive(:post_results).and_return(true, false)
-      reports = {success: ["world"], failure: ["hello"]}
+      allow(LearnTest::GitWip).to receive(:run!).and_return(:git_tree)
+
+      reports = { success: ['world'], failure: ['hello'] }
+
       with_file('retry_failed_reports', reports) do |file|
         reporter.output_path = file.path
         reporter.retry_failed_reports
-        expect(File.exists?(file.path)).to be(true)
-        expect(reporter.failed_reports).to eq({"failure" => ["hello"]})
+
+        expect(File.exist?(file.path)).to be(true)
+        expect(reporter.failed_reports).to eq({ 'failure' => ['hello'] })
       end
     end
   end
@@ -125,6 +162,7 @@ describe LearnTest::Reporter do
   describe '#failed_reports' do
     let(:path) { 'failed_reports' }
     let(:reporter) { described_class.new(strategy, output_path: path) }
+
     subject { reporter.failed_reports }
 
     context 'with no file at location' do
@@ -137,9 +175,9 @@ describe LearnTest::Reporter do
 
     context 'with returns the JSON contents of the file' do
       it {
-        with_file(path, {hello: "world"}) do |file|
+        with_file(path, { hello: 'world' }) do |file|
           reporter.output_path = file.path
-          is_expected.to eq({"hello" => "world"})
+          is_expected.to eq({ 'hello' => 'world' })
         end
       }
     end

data/spec/learn_test/username_parser_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 describe LearnTest::UsernameParser do
   subject { LearnTest::UsernameParser }
 

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

@@ -1,60 +1,61 @@
+# frozen_string_literal: true
+
 describe LearnTest::Strategies::Mocha do
   describe '#missing_dependencies?' do
     let(:package) do
       {
         dependencies: {
-          dep1: "",
-          dep2: ""
+          dep1: '',
+          dep2: ''
         },
         devDependencies: {
-          devDep1: "",
-          devDep2: ""
+          devDep1: '',
+          devDep2: ''
         },
         peerDependencies: {
-          peerDep1: "",
-          peerDep2: ""
+          peerDep1: '',
+          peerDep2: ''
         }
       }
     end
 
-    let(:runner) { double("Runner", options: {}) }
+    let(:runner) { double('Runner', options: {}) }
     let(:strategy) { LearnTest::Strategies::Mocha.new(runner) }
 
-    context "node_modules/ does not exist" do
-      it "returns true" do
-        allow(File).to receive(:exist?).with("node_modules").and_return(false)
+    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(:exist?).and_return(true)
         allow(strategy).to receive(:js_package).and_return(package)
       end
 
-      it "returns true if missing a dependency" do
-        allow(File).to receive(:exist?).with("node_modules/dep2").and_return(false)
+      it 'returns true if missing a dependency' do
+        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
-        allow(File).to receive(:exist?).with("node_modules/devDep2").and_return(false)
+      it 'returns true if missing a devDependency' do
+        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
-        allow(File).to receive(:exist?).with("node_modules/peerDep2").and_return(false)
+      it 'returns true if missing a peerDependency' do
+        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
+      it 'returns false if missing no dependencies' do
         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
+      it 'returns false if there are no dependencies' do
         allow(strategy).to receive(:js_package).and_return({})
         expect(strategy.missing_dependencies?).to eq(false)
       end

data/spec/repo_parser_spec.rb

@@ -1,25 +1,27 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 RSpec.describe LearnTest::RepoParser do
-  context "URLs from github" do
-    let!(:remote) { OpenStruct.new(:url => nil)}
-    let!(:repo) { OpenStruct.new(:remote => remote)}
+  context 'URLs from github' do
+    let!(:remote) { OpenStruct.new(url: nil) }
+    let!(:repo) { OpenStruct.new(remote: remote) }
 
-    it "supports ssh addresses" do
+    it 'supports ssh addresses' do
       expect(Git).to receive(:open).and_return(repo)
-      remote.url = "git@github.com:flatiron-labs/learn-gem.git"
-      expect(described_class.get_repo).to eq("learn-gem")
+      remote.url = 'git@github.com:flatiron-labs/learn-gem.git'
+      expect(described_class.get_repo).to eq('learn-gem')
     end
 
-    it "supports http addresses" do
+    it 'supports http addresses' do
       expect(Git).to receive(:open).and_return(repo)
-      remote.url = "https://github.com/flatiron-labs/learn-gem.git"
-      expect(described_class.get_repo).to eq("learn-gem")
+      remote.url = 'https://github.com/flatiron-labs/learn-gem.git'
+      expect(described_class.get_repo).to eq('learn-gem')
     end
 
-    it "supports http addresses without .git extension" do
+    it 'supports http addresses without .git extension' do
       expect(Git).to receive(:open).and_return(repo)
-      remote.url = "https://github.com/flatiron-labs/learn-gem"
-      expect(described_class.get_repo).to eq("learn-gem")
+      remote.url = 'https://github.com/flatiron-labs/learn-gem'
+      expect(described_class.get_repo).to eq('learn-gem')
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../lib/learn_test'
 RSpec.configure do |config|
   config.expect_with :rspec do |expectations|

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: learn-test
 version: !ruby/object:Gem::Version
-  version: 2.7.0
+  version: 3.2.0
 platform: ruby
 authors:
 - Flatiron School
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-29 00:00:00.000000000 Z
+date: 2020-08-04 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.1.4
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.1.4
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.7'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -86,42 +72,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
@@ -136,40 +122,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
@@ -184,65 +136,12 @@ 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
 executables:
 - learn-test
+- learn-test-wip
 extensions: []
 extra_rdoc_files: []
 files:
@@ -250,18 +149,16 @@ files:
 - ".gitignore"
 - ".rspec"
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
 - bin/learn-test
+- bin/learn-test-wip
 - learn-test.gemspec
 - lib/learn_test.rb
 - 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
@@ -271,6 +168,7 @@ files:
 - lib/learn_test/dependencies/selenium_server.rb
 - lib/learn_test/dependency.rb
 - lib/learn_test/file_finder.rb
+- lib/learn_test/git_wip.rb
 - lib/learn_test/github_interactor.rb
 - lib/learn_test/js_strategy.rb
 - lib/learn_test/learn_oauth_token_parser.rb
@@ -279,27 +177,6 @@ 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
@@ -317,16 +194,13 @@ 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/.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
 - 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/git_spec.rb
 - spec/learn_test/reporter_spec.rb
 - spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb
@@ -345,7 +219,7 @@ 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:
   - - ">="
@@ -355,19 +229,16 @@ requirements: []
 rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
-summary: Runs RSpec, Jasmine, Karma, Mocha, and Python Pytest 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/.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
 - 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/git_spec.rb
 - spec/learn_test/reporter_spec.rb
 - spec/learn_test/username_parser_spec.rb
 - spec/lib/learn_test/strategies/mocha_spec.rb