Ruby 1.9.3 fix

buren · Oct 24, 2014 · fe50d51 · fe50d51
1 parent 0bc7e09
commit fe50d51
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -1,6 +1,6 @@
 # SiteMapper
 [![Code Climate](https://codeclimate.com/github/buren/site_mapper.png)](https://codeclimate.com/github/buren/site_mapper) 
- [![Gem Version](https://badge.fury.io/rb/site_mapper.svg)](http://badge.fury.io/rb/site_mapper) [![Build Status](https://travis-ci.org/buren/site_mapper.svg?branch=master)](https://travis-ci.org/buren/site_mapper) [![Dependency Status](https://gemnasium.com/buren/site_mapper.svg)](https://gemnasium.com/buren/site_mapper)
+ [![Gem Version](https://badge.fury.io/rb/site_mapper.svg)](http://badge.fury.io/rb/site_mapper) [![Build Status](https://travis-ci.org/buren/site_mapper.svg?branch=master)](https://travis-ci.org/buren/site_mapper) [![Dependency Status](https://gemnasium.com/buren/site_mapper.svg)](https://gemnasium.com/buren/site_mapper) [![Coverage Status](https://img.shields.io/coveralls/buren/site_mapper.svg)](https://coveralls.io/r/buren/site_mapper)
 
 Map all links on a given site.  
 SiteMapper will try to respect `/robots.txt`

diff --git a/lib/site_mapper/crawler.rb b/lib/site_mapper/crawler.rb
@@ -6,9 +6,9 @@ module SiteMapper
   class Crawler
     # @param [String] url base url for crawler
     # @param [Hash] resolve (optional false by default)
-    def initialize(url, resolve: false)
-      @base_url     = Request.resolve_url(url)
-      @options     = { resolve: resolve }
+    def initialize(url, options = {})
+      @base_url    = Request.resolve_url(url)
+      @options     = { resolve: false }.merge(options)
       @crawl_url   = CrawlUrl.new(@base_url)
       @fetch_queue = CrawlQueue.new
       @processed   = Set.new

diff --git a/lib/site_mapper/request.rb b/lib/site_mapper/request.rb
@@ -50,9 +50,10 @@ def get_response_body(*args)
       # @example Resolve google.com
       #    resolve_url('google.com')
       #    # => 'https://www.google.com'
-      def resolve_url(url, with_query: true)
+      def resolve_url(url, options = {})
+        options  = { with_query: true }.merge(options)
         resolved = UrlResolver.resolve(url)
-        resolved = remove_query(resolved) unless with_query
+        resolved = remove_query(resolved) unless options[:with_query]
         resolved
       end
 

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,5 +1,6 @@
 require 'simplecov'
 require 'coveralls'
+Coveralls.wear!
 
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
   SimpleCov::Formatter::HTMLFormatter,