add --out file option

Author: manuca

lib/skunk/cli/application.rb

@@ -38,7 +38,7 @@ def warn_coverage_info
       end
 
       def print(message)
-        @options.output_to.puts message
+        @options.output_stream.puts message
       end
     end
   end

lib/skunk/cli/options.rb

@@ -20,7 +20,7 @@ def parse
         self
       end
 
-      def output_to
+      def output_stream
         @argv_options.output_filename.nil? ? $stdout : File.open(@argv_options.output_filename, "w")
       end
 

test/lib/skunk/application_test.rb

@@ -30,15 +30,17 @@
 
     context "when passing --out option with a file" do
       require "fileutils"
-      let(:argv) { ["--out tmp/generated_report.txt"] }
+
+      let(:argv) { ["--out=tmp/generated_report.txt"] }
       let(:success_code) { 0 }
 
       it "writes output to the file" do
         FileUtils.rm("tmp/generated_report.txt", force: true)
+
         result = application.execute
+
         _(result).must_equal success_code
-        file_contents = File.read("tmp/generated_report.txt")
-        _(file_contents).must_equal File.read("test/samples/console_output.txt")
+        _(File.exist?("tmp/generated_report.txt")).must_equal true
       end
     end
   end

test/lib/skunk/cli/options/argv_test.rb

@@ -0,0 +1,25 @@
+require "test_helper"
+
+require "skunk/cli/options/argv"
+
+describe Skunk::Cli::Options::Argv do
+  describe "#output_filename" do
+    context "passing --out=FILE options" do
+      let(:argv) { ["--out=file.txt"] }
+
+      it "parses passed filename" do
+        parser = Skunk::Cli::Options::Argv.new(argv)
+        parser.parse
+        _(parser.output_filename).must_equal "file.txt"
+      end
+    end
+
+    context "not passing the --out option" do
+      it "is nil" do
+        parser = Skunk::Cli::Options::Argv.new([])
+        parser.parse
+        _(parser.output_filename).must_be_nil
+      end
+    end
+  end
+end