guard-copy 0.0.1 → 0.0.2

data/lib/guard/copy/version.rb

@@ -1,5 +1,5 @@
 module Guard
   module CopyVersion
-    VERSION = "0.0.1"
+    VERSION = "0.0.2"
   end
 end

data/lib/guard/copy.rb

@@ -1,24 +1,35 @@
 require 'guard'
 require 'guard/guard'
 require 'guard/copy/version'
+require 'fileutils'
 
 module Guard
   class Copy < Guard
 
+    attr_reader :targets
+
     # Initialize a Guard.
     # @param [Array<Guard::Watcher>] watchers the Guard file watchers
     # @param [Hash] options the custom Guard options
     def initialize(watchers = [], options = {})
       # watchers are currently ignored
       watchers << ::Guard::Watcher.new(%r{#{options[:from]}/.*})
-      @targets = options[:to].is_a?(Array) ? options[:to] : [options[:to]]
-      UI.info("Guard::Copy will copy files from '#{options[:from]}' to '#{@targets.first}'")
+      options[:to] = Array(options[:to]).freeze
       super
     end
 
     # Call once when Guard starts. Please override initialize method to init stuff.
     # @raise [:task_has_failed] when start has failed
     def start
+      validate_from
+      validate_to
+      @targets = resolve_targets
+      if @targets.any?
+        UI.info("Guard::Copy will copy files from:")
+        UI.info("  #{options[:from]}")
+        UI.info("to:")
+        @targets.each { |target| UI.info("  #{target}") }
+      end
     end
 
     # Called when just `enter` is pressed
@@ -31,41 +42,82 @@ module Guard
     # @param [Array<String>] paths the changes files or paths
     # @raise [:task_has_failed] when run_on_change has failed
     def run_on_change(paths)
+      validate_targets
       paths.each do |from_path|
         @targets.each do |target|
-          target_dirs = target_dirs(target)
-          if target_dirs.empty?
-            UI.error("'#{target}' does not match any directories")
-          else
-            copy(from_path, target_dirs)
-          end
+          to_path = from_path.sub(@options[:from], target)
+          validate_to_path(to_path)
+          FileUtils.cp(from_path, to_path)
         end
       end
     end
 
     # Called on file(s) deletions that the Guard watches.
     # @param [Array<String>] paths the deleted files or paths
-    # @raise [:task_has_failed] when run_on_change has failed
+    # @raise [:task_has_failed] when run_on_deletion has failed
     def run_on_deletion(paths)
     end
 
     private
 
-    def target_dirs(target)
-      if @options[:glob] == :newest
-        Dir[target].sort_by { |f| File.mtime(f) }.last(1)
-      else
-        Dir[target]
+    def validate_from
+      unless options[:from]
+        UI.error('Guard::Copy - :from option is required')
+        throw :task_has_failed
+      end
+      if File.file?(options[:from])
+        UI.error("Guard::Copy - '#{options[:from]}' is a file and must be a directory")
+        throw :task_has_failed
+      end
+      unless File.directory?(options[:from])
+        UI.error('Guard::Copy - :from option does not contain a valid directory')
+        throw :task_has_failed
       end
     end
 
-    def copy(from_path, target_dirs)
-      target_dirs.each do |target_dir|
-        if File.directory?(target_dir)
-          to_path = from_path.sub(@options[:from], target_dir)
-          FileUtils.cp(from_path, to_path)
-        else
-          UI.error("'#{target_dir}' is not a directory")
+    def validate_to
+      if options[:to].empty?
+        UI.error('Guard::Copy - :to option is required')
+        throw :task_has_failed
+      end
+      if options[:to].include?(options[:from])
+        UI.error('Guard::Copy - :to must not include :from')
+        throw :task_has_failed
+      end
+    end
+
+    def validate_targets
+      if @targets.empty?
+        UI.error('Guard::Copy - cannot copy, no valid :to directories')
+        throw :task_has_failed
+      end
+    end
+
+    def validate_to_path(to_path)
+      to_dir = File.dirname(to_path)
+      unless File.directory?(to_dir)
+        UI.error('Guard::Copy - cannot copy, directory path does not exist:')
+        UI.error("  #{to_dir}")
+        throw :task_has_failed
+      end
+    end
+
+    def resolve_targets
+      Array.new.tap do |targets|
+        options[:to].each do |to|
+          if @options[:glob] == :newest
+            dirs = Dir[to].sort_by { |f| File.mtime(f) }.last(1)
+          else
+            dirs = Dir[to]
+          end
+          if dirs.any?
+            #dirs.each do |dir|
+              #throw :task_has_failed if File.file?(dir)
+            #end
+            targets.concat(dirs)
+          else
+            UI.warning("Guard::Copy - '#{to}' does not match a valid directory")
+          end
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: guard-copy
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-15 00:00:00.000000000 Z
+date: 2012-05-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard
@@ -91,6 +91,54 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.7.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.7.2
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.4.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.4.0
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.11.4
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.11.4
 description: Guard::Copy automatically copies files.
 email:
 - marc.schwieterman@gmail.com
@@ -115,12 +163,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 964757571042390819
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 964757571042390819
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24