guard-puppet-lint 0.1.1 → 0.2.0

data/Gemfile

@@ -11,5 +11,6 @@ group :development do
   #gem "bundler", "> 1.0.0"
   gem "jeweler", "~> 1.8.3"
   gem "rcov", ">= 0"
+  gem 'libnotify'
   gem "guard", ">= 0"
 end

data/Gemfile.lock

@@ -3,20 +3,21 @@ GEM
   specs:
     ffi (1.0.11)
     git (1.2.5)
-    guard (1.0.1)
+    guard (1.0.3)
       ffi (>= 0.5.0)
-      thor (~> 0.14.6)
+      thor (>= 0.14.6)
     jeweler (1.8.3)
       bundler (~> 1.0)
       git (>= 1.2.5)
       rake
       rdoc
-    json (1.6.6)
+    json (1.7.3)
+    libnotify (0.7.2)
     rake (0.9.2.2)
     rcov (1.0.0)
     rdoc (3.12)
       json (~> 1.4)
-    thor (0.14.6)
+    thor (0.15.2)
 
 PLATFORMS
   ruby
@@ -24,4 +25,5 @@ PLATFORMS
 DEPENDENCIES
   guard
   jeweler (~> 1.8.3)
+  libnotify
   rcov

data/VERSION

@@ -1 +1 @@
-0.1.1
+0.2.0

data/guard-puppet-lint.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "guard-puppet-lint"
-  s.version = "0.1.1"
+  s.version = "0.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Alister Bulman"]
-  s.date = "2012-04-02"
+  s.date = "2012-05-20"
   s.description = "As Puppet manifest files change, run puppet-lint on them"
   s.email = "abulman@gmail.com"
   s.extra_rdoc_files = [
@@ -43,15 +43,18 @@ Gem::Specification.new do |s|
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<libnotify>, [">= 0"])
       s.add_development_dependency(%q<guard>, [">= 0"])
     else
       s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
       s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<libnotify>, [">= 0"])
       s.add_dependency(%q<guard>, [">= 0"])
     end
   else
     s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
     s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<libnotify>, [">= 0"])
     s.add_dependency(%q<guard>, [">= 0"])
   end
 end

data/lib/guard-puppet-lint.rb

@@ -0,0 +1 @@
+require 'guard/puppet-lint'

data/lib/guard/puppet-lint.rb

@@ -1,12 +1,32 @@
 require 'guard'
 require 'guard/guard'
 require 'guard/watcher'
+require 'puppet-lint'
+
+class PuppetLint
+
+  attr_reader :messages
+
+  def clear_messages
+    @messages = []
+  end
+
+  def format_message(message)
+    @messages << (log_format % message)
+  end
+end
 
 module Guard
   class Puppetlint < Guard
 
     VERSION = '0.0.1'
 
+    def initialize(watchers = [], options = {})
+      options = { :syntax_check => true }.merge(options)
+      @linter = PuppetLint.new
+      super
+    end
+
     # Calls #run_all if the :all_on_start option is present.
     def start
       run_all if options[:all_on_start]
@@ -17,17 +37,41 @@ module Guard
       run_on_change(Watcher.match_files(self, Dir.glob('{,**/}*{,.*}').uniq))
     end
 
+    def prepend_filename(msg, file)
+      if msg
+        msg.map {|x| "#{file}: #{x}"}
+      else
+        []
+      end
+    end
+
     # Print the result of the command(s), if there are results to be printed.
     def run_on_change(res)
-      puts res if res
-    end
+      messages = []
+      res.each do |file|
+        file = File.join( options[:watchdir].to_s,file ) if options[:watchdir]
 
-  end
+        if options[:syntax_check]
+          parser_messages = `puppet parser validate #{file} --color=false`.split("\n")
+          parser_messages.reject! { |s| s =~ /puppet help parser validate/ }
+          parser_messages.map! { |s| s.gsub 'err: Could not parse for environment production:', '' }
+
+          messages += prepend_filename(parser_messages, file)
+        end
 
-  class Dsl
-    # Easy method to display a notification
-    def n(msg, title='', image=nil)
-      Notifier.notify(msg, :title => title, :image => image)
+        @linter.file = file
+        @linter.clear_messages
+        @linter.run
+        linter_msg = @linter.messages.reject { |s| !options[:show_warnings] && s =~ /WARNING/ }
+        messages += prepend_filename(linter_msg, file)
+      end
+      if messages.empty?
+        messages = ["Files are ok:"] + res
+        image = :success
+      else
+        image = :failed
+      end
+      Notifier.notify( messages.join("\n"), :title => "Puppet lint", :image => image )
     end
   end
 end

data/lib/guard/puppet-lint/templates/Guardfile

@@ -1,7 +1,9 @@
 # Add files and commands to this file, like the example:
 #   watch(%r{file/path}) { `command(s)` }
 #
-guard 'puppet-lint' do
-  watch(/(.*).pp$/) {|m| `echo #{m[0]}; \
-	puppet-lint #{m[0]} --no-80chars-check --with-filename` }
+guard 'puppet-lint', :watchdir => Guard.options[:watchdir], 
+                     :show_warnings => false, 
+                     :syntax_check => true \
+do
+  watch(/(.*).pp$/)
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: guard-puppet-lint
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: 
   segments: 
   - 0
-  - 1
-  - 1
-  version: 0.1.1
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
 authors: 
 - Alister Bulman
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-04-02 00:00:00 Z
+date: 2012-05-20 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   version_requirements: &id001 !ruby/object:Gem::Requirement 
@@ -57,10 +57,24 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  name: guard
+  name: libnotify
   prerelease: false
   type: :development
   requirement: *id003
+- !ruby/object:Gem::Dependency 
+  version_requirements: &id004 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: guard
+  prerelease: false
+  type: :development
+  requirement: *id004
 description: As Puppet manifest files change, run puppet-lint on them
 email: abulman@gmail.com
 executables: []