Merge pull request #17 from fluxfederation/semvar-comparison

[1.0.17] Rectify semantic version comparison

.rubocop.yml

@@ -0,0 +1,2 @@
+Metrics/BlockLength:
+  ExcludedMethods: ['describe', 'context']

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 ## Unreleased
 ### Updated
 
+## [1.0.17] - 2021-10-26
+### Updated
+- Rectify an issue with SemVer comparison
+
 ## [1.0.16] - 2020-04-15
 ### Updated
 - Proactively sort splittable rspec files so all splittable files are processed first

lib/nitra/workers/rspec.rb

@@ -2,6 +2,8 @@
 
 module Nitra::Workers
   class Rspec < Worker
+    RSPEC_SEM_VER_VERSION = Gem::Version.new(RSpec::Core::Version::STRING)
+
     def self.filename_match?(filename)
       filename =~ /_spec\.rb/
     end
@@ -94,7 +96,7 @@ def runner_for(filename, preloading)
       args << filename
 
       result =
-        if RSpec::Core::const_defined?(:CommandLine) && RSpec::Core::Version::STRING < "2.99"
+        if RSpec::Core::const_defined?(:CommandLine) && RSPEC_SEM_VER_VERSION < Gem::Version.new('2.99')
           RSpec::Core::CommandLine.new(args)
         else
           options = RSpec::Core::ConfigurationOptions.new(args)
@@ -108,7 +110,7 @@ def runner_for(filename, preloading)
     def clean_up
       super
 
-      if RSpec::Core::Version::STRING < "3.2"
+      if RSPEC_SEM_VER_VERSION < Gem::Version.new('3.2')
         # Rspec.reset in 2.6 didn't destroy your rspec_rails fixture loading, we can't use it anymore for it's intended purpose.
         # This means our world object will be slightly polluted by the preload_framework code, but that's a small price to pay
         # to upgrade.

nitra.gemspec

@@ -1,6 +1,6 @@
 spec = Gem::Specification.new do |s|
   s.name = 'nitra'
-  s.version = '1.0.16'
+  s.version = '1.0.17'
   s.platform = Gem::Platform::RUBY
   s.license = "MIT"
   s.homepage = "http://github.com/fluxfederation/nitra"

spec/nitra/workers/rspec_spec.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require_relative '../../spec_helper'
+require_relative '../../../lib/nitra/workers/rspec'
+
+require 'rspec'
+
+describe Nitra::Workers::Rspec do
+  describe '#clean_up' do
+    let(:described_instance) do
+      described_class.new(
+        nil,
+        nil,
+        RSpec.configuration
+      )
+    end
+
+    context 'when the rspec semantic version is < 3.2' do
+      before do
+        stub_const('Nitra::Workers::Rspec::RSPEC_SEM_VER_VERSION', Gem::Version.new('3.0'))
+        allow(::RSpec.configuration).to receive(:reset)
+      end
+
+      it 'calls RSpec.configuration.reset' do
+        described_instance.clean_up
+        expect(::RSpec.configuration).to have_received(:reset)
+      end
+    end
+
+    context 'when the rspec semantic version is > 3.2' do
+      before do
+        stub_const('Nitra::Workers::Rspec::RSPEC_SEM_VER_VERSION', Gem::Version.new('3.10.1'))
+        allow(::RSpec).to receive(:clear_examples)
+      end
+
+      it 'calls RSpec.clear_examples' do
+        described_instance.clean_up
+        expect(::RSpec).to have_received(:clear_examples)
+      end
+    end
+  end
+end

spec/spec_helper.rb

@@ -1,4 +1,8 @@
+# frozen_string_literal: true
+
 require 'minitest/spec'
 require 'minitest/autorun'
 require 'minitest/reporters'
 Minitest::Reporters.use!(Minitest::Reporters::SpecReporter.new)
+
+require_relative '../lib/nitra'