rspec-webservice_matchers 4.8.1 → 4.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ba8fd43304c72ef0e625db7c2f709e2accfb5782
-  data.tar.gz: 6a7a8ae73935d83c4ae0dd074967a9dd8e957fa6
+  metadata.gz: f930e26ec837ba1ca066f2a911608b5c58edfc09
+  data.tar.gz: 0c2be7eae06450c998838d3e1530421ee9dcb6b2
 SHA512:
-  metadata.gz: 7bf79fe28ee47a785d482d14badcb0c016d7960715dd31e1db673ba23ec13973a6f78ecf361d4608d56eb905f9af598ff904b51c366b12cd4e33cbdbf0b4eba9
-  data.tar.gz: a7f7d3973bce00278dd1ef361cce87ec738eaedd873d82a1f48ef07bef0644edde260f362830864a973c158181562b9ae412fd67cf5cc204f9b5449ae0d73522
+  metadata.gz: 702e17e06e648086ce8c75c2838eb3997a7d44ccb741af282d22b46785fc5d44862966713e788c54acbe908f2290469fb170754c0fec303b8218dab3bfa7197b
+  data.tar.gz: 88be96f4ae2de62c765bb0a9fd6bc621e0024b6513ba945700922adeb4e76031f9b3e9ab04e9ed1247cb928bf305ee335679a4c9a96128245e44ab721c7c6ec6

data/HISTORY.md

@@ -1,3 +1,11 @@
+4.9.0
+-----
+Adds new API for `be_fast`.
+
+4.8.1
+-----
+Fixes require bug.
+
 4.8.0
 -----
 Adds the new public API, beginning with the `be_up` matcher.

data/lib/rspec/webservice_matchers/be_fast.rb

@@ -1,11 +1,23 @@
 # frozen_string_literal: true
 require 'cgi'
 require 'json'
+require 'rspec/matchers'
 require 'rspec/webservice_matchers/util'
+require 'validated_object'
 
 module RSpec
   module WebserviceMatchers
     module BeFast
+
+      class TestResult < ValidatedObject::Base
+        attr_accessor :success, :score
+        alias success? success
+
+        validates :success, inclusion: [true, false]
+        validates :score, inclusion: (0..100)
+      end
+
+
       def self.parse(json:)
         response = JSON.parse(json)
         unless response.key?('ruleGroups')
@@ -29,12 +41,21 @@ module RSpec
         BeFast.parse(json: response.body).fetch(:score)
       end
 
+      def self.test(url:)
+        TestResult.new do |r|
+          r.score   = BeFast.page_speed_score(url: url)
+          r.success = r.score >= 85
+        end
+      end
+
+
       RSpec::Matchers.define :be_fast do
         score = nil
 
         match do |url|
-          score = BeFast.page_speed_score(url: url)
-          score >= 85
+          result = BeFast.test(url: url)
+          score = result.score
+          result.success?
         end
 
         failure_message do

data/lib/rspec/webservice_matchers/be_up.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require 'rspec/matchers'
 require 'rspec/webservice_matchers/util'
 require 'validated_object'
 
@@ -18,7 +19,7 @@ module RSpec
 
       def self.test(url:nil, domain:nil)
         raise 'Must specify a url or domain' if url.nil? && domain.nil?
-        
+
         TestResult.new do |r|
           r.status_code = Util.status(url || domain, follow: true)
           r.success =     (r.status_code == 200)

data/lib/rspec/webservice_matchers/have_a_valid_cert.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 require 'rspec/webservice_matchers/util'
+require 'rspec/matchers'
 
 module RSpec
   module WebserviceMatchers

data/lib/rspec/webservice_matchers/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 module RSpec
   module WebserviceMatchers
-    VERSION = '4.8.1'
+    VERSION = '4.9.0'
   end
 end

data/spec/rspec/webservice_matchers/be_fast_spec.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+require 'spec_helper'
+require 'rspec/webservice_matchers/be_fast'
+include RSpec::WebserviceMatchers
+
+describe BeFast do
+  describe BeFast::TestResult do
+    it 'requires :success' do
+      expect {
+        BeFast::TestResult.new {}
+      }.to raise_error(ArgumentError, /success/i)
+    end
+
+    it 'requires :score' do
+      expect {
+        BeFast::TestResult.new { |r| r.success = true }
+      }.to raise_error(ArgumentError, /score/i)
+    end
+  end
+
+  describe '#test' do
+    it 'handles a fast site' do
+      result = BeFast.test url: 'http://nonstop.qa'
+      expect( result.success? ).to be true
+      expect( result.score    ).to be >= 85
+    end
+  end
+end

data/spec/rspec/webservice_matchers/be_up_spec.rb

@@ -2,9 +2,9 @@
 require 'spec_helper'
 require 'rspec/webservice_matchers/be_up'
 
-include RSpec::WebserviceMatchers::BeUp
+include RSpec::WebserviceMatchers
 
-describe RSpec::WebserviceMatchers::BeUp do
+describe BeUp do
   describe '#test' do
     it 'handles a simple 200' do
       result = BeUp.test url: 'http://www.website.com/'
@@ -26,30 +26,30 @@ describe RSpec::WebserviceMatchers::BeUp do
   end
 
 
-  describe TestResult do
+  describe BeUp::TestResult do
     it 'requires :success' do
       expect {
-        TestResult.new { |r| r.status_code = 200 }
+        BeUp::TestResult.new { |r| r.status_code = 200 }
       }.to raise_error(ArgumentError)
     end
 
     it 'requires :status_code' do
       expect {
-        TestResult.new { |r| r.success = true }
+        BeUp::TestResult.new { |r| r.success = true }
       }.to raise_error(ArgumentError)
     end
 
     it 'accepts boolean :success & integer :status_code' do
-      result = TestResult.new do |r|
+      result = BeUp::TestResult.new do |r|
         r.status_code = 404
         r.success = false
       end
-      expect( result ).to be_an_instance_of(TestResult)
+      expect( result ).to be_an_instance_of(BeUp::TestResult)
     end
 
     it 'requires boolean :success' do
       expect {
-        TestResult.new do |r|
+        BeUp::TestResult.new do |r|
           r.status_code = 200
           r.success = 1
         end
@@ -58,7 +58,7 @@ describe RSpec::WebserviceMatchers::BeUp do
 
     it 'requires integer :status_code' do
       expect {
-        TestResult.new do |r|
+        BeUp::TestResult.new do |r|
           r.status_code = '404'
           r.success = false
         end
@@ -67,7 +67,7 @@ describe RSpec::WebserviceMatchers::BeUp do
 
     it 'cannot have status < 100' do
       expect {
-        TestResult.new do |r|
+        BeUp::TestResult.new do |r|
           r.status_code = -5
           r.success = false
         end
@@ -76,7 +76,7 @@ describe RSpec::WebserviceMatchers::BeUp do
 
     it 'cannot have status > 510' do
       expect {
-        TestResult.new do |r|
+        BeUp::TestResult.new do |r|
           r.status_code = 511
           r.success = false
         end
@@ -84,11 +84,11 @@ describe RSpec::WebserviceMatchers::BeUp do
     end
 
     it 'allows status 510' do
-      result = TestResult.new do |r|
+      result = BeUp::TestResult.new do |r|
         r.status_code = 510
         r.success = false
       end
-      expect( result ).to be_an_instance_of(TestResult)
+      expect( result ).to be_an_instance_of(BeUp::TestResult)
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-webservice_matchers
 version: !ruby/object:Gem::Version
-  version: 4.8.1
+  version: 4.9.0
 platform: ruby
 authors:
 - Robb Shecter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-29 00:00:00.000000000 Z
+date: 2016-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -177,6 +177,7 @@ files:
 - lib/rspec/webservice_matchers/version.rb
 - rspec-webservice_matchers.gemspec
 - spec/fixtures/pagespeed.json
+- spec/rspec/webservice_matchers/be_fast_spec.rb
 - spec/rspec/webservice_matchers/be_up_spec.rb
 - spec/rspec/webservice_matchers/page_speed_spec.rb
 - spec/rspec/webservice_matchers/protcol_spec.rb
@@ -210,6 +211,7 @@ specification_version: 4
 summary: Black-box web app configuration testing
 test_files:
 - spec/fixtures/pagespeed.json
+- spec/rspec/webservice_matchers/be_fast_spec.rb
 - spec/rspec/webservice_matchers/be_up_spec.rb
 - spec/rspec/webservice_matchers/page_speed_spec.rb
 - spec/rspec/webservice_matchers/protcol_spec.rb