removing unnecessary puts

Author: zappy-shu

lib/puppet_x/transforms/xdt_transform_insert_after.rb

@@ -17,7 +17,6 @@ def transform(source_node, transform_node)
 
         xpath = @transform_arguments[0]
         insert_after_nodes = source_node.xpath(@transform_arguments[0])
-        puts insert_after_nodes
         if insert_after_nodes.length == 0
             warn "Cannot find xpath '#{xpath}'"
             return

lib/puppet_x/xdt_transformer.rb

@@ -8,7 +8,6 @@ def initialize(source_doc, transform_doc)
         @source_doc = source_doc
         @transform_doc = transform_doc
         @doc_to_transform = source_doc.clone
-        #puts @doc_to_transform
 
         @xdt_namespace = XdtNamespace.new
         @locator_factory = XdtLocatorFactory.new

spec/acceptance/puppet_x/xdt_transformer_spec.rb

@@ -26,9 +26,8 @@ def compare_elements(expected, actual)
     expect(actual.name).to eql(expected.name)
     compare_namespaces(expected.namespace, actual.namespace)
     compare_attributes(expected, actual)
-    
     expect(actual.elements.length).to eql(expected.elements.length)
-        expected.elements.each_with_index do |expected_child, i|
+    expected.elements.each_with_index do |expected_child, i|
         compare_elements(expected_child, actual.elements[i])
     end
 end
@@ -41,6 +40,7 @@ def compare_elements(expected, actual)
                 transform_doc = Nokogiri::XML(File.read("#{Dir.pwd}/spec/resources/transform_only/transform.xml"))
                 expected_doc = Nokogiri::XML(File.read("#{Dir.pwd}/spec/resources/transform_only/expected.xml"))
                 transformed_doc = XdtTransformer.new(source_doc, transform_doc).transform
+                
                 File.write("#{Dir.pwd}/spec/resources/transform_only/.temp.actual.xml", transformed_doc.to_s)
                 compare_elements(expected_doc.root, transformed_doc.root)
             end

spec/resources/expected.xml

@@ -1,18 +1,19 @@
+<?xml version="1.0"?>
 <root>
-	<insertafter a="aaa" b="bbb" />
-	<insertafter_1 a="aaaa" c="cccc" />
-	<insertbefore_2 a="aaaa" c="cccc" />
-	<insertbefore a="aaa" b="bbb" />
-	<removeattributes />
+	<insertafter a="aaa" b="bbb"/>
+	<insertafter_1 a="aaaa" c="cccc"/>
+	<insertbefore_2 a="aaaa" c="cccc"/>
+	<insertbefore a="aaa" b="bbb"/>
+	<removeattributes/>
 	<replace a="aaaa" c="cccc">
-		<replace_1 a="aaaa" c="cccc" />
-		<replace_3 />
+		<replace_1 a="aaaa" c="cccc"/>
+		<replace_3/>
 	</replace>
 	<setattributes a="aaaa" b="bbb" c="cccc">
-		<setattributes_1 />
+		<setattributes_1/>
 	</setattributes>
 	<setallattributes a="aaaa" b="bbb" c="cccc">
-		<setallattributes_1 />
+		<setallattributes_1/>
 	</setallattributes>
-	<insert a="aaaa" c="cccc" />
+	<insert a="aaaa" c="cccc"/>
 </root>

spec/resources/source.xml

@@ -1,12 +1,13 @@
+<?xml version="1.0"?>
 <root>
 	<a:parent_with_prefix xmlns:a="http://prefix_namespace/">
-		<a:child_with_prefix b="aaaa" />
-		<child_without_prefix b="bbbb" />
+		<a:child_with_prefix b="aaaa"/>
+		<child_without_prefix b="bbbb"/>
 	</a:parent_with_prefix>
 	<parent_with_default_namespace xmlns="http://default_namespace/">
-		<child_with_default_namespace a="aaaa" />
+		<child_with_default_namespace a="aaaa"/>
 	</parent_with_default_namespace>
 	<parent_without_namespace>
-		<child_without_namespace a="aaaa" />
+		<child_without_namespace a="aaaa"/>
 	</parent_without_namespace>
 </root>