dead_code_detector 0.0.10 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60644539c351c9c6fba75e96f9a0e0b1509d9d2fb5b392613f5ee396d01fc170
-  data.tar.gz: b56f91fc57521d3405db882b51b2051972132e8af36663e7eb468a75e0633c00
+  metadata.gz: 2b4f7f42c5f2d8764222099b8c490898dfc9c75d8b8223b09c13c065a9d1acc8
+  data.tar.gz: 0407b8f7e3a8e5acc55eb1c8121637ed97017208d868b82d5127f901318aa903
 SHA512:
-  metadata.gz: b18b4fe4c42e4fd445faf30afd2a74110d819b3518f23f7d7ad3803ca6f72bd58edaf4d02abef4991d1f2a171b0a808e15a3edb4a7cd4ac3521e70e9ef8bc343
-  data.tar.gz: 8ddce2288950a82afe6674151918e2c130eb331c37f0b6e32485343b85686a36b1381f44e098f8e2611f7b008b8e925bc2c5dce3f16b64a6100777db77cb3fe0
+  metadata.gz: 7dbe80ea179ff3215fcffd403a9750ef4d5cf3617249beb5b516222dcfa56b3ef65729d67c27d8bf4ae9d89ee70a934e7ea9d1e961406411e6c732fc63035048
+  data.tar.gz: cb047ce6a066dd8884dc13f6f2f0259120ce9071c91662aa58bea84bc77609eb252702af96bfbe2f3ec91ab78e3aa63cf02cf55908f105e678b9c28c5ce46091

data/lib/dead_code_detector/base_method_wrapper.rb

@@ -33,7 +33,9 @@ module DeadCodeDetector
 
     def refresh_cache
       clear_cache
-      DeadCodeDetector.config.storage.add(self.class.record_key(klass.name), default_methods)
+      if default_methods.any?
+        DeadCodeDetector.config.storage.add(self.class.record_key(klass.name), default_methods)
+      end
     end
 
     private

data/lib/dead_code_detector/class_method_wrapper.rb

@@ -53,12 +53,17 @@ module DeadCodeDetector
       return true if DeadCodeDetector.config.ignore_paths.nil?
       source_location = klass.method(method_name).source_location&.first
       return false if source_location.nil?
+      return false if source_location == "(eval)"
       source_location !~ DeadCodeDetector.config.ignore_paths
     end
 
     def owned_method?(method_name)
       original_method = klass.method(method_name)
-      klass.singleton_class <= original_method.owner && !(klass.superclass.singleton_class <= original_method.owner)
+      if klass.respond_to?(:superclass)
+        klass.singleton_class <= original_method.owner && !(klass.superclass.singleton_class <= original_method.owner)
+      else
+        klass.singleton_class <= original_method.owner
+      end
     end
 
   end

data/lib/dead_code_detector/configuration.rb

@@ -1,7 +1,7 @@
 module DeadCodeDetector
   class Configuration
 
-    attr_accessor :redis, :classes_to_monitor, :error_handler, :allowed, :cache_expiry, :ignore_paths
+    attr_accessor :redis, :classes_to_monitor, :error_handler, :allowed, :cache_expiry, :ignore_paths, :max_seconds_to_enable
 
     STORAGE_BACKENDS = {
       memory: Storage::MemoryBackend,
@@ -12,6 +12,7 @@ module DeadCodeDetector
       @allowed = true
       @classes_to_monitor = []
       @cache_expiry = 60 * 60 * 24 * 14
+      @max_seconds_to_enable = 1
     end
 
     def storage=(backend_type)

data/lib/dead_code_detector/initializer.rb

@@ -3,6 +3,8 @@ module DeadCodeDetector
 
     class << self
 
+      attr_accessor :fully_enabled, :last_enabled_class
+
       def refresh_caches
         DeadCodeDetector.config.classes_to_monitor.each do |klass|
           refresh_cache_for(klass)
@@ -10,7 +12,8 @@ module DeadCodeDetector
       end
 
       def refresh_cache_for(klass)
-        @enabled = false
+        self.fully_enabled = false
+        self.last_enabled_class = nil
         classes = [klass, *descendants_of(klass)]
         classes.each do |class_to_enable|
           cache_methods_for(class_to_enable)
@@ -34,13 +37,18 @@ module DeadCodeDetector
       end
 
       def enable_for_cached_classes!
-        return if @enabled
+        return if fully_enabled
         return unless allowed?
-        @enabled = true
-        cached_classes.each do |class_name|
+        classes = cached_classes.sort.to_a
+        starting_index = (classes.index(last_enabled_class) || -1) + 1
+        start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+        classes[starting_index..-1].each do |class_name|
           klass = Object.const_get(class_name) rescue nil
           enable(klass) if klass
+          self.last_enabled_class = class_name
+          return if Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time > DeadCodeDetector.config.max_seconds_to_enable
         end
+        self.fully_enabled = true
       end
 
       def allowed?
@@ -57,7 +65,9 @@ module DeadCodeDetector
 
       private
       def descendants_of(parent_class)
-        ObjectSpace.each_object(parent_class.singleton_class).select { |klass| klass < parent_class }
+        ObjectSpace.each_object(parent_class.singleton_class).select do |klass|
+          klass < parent_class && !klass.name.nil?
+        end
       end
 
       def cache_methods_for(klass)

data/lib/dead_code_detector/instance_method_wrapper.rb

@@ -47,12 +47,17 @@ module DeadCodeDetector
       return true if DeadCodeDetector.config.ignore_paths.nil?
       source_location = klass.instance_method(method_name).source_location&.first
       return false if source_location.nil?
+      return false if source_location == "(eval)"
       source_location !~ DeadCodeDetector.config.ignore_paths
     end
 
     def owned_method?(method_name)
       original_method = klass.instance_method(method_name)
-      klass <= original_method.owner && !(klass.superclass <= original_method.owner)
+      if klass.respond_to?(:superclass)
+        klass <= original_method.owner && !(klass.superclass <= original_method.owner)
+      else
+        klass <= original_method.owner
+      end
     end
 
   end

data/lib/dead_code_detector/version.rb

@@ -1,3 +1,3 @@
 module DeadCodeDetector
-  VERSION = "0.0.10"
+  VERSION = "0.0.11"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dead_code_detector
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors:
 - Kyle d'Oliveira
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-05-15 00:00:00.000000000 Z
+date: 2019-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler