url_hunter 0.0.1 → 0.0.2

data/Rakefile

@@ -1,6 +1,15 @@
-require "bundler/gem_tasks"
+require 'rubygems'
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 desc "Open a shell with local libraries loaded"
 task :shell do
-  exec "irb -I #{File.expand_path('../lib', __FILE__)} -r url_hunter"
+  exec "bundle exec irb -r url_hunter"
 end
+
+desc "Run specs"
+RSpec::Core::RakeTask.new do |t|
+  t.rspec_opts = %w(-fs --color)
+end
+
+task :default => :spec

data/Readme.md

@@ -2,7 +2,7 @@
 
 Before I go anywhere with you, I want to know where you're taking me
 
-### Quick Reoslve ###
+### Quick Resolve ###
     require 'url_hunter'
     UrlHunter.resolve 'http://bit.ly/2SnEZb'
     => "http://www.microsoft.com/en-us/default.aspx"

data/lib/url_hunter.rb

@@ -2,7 +2,7 @@ require 'uri'
 require 'net/http'
 
 class UrlHunter
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
   class Error < StandardError; end
 
   attr_reader :url, :original_url, :limit, :tries
@@ -42,7 +42,7 @@ class UrlHunter
       redirect = res['Location']
       # Append Location re-direct to previous URL if it's relative
       if not redirect =~ /^(http|https):\/\//i
-        redirect = @url + redirect
+        redirect = u.scheme + '://' + u.hostname + redirect
       end
       @url = redirect
       @tries += 1

data/spec/spec_helper.rb

@@ -0,0 +1,2 @@
+$:.unshift File.expand_path('../../lib',__FILE__)
+require 'url_hunter'

data/spec/url_hunter_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe UrlHunter, '#resolve' do
+  it "Should require http:// or https:// prefixes" do
+    lambda { UrlHunter.resolve('no_prefix') }.should raise_error(UrlHunter::Error)
+  end
+
+  it "Should resolve bit.ly urls pretty well" do
+    UrlHunter.resolve('http://bit.ly/DhAoX').should eq('http://www.apple.com/')
+  end
+
+  it "Should resolve the bit.ly url for http://www.apple.com/ in one try" do
+    u = UrlHunter.new('http://bit.ly/DhAoX')
+    u.resolve.should eq('http://www.apple.com/')
+    u.tries.should eq(1)
+  end
+
+  it "Should handle relative paths mixed in Location tags" do
+    u = UrlHunter.new('http://flic.kr/p/bBqyXJ')
+    u.resolve.should_not include('bBqyXJ')
+  end
+end

data/url_hunter.gemspec

@@ -17,4 +17,5 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
+  s.add_development_dependency('rspec')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: url_hunter
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,8 +9,19 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-03 00:00:00.000000000 Z
-dependencies: []
+date: 2012-04-26 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: &70161664070420 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70161664070420
 description: Wait a minute, before I go anywhere with you, I want to know where you're
   taking me.
 email:
@@ -24,6 +35,8 @@ files:
 - Rakefile
 - Readme.md
 - lib/url_hunter.rb
+- spec/spec_helper.rb
+- spec/url_hunter_spec.rb
 - url_hunter.gemspec
 homepage: https://github.com/bshelton229/url_hunter
 licenses: []
@@ -49,4 +62,6 @@ rubygems_version: 1.8.11
 signing_key: 
 specification_version: 3
 summary: Resolves URL re-directs given a limit
-test_files: []
+test_files:
+- spec/spec_helper.rb
+- spec/url_hunter_spec.rb