Use compensated-spec fixtures to test compensated-ruby and `compe…

…nsated-proxy` (#83)

* 🐛Don't double dump the hash structures

See: #68

* `compensated` relies on `compensated-spec` for Stripe test fixtures

See: #68

* Rely on `compensated-spec` in `compensated-proxy` for event fixtures

compensated-proxy/Gemfile

@@ -5,3 +5,4 @@ gemspec
 
 gem "rake", "~> 12.0"
 gem "rspec", "~> 3.0"
+gem "compensated-spec", path: "../compensated-spec"

compensated-proxy/Gemfile.lock

@@ -1,3 +1,10 @@
+PATH
+  remote: ../compensated-spec
+  specs:
+    compensated-spec (0.1.0)
+      compensated
+      jsonpath (~> 1.0)
+
 PATH
   remote: .
   specs:
@@ -9,6 +16,10 @@ GEM
   specs:
     compensated (0.1.0.pre13)
     diff-lcs (1.3)
+    jsonpath (1.0.5)
+      multi_json
+      to_regexp (~> 0.2.1)
+    multi_json (1.14.1)
     rake (12.3.3)
     rspec (3.9.0)
       rspec-core (~> 3.9.0)
@@ -23,12 +34,14 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.9.0)
     rspec-support (3.9.3)
+    to_regexp (0.2.1)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   compensated-proxy!
+  compensated-spec!
   rake (~> 12.0)
   rspec (~> 3.0)
 

compensated-proxy/spec/compensated/proxy_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative '../../../compensated-ruby/spec/fixtures'
-
 require 'compensated/stripe'
 RSpec.describe Compensated::Proxy do
   include Compensated::Spec::Helpers
@@ -10,11 +8,8 @@
     expect(Compensated::Proxy::VERSION).not_to be nil
   end
 
-  let(:interpolate) { {} }
-  # TODO: Maybe pull the spec helpers into a `compensated-spec` gem
-  # that will make it easier to leverage the fixtures when testing compensated.
-  let(:template_file_path) { fixture_path(Pathname.new(__dir__).join('..', '..', '..', 'compensated-ruby', 'spec', 'stripe'), fixture) }
-  let(:request) { fake_request(template_file_path, interpolate: interpolate) }
+  let(:overrides) { [] }
+  let(:request) { compensated_fake_request("stripe/#{fixture}", overrides: overrides) }
   let(:fixture) { 'customer.subscription.updated.api-v2019-12-03.json' }
 
   it 'forwards the standardized body without the raw body' do

compensated-proxy/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'bundler/setup'
+require 'compensated/spec'
 require 'compensated/proxy'
 
 RSpec.configure do |config|

compensated-ruby/Gemfile

@@ -2,3 +2,5 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in compensated-ruby.gemspec
 gemspec
+
+gem "compensated-spec", path: "../compensated-spec"

compensated-ruby/compensated.gemspec

@@ -40,4 +40,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rack"
   spec.add_development_dependency "cucumber"
   spec.add_development_dependency "yard"
+  spec.add_development_dependency "pry"
 end

compensated-ruby/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 require "bundler/setup"
 require "compensated"
-require_relative 'fixtures'
+require "compensated/spec"
+require "pry"
 
 RSpec.configure do |config|
   # Enable flags like --only-failures and --next-failure

compensated-ruby/spec/stripe/event_parser_spec.rb

@@ -3,9 +3,8 @@ module Compensated
   module Stripe
     RSpec.describe EventParser do
       include Compensated::Spec::Helpers
-      let(:interpolate) { {} }
-      let(:template_file_path) { fixture_path(__dir__, fixture) }
-      let(:request) { fake_request(template_file_path, interpolate: interpolate) }
+      let(:interpolate) { [] }
+      let(:request) { compensated_fake_request("stripe/#{fixture}", overrides: interpolate) }
 
       it "Adds itself to the list of event parses available to compensated" do
         expect(Compensated.event_parsers.find { |ep| ep.instance_of?(Stripe::EventParser) }).not_to be_nil
@@ -249,12 +248,12 @@ module Stripe
                 subject(:subscription) { product[:subscription] }
                 it { is_expected.to include(period: { start: Time.at(1586329200), end: Time.at(1588921200) }) }
                 context 'when the subscription does not have a canceled_at value set' do
-                  let(:interpolate) { { data: { object: { canceled_at: nil } } } }
+                  let(:interpolate) { [ {location: "$.data.object.canceled_at", value: nil }] }
                   it { is_expected.to include(status: :active) }
                 end
 
                 context 'when the subscription does have a canceled_at value set' do
-                  let(:interpolate) { { data: { object: { canceled_at: 1234 } } } }
+                  let(:interpolate) { [ {location: "$.data.object.canceled_at", value: 1234 }] }
                   it { is_expected.to include(status: :canceled) }
                 end
 
@@ -284,12 +283,16 @@ module Stripe
               describe ":subscription" do
                 subject(:subscription) { product[:subscription] }
                 context 'when data.object.ended_at is nil and data.object.canceled_at is present' do
-                  let(:interpolate) { { data: { object: { ended_at: nil, canceled_at: 12345 } } } }
+                  let(:interpolate) { [{ location: "$.data.object.ended_at", value: nil },
+                                       { location: "$.data.object.canceled_at", value: 12345 }] }
                   it { is_expected. to include(status: :canceled) }
                 end
 
                 context 'when data.object.ended_at is present and data.object.canceled_at is present' do
-                  let(:interpolate) { { data: { object: { ended_at: 12345, canceled_at: 12345 } } } }
+                  let(:interpolate) do
+                    [{ location: "$.data.object.ended_at", value: 12345 }, { location: "$.data.object.canceled_at", value: 12345 }]
+                  end
+
                   it { is_expected. to include(status: :ended) }
                 end
               end