Merge pull request #3 from patch-technology/bs/integration-test

Remove VCR and test with production sandbox account

Author: bspellacy

.github/workflows/test.yml

@@ -23,7 +23,5 @@ jobs:
 
       - name: Run RSpec
         env:
-          PATCH_RUBY_HOST: 'https://api.staging-patch.com/'
-          PATCH_RUBY_API_KEY: 'key_test_1234'
+          PATCH_RUBY_API_KEY: ${{ secrets.SANDBOX_API_KEY }}
         run: bundle exec rspec
-

Gemfile

@@ -6,5 +6,4 @@ group :development, :test do
   gem 'rake', '~> 13.0.1'
   gem 'pry-byebug'
   gem 'rubocop', '~> 0.66.0'
-  gem 'vcr', '~> 6.0'
 end

Gemfile.lock

@@ -55,7 +55,6 @@ GEM
     typhoeus (1.4.0)
       ethon (>= 0.9.0)
     unicode-display_width (1.5.0)
-    vcr (6.0.0)
 
 PLATFORMS
   ruby
@@ -66,7 +65,6 @@ DEPENDENCIES
   rake (~> 13.0.1)
   rspec (~> 3.6, >= 3.6.0)
   rubocop (~> 0.66.0)
-  vcr (~> 6.0)
 
 BUNDLED WITH
    2.1.4

README.md

@@ -1,12 +1,12 @@
-# Patch ruby
+# Patch Ruby SDK
 ![Test](https://github.com/patch-technology/patch-ruby/workflows/Test/badge.svg)
 [![Gem Version](https://badge.fury.io/rb/patch_ruby.svg)](https://badge.fury.io/rb/patch_ruby)
 
 The official Ruby gem for the [Patch API](https://www.usepatch.com)
 
 ## Documentation
 
-For detailed documentation and examples, see the [Patch API docs](https://www.usepatch.com).
+For detailed documentation and examples, see the [Patch API docs](https://docs.usepatch.com).
 
 ## Installation
 

spec/api_client_spec.rb

@@ -10,43 +10,22 @@
 
 =end
 
-require 'spec_helper'
-
 describe Patch::ApiClient do
   context 'initialization' do
-    context 'URL stuff' do
-      context 'host' do
-        it 'removes http from host' do
-          Patch.configure { |c| c.host = 'http://example.com' }
-          expect(Patch::Configuration.default.host).to eq('example.com')
-        end
-
-        it 'removes https from host' do
-          Patch.configure { |c| c.host = 'https://wookiee.com' }
-          expect(Patch::ApiClient.default.config.host).to eq('wookiee.com')
-        end
-
-        it 'removes trailing path from host' do
-          Patch.configure { |c| c.host = 'hobo.com/v4' }
-          expect(Patch::Configuration.default.host).to eq('hobo.com')
-        end
+    context 'base_path' do
+      it "prepends a slash to base_path" do
+        Patch.configure { |c| c.base_path = 'v4/dog' }
+        expect(Patch::Configuration.default.base_path).to eq('/v4/dog')
+      end
+
+      it "doesn't prepend a slash if one is already there" do
+        Patch.configure { |c| c.base_path = '/v4/dog' }
+        expect(Patch::Configuration.default.base_path).to eq('/v4/dog')
       end
 
-      context 'base_path' do
-        it "prepends a slash to base_path" do
-          Patch.configure { |c| c.base_path = 'v4/dog' }
-          expect(Patch::Configuration.default.base_path).to eq('/v4/dog')
-        end
-
-        it "doesn't prepend a slash if one is already there" do
-          Patch.configure { |c| c.base_path = '/v4/dog' }
-          expect(Patch::Configuration.default.base_path).to eq('/v4/dog')
-        end
-
-        it "ends up as a blank string if nil" do
-          Patch.configure { |c| c.base_path = nil }
-          expect(Patch::Configuration.default.base_path).to eq('')
-        end
+      it "ends up as a blank string if nil" do
+        Patch.configure { |c| c.base_path = nil }
+        expect(Patch::Configuration.default.base_path).to eq('')
       end
     end
   end