response_mate 0.2.1 → 0.2.2

data/.gitignore

@@ -8,7 +8,7 @@ tmp
 .rbenv*
 
 spec/source/responses
-spec/source/responses.last_recording
+spec/source/other_output_dir
 
 output
 oauth.yml

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    response_mate (0.2.0)
+    response_mate (0.2.2)
       activesupport
       addressable
       awesome_print

data/lib/response_mate/cli.rb

@@ -6,6 +6,7 @@ module ResponseMate
 
     desc 'record', 'Perform requests and record their output'
     method_option :requests_manifest, aliases: '-r'
+    method_option :output_dir, aliases: '-o', type: :string
     method_option :keys, aliases: '-k', type: :array, default: []
     def record
       ResponseMate::Commands::Record.new(args, options).run

data/lib/response_mate/commands/record.rb

@@ -9,13 +9,10 @@ module ResponseMate
         manifest = ResponseMate::Manifest.new(options[:requests_manifest], environment)
 
         options[:manifest] = manifest
+
         recorder = ResponseMate::Recorder.new(options)
 
         recorder.record(options[:keys])
-
-        File.open(ResponseMate.configuration.output_dir + '.last_recording', 'w') do |f|
-          f << Time.current
-        end
       end
     end
   end

data/lib/response_mate/recorder.rb

@@ -3,11 +3,12 @@
 module ResponseMate
   # Handles recording requests
   class Recorder
-    attr_accessor :conn, :manifest, :keys
+    attr_accessor :conn, :manifest, :keys, :output_dir
 
     def initialize(args = {})
       @manifest = args[:manifest]
       @conn = ResponseMate::Connection.new
+      @output_dir = args[:output_dir]
     end
 
     def record(keys)
@@ -22,7 +23,11 @@ module ResponseMate
       meta = request.meta
       puts request.to_cli_format
 
-      ResponseMate::Tape.new.write(request.key, request, conn.fetch(request), meta)
+      ResponseMate::Tape.new.write(request.key,
+                                   request,
+                                   conn.fetch(request),
+                                   meta,
+                                   output_dir)
     end
   end
 end

data/lib/response_mate/tape.rb

@@ -1,8 +1,9 @@
 # encoding: utf-8
 
 class ResponseMate::Tape
-  def write(key, request, response, meta = {})
-    output_dir = ResponseMate.configuration.output_dir
+  def write(key, request, response, meta = {}, output_dir = nil)
+    output_dir = output_dir || ResponseMate.configuration.output_dir
+
     output_path = File.join output_dir, "#{key}.yml"
 
     File.open(output_path, 'w') do |f|

data/lib/response_mate/version.rb

@@ -1,5 +1,5 @@
 # coding: utf-8
 
 module ResponseMate
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 end

data/spec/lib/response_mate/commands/record_spec.rb

@@ -18,6 +18,50 @@ describe ResponseMate::Commands::Record do
       end
     end
 
+    context 'with output_dir option specified' do
+      let(:other_output_dir) do
+        File.expand_path('spec/source/other_output_dir')
+      end
+
+      let(:cmd_with_output_dir) do
+        quietly do
+          ResponseMate::Commands::Record.new([], {
+            keys: [],
+            output_dir: [other_output_dir]
+          }).run
+        end
+      end
+
+      context 'when the specified directory exists' do
+        let(:output_files) { ->{ Dir[other_output_dir + '/*'] } }
+
+        before { cmd_with_output_dir }
+        after { output_files.call.each { |file| File.delete(file) } }
+
+        it 'created the tapes in the specified directory' do
+          expect(output_files.call).to have_exactly(2).items
+        end
+      end
+
+      context 'when the specified directory does not exist' do
+        let(:other_output_dir) do
+          File.expand_path('spec/source/i_do_not_exist')
+        end
+
+        it 'raises ResponeMate::OutputDirError' do
+          expect { cmd_with_output_dir }.to raise_error(ResponseMate::OutputDirError)
+        end
+      end
+    end
+
+    context 'with output_dir option unspecified' do
+      it 'creates the tapes in the default output directory' do
+        quietly { ResponseMate::Commands::Record.new([], { keys: [] }).run }
+
+        expect(output_files.call).to have_exactly(2).items
+      end
+    end
+
     context 'with keys option specified' do
       context 'when the requested key exists' do
         before do

data/spec/lib/response_mate/tape_spec.rb

@@ -28,15 +28,45 @@ describe ResponseMate::Tape do
   describe '#write' do
     let(:tape) { YAML.load_file(output_files.call.last) }
 
-    before do
-      ResponseMate::Tape.new.write(key, request, response, meta)
+    let(:output_dir) { nil }
+
+    let(:write) do
+      ResponseMate::Tape.new.write(key, request, response, meta, output_dir)
     end
 
+    subject { write }
+
     it 'creates a new tape with key parameter as the filename' do
+      subject
       expect(File.basename(output_files.call.last)).to eq("#{key}.yml")
     end
 
+    describe 'tape location' do
+      context 'when the specified output_dir exists' do
+        let(:output_dir) { File.expand_path('./spec/source/other_output_dir') }
+        let(:output_files) { ->{ Dir[output_dir + '/*'] } }
+
+        after { output_files.call.each { |file| File.delete(file) } }
+
+        it 'is inside the specified output_dir' do
+          subject
+          expect(tape).to be
+        end
+      end
+
+      context 'when the specified output_dir does not exist' do
+        let(:output_dir) do
+          File.expand_path('spec/source/i_do_not_exist')
+        end
+
+        it 'raises ResponeMate::OutputDirError' do
+          expect { subject }.to raise_error(ResponseMate::OutputDirError)
+        end
+      end
+    end
+
     describe 'the created tape' do
+      before { write }
       subject { tape }
 
       it 'is valid YAML' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: response_mate
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-14 00:00:00.000000000 Z
+date: 2014-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -246,6 +246,7 @@ files:
 - spec/lib/response_mate/tape_spec.rb
 - spec/lib/response_mate_spec.rb
 - spec/source/environment.yml
+- spec/source/other_output_dir/.gitkeep
 - spec/source/requests.yml
 - spec/source/responses/.gitkeep
 - spec/spec_helper.rb
@@ -287,6 +288,7 @@ test_files:
 - spec/lib/response_mate/tape_spec.rb
 - spec/lib/response_mate_spec.rb
 - spec/source/environment.yml
+- spec/source/other_output_dir/.gitkeep
 - spec/source/requests.yml
 - spec/source/responses/.gitkeep
 - spec/spec_helper.rb