opal-rspec 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: d85d8f2c3e91a3277ff22eac597554b6c7c9fad3
-  data.tar.gz: d36a6b8ef94ebe0bb7cbfd5fa342e3beafc660e9
+SHA256:
+  metadata.gz: 75ebf6988b2018f0174da74e1232e9b34fc5d7934d459e1bc58a70afac085af7
+  data.tar.gz: 302016b488d81915ae5d6bda8e249ee4d37eaa948a19f2037a6a2acd2cf49fe7
 SHA512:
-  metadata.gz: 18e1fa24aa9c4647d30f65004e685bdbf1f80d3550749de6e9e703eda078975bce3e81f7497f19a38d470eabe77c8d3d4cb9a04276bf1603f31399e2cf2ce106
-  data.tar.gz: c5d331955aaef00517ece7a419f30918f805382c47c5772fd95f98567c90e34495be196901111092dff019b519092ba29ce25ee3b40647398685a188c2a45ac7
+  metadata.gz: 4c665b19023aecb9310c533e418931c77a47a59b85f9b9d46db5494ad5a50556529b6d600307dcf8efd1623f9bae7d7d4a9ec43374c1350c78e1020a59ccc603
+  data.tar.gz: b0213c7196ef7fa86757420e8d4b4c665db2df1badd0de94fbf4d7ed6bfa85741584116082ebaee0e9275e145bcc9dd94146eae2f62163dd4c0f1b5f2a005cb7

data/.travis.yml

@@ -5,62 +5,30 @@ cache:
   bundler: true
   directories:
   - node_modules
-  - phantom20
+
+env:
+  global:
+    - OPAL_VERSION="~> 0.10.0"
 
 matrix:
   fast_finish: true
 
   include:
-    - rvm: 2.2.4
+    - rvm: 2.4.5
       env: RUN=mri_sanity_check
-    - rvm: 2.3.4
+    - rvm: 2.3.8
       env: RUN=mri_sanity_check
-    - rvm: 2.4.1
+    - rvm: 2.5.3
       env: RUN=mri_sanity_check
-    - rvm: 2.4.1
+    - rvm: 2.5.3
       env: RUN=rake_only
-    - rvm: 2.4.1
-      # PHANTOMJS env variable is not used but it makes it easier to differentiate this build in Travis
-      env: PHANTOMJS=2.1 RUN=rake_only
-      before_install:
-        # Attempt to work around Travis issues and Ruby 2.3 - https://github.com/vcr/vcr/issues/582
-        - gem update --system
-        - "export PATH=`pwd`/node_modules/phantomjs-prebuilt/bin:$PATH"
-        - "echo $PATH"
-        - phantomjs -v | grep 2.1 || npm install phantomjs-prebuilt # get rate limits from bitbucket download source, so get from NPM if we don't have it already
-    - rvm: 2.4.1
+    - rvm: 2.5.3
       env: RUNNER=node RUN=rake_only
-    - rvm: 2.4.1
-      env: RUN=rake_only
-      gemfile: gemfiles/opal_master.gemfile
-    - rvm: 2.4.1
-      env: RUN=rake_only
-      gemfile: gemfiles/opal_0.10_stable.gemfile
-
-  allow_failures:
-    - gemfile: gemfiles/opal_master.gemfile
 
 before_install:
-  # Attempt to work around Travis issues and Ruby 2.3 - https://github.com/vcr/vcr/issues/582
-  - gem update --system
-  - "mkdir -p phantom20"
-  - "export PATH=`pwd`/phantom20:$PATH"
+  - "export PATH=`pwd`/node_modules/phantomjs-prebuilt/bin:$PATH"
   - "echo $PATH"
-  - "pushd phantom20"
-  - "ls phantomjs || curl -L -O https://s3.amazonaws.com/travis-phantomjs/phantomjs-2.0.0-ubuntu-12.04.tar.bz2"
-  - "ls phantomjs || tar xjvf phantomjs-2.0.0-ubuntu-12.04.tar.bz2"
-  - "rm -rf phantomjs-2.0.0-ubuntu-12.04.tar.bz2"
-  - "popd"
-
-# before_script:
-#   - "export DISPLAY=:99.0"
-#   - "sh -e /etc/init.d/xvfb start"
-#   - "phantomjs -v"
-#   - "node -v"
-#   - "firefox --version"
-
-notifications:
-  irc: "irc.freenode.org#opal"
+  - phantomjs -v | grep 2.1 || npm install phantomjs-prebuilt # get rate limits from bitbucket download source, so get from NPM if we don't have it already
 
 script:
   - "bundle exec rake $RUN"

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## 0.6.2 - 2018-11-09
+
+- Updated the homepage url
+
+- Only test against Opal v0.10, v0.11 will be supported in opal-rspec v0.7
+
+- Add support for multiple formats in `SPEC_OPTS` and allow both `--format` and `-f`
+
+
 ## 0.6.1 - 2017-05-25
 
 - Fixed compatibility with Rake 12

data/Gemfile

@@ -5,15 +5,7 @@ unless Dir['rspec{,-{core,expectations,mocks,support}}'].any?
   warn 'Run: "git submodule update --init" to get RSpec sources'
 end
 
-case (opal_version = ENV['OPAL_VERSION'])
-when 'master'
-  gem 'opal', github: 'opal/opal', branch: 'master'
-  gem 'opal-sprockets', github: 'opal/opal-sprockets'
-when nil
-  gem 'opal' # let bundler pick a version
-else
-  gem 'opal', opal_version
-end
+gem "opal", git: "https://github.com/opal/opal.git", branch: "0-10-stable"
 
 # These need to come from our local path in order for create_requires.rb to work properly
 gem 'rspec',              path: 'rspec'

data/Rakefile

@@ -76,7 +76,7 @@ task :verify_rspec_specs => [
 
 desc 'Verifies other_spec_dir task ran correctly'
 task :verify_other_specs do
-  test_output = `rake other_specs`
+  test_output = `rake other_specs --trace`
   unless /1 example, 0 failures/.match(test_output)
     raise "Expected 1 passing example, but got output '#{test_output}'"
   end
@@ -85,11 +85,21 @@ end
 
 desc 'Will run a spec suite (rake opal_specs) and check for expected combination of failures and successes'
 task :verify_opal_specs do
-  test_output = `rake opal_specs`
+  require 'tempfile'
+
+  test_output = Tempfile.create('opal_specs') do |f|
+    `bundle exec rake opal_specs --trace > #{f.path}`
+    File.read f.path
+  end
   test_output.force_encoding 'UTF-8'
   raise "Expected test runner to fail due to failed tests, but got return code of #{$?.exitstatus}" if $?.success?
   count_match = /(\d+) examples, (\d+) failures, (\d+) pending/.match(test_output)
-  raise 'Expected a finished count of test failures/success/etc. but did not see it' unless count_match
+
+  unless count_match
+    raise "Expected a finished count of test failures/success/etc. but did not see it\n" +
+    "Expected /(\d+) examples, (\d+) failures, (\d+) pending/ to match:\n\n> #{test_output.gsub("\n", "\n> ")}"
+  end
+
   total, failed, pending = count_match.captures
 
   actual_failures = []

data/lib/opal/rspec/rake_task.rb

@@ -18,47 +18,70 @@ module Opal
         setting == :enabled
       end
 
-      def launch_phantom(timeout_value)
-        command_line = %Q{phantomjs #{RUNNER} "#{URL}"#{timeout_value ? " #{timeout_value}" : ''}}
-        puts "Running #{command_line}"
-        system command_line
-        success = $?.success?
+      def launch_phantom(sprockets_env, main, &config_block)
+        app = Opal::Server.new(sprockets: sprockets_env) { |server|
+          server.main = main
+          server.debug = false
+          config_block.call(server)
+        }
+
+        server = Thread.new do
+          require 'rack'
+          require 'webrick'
+          Thread.current.abort_on_exception = true
+          Rack::Server.start(
+            :app => app,
+            :Port => PORT,
+            :AccessLog => [],
+            :Logger => WEBrick::Log.new("/dev/null"),
+          )
+        end
 
-        exit 1 unless success
+        wait_for_server
+
+        version_output = begin
+          `phantomjs -v`.strip
+        rescue
+          warn 'Could not find phantomjs command on path!'
+          exit 1
+        end
+
+        if version_output.to_f < 2
+          warn "PhantomJS >= 2.0 is required but version #{version_output} is installed!"
+          exit 1
+        end
+
+        begin
+          command_line = %Q{phantomjs #{RUNNER} "#{URL}"#{@timeout ? " #{@timeout}" : ''}}
+          puts "Running #{command_line}"
+          system command_line
+          success = $?.success?
+
+          exit 1 unless success
+        ensure
+          server.kill
+        end
       end
 
       def runner
         ((via_env = ENV['RUNNER']) && via_env.to_sym) || @runner || :phantom
       end
 
-      def get_load_asset_code(server)
-        sprockets = server.sprockets
-        name = server.main
-        asset = sprockets[name]
-        raise "Cannot find asset: #{name}" if asset.nil?
-        Opal::Sprockets.load_asset name, sprockets
-      end
+      def launch_node(sprockets, main, &config_block)
+        Opal.paths.each { |p| sprockets.append_path(p) } # Opal::Server does this
 
-      # TODO: Avoid the Rack server and compile directly
-      def launch_node(server)
-        compiled = Tempfile.new 'opal_rspec.js'
-        begin
-          require 'net/http'
-          uri = URI(URL)
-          Net::HTTP.start uri.hostname, uri.port do |http|
-            resp = http.get File.join('/assets', server.main)
-            compiled.write resp.body
-            load_asset_code = get_load_asset_code server
-            compiled.write load_asset_code
-            compiled.close
-          end
-          command_line = "node #{compiled.path} 2>&1"
-          puts "Running #{command_line}"
-          system command_line
+        config_block.call sprockets
+
+        asset = sprockets[main]
+        raise "Cannot find asset: #{main} in #{sprockets.inspect}" if asset.nil?
+
+        Tempfile.create [main.to_s.gsub(/\W/, '.'), '.opal_rspec.js'] do |file|
+          File.write file.path, asset.to_s + Opal::Sprockets.load_asset(main)
+
+          command = "node #{file.path} 2>&1"
+          puts "~~> Running #{command}"
+          system command
           exit 1 unless $?.success?
-        ensure
-          compiled.close unless compiled.closed?
-          compiled.unlink
         end
       end
 
@@ -87,56 +110,27 @@ module Opal
       def initialize(name = 'opal:rspec', &block)
         desc 'Run opal specs in phantomjs/node'
         task name do
-          require 'rack'
-          require 'webrick'
-
           sprockets_env = Opal::RSpec::SprocketsEnvironment.new
-          app = Opal::Server.new(sprockets: sprockets_env) { |s|
-            s.main = 'opal/rspec/sprockets_runner'
-            s.debug = false
-
-            block.call s, self if block
-            sprockets_env.spec_pattern = self.pattern if self.pattern
-            sprockets_env.spec_exclude_pattern = self.exclude_pattern
-            sprockets_env.spec_files = self.files
-            sprockets_env.default_path = self.default_path if self.default_path
-            raise 'Cannot supply both a pattern and files!' if self.files and self.pattern
+          main = 'opal/rspec/sprockets_runner'
+          current_task = self
+
+          config_block = -> *args {
+            args.insert(1, current_task)
+            block.call *args if block
+
+            sprockets_env.spec_pattern = current_task.pattern if current_task.pattern
+            sprockets_env.spec_exclude_pattern = current_task.exclude_pattern
+            sprockets_env.spec_files = current_task.files
+            sprockets_env.default_path = current_task.default_path if current_task.default_path
+            raise 'Cannot supply both a pattern and files!' if current_task.files and current_task.pattern
             sprockets_env.add_spec_paths_to_sprockets
+            Opal::Config.arity_check_enabled = arity_checking?
           }
 
-          Opal::Config.arity_check_enabled = arity_checking?
-
-          # TODO: Once Opal 0.9 compatibility is established, if we're running node, add in the node stdlib requires in so RSpec can use them, also add NODE_PATH to the runner command above
-
-          server = Thread.new do
-            Thread.current.abort_on_exception = true
-            Rack::Server.start(
-              :app => app,
-              :Port => PORT,
-              :AccessLog => [],
-              :Logger => WEBrick::Log.new("/dev/null"),
-            )
-          end
-
-          wait_for_server
-          is_phantom = runner == :phantom
-          if is_phantom
-            version_output = begin
-              `phantomjs -v`.strip
-            rescue
-              warn 'Could not find phantomjs command on path!'
-              exit 1
-            end
-            if version_output.to_f < 2
-              warn "PhantomJS >= 2.0 is required but version #{version_output} is installed!"
-              exit 1
-            end
-          end
-
-          begin
-            is_phantom ? launch_phantom(timeout) : launch_node(app)
-          ensure
-            server.kill
+          case runner
+          when :node then launch_node(sprockets_env, main, &config_block)
+          when :phantom then launch_phantom(sprockets_env, main, &config_block)
+          else raise "unknown runner type: #{runner.inspect}"
           end
         end
       end

data/lib/opal/rspec/version.rb

@@ -1,5 +1,5 @@
 module Opal
   module RSpec
-    VERSION = '0.6.1'
+    VERSION = '0.6.2'
   end
 end

data/opal-rspec.gemspec

@@ -6,7 +6,7 @@ Gem::Specification.new do |s|
   s.version = Opal::RSpec::VERSION
   s.author = 'Adam Beynon'
   s.email = 'adam.beynon@gmail.com'
-  s.homepage = 'http://opalrb.org'
+  s.homepage = 'https://github.com/opal-rspec'
   s.summary = 'RSpec for Opal'
   s.description = 'Opal compatible rspec library'
 
@@ -17,6 +17,7 @@ Gem::Specification.new do |s|
   s.add_dependency 'opal', ['>= 0.10.0', '< 0.12']
   s.add_development_dependency 'rake'
   s.add_development_dependency 'yard'
+  s.add_development_dependency 'puma'
   s.add_development_dependency 'poltergeist'
   s.add_development_dependency 'capybara'
   s.add_development_dependency 'launchy'

data/opal/opal/rspec/fixes/rspec/core/notifications/summary_notification.rb

@@ -2,11 +2,10 @@ module ::RSpec::Core::Notifications
   # SummaryNotification is a Struct
   SummaryNotification.class_eval do
     def totals_line
+      # string mutations
       summary = Formatters::Helpers.pluralize(example_count, "example")
-      # 2 string mutations
-      summary = summary + ", " + Formatters::Helpers.pluralize(failure_count, "failure")
-      # string mutation
-      summary = summary + ", #{pending_count} pending" if pending_count > 0
+      summary += ", " + Formatters::Helpers.pluralize(failure_count, "failure")
+      summary += ", #{pending_count} pending" if pending_count > 0
       summary
     end
 

data/opal/opal/rspec/spec_opts.rb.erb

@@ -19,10 +19,9 @@
 
 <%          end
         end
-        if (match = /--format (\S+)/.match(spec_opts))
-            formatter = match.captures[0]
+        if (match = spec_opts.scan(/(?:--format (\S+)|-f ?(\S+))/).flatten.compact).any?
 %>
-            config.formatter = "<%=formatter%>"
+            <%= match.inspect %>.each { |f| config.add_formatter f }
 <%      end %>
      end
 <% end %>

data/spec/mri/unit/opal/rspec/rake_task_spec.rb

@@ -6,14 +6,13 @@ require_relative 'temp_dir_helper'
 describe Opal::RSpec::RakeTask do
   include_context :temp_dir
   let(:captured_opal_server) { {} }
+  let(:runner) { :phantom }
+  before { ENV['RUNNER'] = runner.to_s }
 
-  RSpec::Matchers.define :invoke_runner do |expected, timeout_value=nil|
-    match do
-      invoked_runners == [{
-                            type: expected,
-                            timeout_value: timeout_value
-                          }]
-    end
+  RSpec::Matchers.define :invoke_runner do |expected|
+    expected_value = [expected]
+    match { expect(invoked_runners).to eq(expected_value) }
+    failure_message { {expected: expected_value, got: invoked_runners}.inspect }
   end
 
   RSpec::Matchers.define :enable_arity_checking do
@@ -24,7 +23,7 @@ describe Opal::RSpec::RakeTask do
 
   RSpec::Matchers.define :require_opal_specs do |matcher|
     def actual
-      captured_opal_server[:server].sprockets.cached.get_opal_spec_requires
+      captured_opal_server[:sprockets].cached.get_opal_spec_requires
     end
 
     match do
@@ -42,7 +41,7 @@ describe Opal::RSpec::RakeTask do
 
   RSpec::Matchers.define :append_opal_path do |expected_path|
     def actual
-      captured_opal_server[:server].sprockets.paths
+      captured_opal_server[:sprockets].paths
     end
 
     abs_expected = lambda { File.expand_path(expected_path) }
@@ -66,8 +65,8 @@ describe Opal::RSpec::RakeTask do
       task.clear
       task.reenable
     end
-    allow(Rack::Server).to receive(:start) do |config| # don't want to actually run specs
-      captured_opal_server[:server] = config[:app]
+    allow(Rack::Server).to receive(:start) do |config|
+      # don't want to actually run specs
     end
     thread_double = instance_double Thread
     allow(thread_double).to receive :kill
@@ -75,23 +74,26 @@ describe Opal::RSpec::RakeTask do
       block.call
       thread_double
     end
-    allow(task_definition).to receive(:launch_phantom) do |timeout_value|
-      invoked_runners << {
-        type: :phantom,
-        timeout_value: timeout_value
-      }
+
+    dummy_launch = -> (runner_name, sprockets, main, &config_block) {
+      config_block.call
+      captured_opal_server[:sprockets] = sprockets
+      captured_opal_server[:main] = main
+      invoked_runners << runner_name
       nil
+    }
+
+    allow(task_definition).to receive(:launch_phantom) do |*args, &block|
+      dummy_launch.call(:phantom, *args, &block)
     end
-    allow(task_definition).to receive(:launch_node) do
-      invoked_runners << {
-        type: :node,
-        timeout_value: nil
-      }
-      nil
+
+    allow(task_definition).to receive(:launch_node) do |*args, &block|
+      dummy_launch.call(:node, *args, &block)
     end
-    expect(task_definition).to receive(:wait_for_server) if expected_to_run
   end
 
+  after { raise "did not run" if invoked_runners.empty? if expected_to_run }
+
   let(:invoked_runners) { [] }
   let(:task_name) { :foobar }
   let(:expected_to_run) { true }
@@ -112,6 +114,7 @@ describe Opal::RSpec::RakeTask do
     end
 
     around do |example|
+      expect(runner).to eq(:phantom)
       # in case we're running on travis, etc.
       current_env_runner = ENV['RUNNER']
       ENV['RUNNER'] = nil
@@ -195,9 +198,8 @@ describe Opal::RSpec::RakeTask do
     context 'ENV' do
       TEST_RUNNERS.each do |runner|
         context runner do
-          before do
-            ENV['RUNNER'] = runner.to_s
-          end
+          let(:runner) { runner }
+          before { ENV['RUNNER'] = runner.to_s }
 
           include_context :explicit, runner
         end
@@ -310,6 +312,9 @@ describe Opal::RSpec::RakeTask do
     it { is_expected.to have_attributes pattern: nil }
     it { is_expected.to append_opal_path 'spec' }
     it { is_expected.to require_opal_specs eq ['something/dummy_spec'] }
-    it { is_expected.to invoke_runner :phantom, timeout_value=40000 }
+    it { is_expected.to invoke_runner :phantom }
+    it 'has timeout set' do
+      expect(subject.timeout).to eq(40000)
+    end
   end
 end

data/spec/rspec/core/config.rb

@@ -1,4 +1,4 @@
-require 'opal/progress_json_formatter' # verify case uses this
+require 'opal/separator_formatter' # verify case uses this
 
 RSpec.configure do |c|
   #c.full_description = 'uses the default color for the shared example backtrace line'

data/spec/rspec/expectations/config.rb

@@ -1,4 +1,4 @@
-require 'opal/progress_json_formatter' # verify case uses this
+require 'opal/separator_formatter' # verify case uses this
 
 # Only doing this because this spec causes the runner itself to break
 CAUSES_SPECS_TO_CRASH = [

data/spec/rspec/mocks/spec_helper_opal.rb

@@ -3,7 +3,7 @@ require 'rspec/support/spec'
 require 'rspec/support/ruby_features'
 require 'opal-parser'
 
-require 'opal/progress_json_formatter' # verify case uses this
+require 'opal/separator_formatter' # verify case uses this
 # Only doing this because any_instance causes the runner itself to break
 CAUSES_SPECS_TO_CRASH = [
     /.*allow_any_instance.*/,

data/spec/rspec/opal_rspec_spec_loader.rb

@@ -146,7 +146,7 @@ module Opal
           missing << path unless matching
           temp_filename
         end
-        raise "Expected to #{description} in #{fix_these_files} but we didn't find any expressions in #{missing}. Check if RSpec has been upgraded" if missing.any?
+        raise "~~> Expected to #{description} in #{fix_these_files} but we didn't find any expressions in #{missing}. Check if RSpec has been upgraded" if missing.any?
         files_we_left_alone = starting_file_set - fix_these_files
         files_we_left_alone + fixed_temp_files
       end
@@ -162,7 +162,7 @@ module Opal
                         .source
                         .gsub('/', '\/')
           replace = "/#{escaped}/m"
-          puts "Replacing multiline regex with #{replace} in new temp file #{temp_filename}"
+          puts "~~> Replacing multiline regex with #{replace} in new temp file #{temp_filename}"
           replace
         end
       end
@@ -204,46 +204,42 @@ module Opal
           missing << path unless found_blackslash
           temp_filename
         end
-        raise "Expected to fix blackslash continuation in #{fix_these_files} but we didn't find any backslashes in #{missing}. Check if RSpec has been upgraded (maybe those blackslashes are gone??)" if missing.any?
+        raise "~~> Expected to fix blackslash continuation in #{fix_these_files} but we didn't find any backslashes in #{missing}. Check if RSpec has been upgraded (maybe those blackslashes are gone??)" if missing.any?
         files_we_left_alone = files - fix_these_files
         files_we_left_alone + fixed_temp_files
       end
 
       def execute_specs(name)
-        command_line = "SPEC_OPTS=\"--format Opal::RSpec::ProgressJsonFormatter\" rake #{name}"
-        puts "Running #{command_line}"
-        pinger = Thread.new {
-          while true
-            sleep 60
-            puts 'still alive' # travis/keep alive
-          end
-        }
-        example_info = []
-        state = :progress
-        IO.popen(command_line).each do |line|
-          line.force_encoding 'UTF-8'
-          case state
-            when :progress
-              puts line
-            when :example_info
-              example_info << line
-          end
-          state = case line
-                    when /BEGIN JSON/
-                      :example_info
-                    else
-                      state
-                  end
-        end.close
+        require 'tempfile'
+        file = Tempfile.new([name.to_s, '.json'])
+        command = "SPEC_OPTS=\"--no-color --format progress --format Opal::RSpec::SeparatorFormatter --format json\" rake #{name} --trace > #{file.path}"
+        puts
+        puts "~~> Running #{command}"
+
+        # travis/keep alive
+        pinger = Thread.new { loop { sleep 60; print '.' } }
+        success = system(command)
         pinger.exit
+
+        file.rewind
+        ouput = file.read
+        ouput.force_encoding 'UTF-8'
+        progress, example_info = ouput.split('~~~SEPARATOR~~~', 2)
+
         {
-          example_info: example_info,
-          success: $?.success?
+          example_info: [example_info],
+          success: success
         }
       end
 
       def parse_results(results)
         JSON.parse results[:example_info].join("\n")
+      rescue
+        warn "JSON PARSING FAILED"
+        warn "-------------------"
+        warn results[:example_info].join("\n")
+        warn "-------------------"
+        raise
       end
 
       def rake_tasks_for(name)
@@ -254,7 +250,8 @@ module Opal
           task.files = sub_in_files
           task.default_path = default_path
           append_additional_load_paths server
-          server.debug = ENV['OPAL_DEBUG']
+
+          server.debug = ENV['OPAL_DEBUG'] if server.respond_to?(:debug=)
         end
 
         desc "Verifies that #{name} work correctly"
@@ -312,23 +309,23 @@ module Opal
           passing = total - failed - pending
           percentage = ((passing.to_f / total) * 100).round(1)
           if reasons.empty?
-            puts 'Test successful!'
-            puts "#{total} total specs, #{failed} expected failures, #{pending} expected pending"
-            puts "Passing percentage: #{percentage}%"
+            puts '~~> Test successful!'
+            puts "~~> #{total} total specs, #{failed} expected failures, #{pending} expected pending"
+            puts "~~> Passing percentage: #{percentage}%"
           else
-            puts "Test FAILED for the following reasons:\n"
+            puts "~~> Test FAILED for the following reasons:\n"
             puts reasons.join "\n\n"
             if remaining_failures.any?
               puts
-              puts "Unexpected failures:\n\n#{remaining_failures.join("\n")}\n"
+              puts "~~> Unexpected failures:\n\n#{remaining_failures.join("\n")}\n"
             end
-            puts '-----------Summary-----------'
-            puts "Total passed count: #{passing}"
-            puts "Pending count #{pending}"
-            puts "Total 'failure' count: #{actual_failures.length}"
-            puts "Passing percentage: #{percentage}%"
-            puts "Unexpected failure count: #{remaining_failures.length}"
-            raise 'Test failed!'
+            puts '~~> -----------Summary-----------'
+            puts "~~> Total passed count: #{passing}"
+            puts "~~> Pending count #{pending}"
+            puts "~~> Total 'failure' count: #{actual_failures.length}"
+            puts "~~> Passing percentage: #{percentage}%"
+            puts "~~> Unexpected failure count: #{remaining_failures.length}"
+            raise '~~> Test failed!'
           end
         end
       end

data/spec/rspec/shared/opal/separator_formatter.rb

@@ -0,0 +1,14 @@
+module Opal::RSpec
+  # a good compromise between not mucking with the code we're testing but making it more machine readable
+  class SeparatorFormatter < ::RSpec::Core::Formatters::BaseFormatter
+    ::RSpec::Core::Formatters.register self, :close
+    SEP = "~~~SEPARATOR~~~"
+
+    def close(_notification)
+      output.puts # our dots will not have closed out with a CR
+      output.puts SEP
+      output.sync = true
+      output.puts # Need a CR for popen to know we're done
+    end
+  end
+end

data/spec/rspec/support/config.rb

@@ -1,4 +1,4 @@
-require 'opal/progress_json_formatter' # verify case uses this
+require 'opal/separator_formatter' # verify case uses this
 
 RSpec::configure do |config|
   #config.full_description = '.*yield.*has a description'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opal-rspec
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Adam Beynon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-25 00:00:00.000000000 Z
+date: 2018-11-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opal
@@ -58,6 +58,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: puma
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: poltergeist
   requirement: !ruby/object:Gem::Requirement
@@ -126,7 +140,6 @@ files:
 - ".rspec"
 - ".travis.yml"
 - ".yardopts"
-- Appraisals
 - CHANGELOG.md
 - Gemfile
 - README.md
@@ -137,8 +150,6 @@ files:
 - example/Rakefile
 - example/opal/user.rb
 - example/spec/user_spec.rb
-- gemfiles/opal_0.10_stable.gemfile
-- gemfiles/opal_master.gemfile
 - lib/opal-rspec.rb
 - lib/opal/rspec.rb
 - lib/opal/rspec/cached_environment.rb
@@ -764,7 +775,7 @@ files:
 - spec/rspec/opal_rspec_spec_loader.rb
 - spec/rspec/shared/opal/fixes/deprecation_helpers.rb
 - spec/rspec/shared/opal/fixes/rspec_helpers.rb
-- spec/rspec/shared/opal/progress_json_formatter.rb
+- spec/rspec/shared/opal/separator_formatter.rb
 - spec/rspec/support/config.rb
 - spec/rspec/support/config.ru
 - spec/rspec/support/filter/bugs/recursive_const_methods.txt
@@ -781,7 +792,7 @@ files:
 - spec/rspec/support/support_spec_loader.rb
 - util/create_requires.rb
 - vendor/spec_runner.js
-homepage: http://opalrb.org
+homepage: https://github.com/opal-rspec
 licenses: []
 metadata: {}
 post_install_message: 
@@ -800,7 +811,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.7.7
 signing_key: 
 specification_version: 4
 summary: RSpec for Opal

data/Appraisals

@@ -1,8 +0,0 @@
-appraise 'opal-0.10-stable' do
-  gem 'opal', git: 'https://github.com/opal/opal.git', branch: '0-10-stable'
-end
-
-appraise 'opal-master' do
-  gem 'opal', git: 'https://github.com/opal/opal.git'
-  gem 'opal-sprockets', github: 'opal/opal-sprockets'
-end

data/gemfiles/opal_0.10_stable.gemfile

@@ -1,12 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "opal", git: "https://github.com/opal/opal.git", branch: "0-10-stable"
-gem "rspec", path: "../rspec"
-gem "rspec-support", path: "../rspec-support"
-gem "rspec-core", path: "../rspec-core"
-gem "rspec-mocks", path: "../rspec-mocks"
-gem "rspec-expectations", path: "../rspec-expectations"
-
-gemspec path: "../"

data/gemfiles/opal_master.gemfile

@@ -1,13 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "opal", git: "https://github.com/opal/opal.git"
-gem "rspec", path: "../rspec"
-gem "rspec-support", path: "../rspec-support"
-gem "rspec-core", path: "../rspec-core"
-gem "rspec-mocks", path: "../rspec-mocks"
-gem "rspec-expectations", path: "../rspec-expectations"
-gem "opal-sprockets", github: "opal/opal-sprockets"
-
-gemspec path: "../"

data/spec/rspec/shared/opal/progress_json_formatter.rb

@@ -1,29 +0,0 @@
-module Opal::RSpec
-  # a good compromise between not mucking with the code we're testing but making it more machine readable
-  class ProgressJsonFormatter < ::RSpec::Core::Formatters::JsonFormatter
-    ::RSpec::Core::Formatters.register self, :message, :dump_summary, :dump_profile, :example_passed, :example_pending, :example_failed, :close, :stop
-
-    def example_passed(_notification)
-      output.print ::RSpec::Core::Formatters::ConsoleCodes.wrap('.', :success)
-    end
-
-    def example_pending(_notification)
-      output.print ::RSpec::Core::Formatters::ConsoleCodes.wrap('*', :pending)
-    end
-
-    def example_failed(_notification)
-      output.print ::RSpec::Core::Formatters::ConsoleCodes.wrap('F', :failure)
-    end
-
-    def start_dump(_notification)
-      output.puts
-    end
-
-    def close(_notification)
-      output.puts # our dots will not have closed out with a CR
-      output.puts 'BEGIN JSON'
-      super
-      output.puts # Need a CR for popen to know we're done
-    end
-  end
-end