utterson 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e8d588f2917ca5d7bfdeb95bae81150afbf47930
-  data.tar.gz: a38b9ea0380228eb434acf6bffac68a2c1b8d8ce
+  metadata.gz: 535d62e57f06b6f95ca5948cf2f235ff41e0dc05
+  data.tar.gz: 1e8098ec08f238c880596326d689d9272855793e
 SHA512:
-  metadata.gz: ae2f3b547a9a7c1a602cd293d6e0a7eba83b8d8747671198635559793bf40d5363115eb6a873ac679729a14f939f16bfe94ad3a949fd0397c248c88c225f5819
-  data.tar.gz: ede353e24c144ea04d6a8fd724ec37c449cc865bf84708ab05d83ad1a39a6f4f69ce0c34cb6e6f4e76586eeca9e1c2f4bf237d24c258401e5b8e006bb77fee62
+  metadata.gz: dcbfe2cb045061aedae22640f0042529a934c9dce5ca5bc59e7c1e6ca1419f7d04aadf28581524a15cc7b4ef63954c4d105ef6e2862e09bdc495fc14d3fce193
+  data.tar.gz: 2f11619c7568d1bf885493eb384aafc3a540ef8de6a6e109734bb04ece5027dbf6fc5016dab6c0fe872e20f0b5f9c991329deea6813a7e8a7470cd0f3eaa3864

data/README.md

@@ -1,6 +1,6 @@
 Simple utility to traverse directory of html files and check links in them.
 
-[![Build Status](https://travis-ci.org/iiska/utterson.png)](https://travis-ci.org/iiska/utterson)
+[![Build Status](https://travis-ci.org/iiska/utterson.png)](https://travis-ci.org/iiska/utterson) [![Dependency Status](https://gemnasium.com/iiska/utterson.png)](https://gemnasium.com/iiska/utterson)
 
 ## Getting started
 

data/spec/utterson_spec.rb

@@ -34,105 +34,115 @@ describe Utterson do
 
   describe "#check_uri" do
     let(:u) {Utterson.new}
+    let(:html_file) {"file.html"}
 
     it "should use remote checking for http protocol" do
+      url = "http://example.com"
       u.stub(:check_remote_uri) {}
-      u.should_receive(:check_remote_uri).with("http://example.com", "file.html")
-      u.check_uri("http://example.com", "file.html")
+      u.should_receive(:check_remote_uri).with(url, html_file)
+      u.check_uri(url, html_file)
     end
 
     it "should use remote checking for https protocol" do
+      url = "https://example.com"
       u.stub(:check_remote_uri) {}
-      u.should_receive(:check_remote_uri).with("https://example.com", "file.html")
-      u.check_uri("https://example.com", "file.html")
+      u.should_receive(:check_remote_uri).with(url, html_file)
+      u.check_uri(url, html_file)
     end
 
     it "should use remote checking when only // is specified" do
+      url = "//example.com"
       u.stub(:check_remote_uri) {}
-      u.should_receive(:check_remote_uri).with("//example.com", "file.html")
-      u.check_uri("//example.com", "file.html")
+      u.should_receive(:check_remote_uri).with(url, html_file)
+      u.check_uri(url, html_file)
     end
 
     it "should use local checking for relative uris" do
+      url = "../file.html"
       u.stub(:check_local_uri) {}
-      u.should_receive(:check_local_uri).with("../file.html", "file.html")
-      u.check_uri("../file.html", "file.html")
+      u.should_receive(:check_local_uri).with(url, html_file)
+      u.check_uri(url, html_file)
     end
   end
 
   describe "#check_local_uri" do
     let(:u) {Utterson.new(dir: "spec/fixtures/dir-structure")}
+    let(:html_file) {"spec/fixtures/dir-structure/1.htm"}
 
     it "should not assign error info if file exists" do
-      u.check_local_uri("../sample.html", "spec/fixtures/dir-structure/1.htm")
+      u.check_local_uri("../sample.html", html_file)
       u.errors.should be_empty
     end
 
     it "should assign error info if file doesn't exist" do
-      u.check_local_uri("../sample_not_found.html", "spec/fixtures/dir-structure/1.htm")
-      u.errors["spec/fixtures/dir-structure/1.htm"].should == {"../sample_not_found.html" => "File not found"}
+      url = "../sample_not_found.html"
+      u.check_local_uri(url, html_file)
+      u.errors[html_file].should == {url => "File not found"}
     end
 
     it "should use root directory when urls start with /" do
-      u2 = Utterson.new(dir: "spec/fixtures/dir-structure", root: "spec/fixtures")
-      u2.check_local_uri("/sample.html", "spec/fixtures/dir-structure/1.htm")
+      u2 = Utterson.new(dir: "spec/fixtures/dir-structure",
+                        root: "spec/fixtures")
+      u2.check_local_uri("/sample.html", html_file)
       u2.errors.should be_empty
     end
 
     it "should handle target directory as root for urls starting with / if root is no available" do
-      u.check_local_uri("/2.html", "spec/fixtures/dir-structure/1.htm")
+      u.check_local_uri("/2.html", html_file)
       u.errors.should be_empty
     end
 
     it "should ignore query string when checking local files" do
-      u.check_local_uri("2.html?queryparam=value", "spec/fixtures/dir-structure/1.htm")
+      u.check_local_uri("2.html?queryparam=value", html_file)
       u.errors.should be_empty
     end
   end
 
   describe "#check_remote_uri" do
     let(:u) {Utterson.new}
+    let(:html_file) {"test.html"}
+    let(:url) {"http://example.com/index.html"}
 
     it "should not assign error info if request is successfull" do
-      stub_request(:head, "http://example.com/index.html").
+      stub_request(:head, url).
         with(:headers => {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
         to_return(:status => 200, :body => "", :headers => {})
-      u.check_remote_uri("http://example.com/index.html", "test.html")
+      u.check_remote_uri(url, html_file)
       u.errors.should be_empty
     end
 
     it "should assign error info if there is error response" do
-      stub_request(:head, "http://example.com/404.html").
+      stub_request(:head, url).
         with(:headers => {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
         to_return(:status => 404, :body => "", :headers => {})
-      u.check_remote_uri("http://example.com/404.html", "test.html")
+      u.check_remote_uri(url, html_file)
       puts u.errors.inspect
-      u.errors["test.html"].should_not be_empty
-      u.errors["test.html"]["http://example.com/404.html"].instance_of?(Net::HTTPNotFound).should be_true
+      u.errors[html_file].should_not be_empty
+      u.errors[html_file][url].instance_of?(Net::HTTPNotFound).should be_true
     end
 
     it "should add error status from buffer timeouts" do
-      stub_request(:head, "http://example.com/index.html").to_timeout
-      u.check_remote_uri("http://example.com/index.html", "test.html")
+      stub_request(:head, url).to_timeout
+      u.check_remote_uri(url, html_file)
       u.errors.should_not be_empty
     end
 
     it "should add error status from connection timeouts" do
-      stub_request(:head, "http://example.com/index.html").to_raise(Errno::ETIMEDOUT)
-      u.check_remote_uri("http://example.com/index.html", "test.html")
+      stub_request(:head, url).to_raise(Errno::ETIMEDOUT)
+      u.check_remote_uri(url, html_file)
       u.errors.should_not be_empty
     end
 
     it "shoud add error status from name resolution errors" do
-      stub_request(:head, "http://example.com/index.html").
+      stub_request(:head, url).
         to_raise(SocketError.new('getaddrinfo: Name or service not known'))
-      u.check_remote_uri("http://example.com/index.html", "test.html")
+      u.check_remote_uri(url, html_file)
       u.errors.should_not be_empty
     end
 
     it "shoud add error status when invalid URI" do
       URI.stub(:new).and_raise(URI::InvalidURIError)
-      u.check_remote_uri("http://invalid_uri", "test.html")
+      u.check_remote_uri("http://invalid_uri", html_file)
       u.errors.should_not be_empty
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: utterson
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Juhamatti Niemelä
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-03 00:00:00.000000000 Z
+date: 2013-11-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trollop
@@ -122,7 +122,7 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ~>
+  - - '>='
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement