devformance 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28211e6e59e94ed6dab36b693c72585d084491a2bb0d6d3dc4a7b37c8c96fb40
-  data.tar.gz: 13a835b26dbbac8c9179eca106f098cb0f02c046adef66d9e3d63c28147780a5
+  metadata.gz: d4dbeb244e125c274d11edc1076f81967b4f2939a594094abd774384704789cf
+  data.tar.gz: a2f2499f058d55968708385748d7e9acd2bab7acccb2c27c26a9972723cab95c
 SHA512:
-  metadata.gz: 40639e968973f0e9ef89ee401da59633c5c25cc99ddd3ade2cd489d394bdbdd952ea727ffbd16b26cad1ce133529a1a56a07451f46c5859e055de401945772d0
-  data.tar.gz: cd4251f047e66bdb0fce61ea5d189e8026db679f47090b16be8d2c4e64807e6b114f0d4604b68c198f4e97935324d1b0cbc21abfb56cd28d85e91b4851bd7bfa
+  metadata.gz: d5be802a36a8ca2782acdc599f461d3006a75194498c603e66e6446b33881409c03f544258ba1b43a170edd5621bb1efaba518d7f1ff4d748e399285008b529b
+  data.tar.gz: 36aa0ed34f200ab7de0ab555427aec0ac3cc4763eddcc92a983c595de745a3c2a04b7d002b6e44809ee0ac1aa2cc39ad9b7d70c925f735c11d530697545c7454

data/app/models/devformance/file_result.rb

@@ -7,8 +7,14 @@ module Devformance
     belongs_to :run, foreign_key: :run_id, primary_key: :run_id,
                class_name: "Devformance::Run"
 
+    validates :file_key, uniqueness: { scope: :run_id }
+
     def self.file_key_for(file_path)
-      File.basename(file_path, ".rb").gsub(/[^a-z0-9_]/, "_")
+      relative_path = Pathname.new(file_path).relative_path_from(Rails.root).to_s
+
+      relative_path.gsub(/[^a-z0-9_]+/i, "_").downcase.sub(/_rb\z/, "")
+    rescue ArgumentError
+      file_path.to_s.gsub(/[^a-z0-9_]+/i, "_").downcase.sub(/_rb\z/, "")
     end
   end
 end

data/lib/devformance/version.rb

@@ -1,3 +1,3 @@
 module Devformance
-  Version = "0.1.0"
+  Version = "0.1.1"
 end

data/spec/fixtures/devformance/file_result.rb

@@ -13,22 +13,4 @@ FactoryBot.define do
     duration_ms { nil }
     log_path { nil }
   end
-
-  trait :passed do
-    status { :passed }
-    total_tests { 10 }
-    passed_tests { 10 }
-    failed_tests { 0 }
-    coverage { 95.5 }
-    duration_ms { 2500 }
-  end
-
-  trait :failed do
-    status { :failed }
-    total_tests { 10 }
-    passed_tests { 7 }
-    failed_tests { 3 }
-    coverage { 85.0 }
-    duration_ms { 3000 }
-  end
 end

data/spec/models/devmetrics/file_result_spec.rb

@@ -18,8 +18,8 @@ RSpec.describe Devformance::FileResult, type: :model do
     context "with a standard spec path" do
       let(:file_path) { "spec/requests/users_spec.rb" }
 
-      it "returns the basename without .rb extension" do
-        expect(key).to eq("users_spec")
+      it "returns a sanitized key from the relative path" do
+        expect(key).to eq("spec_requests_users_spec")
       end
     end
 
@@ -27,15 +27,32 @@ RSpec.describe Devformance::FileResult, type: :model do
       let(:file_path) { "spec/requests/my-complex_spec.rb" }
 
       it "replaces non-alphanumeric characters with underscores" do
-        expect(key).to eq("my_complex_spec")
+        expect(key).to eq("spec_requests_my_complex_spec")
       end
     end
 
     context "with a deeply nested path" do
-      let(:file_path) { "/home/user/app/spec/requests/api/v1/orders_spec.rb" }
+      let(:file_path) { Rails.root.join("spec/requests/api/v1/orders_spec.rb").to_s }
 
-      it "uses only the basename" do
-        expect(key).to eq("orders_spec")
+      it "uses the full relative path" do
+        expect(key).to eq("spec_requests_api_v1_orders_spec")
+      end
+    end
+
+    it "distinguishes files with the same basename in different directories" do
+      first_path = Rails.root.join("spec/requests/api/v1/categories_petitions/list_spec.rb").to_s
+      second_path = Rails.root.join("spec/requests/api/v1/core/document_types/list_spec.rb").to_s
+
+      expect(described_class.file_key_for(first_path)).to eq("spec_requests_api_v1_categories_petitions_list_spec")
+      expect(described_class.file_key_for(second_path)).to eq("spec_requests_api_v1_core_document_types_list_spec")
+      expect(described_class.file_key_for(first_path)).not_to eq(described_class.file_key_for(second_path))
+    end
+
+    context "when the path cannot be made relative to Rails.root" do
+      let(:file_path) { "/tmp/external/list_spec.rb" }
+
+      it "falls back to sanitizing the full path" do
+        expect(key).to eq("_tmp_external_list_spec")
       end
     end
   end

data/spec/rails_helper.rb

@@ -1,7 +1,7 @@
 ENV['RAILS_ENV'] ||= 'test'
 require_relative '../config/environment'
 require 'rspec/rails'
-require 'factory_bot'
+require 'factory_bot_rails'
 require 'simplecov'
 SimpleCov.start 'rails' do
   enable_coverage :branch
@@ -16,5 +16,5 @@ end
 
 Dir[Rails.root.join("spec/fixtures/devformance/**/*.rb")].each { |f| require f }
 
-require_relative '../spec/support/devformance_formatter'
-require_relative '../spec/support/devformance_metrics'
+require_relative '../spec/support/devmetrics_formatter'
+require_relative '../spec/support/devmetrics_metrics'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: devformance
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Devformance Contributors
@@ -166,7 +166,6 @@ files:
 - spec/lib/devmetrics/sql_instrumentor_spec.rb
 - spec/models/devmetrics/file_result_spec.rb
 - spec/models/devmetrics/run_spec.rb
-- spec/models/query_log_spec.rb
 - spec/rails_helper.rb
 - spec/requests/devmetrics/metrics_controller_spec.rb
 - spec/requests/devmetrics_pages_spec.rb
@@ -176,10 +175,12 @@ files:
 - spec/support/devmetrics_formatter.rb
 - spec/support/devmetrics_metrics.rb
 - spec/support/factory_bot.rb
-homepage: https://github.com/yourusername/devformance
+homepage: https://github.com/anyelopetit/devformance
 licenses:
 - MIT
-metadata: {}
+metadata:
+  source_code_uri: https://github.com/anyelopetit/devformance
+  bug_tracker_uri: https://github.com/anyelopetit/devformance/issues
 rdoc_options: []
 require_paths:
 - lib

data/spec/models/query_log_spec.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-require "rails_helper"
-
-RSpec.describe QueryLog, type: :model do
-  describe "#initialize" do
-    subject(:instance) { described_class.new(query: "SELECT 1") }
-
-    it "stores the query attribute" do
-      expect(instance.query).to eq("SELECT 1")
-    end
-  end
-
-  describe "validations" do
-    subject(:log) { described_class.new(query: "SELECT * FROM users", duration: 150, user_id: 1) }
-
-    it "is valid with basic attributes" do
-      expect(log).to be_valid
-    end
-  end
-end