mock5 1.0.5 → 1.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5e3ff7dc93593a4f21dd450ed0874f436d1493c9
-  data.tar.gz: 1599500a3e285e92e7931b4e24ce5790d0f6bc25
+  metadata.gz: f46073471999d56560318ffd53aae8d68b24af17
+  data.tar.gz: 9cc7ed0b830b9a56ad02f465cd1c4402b74bd3f3
 SHA512:
-  metadata.gz: 6e93dd71cc6c4b953de995bd642764b17298a0e24c16ea2ede62d7c56dc0f70c71ad3397d9c456c43384f74f5c0f4b5dff1c1268d7a67a85026360a3f55b0005
-  data.tar.gz: 961aa86904f8e5f6fd54007202f4f107f4301174e01a864c9d72d94e180d0c931042b068b9cd21012148daafc98972cef47bfc126a0306120ab480f97eb9b884
+  metadata.gz: cdef01f636598d823789ed147073b681ff4b6facdb8f836a3126674258496ae4815ef2eb550362e957a6743f9897aa57c11df2457e928066fb2da8d26c618c04
+  data.tar.gz: fc34feb90c6933d86ff0b7f82a1db399b53bd412d5d13954115c00b15672d6db1db36c974ca3bc0b4f33ef90692582a4ec5c21a67f338f1f84e0d158a2eb97b7

data/.travis.yml

@@ -1,4 +1,4 @@
 rvm:
   - 1.9.3
   - 2.0.0
-  - 2.1.1
+  - 2.1.2

data/Gemfile

@@ -1,10 +1,9 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in mock5.gemspec
 gemspec
 
-gem "bundler", "~> 1.5"
-gem "rspec",   "~> 2.14"
-gem "rake",    "~> 10.1"
-
-gem "pry-debugger"
+gem "bundler", "~> 1.6"
+gem "rspec",   "~> 3.0"
+gem "rake",    "~> 10.3"
+gem "pry"

data/README.md

@@ -2,7 +2,7 @@
 [![Gem Version](https://img.shields.io/gem/v/mock5.svg)](https://rubygems.org/gems/mock5)
 [![Build Status](https://img.shields.io/travis/rwz/mock5.svg)](http://travis-ci.org/rwz/mock5)
 [![Code Climate](https://img.shields.io/codeclimate/github/rwz/mock5.svg)](https://codeclimate.com/github/rwz/mock5)
-[![Inline docs](http://inch-pages.github.io/github/rwz/mock5.svg)](http://inch-pages.github.io/github/rwz/mock5)
+[![Inline docs](http://inch-ci.org/github/rwz/mock5.svg)](http://inch-ci.org/github/rwz/mock5)
 
 Mock5 allows to mock external APIs with simple Sinatra Rack apps.
 

data/lib/mock5.rb

@@ -1,5 +1,3 @@
-require "mock5/version"
-require "mock5/api"
 require "set"
 
 # The main module of the gem, exposing all API management methods.
@@ -7,6 +5,9 @@ require "set"
 module Mock5
   extend self
 
+  autoload :VERSION, "mock5/version"
+  autoload :Api, "mock5/api"
+
   # Returns a set of currently mounted APIs
   #
   # @return [Set] a list of currently mounted APIs
@@ -49,6 +50,7 @@ module Mock5
   # @return [Set] a list of APIs actually mounted
   def mount(*apis)
     apis.to_set.subtract(mounted_apis).each do |api|
+      check_api api
       mounted_apis.add api
       registry.register_request_stub api.request_stub
     end
@@ -125,4 +127,8 @@ module Mock5
   def registry
     WebMock::StubRegistry.instance
   end
+
+  def check_api(api)
+    fail ArgumentError, "expected an instance of Mock5::Api" unless Api === api
+  end
 end

data/lib/mock5/version.rb

@@ -1,3 +1,3 @@
 module Mock5
-  VERSION = "1.0.5".freeze
+  VERSION = "1.0.6".freeze
 end

data/spec/mock5_api_spec.rb

@@ -22,7 +22,7 @@ describe Mock5::Api do
         .to raise_error(ArgumentError, "Endpoint URL should not include path")
     end
 
-    it "can not be specified as a invalid url string" do
+    it "can not be specified as an invalid url string" do
       expect{ described_class.new("foo") }
         .to raise_error(ArgumentError, "Endpoint should be a valid URL")
     end

data/spec/mock5_spec.rb

@@ -3,9 +3,10 @@ require "mock5"
 describe Mock5 do
   describe ".mock" do
     it "creates an Api" do
-      block = ->{}
-      expect(described_class::Api).to receive(:new).with(/foo/, &block)
-      described_class.mock /foo/, &block
+      expect(described_class::Api).to receive(:new).with(/foo/).and_yield
+      described_class.mock /foo/ do
+        # mock definition goes here
+      end
     end
 
     it "returns an Api" do
@@ -18,16 +19,22 @@ describe Mock5 do
       WebMock::StubRegistry.instance.reset!
       described_class.instance_exec do
         if instance_variable_defined?(:@_mounted_apis)
-          remove_instance_variable(:@_mounted_apis)
+          remove_instance_variable :@_mounted_apis
         end
       end
     end
 
     let(:mounted_apis){ described_class.mounted_apis }
+    let(:mounted_apis_qty){ mounted_apis.size }
     let(:api){ described_class.mock }
     let(:another_api){ described_class.mock }
 
     describe ".mount" do
+      it "raises ArgumentError when passed an invalid argument" do
+        action = ->{ described_class.mount nil }
+        message = "expected an instance of Mock5::Api"
+        expect(&action).to raise_error(ArgumentError, message)
+      end
       it "mounts an api" do
         described_class.mount api
         expect(mounted_apis).to include(api)
@@ -35,7 +42,7 @@ describe Mock5 do
 
       it "mounts an api only once" do
         10.times{ described_class.mount api }
-        expect(mounted_apis).to have(1).item
+        expect(mounted_apis_qty).to eq(1)
       end
 
       it "mounts several APIs at once" do
@@ -65,7 +72,7 @@ describe Mock5 do
 
       it "unmounts several APIs at once" do
         described_class.mount another_api
-        expect(mounted_apis).to have(2).apis
+        expect(mounted_apis_qty).to eq(2)
         described_class.unmount api, another_api
         expect(mounted_apis).to be_empty
       end
@@ -88,13 +95,13 @@ describe Mock5 do
       end
 
       it "unmounts all currently mounted apis" do
-        expect(mounted_apis).to have(3).apis
+        expect(mounted_apis_qty).to eq(3)
         described_class.unmount_all!
         expect(mounted_apis).to be_empty
       end
 
       it "has .reset! alias" do
-        expect(mounted_apis).to have(3).apis
+        expect(mounted_apis_qty).to eq(3)
         described_class.reset!
         expect(mounted_apis).to be_empty
       end
@@ -104,11 +111,11 @@ describe Mock5 do
       before{ described_class.mount api }
 
       it "returns true if api is currently mounted" do
-        expect(described_class.mounted?(api)).to be_true
+        expect(described_class.mounted?(api)).to be_truthy
       end
 
       it "returns false if api is currently not mounted" do
-        expect(described_class.mounted?(another_api)).to be_false
+        expect(described_class.mounted?(another_api)).to be_falsy
       end
 
       it "returns true only when ALL api are mounted" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mock5
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.6
 platform: ruby
 authors:
 - Pavel Pravosud
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-29 00:00:00.000000000 Z
+date: 2014-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: webmock
@@ -84,4 +84,3 @@ summary: Mock APIs using Sinatra
 test_files:
 - spec/mock5_api_spec.rb
 - spec/mock5_spec.rb
-has_rdoc: