cruft_tracker 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e11661e18988d5a678417bb2295abf479c7eecd17644ab9e3fa3386188e6a25a
-  data.tar.gz: 104b7dc62a4590837655618f9eb82786f609e7a0b5733b629afc85628088ea9b
+  metadata.gz: f3460be9573ae2b54fe73ce702f82a7927f447e5495de026bfe4e5690111fe26
+  data.tar.gz: d26d664335c9da97deb9365e291a00ca4a3061e4e11717d2bfd1f70a862a53e6
 SHA512:
-  metadata.gz: 635d585bc91bc042b6713f40e2da1c782d4e42664c5a69e3b6377089f38cf5bb91744a6f202ff9ef61020d2ce6170b507644d5ec45908ee88b682f685f8a7fa0
-  data.tar.gz: ec22b4e3e545e2adec8094f4a5e6b46156d9d2cf35d0ef3f20f61c347f50e7c2160b98767fd8a1943ba366df13d0459de8391bbfa34f78e2dfa0f79e99d04904
+  metadata.gz: 3a24be83b7b4fef793b01ddd18868b4d161c321791c942b8bd0bf6358d545328d0edc8c8ebf9f8ce72f5bccff22e4733c09f02f145c0881aa02e15069ade13e8
+  data.tar.gz: 9c97c8ea00fa75bcff566ec04677ba811692c7ad70e763a513bc187f921216bf2cfc74baa73b26472ca611f45077bdd1e0f77bea60ff34a9903a2af25177e6d3

data/app/services/cruft_tracker/record_arguments.rb

@@ -24,22 +24,23 @@ module CruftTracker
     def arguments_record
       @arguments_record ||=
         begin
-          return find_existing_arguments_record if max_records_reached?
+          arguments_record = CruftTracker::Argument.find_by(
+            method: method,
+            arguments_hash: arguments_hash
+          )
+
+          if arguments_record.present? || max_records_reached?
+            return arguments_record
+          end
 
           CruftTracker::Argument.create(
             method: method,
             arguments_hash: arguments_hash,
             arguments: transformed_arguments
           )
-        rescue ActiveRecord::RecordNotUnique
-          find_existing_arguments_record
         end
     end
 
-    def find_existing_arguments_record
-      CruftTracker::Argument.find_by(arguments_hash: arguments_hash)
-    end
-
     def arguments_hash
       Digest::MD5.hexdigest(transformed_arguments.to_json)
     end

data/app/services/cruft_tracker/record_backtrace.rb

@@ -22,22 +22,23 @@ module CruftTracker
     def backtrace_record
       @backtrace_record ||=
         begin
-          return find_existing_backtrace_record if max_records_reached?
+          backtrace_record = CruftTracker::Backtrace.find_by(
+            traceable: method,
+            trace_hash: backtrace_hash
+          )
+
+          if backtrace_record.present? || max_records_reached?
+            return backtrace_record
+          end
 
           CruftTracker::Backtrace.create(
             traceable: method,
             trace_hash: backtrace_hash,
             trace: filtered_backtrace
           )
-        rescue ActiveRecord::RecordNotUnique
-          find_existing_backtrace_record
         end
     end
 
-    def find_existing_backtrace_record
-      CruftTracker::Backtrace.find_by(trace_hash: backtrace_hash)
-    end
-
     def backtrace_hash
       Digest::MD5.hexdigest(filtered_backtrace.to_json)
     end

data/app/services/cruft_tracker/record_render_metadata.rb

@@ -25,22 +25,23 @@ module CruftTracker
     def render_metadata_record
       @render_metadata_record ||=
         begin
-          return find_existing_render_metadata_record if max_records_reached?
+          render_metadata_record = CruftTracker::RenderMetadata.find_by(
+            view_render: view_render,
+            metadata_hash: metadata_hash
+          )
+
+          if render_metadata_record.present? || max_records_reached?
+            return render_metadata_record
+          end
 
           CruftTracker::RenderMetadata.create(
             view_render: view_render,
             metadata_hash: metadata_hash,
             metadata: metadata
           )
-        rescue ActiveRecord::RecordNotUnique
-          find_existing_render_metadata_record
         end
     end
 
-    def find_existing_render_metadata_record
-      CruftTracker::RenderMetadata.find_by(metadata_hash: metadata_hash)
-    end
-
     def metadata_hash
       Digest::MD5.hexdigest([view_render.render_hash, metadata].to_json)
     end

data/app/services/cruft_tracker/record_view_render.rb

@@ -52,7 +52,14 @@ module CruftTracker
     def view_render_record
       @view_render_record ||=
         begin
-          return find_existing_view_render_record if max_records_reached?
+          view_render_record = CruftTracker::ViewRender.find_by(
+            view: view,
+            render_hash: render_hash
+          )
+
+          if view_render_record.present? || max_records_reached?
+            return view_render_record
+          end
 
           CruftTracker::ViewRender.create(
             view: view,
@@ -63,15 +70,9 @@ module CruftTracker
             http_method: http_method,
             render_stack: render_stack
           )
-        rescue ActiveRecord::RecordNotUnique
-          find_existing_view_render_record
         end
     end
 
-    def find_existing_view_render_record
-      CruftTracker::ViewRender.find_by(render_hash: render_hash)
-    end
-
     def render_hash
       Digest::MD5.hexdigest(
         {

data/app/services/cruft_tracker/track_method.rb

@@ -77,17 +77,19 @@ module CruftTracker
     end
 
     def create_or_find_method_record
-      CruftTracker::Method.create(
+      method_record = CruftTracker::Method.find_by(
         owner: owner.name,
         name: name,
-        method_type: method_type,
-        comment: comment
+        method_type: method_type
       )
-    rescue ActiveRecord::RecordNotUnique
-      CruftTracker::Method.find_by(
+
+      return method_record if method_record.present?
+
+      CruftTracker::Method.create(
         owner: owner.name,
         name: name,
-        method_type: method_type
+        method_type: method_type,
+        comment: comment
       )
     end
 

data/app/services/cruft_tracker/track_view.rb

@@ -52,9 +52,11 @@ module CruftTracker
     end
 
     def create_or_find_view_record
+      view_record = CruftTracker::View.find_by(view: view)
+
+      return view_record if view_record.present?
+
       CruftTracker::View.create(view: view, comment: comment)
-    rescue ActiveRecord::RecordNotUnique
-      CruftTracker::View.find_by(view: view)
     end
   end
 end

data/lib/cruft_tracker/version.rb

@@ -1,3 +1,3 @@
 module CruftTracker
-  VERSION = '0.2.2'
+  VERSION = '0.2.3'
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: cruft_tracker
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors:
 - Adwerx Inc.
 - Doug Hughes
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-07-25 00:00:00.000000000 Z
+date: 2022-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_interaction
@@ -313,7 +313,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/AdWerx/cruft-tracker
   source_code_uri: https://github.com/AdWerx/cruft-tracker
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -328,8 +328,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key:
+rubygems_version: 3.3.17
+signing_key: 
 specification_version: 4
 summary: Provides a system to track Ruby method usage somewhat unobtrusively.
 test_files: []