devloop 0.0.6 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cdbcf2a0d7f3ee56a98ad2db128f8835b348f990efab6fd77ab9a57dbc2f6aa3
-  data.tar.gz: 7b041ec78b9ac2c25e6423e9d3dc4b8be64c1283668d88a259200efa4feade24
+  metadata.gz: c6e087fe2cd945782777fe0a1a87fd21b3022f0348a38b1c789f8ba6c7817f49
+  data.tar.gz: 1718ef614659f6ba55a3a7ca1d60d2d7b676a70919d1a93e7bd4f0c5ff1a46ba
 SHA512:
-  metadata.gz: 381c4b59391e866838d90084e8251f193d7d3ba31da4ddc7905e28311e6e0d7ab42fb2a5ca18773fe58706f62a270e8a6b7ff9d4412e842954dbb4a86e6a9108
-  data.tar.gz: 6f1d6304053545f310d5416a5649e704d84481a1d654d0c2e4530e64b5f0ee429d41c5106f5465a4c174f097df8a9f5402c0c6890491fe4f29a5283f50139c0a
+  metadata.gz: 3c0948c9527bc7bed104aa6cac0a65bf6c67618e5d3afa31d2245d90565ca32546cb4bdc8c084ef2103c0919c648f4a315c923aed8ca1b1ec7b636fa394b83fa
+  data.tar.gz: 59775a6c5a08119a319cd541bd8e45a7bc1432fb57b498a0e788eb22018f7ef864f316fd412c788a8574359144607ca695bb75b7e38bbe132e51ae68b712ecde

data/README.md

@@ -2,6 +2,8 @@
 
 Devloop is an automated Rspec runner for Rails apps inspired by [TLDR](https://github.com/tendersearls/tldr). The purpose of this tool is to provide continuous and instant feedback when working on the Rails app. It runs only specs from _lines_ modified in the recent git commits. Even if you have a large `spec/user_spec.rb` file, you'll receive specs feedback in ~second on each file save.
 
+Optionally, you can edit first line of any spec file (i.e. add `#`) to run all the tests from it.
+
 ## Installation 
 
 It uses [fswatch](https://github.com/emcrisostomo/fswatch) so make sure to install it first:

data/lib/devloop/diff_parser.rb

@@ -5,18 +5,37 @@ module Devloop
     end
 
     def call(diff)
-      lines = diff.split("\n")
-      results = []
-      file = ""
-      lines.each_with_index do |line, index|
+      _, results = diff.split("\n").reduce(["", []]) do |(file, results), line|
         if line.start_with?("+++ b/")
-          file = line[6..-1]
+          [line[6..-1], results]
         elsif line.start_with?("@@ -")
           line_number = line.match(/@@ -(\d+)/)[1]
-          results << "#{file}:#{line_number}"
+          if line_number == "1"
+            [file, results << "#{relative_path(file)}"]
+          else
+            [file, results << "#{relative_path(file)}:#{line_number}"]
+          end
+        else
+          [file, results]
         end
-      end
-      results
+      end.uniq
+
+      # Remove filenames with line number if filename without line number is present
+      results.reject { |result| results.include?(result.split(":").first) && result.include?(":") }
+    end
+
+    private
+
+    def relative_path(path)
+      path.gsub(project_path, "")
+    end
+
+    def project_path
+      @project_path ||= Dir.pwd.gsub("#{git_root_path}/", "") + "/"
+    end
+
+    def git_root_path
+      @git_root_path ||= `git rev-parse --show-toplevel`.strip
     end
   end
 end

data/lib/devloop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Devloop
-  VERSION = "0.0.6"
+  VERSION = "0.0.8"
 end

data/spec/diff_parser_spec.rb

@@ -4,46 +4,96 @@ require "spec_helper"
 require "devloop/diff_parser"
 
 describe Devloop::DiffParser do
-  let(:diff1) do
-    <<~DIFF
-      diff --git a/spec/models/team_spec.rb b/spec/models/team_spec.rb
-      index 19772f2a..a32824f9 100644
-      --- a/spec/models/team_spec.rb
-      +++ b/spec/models/team_spec.rb
-      @@ -10,2 +10,2 @@ describe Team do
-      -  it "has a valid factory" do
-      -    expect(team).to be_valid
-      +  it "has valid factory" do
-      +    expect(team).not_to be_valid
-      diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
-      index 410ffa8c..610d9e19 100644
-      --- a/spec/models/user_spec.rb
-      +++ b/spec/models/user_spec.rb
-      @@ -167 +167 @@ describe User do
-      -          web_auth_token_generated_at: 50.minutes.ago,
-      +          web_auth_token_generated_at: 10.minutes.ago,
-    DIFF
+  context "first line of a file was edited" do
+    let(:diff) do
+      <<~DIFF
+            diff --git a/spec/models/team_spec.rb b/spec/models/team_spec.rb
+        index 19772f2a..9f614e20 100644
+        --- a/spec/models/team_spec.rb
+        +++ b/spec/models/team_spec.rb
+        @@ -1 +1,3 @@ describe Team do
+        -    it "has_many emojis" do
+        +    it "has_many eojis" do
+        @@ -24,2 +24,2 @@ describe Team do
+        -  describe "normalize attributes" do
+        -    it "does not allow empty string values" do
+        +  describe "normalize attrbtes" do
+        +    it "does not allw empty string values" do
+      DIFF
+    end
+
+    it "will run the whole file" do
+      expect(Devloop::DiffParser.call(diff)).to eq(["spec/models/team_spec.rb"])
+    end
   end
 
-  let(:diff2) do
-    <<~DIFF
-          diff --git a/spec/models/team_spec.rb b/spec/models/team_spec.rb
-      index 19772f2a..9f614e20 100644
-      --- a/spec/models/team_spec.rb
-      +++ b/spec/models/team_spec.rb
-      @@ -19 +19 @@ describe Team do
-      -    it "has_many emojis" do
-      +    it "has_many eojis" do
-      @@ -24,2 +24,2 @@ describe Team do
-      -  describe "normalize attributes" do
-      -    it "does not allow empty string values" do
-      +  describe "normalize attrbtes" do
-      +    it "does not allw empty sting values" do
-    DIFF
+  context "the project root is the same as git root" do
+    let(:diff1) do
+      <<~DIFF
+        diff --git a/spec/models/team_spec.rb b/spec/models/team_spec.rb
+        index 19772f2a..a32824f9 100644
+        --- a/spec/models/team_spec.rb
+        +++ b/spec/models/team_spec.rb
+        @@ -10,2 +10,2 @@ describe Team do
+        -  it "has a valid factory" do
+        -    expect(team).to be_valid
+        +  it "has valid factory" do
+        +    expect(team).not_to be_valid
+        diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
+        index 410ffa8c..610d9e19 100644
+        --- a/spec/models/user_spec.rb
+        +++ b/spec/models/user_spec.rb
+        @@ -167 +167 @@ describe User do
+        -          web_auth_token_generated_at: 50.minutes.ago,
+        +          web_auth_token_generated_at: 10.minutes.ago,
+      DIFF
+    end
+
+    let(:diff2) do
+      <<~DIFF
+            diff --git a/spec/models/team_spec.rb b/spec/models/team_spec.rb
+        index 19772f2a..9f614e20 100644
+        --- a/spec/models/team_spec.rb
+        +++ b/spec/models/team_spec.rb
+        @@ -19 +19 @@ describe Team do
+        -    it "has_many emojis" do
+        +    it "has_many eojis" do
+        @@ -24,2 +24,2 @@ describe Team do
+        -  describe "normalize attributes" do
+        -    it "does not allow empty string values" do
+        +  describe "normalize attrbtes" do
+        +    it "does not allw empty string values" do
+      DIFF
+    end
+
+    it "parses the diff correctly" do
+      expect(Devloop::DiffParser.call(diff1)).to eq(["spec/models/team_spec.rb:10", "spec/models/user_spec.rb:167"])
+      expect(Devloop::DiffParser.call(diff2)).to eq(["spec/models/team_spec.rb:19", "spec/models/team_spec.rb:24"])
+    end
   end
 
-  it "parses the diff correctly" do
-    expect(Devloop::DiffParser.call(diff1)).to eq(["spec/models/team_spec.rb:10", "spec/models/user_spec.rb:167"])
-    expect(Devloop::DiffParser.call(diff2)).to eq(["spec/models/team_spec.rb:19", "spec/models/team_spec.rb:24"])
+  context "the project root is different from the git root" do
+    let(:diff) do
+      <<~DIFF
+        diff --git a/src/spec/models/team_spec.rb b/src/spec/models/team_spec.rb
+        index 19772f2a..9f614e20 100644
+        --- a/src/spec/models/team_spec.rb
+        +++ b/src/spec/models/team_spec.rb
+        @@ -19 +19 @@ describe Team do
+        -    it "has_many emojis" do
+        +    it "has_many eojis" do
+        @@ -24,2 +24,2 @@ describe Team do
+        -  describe "normalize attributes" do
+        -    it "does not allow empty string values" do
+        +  describe "normalize attrbtes" do
+        +    it "does not allw empty string values" do
+      DIFF
+    end
+
+    it "parses the diff correctly" do
+      allow_any_instance_of(Devloop::DiffParser).to receive(:git_root_path).and_return("/Users/username/projects/")
+      allow_any_instance_of(Devloop::DiffParser).to receive(:project_path).and_return("src/")
+      expect(Devloop::DiffParser.call(diff)).to eq(["spec/models/team_spec.rb:19", "spec/models/team_spec.rb:24"])
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: devloop
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.8
 platform: ruby
 authors:
 - pawurb
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-07 00:00:00.000000000 Z
+date: 2024-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake