From 81faff85fbd3bb008474446951a37b8c4c514827 Mon Sep 17 00:00:00 2001 From: Chris Hunt Date: Sun, 19 Jan 2014 12:09:25 -0800 Subject: [PATCH 1/3] Add mute as development dependency --- github-auth.gemspec | 1 + 1 file changed, 1 insertion(+) diff --git a/github-auth.gemspec b/github-auth.gemspec index a5bacac..1cbb05b 100644 --- a/github-auth.gemspec +++ b/github-auth.gemspec @@ -22,6 +22,7 @@ Gem::Specification.new do |spec| spec.add_development_dependency 'cane', '~> 2.6.1' spec.add_development_dependency 'cane-hashcheck', '~> 1.2.0' spec.add_development_dependency 'coveralls', '~> 0.7.0' + spec.add_development_dependency 'mute', '~> 1.0.0' spec.add_development_dependency 'pry', '~> 0.9.12.4' spec.add_development_dependency 'rake', '~> 10.1.1' spec.add_development_dependency 'rspec', '~> 2.14.1' From b1da523e0ddf66f466597a5d0bddd6191789dcb3 Mon Sep 17 00:00:00 2001 From: Chris Hunt Date: Sun, 19 Jan 2014 12:09:38 -0800 Subject: [PATCH 2/3] Require mute for all specs --- spec/spec_helper.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index 0894e9b..2d030fd 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -1,3 +1,4 @@ require 'pry' +require 'mute' require 'coveralls' Coveralls.wear! From ba515fbf1902409b851c056339cb668b76c144bd Mon Sep 17 00:00:00 2001 From: Chris Hunt Date: Sun, 19 Jan 2014 12:12:53 -0800 Subject: [PATCH 3/3] Use Mute::IO to mute cli acceptance specs --- spec/acceptance/github/auth/cli_spec.rb | 26 +++++++++++++++++-------- spec/support/capture_stdout.rb | 11 ----------- 2 files changed, 18 insertions(+), 19 deletions(-) delete mode 100644 spec/support/capture_stdout.rb diff --git a/spec/acceptance/github/auth/cli_spec.rb b/spec/acceptance/github/auth/cli_spec.rb index 889af5e..3df66a1 100644 --- a/spec/acceptance/github/auth/cli_spec.rb +++ b/spec/acceptance/github/auth/cli_spec.rb @@ -1,5 +1,4 @@ require 'spec_helper' -require 'support/capture_stdout' require 'support/mock_github_server' require 'github/auth' @@ -20,13 +19,17 @@ def cli(args = []) end it 'adds and removes keys from the keys file' do - cli %w(add --users=chrishunt) + Mute::IO.capture_stdout do + cli %w(add --users=chrishunt) + end keys_file.read.tap do |keys_file_content| keys.each { |key| expect(keys_file_content).to include key.to_s } end - cli %w(remove --users=chrishunt) + Mute::IO.capture_stdout do + cli %w(remove --users=chrishunt) + end expect(keys_file.read).to be_empty @@ -34,9 +37,11 @@ def cli(args = []) end it 'lists users from the keys file' do - cli %w(add --users=chrishunt) + Mute::IO.capture_stdout do + cli %w(add --users=chrishunt) + end - output = capture_stdout do + output = Mute::IO.capture_stdout do cli %w(list) end @@ -44,18 +49,23 @@ def cli(args = []) end it 'supports ssh commands' do - cli %w(add --users=chrishunt) << '--command=tmux attach' + Mute::IO.capture_stdout do + cli %w(add --users=chrishunt) << '--command=tmux attach' + end expect(keys_file.read).to include 'command="tmux attach"' keys_file.rewind - cli %w(remove --users=chrishunt) + + Mute::IO.capture_stdout do + cli %w(remove --users=chrishunt) + end expect(keys_file.read.strip).to be_empty end it 'prints version information' do - output = capture_stdout do + output = Mute::IO.capture_stdout do cli %w(version) end diff --git a/spec/support/capture_stdout.rb b/spec/support/capture_stdout.rb deleted file mode 100644 index 3892054..0000000 --- a/spec/support/capture_stdout.rb +++ /dev/null @@ -1,11 +0,0 @@ -require 'stringio' - -def capture_stdout - captured_output = StringIO.new - real_stdout = $stdout - $stdout = captured_output - yield - captured_output.string -ensure - $stdout = real_stdout -end