filewatcher 1.1.1 → 2.0.0.beta5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 805761615b226e14eb6431116a32bdc34e8b13cf
-  data.tar.gz: 4c1523220b4c242d33750e145091e1e2568766ac
+SHA256:
+  metadata.gz: c4552a19cac521012fb0cca2c232a14f501530262e02639ed017c0897e5f985f
+  data.tar.gz: f3d98c14badcbd439a9dca5ce289b40cc80aecb1c377d8ec08f80e76c398aaf5
 SHA512:
-  metadata.gz: 6d6f196a0ce277f920732a6c4ff0071b82e5b6eb3cf0683834041e973bb3a10a6c914aba6495d0bbbdfd83d241f983f697b45aa8ddd62ad9c973e3b4294fbffa
-  data.tar.gz: 7a75c0bb5e79efa44207f20052f8907790e2292195f65336f3d06303a8618ebe84fc658e32fec8333da461d7ccc979c74df061c735692ff7dd2647fac70b0f91
+  metadata.gz: 85e030c3e12237aff4dffddfb80fdf4b288b462a35e21a63d254e1aea4ca40cd2d5ba27cb21e39e3847f16936d4291398a106bd55a3ef40205f35d75c401603c
+  data.tar.gz: 7cec237639d7c760b963fe42b7c7d6ef40b989dde617308434e7d29037f73aad3f6fc87f971f60b4e47f420aec779e36ab76e5a6a2f27cf4f1c0849719b0b286

data/lib/filewatcher.rb

@@ -1,20 +1,26 @@
 # frozen_string_literal: true
 
+require 'logger'
 require_relative 'filewatcher/cycles'
+require_relative 'filewatcher/snapshots'
 
 # Simple file watcher. Detect changes in files and directories.
 #
-# Issues: Currently doesn't monitor changes in directorynames
+# Issues: Currently doesn't monitor changes in directory names
 class Filewatcher
   include Filewatcher::Cycles
+  include Filewatcher::Snapshots
 
   attr_accessor :interval
   attr_reader :keep_watching
 
-  def update_spinner(label)
-    return unless @show_spinner
-    @spinner ||= %w[\\ | / -]
-    print "#{' ' * 30}\r#{label}  #{@spinner.rotate!.first}\r"
+  class << self
+    def print_version
+      system 'ruby -v'
+      puts "Filewatcher #{self::VERSION}"
+
+      super if defined? super
+    end
   end
 
   def initialize(unexpanded_filenames, options = {})
@@ -23,9 +29,10 @@ class Filewatcher
     @keep_watching = false
     @pausing = false
     @immediate = options[:immediate]
-    @show_spinner = options[:spinner]
     @interval = options.fetch(:interval, 0.5)
-    @every = options[:every]
+    @logger = options.fetch(:logger, Logger.new($stdout, level: :info))
+
+    after_initialize unexpanded_filenames, options
   end
 
   def watch(&on_update)
@@ -37,28 +44,31 @@ class Filewatcher
 
     @on_update = on_update
     @keep_watching = true
-    yield('', '') if @immediate
+    yield({ '' => '' }) if @immediate
 
     main_cycle
 
-    @end_snapshot = mtime_snapshot
+    @end_snapshot = current_snapshot
     finalize(&on_update)
   end
 
   def pause
     @pausing = true
-    update_spinner('Initiating pause')
+
+    before_pause_sleep
+
     # Ensure we wait long enough to enter pause loop in #watch
     sleep @interval
   end
 
   def resume
-    if !@keep_watching || !@pausing
-      raise "Can't resume unless #watch and #pause were first called"
-    end
-    @last_snapshot = mtime_snapshot # resume with fresh snapshot
+    raise "Can't resume unless #watch and #pause were first called" if !@keep_watching || !@pausing
+
+    @last_snapshot = current_snapshot # resume with fresh snapshot
     @pausing = false
-    update_spinner('Resuming')
+
+    before_resume_sleep
+
     sleep @interval # Wait long enough to exit pause loop in #watch
   end
 
@@ -66,7 +76,9 @@ class Filewatcher
   # Used mainly in multi-threaded situations.
   def stop
     @keep_watching = false
-    update_spinner('Stopping')
+
+    after_stop
+
     nil
   end
 
@@ -74,65 +86,39 @@ class Filewatcher
   # current snapshot are dealt with
   def finalize(&on_update)
     on_update = @on_update unless block_given?
-    while filesystem_updated?(@end_snapshot || mtime_snapshot)
-      update_spinner('Finalizing')
+
+    while file_system_updated?(@end_snapshot || current_snapshot)
+      finalizing
       trigger_changes(on_update)
     end
-    @end_snapshot = nil
-  end
 
-  def last_found_filenames
-    last_snapshot.keys
+    @end_snapshot = nil
   end
 
   private
 
-  def last_snapshot
-    @last_snapshot ||= mtime_snapshot
+  def debug(data)
+    @logger.debug "Thread ##{Thread.current.object_id} #{data}"
   end
 
-  # Takes a snapshot of the current status of watched files.
-  # (Allows avoidance of potential race condition during #finalize)
-  def mtime_snapshot
-    snapshot = {}
-    filenames = expand_directories(@unexpanded_filenames)
-
-    # Remove files in the exclude filenames list
-    filenames -= expand_directories(@unexpanded_excluded_filenames)
-
-    filenames.each do |filename|
-      mtime = File.exist?(filename) ? File.mtime(filename) : Time.new(0)
-      snapshot[filename] = mtime
-    end
-    snapshot
+  def after_initialize(*)
+    super if defined?(super)
   end
 
-  def filesystem_updated?(snapshot = mtime_snapshot)
-    @changes = {}
-
-    (snapshot.to_a - last_snapshot.to_a).each do |file, _mtime|
-      @changes[file] = last_snapshot[file] ? :updated : :created
-    end
+  def before_pause_sleep
+    super if defined?(super)
+  end
 
-    (last_snapshot.keys - snapshot.keys).each do |file|
-      @changes[file] = :deleted
-    end
+  def before_resume_sleep
+    super if defined?(super)
+  end
 
-    @last_snapshot = snapshot
-    @changes.any?
+  def after_stop
+    super if defined?(super)
   end
 
-  def expand_directories(patterns)
-    patterns = Array(patterns) unless patterns.is_a? Array
-    expanded_patterns = patterns.map do |pattern|
-      pattern = File.expand_path(pattern)
-      Dir[
-        File.directory?(pattern) ? File.join(pattern, '**', '*') : pattern
-      ]
-    end
-    expanded_patterns.flatten!
-    expanded_patterns.uniq!
-    expanded_patterns
+  def finalizing
+    super if defined?(super)
   end
 end
 

data/lib/filewatcher/cycles.rb

@@ -7,7 +7,7 @@ class Filewatcher
 
     def main_cycle
       while @keep_watching
-        @end_snapshot = mtime_snapshot if @pausing
+        @end_snapshot = current_snapshot if @pausing
 
         pausing_cycle
 
@@ -21,27 +21,36 @@ class Filewatcher
 
     def pausing_cycle
       while @keep_watching && @pausing
-        update_spinner('Pausing')
+        before_pausing_sleep
+
         sleep @interval
       end
     end
 
+    def before_pausing_sleep
+      super if defined?(super)
+    end
+
     def watching_cycle
-      while @keep_watching && !filesystem_updated? && !@pausing
-        update_spinner('Watching')
+      @last_snapshot ||= current_snapshot
+      loop do
+        before_watching_sleep
+
+        debug "#{__method__} sleep #{@interval}"
         sleep @interval
+        break if !@keep_watching || file_system_updated? || @pausing
       end
     end
 
+    def before_watching_sleep
+      super if defined?(super)
+    end
+
     def trigger_changes(on_update = @on_update)
-      thread = Thread.new do
-        changes = @every ? @changes : @changes.first(1)
-        changes.each do |filename, event|
-          on_update.call(filename, event)
-        end
-        @changes.clear
-      end
-      thread.join
+      debug __method__
+      on_update.call(@changes.dup) unless @changes.empty?
+      @changes.clear
+      debug '@changes cleared'
     end
   end
 end

data/lib/filewatcher/snapshot.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+
+class Filewatcher
+  # Class for snapshots of file system
+  class Snapshot
+    extend Forwardable
+    def_delegators :@data, :[], :each, :keys
+
+    def initialize(filenames)
+      @data = filenames.each_with_object({}) do |filename, data|
+        data[filename] = SnapshotFile.new(filename)
+      end
+    end
+
+    def -(other)
+      changes = {}
+
+      each do |filename, snapshot_file|
+        changes[filename] = snapshot_file - other[filename]
+      end
+
+      other.each do |filename, _snapshot_file|
+        changes[filename] = :deleted unless self[filename]
+      end
+
+      changes.tap(&:compact!)
+    end
+
+    # Class for one file from snapshot
+    class SnapshotFile
+      STATS = %i[mtime].freeze
+
+      attr_reader(*STATS)
+
+      def initialize(filename)
+        @filename = filename
+        STATS.each do |stat|
+          time = File.public_send(stat, filename) if File.exist?(filename)
+          instance_variable_set :"@#{stat}", time || Time.new(0)
+        end
+      end
+
+      def -(other)
+        if other.nil?
+          :created
+        elsif other.mtime < mtime
+          :updated
+        end
+      end
+
+      def inspect
+        <<~OUTPUT
+          #<Filewatcher::Snapshot::SnapshotFile:#{object_id}
+            @filename=#{@filename.inspect}, mtime=#{mtime.strftime('%F %T.%9N').inspect}
+          >
+        OUTPUT
+      end
+    end
+
+    private_constant :SnapshotFile
+  end
+end

data/lib/filewatcher/snapshots.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require_relative 'snapshot'
+
+# Helpers in Filewatcher class itself
+class Filewatcher
+  # Module for snapshot logic inside Filewatcher
+  module Snapshots
+    def found_filenames
+      current_snapshot.keys
+    end
+
+    private
+
+    # Takes a snapshot of the current status of watched files.
+    # (Allows avoidance of potential race condition during #finalize)
+    def current_snapshot
+      Filewatcher::Snapshot.new(
+        expand_directories(@unexpanded_filenames) -
+          expand_directories(@unexpanded_excluded_filenames)
+      )
+    end
+
+    def expand_directories(patterns)
+      patterns = Array(patterns) unless patterns.is_a? Array
+      expanded_patterns = patterns.map do |pattern|
+        pattern = File.expand_path(pattern)
+        Dir[
+          File.directory?(pattern) ? File.join(pattern, '**', '*') : pattern
+        ]
+      end
+      expanded_patterns.flatten!
+      expanded_patterns.uniq!
+      expanded_patterns
+    end
+
+    def file_system_updated?(snapshot = current_snapshot)
+      debug __method__
+
+      @changes = snapshot - @last_snapshot
+
+      @last_snapshot = snapshot
+
+      @changes.any?
+    end
+  end
+end

data/lib/filewatcher/spec_helper.rb

@@ -0,0 +1,89 @@
+# frozen_string_literal: true
+
+require 'logger'
+
+begin
+  require 'pry-byebug'
+rescue LoadError
+  nil
+end
+
+require_relative 'spec_helper/watch_run'
+
+class Filewatcher
+  ## Helper for common spec features between plugins
+  module SpecHelper
+    ENVIRONMENT_SPECS_COEFFICIENTS = {
+      -> { ENV['CI'] } => 1,
+      -> { RUBY_PLATFORM == 'java' } => 1,
+      -> { Gem::Platform.local.os == 'darwin' } => 1
+    }.freeze
+
+    def logger
+      @logger ||= Logger.new($stdout, level: :debug)
+    end
+
+    def environment_specs_coefficients
+      ENVIRONMENT_SPECS_COEFFICIENTS
+    end
+
+    def wait(seconds: 1, interval: 1, &block)
+      environment_specs_coefficients.each do |condition, coefficient|
+        next unless instance_exec(&condition)
+
+        interval *= coefficient
+        seconds *= coefficient
+      end
+
+      if block
+        wait_with_block seconds, interval, &block
+      else
+        wait_without_block seconds
+      end
+    end
+
+    def wait_with_block(seconds, interval, &_block)
+      (seconds / interval).ceil.times do
+        break if yield
+
+        debug "sleep interval #{interval}"
+        sleep interval
+      end
+    end
+
+    def wait_without_block(seconds)
+      debug "sleep without intervals #{seconds}"
+      sleep seconds
+    end
+
+    def debug(string)
+      logger.debug "Thread ##{Thread.current.object_id} #{string}"
+    end
+
+    def system_stat(filename)
+      case (host_os = RbConfig::CONFIG['host_os'])
+      when /linux(-gnu)?/
+        `stat --printf 'Modification: %y, Change: %z\n' #{filename}`
+      when /darwin\d*/
+        `stat #{filename}`
+      when *Gem::WIN_PATTERNS
+        system_stat_windows filename
+      else
+        "Unknown OS `#{host_os}` for system's `stat` command"
+      end
+    end
+
+    def system_stat_windows(filename)
+      filename = filename.gsub('/', '\\\\\\')
+      properties = 'CreationDate,InstallDate,LastModified,LastAccessed'
+      command = "wmic datafile where Name=\"#{filename}\" get #{properties}"
+      # debug command
+      `#{command}`
+    end
+
+    ## https://github.com/rubocop/ruby-style-guide/issues/556#issuecomment-828672008
+    # rubocop:disable Style/ModuleFunction
+    extend self
+    # rubocop:enable Style/ModuleFunction
+  end
+end