constable-rails 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '092c12ca7debd9d093f51e3ce6d90332aef25dcb9c16a5cf79b67f716e15a6a5'
-  data.tar.gz: 0b82a9db3e6d66228103161d46635e394591223956aa67a4bc5f929075701954
+  metadata.gz: e3b589fed13c38522dab7443ab1bb967bedeaec7abb34beaecd5c4f2c9a4298a
+  data.tar.gz: b646c7160830f81bc2c6c5af9fe60728a8c7e1f40236e594fb64a51eed04080f
 SHA512:
-  metadata.gz: 9bbf3170f8fd728513e0384d36477d4281535276446c86d8e97b8d49b32d956101d1fe4fe30d19688b9a37f36d8159f8ee3061ebc1b2801a6ad55f718d721d27
-  data.tar.gz: 300f8c3ecbfb7075492638c355aceb83fc242808e8296ec21798bea71e23bee98ec6b28ffeaad84a11206fd0e6e65b9771c56a441f4e1d12cc730a969a4340ba
+  metadata.gz: ee0579e80cb81345601ce2c40090db63b87b1498698a77fbc8c34d0df478d36f3210295b3909bd8290835a1f245f2522952f491caca6776d7ac28981dcae7b74
+  data.tar.gz: aea8a0a67d5bdc8868706db9820af271ded921871da905c9e09fe83ac74a359c8c5e9ef8649d737367075f8faa9fabc32fff13156a8f2a9f6e9f69516464441a

data/CHANGELOG.md

@@ -5,6 +5,72 @@ All notable changes to this project are documented here. This project adheres to
 
 ## [Unreleased]
 
+## [1.3.1]
+
+### Cold cases now read `.rspec`
+
+The bug that mattered. `.rspec` is where an RSpec suite says what to load before any spec
+file:
+
+```
+--require spec_helper
+--require rails_helper
+```
+
+That is what `rspec --init` generates, and it is why a real spec file usually has no
+`require` line of its own — there is nothing for it to repeat. RSpec's own runner reads
+those files. Constable's cold-case driver did not.
+
+So a cold case ran with no `rails_helper` at all: no FactoryBot, no shoulda-matchers, no
+`spec/support/**`. On a 1,277-file suite that is **entirely green under
+`bundle exec rspec`**, `constable test spec/models --full` reported:
+
+```
+✓ 774 passed   ✗ 3589 failed
+```
+
+2,637 of them `undefined method 'create'`, the rest `belong_to`,
+`validate_presence_of`, missing support constants. Every one of them Constable's fault,
+and every one of them looking like the user's.
+
+Cold cases now apply the `--require` directives from `.rspec`, `~/.rspec`, `.rspec-local`
+and `SPEC_OPTS`, parsed by RSpec itself so the precedence is its own rather than a guess.
+Only `--require` is taken: formatters, colour and output streams belong to Constable's
+reporter, ordering is Constable's job, and a `--tag` filter meant for a different run
+should not silently drop tests from this one.
+
+The same file is now 14 passed, 0 failed — matching `bundle exec rspec` exactly.
+
+This never showed up before because the app it was developed against wrote
+`require "rails_helper"` at the top of every spec, which is the one arrangement that
+hides it.
+
+### `constable import` says what it actually did
+
+It reported `did reopen 1277 file(s)`. "Reopen" is this code's internal word and means
+nothing to a reader, and "import" on its own suggests the files were copied somewhere —
+which is the opposite of what happened. A real user read it exactly that way and asked
+why their specs had not moved into `test/`.
+
+```
+Adopted 1277 rspec files as cold cases.
+
+  Your rspec files stay exactly where they are and are not changed.
+  One line was added to .constable/config.yml:
+
+    cold_cases:
+      - spec/**/*_spec.rb
+
+  Constable runs them from there, through real RSpec, and folds
+  the results into its own reporting, flake history and CI gate.
+
+  Next:  constable test --full        run everything, cold and native
+         constable test --unsafe      run only these
+         constable modernize PATH     see what one file would look like
+                                      as a native case (writes nothing)
+```
+
+
 ## [1.3.0]
 
 Adoption at scale. Everything here came from installing 1.2.0 into a 1,277-spec-file
@@ -462,7 +528,8 @@ Initial release.
 - Diff-based coverage gate — only lines changed in the current diff are held to the
   threshold. `constable beat` for the full picture, `--html` for a browsable report.
 
-[Unreleased]: https://github.com/Ray-Hughes/constable/compare/v1.3.0...HEAD
+[Unreleased]: https://github.com/Ray-Hughes/constable/compare/v1.3.1...HEAD
+[1.3.1]: https://github.com/Ray-Hughes/constable/compare/v1.3.0...v1.3.1
 [1.3.0]: https://github.com/Ray-Hughes/constable/compare/v1.2.0...v1.3.0
 [1.2.0]: https://github.com/Ray-Hughes/constable/compare/v1.1.0...v1.2.0
 [1.1.0]: https://github.com/Ray-Hughes/constable/compare/v1.0.0...v1.1.0

data/lib/constable/cli.rb

@@ -168,8 +168,8 @@ module Constable
         strategy: options[:strategy].to_sym
       )
 
+      # The summary says this in full now, in the reader's own terms.
       say result.summary
-      say "\nNothing was rewritten -- cold cases run through their own engine, unchanged." if result.any_changes?
       exit(EXIT_CLEAN)
     end
 

data/lib/constable/cold_case/rspec.rb

@@ -278,9 +278,53 @@ module Constable
           # rspec-core's own default; restated because rspec-rails turns it off and a
           # verbatim legacy file may well open with a bare `describe`.
           configuration.expose_dsl_globally = true if configuration.respond_to?(:expose_dsl_globally=)
+          apply_rspec_options(configuration)
           @session_prepared = true
         end
 
+        # `.rspec` is where a suite says what to load before any spec file.
+        #
+        #   --require spec_helper
+        #   --require rails_helper
+        #
+        # That is what `rspec --init` generates, and it is why a real spec file usually
+        # has no `require` line of its own -- there is nothing for it to repeat. RSpec's
+        # own runner reads those files; driving example groups directly does not, so
+        # without this a cold case runs with no rails_helper at all: no FactoryBot, no
+        # shoulda-matchers, no spec/support. In one real suite that was 2,637 tests
+        # failing on `undefined method 'create'` -- a suite that is entirely green under
+        # `bundle exec rspec`.
+        #
+        # Only `--require` is taken. Formatters, colour and output streams belong to
+        # Constable's reporter, ordering is Constable's job, and a `--tag` filter from a
+        # file RSpec would apply to its own run should not silently drop tests from this
+        # one. Requires are the part that decides whether the suite can run at all.
+        def apply_rspec_options(configuration)
+          requires = rspec_option_requires
+          return if requires.empty?
+
+          # `requires=` rather than plain Kernel#require: it is RSpec's own accessor, and
+          # it puts `lib` and the default path (`spec`) on the load path first, which is
+          # what makes a bare `require "rails_helper"` resolve.
+          configuration.requires = requires
+        rescue StandardError => e
+          # A helper that will not load is the suite's problem to fix, and it will say so
+          # loudly on the first file. Constable's job here is not to disappear.
+          Constable.warn!("could not load what .rspec requires (#{e.class}: #{e.message}). " \
+                          "Cold cases will run without it.", kind: :cold_case)
+        end
+
+        # Parsed by RSpec itself, so `.rspec`, `~/.rspec`, `.rspec-local` and SPEC_OPTS are
+        # all read with its precedence rather than a guess at the format.
+        def rspec_option_requires
+          return [] unless defined?(::RSpec::Core::ConfigurationOptions)
+
+          options = ::RSpec::Core::ConfigurationOptions.new([]).options
+          Array(options[:requires])
+        rescue StandardError
+          []
+        end
+
         def clear_examples
           world = ::RSpec.instance_variable_get(:@world)
           world.reset if world.respond_to?(:reset)

data/lib/constable/importer/reopener.rb

@@ -114,12 +114,26 @@ module Constable
           }
         end
 
+        # Written for somebody who has just typed `constable import` and does not yet know
+        # what a cold case is. "reopen" is the word this code uses internally and it means
+        # nothing to a reader; "import" itself suggests files were copied somewhere, which
+        # is the opposite of what happened. So: say where the files are, say what changed,
+        # and say what to do next.
         def summary
-          verb = dry_run? ? "would" : "did"
-          lines = ["#{@from} import (#{@strategy}) -- #{verb} reopen #{imported_count} file(s)"]
+          verb = dry_run? ? "Would adopt" : "Adopted"
+          noun = imported_count == 1 ? "file" : "files"
+          lines = ["#{verb} #{imported_count} #{@from} #{noun} as cold cases."]
+
           unless @globs_added.empty?
-            lines << "  config path match: added #{@globs_added.size} glob(s) to #{@config_path}"
-            @globs_added.each { |glob| lines << "    - #{glob}   (#{covered_by(glob).size} files, 0 file changes)" }
+            lines << ""
+            lines << "  Your #{@from} files stay exactly where they are and are not changed."
+            lines << "  #{dry_run? ? "One line would be added to" : "One line was added to"} " \
+                     "#{@config_path}:"
+            lines << ""
+            @globs_added.each { |glob| lines << "    cold_cases:\n      - #{glob}" }
+            lines << ""
+            lines << "  Constable runs them from there, through real #{engine_label}, and folds"
+            lines << "  the results into its own reporting, flake history and CI gate."
             unless comments_preserved?
               lines << "  note: config.yml was rewritten from parsed YAML; comments were not preserved"
             end
@@ -133,9 +147,20 @@ module Constable
             @skipped.each { |s| lines << "    - #{s[:path]} (#{s[:reason]})" }
           end
           @errors.each { |e| lines << "  error: #{e[:path]} -- #{e[:message]}" }
+
+          unless dry_run? || imported_count.zero?
+            lines << ""
+            lines << "  Next:  constable test --full        run everything, cold and native"
+            lines << "         constable test --unsafe      run only these"
+            lines << "         constable modernize PATH     see what one file would look like"
+            lines << "                                      as a native case (writes nothing)"
+          end
+
           lines.join("\n")
         end
 
+        def engine_label = @from.to_s == "rspec" ? "RSpec" : "Minitest"
+
         def superclass_name = ENGINES.fetch(@from)[:superclass]
 
         def covered_by(glob)

data/lib/constable/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Constable
-  VERSION = "1.3.0"
+  VERSION = "1.3.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: constable-rails
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Ray Hughes