Merge pull request #56 from mgrecar/feature/html_using_launchy

Changed CLI to utilize Launchy to open the graph

.gitignore

@@ -1,3 +1,4 @@
 .DS_Store
 .bundle
 pkg
+Gemfile.lock

Gemfile

@@ -2,8 +2,6 @@ source 'https://rubygems.org'
 
 gemspec
 
-gem 'mixlib-cli'
-
 group :test do
   gem 'rspec'
 end

lib/sandi_meter/cli.rb

@@ -9,6 +9,7 @@
 require 'sandi_meter/html_generator'
 require 'yaml'
 require 'json'
+require 'launchy'
 
 module SandiMeter
   class CommandParser
@@ -111,7 +112,7 @@ def execute
             html_generator.generate_details!(cli.config[:path], data)
 
             index_html_path = File.join(cli.config[:path], 'sandi_meter/index.html')
-            system "open #{index_html_path}"
+            open_in_browser(index_html_path)
           else
             puts "WARNING!!! HTML mode works only if you scan folder."
           end
@@ -154,6 +155,10 @@ def version_info
         # stolen from gem 'bubs' :)
         "SandiMeter ".tr('A-Za-z1-90', 'Ⓐ-Ⓩⓐ-ⓩ①-⑨⓪').split('').join(' ') + SandiMeter::VERSION
       end
+
+      def open_in_browser(url)
+        Launchy.open(url)
+      end
     end
   end
 end

sandi_meter.gemspec

@@ -24,7 +24,9 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec", "~> 2.13"
 
   spec.add_runtime_dependency "mixlib-cli"
   spec.add_runtime_dependency "json"
+  spec.add_runtime_dependency "launchy"
 end

spec/cli_spec.rb

@@ -0,0 +1,28 @@
+require 'test_helper'
+require_relative '../lib/sandi_meter/cli'
+
+describe SandiMeter::CLI do
+  let(:cli) { SandiMeter::CLI }
+
+  describe '#execute' do
+    before do 
+      @original_argv = ARGV
+      ARGV.clear
+    end
+
+    after do 
+      ARGV.clear
+      ARGV.concat(@original_argv)
+    end
+
+    context 'with the graph flag passed in' do
+      before { ARGV.push('-g') }
+      after { ARGV.pop }
+
+      it 'opens the graph in a web browser' do
+        cli.should_receive(:open_in_browser)
+        expect { cli.execute }.to raise_error(SystemExit)
+      end
+    end
+  end
+end