Merge pull request #15 from lsst-it/IT-3965/missing-newline

fix #verbose_list not appending a new line when items arg is empty

exe/foreman_envsync

@@ -31,9 +31,9 @@ def verbose_list(msg, items)
   return unless @options[:verbose] && !items.nil?
 
   printf(msg, items.count)
+  puts
   return if items.empty?
 
-  puts
   # do not attempt to sorry Array of Hashes
   if items.is_a?(Array) && items.first.is_a?(Hash)
     puts "#{YAML.dump(items)}\n"

lib/foreman_envsync/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanEnvsync
-  VERSION = "1.4.0"
+  VERSION = "1.4.1"
 end

spec/unit/foreman_envsync_verbose_list_spec.rb

@@ -7,7 +7,7 @@
     before { @options = { verbose: true } }
 
     context "with array of hash" do
-      let(:aofh) { [{ a: 1 }, { b: 2 }] }
+      let(:items) { [{ a: 1 }, { b: 2 }] }
       let(:foo_output) do
         <<~FOO
           foo
@@ -18,7 +18,18 @@
         FOO
       end
 
-      it { expect { verbose_list("foo", aofh) }.to output(foo_output).to_stdout }
+      it { expect { verbose_list("foo", items) }.to output(foo_output).to_stdout }
+    end
+
+    context "with empty array" do
+      let(:items) { [] }
+      let(:foo_output) do
+        <<~FOO
+          foo
+        FOO
+      end
+
+      it { expect { verbose_list("foo", items) }.to output(foo_output).to_stdout }
     end
   end
 end