puppet-ghostbuster 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e640683fdc2081827655309a5b74b8789f64fc0b
-  data.tar.gz: 35dc19ede6aead0efa888b524bf8cfa7e82f33a2
+  metadata.gz: 48db269b56e5bd75a9c1aa3349c578bf244957b3
+  data.tar.gz: caf14c62cef78d4f307302b5afc38d012f39824d
 SHA512:
-  metadata.gz: f371c80c10c50346e92f7f26a4d6c34aa986fb43a261475c873d9c05bfa028e107188cb642935270850a6a2253ae78c82d383afb583649b9fc2a96f09e4fe362
-  data.tar.gz: 5183186cf01b3c9bb5be41f3321113cd9b8df23d21139ac9295a11de7f8a245252616758cf56455af633b28777a1a3f39c13a773351ad0b68a20fca6f4c29c1a
+  metadata.gz: 0084173d38ed6fb21824f3ec750a70c9fe290a2f8a9c4d53bb45cbd9dbef64574d453f8dec6ef34ff9c85b900f04465dfed5cab9023b6771b507c8028d4d1ffe
+  data.tar.gz: ea9d82fd853ef6c0e20360729e700b3bf04d92dbdf4885217b5db56d9bb439f66c4b4ecc38dd255b87ecc0b649984df55e59b8ceae1b6f257c50887c1c016de9

data/lib/puppet-ghostbuster/version.rb

@@ -1,3 +1,3 @@
 class PuppetGhostbuster
-  VERSION = '0.0.5'
+  VERSION = '0.0.6'
 end

data/lib/puppet-ghostbuster.rb

@@ -49,34 +49,69 @@ class PuppetGhostbuster
     )
   end
 
-  def initialize
-    Puppet.initialize_settings
+  def find_unused_classes
     Dir["./**/manifests/**/*.pp"].each do |file|
       if c = File.readlines(file).grep(/^class\s+([^\s\(\{]+)/){$1}[0]
         class_name = c.split('::').map(&:capitalize).join('::')
         count = self.class.used_classes.select { |klass| klass == class_name }.size
-        puts "#{count} Class[#{class_name}]"
-      elsif d = File.readlines(file).grep(/^define\s+([^\s\(\{]+)/){$1}[0]
+        puts "Class #{class_name} not used" if count == 0
+      end
+    end
+  end
+
+  def find_unused_defines
+    Dir["./**/manifests/**/*.pp"].each do |file|
+      if d = File.readlines(file).grep(/^define\s+([^\s\(\{]+)/){$1}[0]
         define_name = d.split('::').map(&:capitalize).join('::')
         count = self.class.client.request('resources', [:'=', 'type', define_name]).data.size
-        puts "#{count} #{define_name}"
+        puts "Define #{define_name} not used" if count == 0
       end
     end
+  end
+
+  def find_unused_templates
     Dir['./**/templates/*'].each do |template|
       next unless File.file?(template)
       module_name, template_name = template.match(/.*\/([^\/]+)\/templates\/(.+)$/).captures
-      found = false
+      count = 0
       Dir["./**/manifests/**/*.pp"].each do |manifest|
         if match = manifest.match(/.*\/([^\/]+)\/manifests\/.+$/)
           manifest_module_name = match.captures[0]
-          found = File.readlines(manifest).grep(/["']\$\{module_name\}\/#{template_name}["']/).size > 0 if manifest_module_name == module_name
-          break if found
+          count += File.readlines(manifest).grep(/["']\$\{module_name\}\/#{template_name}["']/).size if manifest_module_name == module_name
         end
-        found = File.readlines(manifest).grep(/["']#{module_name}\/#{template_name}["']/).size > 0
-        break if found
+        count += File.readlines(manifest).grep(/["']#{module_name}\/#{template_name}["']/).size
       end
-      puts "#{template} not used" unless found
+      puts "Template #{template} not used" if count == 0
     end
   end
 
+  def find_unused_files
+    Dir['./**/files/*'].each do |file|
+      next unless File.file?(file)
+      module_name, file_name = file.match(/.*\/([^\/]+)\/files\/(.+)$/).captures
+      count = 0
+      Dir["."].each do |caller_file|
+        next unless File.file?(caller_file)
+        if caller_file =~ /\.pp$/
+          if match = manifest.match(/.*\/([^\/]+)\/manifests\/.+$/)
+            manifest_module_name = match.captures[0]
+            if manifest_module_name == module_name
+              count += File.readlines(caller_file).grep(/["']\$\{module_name\}\/#{file_name}["']/).size
+            end
+          end
+        end
+        count += File.readlines(caller_file).grep(/#{module_name}\/#{file_name}/).size
+      end
+      puts "File #{file} not used" if count == 0
+    end
+  end
+
+  def initialize
+    Puppet.initialize_settings
+    find_unused_classes
+    find_unused_defines
+    find_unused_templates
+    find_unused_files
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: puppet-ghostbuster
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Camptocamp