quick_check 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 896e5b8930fe9d6f9c090ef32c1bbc70a57a05d415c7ad6996e5a4e211868963
-  data.tar.gz: ae0e8f1831e674357081df078d68cfdbf3966c22d09f6c826becaa7d6a21bd90
+  metadata.gz: 4ea26165117cca41f73e40de4f2975ac21bfa9d86b3d097dabfe43fbf3147416
+  data.tar.gz: 85d0174ff222df297448f18b3d6333b2cecc6aec6a82afacd86076568843f9f0
 SHA512:
-  metadata.gz: cddc454b92ad6e3c2c1f9afe08dab40305c9ee29bfe91d35a44fe578b7c52bb27cc09dbd1806c13d6cb9350bb4e945a0e2e1c24c757bbb005865739b2cd9281f
-  data.tar.gz: 7952bbeb5140d87b51b189b464ee7309367a49fc687e099bdcd124b9d5744b0914735d0336b36d500a6dbe781b2bae5101101519a9446af319f2bbb442256fc4
+  metadata.gz: 4c4631164f1ce412d3ee1c3ee1d100197ea2311c3ea28d246a3edc3854466194670a4faabcfae939e44c97caeed2552ed1a5cc1e09f9b5ec46c0f307d82bfe1b
+  data.tar.gz: 48b7538d2f0600a3381d8847eeab3455630612a715f0dbe70d996cd312fb9b7c15bd05083323eb9cc9f0eab788e1f40452dfa10332f623ef30ac9381c268d4e2

data/README.md

@@ -99,10 +99,10 @@ If no config is present, `qc` will use the first existing branch among `main` or
 
 ## How it works
 
-- Unstaged: `git diff --name-only --diff-filter=ACMR`
-- Staged: `git diff --name-only --cached --diff-filter=ACMR`
+- Unstaged: `git diff --name-only --diff-filter=AM`
+- Staged: `git diff --name-only --cached --diff-filter=AM`
 - Untracked: `git ls-files --others --exclude-standard`
-- Committed vs base: `git diff --name-only --diff-filter=ACMR <base>...HEAD`
+- Committed vs base: `git diff --name-only --diff-filter=AM <base>...HEAD`
 
 Files are filtered to `spec/**/*_spec.rb` and/or `test/**/*_test.rb`, de-duplicated, sorted, and then:
 

data/lib/quick_check/cli.rb

@@ -17,7 +17,7 @@ module QuickCheck
       @argv = argv
       @options = {
         base_branch: nil,
-        include_committed_diff: true,
+        include_committed_diff: false,
         include_staged: true,
         include_unstaged: true,
         custom_command: nil,
@@ -126,12 +126,12 @@ module QuickCheck
       files = []
 
       if @options[:include_unstaged]
-        files.concat(git_diff_name_only(["--name-only", "--diff-filter=ACMR"]))
+        files.concat(git_diff_name_only(["--name-only", "--diff-filter=AM"]))
         files.concat(git_untracked_files)
       end
 
       if @options[:include_staged]
-        files.concat(git_diff_name_only(["--name-only", "--cached", "--diff-filter=ACMR"]))
+        files.concat(git_diff_name_only(["--name-only", "--cached", "--diff-filter=AM"]))
       end
 
       if @options[:include_committed_diff]
@@ -139,7 +139,7 @@ module QuickCheck
         if current_branch && base_branch && current_branch != base_branch
           # Include files changed on this branch vs base
           range = diff_range_against_base(base_branch)
-          files.concat(git_diff_name_only(["--name-only", "--diff-filter=ACMR", range])) if range
+          files.concat(git_diff_name_only(["--name-only", "--diff-filter=AM", range])) if range
         end
       end
 
@@ -147,16 +147,7 @@ module QuickCheck
       rspec_specs = files.select { |f| f.match?(%r{\Aspec/.+_spec\.rb\z}) }
       minitest_tests = files.select { |f| f.match?(%r{\Atest/.+_test\.rb\z}) }
 
-      # Infer tests from source changes (e.g., app/models/user.rb -> spec/models/user_spec.rb)
-      source_files = files.reject { |f| f.match?(%r{\A(spec/|test/)}) }
-      unless source_files.empty?
-        inferred_rspec = source_files.flat_map { |src| infer_rspec_paths_for_source(src) }
-        inferred_minitest = source_files.flat_map { |src| infer_minitest_paths_for_source(src) }
-        rspec_specs.concat(inferred_rspec)
-        minitest_tests.concat(inferred_minitest)
-      end
-
-      { rspec: rspec_specs.uniq, minitest: minitest_tests.uniq }
+      { rspec: rspec_specs.sort, minitest: minitest_tests.sort }
     end
 
     def ensure_git_repo!
@@ -224,10 +215,12 @@ module QuickCheck
     end
 
     def diff_range_against_base(base)
-      return nil unless branch_exists?(base)
-      # If upstream exists, prefer merge-base to HEAD to include all branch commits
-      # The symmetric range base...HEAD ensures we include commits unique to the branch
-      "#{base}...HEAD"
+      # Prefer the symmetric range base...HEAD if base exists
+      if branch_exists?(base)
+        "#{base}...HEAD"
+      else
+        nil
+      end
     end
 
     def git_diff_name_only(args)
@@ -245,52 +238,6 @@ module QuickCheck
       out.split("\n").map(&:strip).reject(&:empty?)
     end
 
-    def infer_rspec_paths_for_source(source_path)
-      candidates = []
-      if source_path.start_with?("app/")
-        rel = source_path.sub(/^app\//, "")
-        # Prefer request specs for controllers
-        if rel.start_with?("controllers/")
-          sub_rel = rel.sub(/^controllers\//, "")
-          dir = File.dirname(sub_rel)
-          dir = "" if dir == "."
-          base_with_controller = File.basename(sub_rel, ".rb")
-          base_without_controller = base_with_controller.sub(/_controller\z/, "")
-          # Common conventions for request specs
-          candidates << File.join("spec", "requests", dir, "#{base_without_controller}_spec.rb")
-          candidates << File.join("spec", "requests", dir, "#{base_with_controller}_spec.rb")
-        end
-        candidates << File.join("spec", rel).sub(/\.rb\z/, "_spec.rb")
-      elsif source_path.start_with?("lib/")
-        rel = source_path.sub(/^lib\//, "")
-        candidates << File.join("spec", "lib", rel).sub(/\.rb\z/, "_spec.rb")
-      end
-
-      candidates.select { |p| File.file?(p) }
-    end
-
-    def infer_minitest_paths_for_source(source_path)
-      candidates = []
-      if source_path.start_with?("app/")
-        rel = source_path.sub(/^app\//, "")
-        # Prefer integration/request-style tests for controllers when present
-        if rel.start_with?("controllers/")
-          sub_rel = rel.sub(/^controllers\//, "")
-          dir = File.dirname(sub_rel)
-          dir = "" if dir == "."
-          base = File.basename(sub_rel, ".rb").sub(/_controller\z/, "")
-          integration_test = File.join("test", "integration", dir, "#{base}_test.rb")
-          candidates << integration_test
-        end
-        candidates << File.join("test", rel).sub(/\.rb\z/, "_test.rb")
-      elsif source_path.start_with?("lib/")
-        rel = source_path.sub(/^lib\//, "")
-        candidates << File.join("test", "lib", rel).sub(/\.rb\z/, "_test.rb")
-      end
-
-      candidates.select { |p| File.file?(p) }
-    end
-
     def build_command_for(framework, files)
       return (@options[:custom_command] + files) if @options[:custom_command]
 
@@ -310,13 +257,11 @@ module QuickCheck
     end
 
     def rails_available?
-      bin_rails = File.join(Dir.pwd, "bin", "rails")
-      File.executable?(bin_rails) || File.file?(bin_rails) || gemfile_includes?("rails")
+      File.executable?(File.join(Dir.pwd, "bin", "rails")) || gemfile_includes?("rails")
     end
 
     def rails_cmd
-      bin_rails = File.join(Dir.pwd, "bin", "rails")
-      if File.executable?(bin_rails) || File.file?(bin_rails)
+      if File.executable?(File.join(Dir.pwd, "bin", "rails"))
         [File.join("bin", "rails")]
       else
         ["bundle", "exec", "rails"]

data/lib/quick_check/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module QuickCheck
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quick_check
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Kasvit
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-22 00:00:00.000000000 Z
+date: 2025-08-27 00:00:00.000000000 Z
 dependencies: []
 description: Adds the `qc` command to run only changed or newly added RSpec files
   from uncommitted changes and vs base branch (main/master).