puffing-billy 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5097ebc685dd2798f2c9e7dbfed3b3582c790b77fc07f94623700c077e143ce7
-  data.tar.gz: c30e447c5ed810078fd484151d0c7e3f58f081ec1fe08e093ce3b9f626894a9b
+  metadata.gz: d6e9bb7a75fe76a33e705c89f01e30fa987d1df3f4cfee135bb53393f28a4398
+  data.tar.gz: 47e189ad311796a8a192479fc60034d2651f1879d8e17c42afb174f6296a491f
 SHA512:
-  metadata.gz: 80c4f30830aeeaccb1da191d4035ff98149bb297410c9d5717aa9214b879cb314bcea019fe1124cd62a111224c8e6602e0cd8cce3a2e3b337e238a16dc50f0d7
-  data.tar.gz: 237e7116829180c69d2263ae2cfd02f28c23a4e72f414b0a298f307047798dbb86c591a9cd01c10f0b1aec6d80d43caaae5c93ba8408efd4d3644120b4bbcfaf
+  metadata.gz: 21e38be3cbfe954494685117ec5828cc989b9d2997170f33bfe08e2259bfdaec52e3468e7483db6a8883f5505b06dd7b7a5c1d8ccfe81447ab866d5da94ef323
+  data.tar.gz: 8fc3ba732c1ab60daa5599c0eb77e3c02d4a5a083b43941680e2798898ae6b11addd0f0060de6aa5cabb1c7f5cd2b598ca5d485fe6019da0e5ca12549a46037c

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+v2.1.0, 2019-03-17
+-------------------
+  * Allow stubbing all request methods [#263](https://github.com/oesmith/puffing-billy/pull/263)
+  * Accept all encodings [#265](https://github.com/oesmith/puffing-billy/pull/265)
+
 v2.0.0, 2019-01-30
 -------------------
   * Add driver registration for apparition driver [#258](https://github.com/oesmith/puffing-billy/pull/258)

data/README.md

@@ -118,8 +118,9 @@ proxy.stub('https://example.com/proc/').and_return(Proc.new { |params, headers,
 # a Proc and use the pass_request method. You can manipulate the request
 # (headers, URL, HTTP method, etc) and also the response from the upstream
 # server. The scope of the delivered callable is the user scope where
-# it was defined.
-proxy.stub('http://example.com/').and_return(Proc.new { |*args|
+# it was defined. Setting method to 'all' will intercept requests regardless of
+# the method.
+proxy.stub('http://example.com/', method => 'all').and_return(Proc.new { |*args|
   response = Billy.pass_request(*args)
   response[:headers]['Content-Type'] = 'text/plain'
   response[:body] = 'Hello World!'
@@ -519,10 +520,11 @@ RSpec.configure do |config|
 end
 ```
 
-## Separate Cache Directory for Each Test (in Cucumber)
-
-If you want the cache for each test to be independent, i.e. have it's own directory where the cache files are stored, you can use a Before tag like so:
+## Separate Cache Directory for Each Test
+If you want the cache for each test to be independent, i.e. have it's own directory where the cache files are stored, you can do so.
 
+### in Cucumber
+use a Before tag:
 ```rb
 Before('@javascript') do |scenario, block|
   Billy.configure do |c|
@@ -534,6 +536,29 @@ Before('@javascript') do |scenario, block|
 end
 ```
 
+### in Rspec
+use a before(:each) block:
+```rb
+RSpec.configure do |config|
+  base_cache_path = Billy.config.cache_path
+  base_certs_path = Billy.config.certs_path
+  config.before :each do |x|
+    feature_name = x.metadata[:example_group][:description].underscore.gsub(' ', '_')
+    scenario_name = x.metadata[:description].underscore.gsub(' ', '_')
+    Billy.configure do |c|
+      cache_scenario_folder_path = "#{base_cache_path}/#{feature_name}/#{scenario_name}/"
+      FileUtils.mkdir_p(cache_scenario_folder_path) unless File.exist?(cache_scenario_folder_path)
+      c.cache_path = cache_scenario_folder_path
+
+      certs_scenario_folder_path = "#{base_certs_path}/#{feature_name}/#{scenario_name}/"
+      FileUtils.mkdir_p(certs_scenario_folder_path) unless File.exist?(certs_scenario_folder_path)
+      c.certs_path = certs_scenario_folder_path
+    end
+  end
+end
+```
+
+
 ## Stub requests recording
 
 If you want to record requests to stubbed URIs, set the following configuration option:

data/lib/billy/handlers/proxy_handler.rb

@@ -67,7 +67,7 @@ module Billy
 
     def build_request_options(url, headers, body)
       headers = Hash[headers.map { |k, v| [k.downcase, v] }]
-      headers.delete('accept-encoding')
+      headers['accept-encoding'] = ''
 
       uri = Addressable::URI.parse(url)
       headers.merge!({'authorization' => [uri.user, uri.password]}) if uri.userinfo

data/lib/billy/proxy_request_stub.rb

@@ -62,7 +62,7 @@ module Billy
     end
 
     def matches?(method, url)
-      if method == @method
+      if @method == 'ALL' || method == @method
         if @url.is_a?(Regexp)
           url.match(@url)
         else

data/lib/billy/version.rb

@@ -1,3 +1,3 @@
 module Billy
-  VERSION = '2.0.0'
+  VERSION = '2.1.0'
 end

data/spec/lib/billy/proxy_request_stub_spec.rb

@@ -43,6 +43,17 @@ describe Billy::ProxyRequestStub do
       expect(stub.matches?('GET', 'http://example.com/foo/bar/')).to be
       expect(stub.matches?('GET', 'http://example.com/foo/bar/?baz=bap')).to be
     end
+
+    it 'should match all methods' do
+      expect(Billy::ProxyRequestStub.new('http://example.com', method: :all)
+        .matches?('GET', 'http://example.com')).to be
+      expect(Billy::ProxyRequestStub.new('http://example.com', method: :all)
+        .matches?('POST', 'http://example.com')).to be
+      expect(Billy::ProxyRequestStub.new('http://example.com', method: :all)
+        .matches?('OPTIONS', 'http://example.com')).to be
+      expect(Billy::ProxyRequestStub.new('http://example.com', method: :all)
+        .matches?('HEAD', 'http://example.com')).to be
+    end
   end
 
   context "#matches? (with strip_query_params false in config)" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puffing-billy
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Olly Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-30 00:00:00.000000000 Z
+date: 2019-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec