multisync 0.3.7 → 0.4.0

data/lib/multisync/definition/entity.rb

@@ -1,18 +1,19 @@
-
 class Multisync::Definition::Entity
-  
   include Multisync::Definition::Dsl
 
-  # The parent of the group
+  # The parent of the entity
   attr_reader :parent
-  
-  # The name of the group
+
+  # The name of the entity
   attr_reader :name
-  
-  # All members (groups or syncs) of this group
+
+  # The level of the entity
+  attr_reader :level
+
+  # All members (groups or syncs) of this entity
   attr_reader :members
-  
-  # Collected results after run as Hash
+
+  # Collected result after run
   #  {
   #    cmd: 'rsync --stats -v source destination',
   #    action: :run,
@@ -21,96 +22,82 @@ class Multisync::Definition::Entity
   #    stderr: '',
   #    skip_message: 'host not reachable',
   #  }
-  
+
   attr_reader :result
 
   def initialize parent, name, &block
     @members = []
-    @name = name.to_s
+    @name = name
     @parent = parent
+    @level = parent.level + 1
     parent.register self
     instance_eval(&block) if block_given?
     @result = {}
   end
-  
-  def to_h
-    {
-      fullname: fullname,
-      default: default?,
-      source: {
-        path: source,
-        description: source_description,
-        check: check_source?,
-      },
-      destination: {
-        path: destination,
-        description: destination_description,
-        check: check_destination?,
-      },
-      checks: checks,
-      rsync_options: rsync_options,
-    }
-  end
-  
+
   # Make the definition visitable
-  def accept visitor, level=0
-    visitor.visit self, level
+  def accept visitor
+    visitor.visit self
     members.map do |member|
-      member.accept visitor, level+1
+      member.accept visitor
     end
   end
-  
+
   def register member
     members << member
   end
-  
+
   # The name including all parents separated by "/"
   def fullname
-    [parent.fullname, name].join '/'
+    [parent.fullname, name].reject(&:empty?).join("/")
   end
-  
+
   # rsync source
   def source
     @from_value || parent.source
   end
-  
+
   def source_description
     @from_description || @from_value || parent.source_description
   end
-  
+
   # rsync destination
   def destination
     @to_value || parent.destination
   end
-  
+
   def destination_description
     @to_description || @to_value || parent.destination_description
   end
-  
+
   # rsync options
   def rsync_options
     opts = @rsync_options || []
     return opts if @rsync_options_mode == :override
     parent.rsync_options + opts
   end
-  
+
   # Is this group/sync defined as default
   def default?
     @default || parent.default?
   end
-  
+
+  def executeable?
+    members.none?
+  end
+
   # All checks from parent to child
   def checks
     (parent.checks + [@check]).compact
   end
-  
+
   # Should source's host or path be checked before sync?
   def check_source?
     @from_check.nil? ? parent.check_source? : @from_check
   end
-  
+
   # Should destination's host or path be checked before sync?
   def check_destination?
     @to_check.nil? ? parent.check_destination? : @to_check
   end
-end
+end

data/lib/multisync/definition/null.rb

@@ -1,14 +1,13 @@
-
 class Multisync::Definition::Null < Multisync::Definition::Entity
-  
   def initialize
+    @level = -1
   end
-  
+
   def register member
   end
 
   def fullname
-    nil
+    ""
   end
 
   def rsync_options
@@ -19,34 +18,33 @@ class Multisync::Definition::Null < Multisync::Definition::Entity
   def source
     raise "no source (from) defined"
   end
-  
+
   def source_description
-    ''
+    ""
   end
-  
+
   # to (destination) is a required option and should be set at least at root level
   def destination
     raise "no destination (to) defined"
   end
-  
+
   def destination_description
-    ''
+    ""
   end
-  
+
   def default?
     false
   end
-  
+
   def checks
     []
   end
 
   def check_source?
     false
-  end  
+  end
 
   def check_destination?
     false
   end
 end
-

data/lib/multisync/definition/template.rb

@@ -1,26 +1,25 @@
-
 class Multisync::Definition::Template
   include Multisync::Definition::Dsl
-  
+
   @registered = []
-  
+
   def self.register instance
     @registered << instance
   end
-  
+
   def self.lookup name
-    @registered.find {|instance| instance.name == name }
+    @registered.find { _1.name == name }
   end
-  
+
   # The name of the template
   attr_reader :name
-  
+
   # The block the template holds
   attr_reader :block
-  
+
   def initialize name, &block
     @name = name
     self.class.register self
     @block = block
   end
-end
+end

data/lib/multisync/definition.rb

@@ -1,7 +1,6 @@
-
 class Multisync::Definition
-  autoload :Dsl, 'multisync/definition/dsl'
-  autoload :Entity, 'multisync/definition/entity'
-  autoload :Null, 'multisync/definition/null'
-  autoload :Template, 'multisync/definition/template'
-end
+  autoload :Dsl, "multisync/definition/dsl"
+  autoload :Entity, "multisync/definition/entity"
+  autoload :Null, "multisync/definition/null"
+  autoload :Template, "multisync/definition/template"
+end

data/lib/multisync/list.rb

@@ -1,44 +1,47 @@
-
-require 'rainbow/ext/string'
-
 class Multisync::List
-  
+  include Multisync::Colors
+
   # Given catalog
   attr_reader :catalog
-  
+
   # Tasks
   attr_reader :tasks
 
-  def initialize catalog
-    @catalog = catalog
-    @tasks = []
+  def initialize tasks
+    @tasks = tasks
   end
-  
+
   def to_s
-    catalog.traverse self
-    table.to_s
+    "\n" + table.to_s
   end
-  
+
   def table
-    Terminal::Table.new(rows: tasks, style: table_style)
+    Terminal::Table.new(rows: rows, style: table_style)
   end
-  
-  def visit subject, level
-    if level > 0
-      tab = ''.ljust(2*(level-1), ' ')
-      default = subject.default? ? ' *' : ''
-      name = "#{tab}#{subject.name}#{default}"
-      tasks << [name, *description(subject).map(&:faint)]
-      # puts "#{name.ljust(32, ' ')}#{description(subject)}"
+
+  def rows
+    tasks.map do |task|
+      next unless task.level > 0
+
+      indent = "".ljust(2 * (task.level - 1), " ")
+      name = task.executeable? ? task.name : "#{task.name}#{as_note("/")}"
+      default = task.default? ? as_note(" *") : ""
+      [
+        [indent, name, default].join,
+        *descriptions(task)
+      ]
     end
   end
-  
-  def description subject
-    desc = [subject.source_description, subject.destination_description]
-    desc.any?(&:empty?) ? [] : [desc.first, ['--> ', desc.last].join]
+
+  def descriptions task
+    if [task.source_description, task.destination_description].any?(&:empty?)
+      ["", "", ""]
+    else
+      [task.source_description, "-->", task.destination_description].map(&method(:as_note))
+    end
   end
 
   def table_style
-    { border_top: false,  border_bottom: false, border_x: '–', border_y: '', border_i: '', padding_left: 0, padding_right: 3 }
+    {border_x: as_note("─"), border_y: "", border_i: "", border_top: false, border_bottom: false, padding_left: 0, padding_right: 3}
   end
 end

data/lib/multisync/rsync_stat.rb

@@ -1,7 +1,6 @@
 require "filesize"
 
 class Multisync::RsyncStat
-
   # Keep track of totals
   def self.total
     @total ||= Hash.new 0
@@ -31,18 +30,18 @@ class Multisync::RsyncStat
 
   # extracted returns a hash with labels as keys and extracted strings as values
   #   {
-  #     "Number of files" => "35,648", 
-  #     "Number of created files" => "2,120", 
+  #     "Number of files" => "35,648",
+  #     "Number of created files" => "2,120",
   #     ...
   #   }
   def extracted
-    @extraced ||= @output.scan(/(#{labels.join('|')}):\s+([,0-9]+)/).to_h
+    @extraced ||= @output.scan(/(#{labels.join("|")}):\s+([,0-9]+)/).to_h
   end
 
   # stats returns a hash with the follwing keys (and updates class total)
   #   {
-  #     "Number of files" => 35648, 
-  #     "Number of created files" => 2120, 
+  #     "Number of files" => 35648,
+  #     "Number of created files" => 2120,
   #     "Number of deleted files" => 37,
   #     "Number of regular files transferred" => 394,
   #     "Total file size" => 204936349,
@@ -81,12 +80,12 @@ class Multisync::RsyncStat
 
   def self.format_map
     {
-      'Number of files' => to_numbers,
-      'Number of created files' => to_numbers,
-      'Number of deleted files' => to_numbers,
-      'Number of regular files transferred' => to_numbers,
-      'Total file size' => to_filesize,
-      'Total transferred file size' => to_filesize,
+      "Number of files" => to_numbers,
+      "Number of created files" => to_numbers,
+      "Number of deleted files" => to_numbers,
+      "Number of regular files transferred" => to_numbers,
+      "Total file size" => to_filesize,
+      "Total transferred file size" => to_filesize
     }
   end
 
@@ -97,4 +96,4 @@ class Multisync::RsyncStat
   def self.to_filesize
     ->(x) { Filesize.new(x).pretty }
   end
-end
+end

data/lib/multisync/runtime.rb

@@ -1,82 +1,77 @@
-
-require 'mixlib/shellout'
+require "mixlib/shellout"
 
 class Multisync::Runtime
+  include Multisync::Colors
 
   # Runtime options
   #   dryrun: true|false
   #   show: true|false
   attr_reader :options
-  
+
   def initialize options
     @options = options
   end
-  
-  def dryrun?
-    options[:dryrun]
-  end
-  
-  def show_only?
-    options[:print]
-  end
-  
-  def quiet?
-    options[:quiet]
-  end
-  
-  def timeout
-    options[:timeout]
-  end
-  
+
+  def dryrun? = options[:dryrun]
+
+  def show_only? = options[:print]
+
+  def quiet? = options[:quiet]
+
+  def timeout = options[:timeout]
+
   def run sync
     rsync_options = sync.rsync_options.dup
-    rsync_options.unshift '--stats'
-    rsync_options.unshift '--verbose' unless quiet?
-    rsync_options.unshift '--dry-run' if dryrun?
+    rsync_options.unshift "--stats"
+    rsync_options.unshift "--verbose" unless quiet?
+    rsync_options.unshift "--dry-run" if dryrun?
 
     # escape path by hand, shellescape escapes also ~, but we want to keep its
     # special meaning for home, instead of passing it as literal char
-    source, destination = [sync.source, sync.destination].map {|path| path.gsub(/\s+/, '\\ ') }
-    cmd = "rsync #{rsync_options.join(' ')} #{source} #{destination}"
-    cmd_options = { timeout: timeout }
-    cmd_options.merge!({live_stdout: $stdout, live_stderr: $stderr}) unless quiet?
+    source, destination = [sync.source, sync.destination].map { _1.gsub(/\s+/, "\\ ") }
+    cmd = "rsync #{rsync_options.join(" ")} #{source} #{destination}"
+    cmd_options = {timeout: timeout}
+    unless quiet?
+      cmd_options[:live_stdout] = $stdout
+      cmd_options[:live_stderr] = $stderr
+    end
     rsync = Mixlib::ShellOut.new(cmd, cmd_options)
     sync.result[:cmd] = rsync.command
 
     unless quiet?
       puts
-      puts [sync.source_description, sync.destination_description].join(' --> ').color(:cyan)
+      puts as_main([sync.source_description, sync.destination_description].join(" --> "))
     end
-    
+
     # Perform all only_if checks, from top to bottom
     sync.checks.each do |check|
       next unless Mixlib::ShellOut.new(check[:cmd]).run_command.error?
 
-      puts check[:cmd] + ' (failed)'
-      puts "Skip: ".color(:yellow) + rsync.command
+      puts as_skipped("#{check[:cmd]} (failed)")
+      puts as_note("Skip: #{rsync.command}")
       sync.result[:action] = :skip
       sync.result[:skip_message] = check[:message]
-      return
+      return false
     end
-    
+
     # source check
-    if sync.check_source? && ! check_path(sync.source, :source)
-      puts "Source #{sync.source} is not accessible"
-      puts "Skip: ".color(:yellow) + rsync.command
+    if sync.check_source? && !check_path(sync.source, :source)
+      puts as_skipped("Source #{sync.source} is not accessible")
+      puts as_note("Skip: #{rsync.command}")
       sync.result[:action] = :skip
       sync.result[:skip_message] = "Source is not accessible"
       return
     end
-    
+
     # target check
-    if sync.check_destination? && ! check_path(sync.destination, :destination)
-      puts "Destination #{sync.destination} is not accessible"
-      puts "Skip: ".color(:yellow) + rsync.command
+    if sync.check_destination? && !check_path(sync.destination, :destination)
+      puts as_skipped("Destination #{sync.destination} is not accessible")
+      puts as_note("Skip: #{rsync.command}")
       sync.result[:action] = :skip
       sync.result[:skip_message] = "Destination is not accessible"
       return
     end
-      
+
     if show_only?
       puts rsync.command
     else
@@ -88,20 +83,20 @@ class Multisync::Runtime
       sync.result[:stderr] = rsync.stderr
     end
   end
-  
+
   # checks a path
   # if path includes a host, the reachability of the host will be checked
   # the existence of the remote path will not be checked
   # if path is a local source path, its existence will be checked
   # if path is a local destination path, the existence of the parent will be checked
   def check_path path, type = :source
-    if path.include? ':'
-      host = path.split(':').first.split('@').last
+    if path.include? ":"
+      host = path.split(":").first.split("@").last
       Mixlib::ShellOut.new("ping -o -t 1 #{host}").run_command.status.success?
     else
-      abs_path = File.expand_path path
-      abs_path = File.dirname abs_path if type == :destination
-      File.exist? abs_path
+      File.expand_path(path)
+        .then { (type == :destination) ? File.dirname(_1) : _1 }
+        .then { File.exist? _1 }
     end
   end
-end
+end

data/lib/multisync/selector.rb

@@ -1,36 +1,58 @@
-
 class Multisync::Selector
-  
   # Given catalog
   attr_reader :catalog
-  
-  # Given set names
-  attr_reader :sets
-  
+
+  # Given queries
+  attr_reader :queries
+
   # Selected tasks
-  attr_reader :result
+  attr_reader :results
 
-  def initialize catalog, sets
+  def initialize catalog, queries
     @catalog = catalog
-    @sets = sets
-    @result = []
+    @queries = queries
+    @results = []
+    @all_subjects = []
+    @subjects_by_name = []
   end
-  
-  def tasks
+
+  def tasks parents: false
     catalog.traverse self
-    result
+    parents ? selected_with_parents : selected
   end
-  
-  def visit subject, _level
-    result << subject if selected?(subject)
+
+  def visit subject
+    results << subject
+  end
+
+  def selected_with_parents
+    @selected_with_parents ||= results.select { selected_or_parent_of_selected? _1 }
+  end
+
+  def selected_or_parent_of_selected? subject
+    !subject.fullname.empty? &&
+      selected_fullnames.any? { %r{^#{subject.fullname}(?:/|$)}.match _1 }
   end
-  
+
+  def selected_fullnames
+    @selected_fullnames ||= selected.map(&:fullname)
+  end
+
+  def selected
+    @selected ||= results.select { selected? _1 }
+  end
+
   def selected? subject
+    # return only subjects with a fullname
+    return false if subject.fullname.empty?
+
+    # no queries defined, but subject is in the default set
+    return true if queries.empty? && subject.default?
+
     # only return the leaves of the definition tree
-    return false unless subject.members.empty?
-    # no sets defined, but subject is in the default set
-    return true if sets.empty? && subject.default?
-    # subject matches any of the given sets
-    sets.any? {|set| /\b#{set}\b/.match subject.fullname }
+    # return false unless subject.members.any?
+
+    # subject matches any of the given queries
+    queries.any? { /\b#{_1}\b/.match subject.fullname }
   end
 end