html-proofer 0.0.16 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cb67833a18915e42dd102d396dae588ff9ece975
-  data.tar.gz: 15a1a7c04a8557835b23aaffc03aeb26f0db1b06
+  metadata.gz: bb18390b6e8107e566191736a04b0bd6209ca1fe
+  data.tar.gz: b82303dfc0e7972ffd8145e6465cfa03a03d3397
 SHA512:
-  metadata.gz: 1f9982a72dee460c4a9f24c98111973cfc50de06c8e76b249a94fd896c86349f1e3f8d4c409fa43282e9565366d3be9ae32b40e2f37fd35086cf021c8c9187d4
-  data.tar.gz: 3484b0ee9558bd807432e7563543846771baca7e65428b76b1705329a9f252ef95fd4eb09740a8a9781ac33f3991aa2e4f12b17424c1a1b7300228e3d2849acc
+  metadata.gz: 1fff02cdf9e240950924f36ce0eef6333e82c58075dbaf17e83f52a7314fe2a18d42ce619a5b887d338312699a24df83eb5853142e4d7ed91c0fef00213005c2
+  data.tar.gz: 125e1869d4ca0b448f36a0c36da4e4f601e9c8a7c959d8b44fa8532a491b52815b7e941db7d7c9623ce7f6106f7ba2b6d19125510a4d773f8a7d883e0d7b4965

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    html-proofer (0.0.16)
+    html-proofer (0.1.0)
       colored (~> 1.2)
       nokogiri (= 1.6.0)
       typhoeus (~> 0.6.3)

data/lib/html/proofer.rb

@@ -1,4 +1,6 @@
 require 'nokogiri'
+require 'find'
+require 'html/proofer/checkable'
 require 'html/proofer/checks'
 
 module HTML
@@ -16,7 +18,7 @@ module HTML
         Find.find(@srcDir) do |path|
           if File.extname(path) == @options[:ext]
             html = HTML::Proofer.create_nokogiri(path)
-            check = klass.new(path, html, @options)
+            check = klass.new(@srcDir, path, html, @options)
             check.run
             check.hydra.run
             self.print_issues(klass, check.issues)
@@ -24,7 +26,10 @@ module HTML
         end
       end
 
-      if !@failedTests.empty?
+      if @failedTests.empty?
+        puts "Tests executed sucesfully.".green
+        exit 0
+      else
         # make the hash default to 0 so that += will work correctly
         count = Hash.new(0)
 
@@ -41,7 +46,7 @@ module HTML
     end
 
     def self.create_nokogiri(path)
-      path << "index.html" if File.directory? path #support for Jekyll-style links
+      path << "/index.html" if File.directory? path #support for Jekyll-style links
       Nokogiri::HTML(File.read(path))
     end
 

data/lib/html/proofer/check.rb

@@ -10,9 +10,10 @@ class HTML::Proofer::Checks
 
   class Check
 
-    attr_reader :issues, :hydra
+    attr_reader :issues, :hydra, :src, :path, :options, :additional_href_ignores
 
-    def initialize(path, html, opts={})
+    def initialize(src, path, html, opts={})
+      @src    = src
       @path   = path
       @html   = html
       @options = opts
@@ -27,31 +28,13 @@ class HTML::Proofer::Checks
     end
 
     def add_issue(desc)
-      @issues << desc
+      @issues << "#{@path.blue}: #{desc}"
     end
 
     def output_filenames
       Dir[@site.config[:output_dir] + '/**/*'].select{ |f| File.file?(f) }
     end
 
-    def external_href?(href)
-      uri = URI.parse(href)
-      %w( http https ).include?(uri.scheme)
-    rescue URI::BadURIError
-      false
-    rescue URI::InvalidURIError
-      false
-    end
-
-    def ignore_href?(href)
-      uri = URI.parse(href)
-      %w( mailto ).include?(uri.scheme) || @additional_href_ignores.include?(href)
-    rescue URI::BadURIError
-      false
-    rescue URI::InvalidURIError
-      false
-    end
-
     def validate_url(href, issue_text)
       request = Typhoeus::Request.new(href, {:followlocation => true})
       request.on_complete do |response|
@@ -91,5 +74,6 @@ class HTML::Proofer::Checks
 
       classes
     end
+
   end
 end

data/lib/html/proofer/checkable.rb

@@ -0,0 +1,106 @@
+module HTML
+  class Proofer
+    class Checkable
+
+      def initialize(obj, check)
+        @src = obj['src']
+        @href = obj['href']
+        @alt = obj['alt']
+        @name = obj['name']
+        @id = obj['id']
+        @check = check
+
+        if @href && @check.options[:href_swap]
+          @options[:href_swap].each do |link, replace|
+            @href = @href.gsub(link, replace)
+          end
+        end
+
+      end
+
+      def url
+        @src || @href || ""
+      end
+
+      def valid?
+        begin
+          URI.parse url
+        rescue
+          false
+        end
+      end
+
+      def parts
+         URI.parse url
+      end
+
+      def path
+        parts.path
+      end
+
+      def hash
+        parts.fragment
+      end
+
+      # path is to an external server
+      def remote?
+        uri = URI.parse url
+        %w( http https ).include?(uri.scheme)
+      rescue URI::BadURIError
+        false
+      rescue URI::InvalidURIError
+        false
+      end
+
+      def ignore?
+        uri = URI.parse url
+        %w( mailto ).include?(uri.scheme) || @check.additional_href_ignores.include?(href)
+      rescue URI::BadURIError
+        false
+      rescue URI::InvalidURIError
+        false
+      end
+
+      # path is external to the file
+      def external?
+        !internal?
+      end
+
+      # path is an anchor
+      def internal?
+        url[0] == "#"
+      end
+
+      def file_path
+
+        return if path.nil?
+
+        if path =~ /^\// #path relative to root
+          base = @check.src
+        elsif File.exist? File.expand_path path, @check.src #relative links, path is a file
+          base = File.dirname @check.path
+        else #relative link, path is a directory
+          base = @check.path
+        end
+
+        file = File.join base, path
+
+        # implicit /index.html support, with support for tailing slashes
+        file = File.join path, "index.html" if File.directory? File.expand_path file, @check.src
+
+        file
+      end
+
+      # checks if a file exists relative to the current pwd
+      def exists?
+        File.exist? absolute_path
+      end
+
+      def absolute_path
+        path = file_path || @check.path
+        File.expand_path path, Dir.pwd
+      end
+
+    end
+  end
+end

data/lib/html/proofer/checks/images.rb

@@ -1,30 +1,50 @@
 # encoding: utf-8
 
+class Image < ::HTML::Proofer::Checkable
+
+  SCREEN_SHOT_REGEX = /Screen(?: |%20)Shot(?: |%20)\d+-\d+-\d+(?: |%20)at(?: |%20)\d+.\d+.\d+/
+
+  def valid_alt_tag?
+    @alt and !@alt.empty?
+  end
+
+  def terrible_filename?
+    @src =~ SCREEN_SHOT_REGEX
+  end
+
+  def src
+    @src unless @src.nil? || @src.empty?
+  end
+
+  def missing_src?
+    !src
+  end
+
+end
+
 class Images < ::HTML::Proofer::Checks::Check
 
   def run
     @html.css('img').each do |img|
-      src = img['src']
-
-      # check image sources
-      if src && src.length > 0
-        if !external_href?(src)
-          self.add_issue("#{@path}".blue + ": internal image #{src} does not exist") unless src[0] != "/" and File.exist?(File.join(File.dirname(@path), src))
-        else
-          validate_url(src, "#{@path}".blue + ": external image #{src} does not exist")
-        end
+
+      img = Image.new img, self
+
+      # screenshot filenames, return because invalid URL
+      return self.add_issue "image has a terrible filename (#{img.src})" if img.terrible_filename?
+
+      # does the image exist?
+      if img.missing_src?
+        self.add_issue "image has no src attribute"
+      elsif img.remote?
+        validate_url img.src, "external image #{img.src} does not exist"
       else
-        self.add_issue("#{@path}".blue + ": image has no src attribute")
+        self.add_issue("internal image #{img.src} does not exist") unless img.exists?
       end
 
       # check alt tag
-      self.add_issue("#{@path}".blue + ": image #{src} does not have an alt attribute") unless img['alt'] and !img['alt'].empty?
-      
-      screenShotRegExp = /Screen(?: |%20)Shot(?: |%20)\d+-\d+-\d+(?: |%20)at(?: |%20)\d+.\d+.\d+/
+      self.add_issue "image #{img.src} does not have an alt attribute" unless img.valid_alt_tag?
+
 
-      if src =~ screenShotRegExp
-        self.add_issue("#{@path}".blue + ": image has a terrible filename (#{src})")
-      end
     end
   end
 end

data/lib/html/proofer/checks/links.rb

@@ -1,57 +1,48 @@
 # encoding: utf-8
 
+class Link < ::HTML::Proofer::Checkable
+
+  def href
+    @href unless @href.nil? || @href.empty?
+  end
+
+  def missing_href?
+    href.nil? and @name.nil? and @id.nil?
+  end
+
+end
+
 class Links < ::HTML::Proofer::Checks::Check
 
   def run
-    @html.css('a').each do |a|
-      href = a['href']
+    @html.css('a').each do |link|
 
-      if href && href.length > 0
-        if @options[:href_swap]
-          @options[:href_swap].each do |link, replace|
-            href = href.gsub(link, replace)
-          end
-        end
+      link = Link.new link, self
+      return if link.ignore?
 
-        return if ignore_href?(href)
+      # is there even a href?
+      return self.add_issue("link has no href attribute") if link.missing_href?
 
-        if href.include? '#'
-          href_split = href.split('#')
-        end
-        if !external_href?(href)
-          # an internal link, with a hash
-          if href_split && !href_split.empty?
-            href_file = href_split[0]
-            href_hash = href_split[1]
+      # is it even a valid URL?
+      return self.add_issue "#{link.href} is an invalid URL" unless link.valid?
 
-            # it's not an internal hash; it's pointing to some other file
-            if href_file.length > 0
-              href_location = resolve_path File.join(File.dirname(@path), href_file)
-              if !File.exist?(href_location)
-                self.add_issue("#{@path}".blue + ": internal link #{href_location} does not exist")
-              else
-                href_html = HTML::Proofer.create_nokogiri(href_location)
-                found_hash_match = false
-                unless hash_check(href_html, href_hash)
-                  self.add_issue("#{@path}".blue + ": linking to #{href}, but #{href_hash} does not exist")
-                end
-              end
-            # it is an internal link, with an internal hash
-            else
-              unless hash_check(@html, href_hash)
-                self.add_issue("#{@path}".blue + ": linking to an internal hash called #{href_hash} that does not exist")
-              end
-            end
-          # internal link, no hash
-          else
-            href_location = resolve_path File.join(File.dirname(@path), href)
-            self.add_issue("#{@path}".blue + ": internally linking to #{href_location}, which does not exist") unless File.exist?(href_location)
-          end
-        else
-          validate_url(href, "#{@path}".blue + ": externally linking to #{href}, which does not exist")
-        end
+      # does the file even exist?
+      if link.remote?
+        validate_url link.href, "externally linking to #{link.href}, which does not exist"
       else
-        self.add_issue("#{@path}".blue + ": link has no href attribute") unless a['name'] || a['id']
+        self.add_issue "internally linking to #{link.href}, which does not exist" unless link.exists?
+      end
+
+      # verify the target hash
+      if link.hash
+        if link.remote?
+          #not yet checked
+        elsif link.internal?
+          self.add_issue "linking to internal hash ##{link.hash} that does not exist" unless hash_check @html, link.hash
+        elsif link.external?
+          target_html = HTML::Proofer.create_nokogiri link.absolute_path
+          self.add_issue "linking to #{link.href}, but #{link.hash} does not exist" unless hash_check target_html, link.hash
+        end
       end
     end
   end
@@ -60,9 +51,4 @@ class Links < ::HTML::Proofer::Checks::Check
     html.xpath("//*[@id='#{href_hash}']", "//*[@name='#{href_hash}']").length > 0
   end
 
-  #support for implicit /index.html in URLs
-  def resolve_path(path)
-    path << "index.html" if File.directory? path
-    path
-  end
 end

data/lib/html/proofer/version.rb

@@ -1,5 +1,5 @@
 module HTML
   class Proofer
-    VERSION = "0.0.16"
+    VERSION = "0.1.0"
   end
 end

data/spec/html/proofer/fixtures/brokenLinkWithNumber.html

@@ -2,7 +2,7 @@
 
 <body>
 
-	<p>Blah blah blah. <a name="25-method-not-allowed" class="anchor" href="#25-method-not-allowed"><span class="mini-icon mini-icon-link"></span></a>25 Method not allowed</h4></p>
+	<p>Blah blah blah. <a class="anchor" href="#25-method-not-allowed"><span class="mini-icon mini-icon-link"></span></a>25 Method not allowed</h4></p>
 </body>
 
-</html>
+</html>

data/spec/html/proofer/fixtures/index.html

@@ -1 +1 @@
-root.
+<h1 id="anchor">root.</a>

data/spec/html/proofer/fixtures/relativeLinks.html

@@ -0,0 +1,4 @@
+<a href="/index.html">Relative to root</a>
+<a href="index.html">Relative to self</a>
+<a href="folder/">Folder relative to self</a>
+<a href="index.html#anchor">Anchor relative to self</a>

data/spec/html/proofer/fixtures/rootLink.html

@@ -3,6 +3,7 @@
 <body>
 
   <p>Blah blah blah. <a href="/">I am root.</a></p>
+  <p>Blah blah blah. <a href="/folder">I am relative to root.</a></p>
 
 </body>
 

data/spec/html/proofer/images_spec.rb

@@ -3,35 +3,28 @@ require "spec_helper"
 describe "Image tests" do
   it "passes for existing external images" do
     externalImageFilepath = "#{FIXTURES_DIR}/existingImageExternal.html"
-    @imageCheck = Images.new(externalImageFilepath, HTML::Proofer.create_nokogiri(externalImageFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", externalImageFilepath, HTML::Proofer.create_nokogiri(externalImageFilepath))
     @imageCheck.run
     @imageCheck.issues[0].should eq(nil)
   end
 
   it "fails for image without alt attribute" do
     missingAltFilepath = "#{FIXTURES_DIR}/missingImageAlt.html"
-    @imageCheck = Images.new(missingAltFilepath, HTML::Proofer.create_nokogiri(missingAltFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", missingAltFilepath, HTML::Proofer.create_nokogiri(missingAltFilepath))
     @imageCheck.run
     @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/missingImageAlt.html".blue + ": image ./gpl.png does not have an alt attribute")
   end
 
   it "fails for image with an empty alt attribute" do
     missingAltFilepath = "#{FIXTURES_DIR}/missingImageAltText.html"
-    @imageCheck = Images.new(missingAltFilepath, HTML::Proofer.create_nokogiri(missingAltFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", missingAltFilepath, HTML::Proofer.create_nokogiri(missingAltFilepath))
     @imageCheck.run
     @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/missingImageAltText.html".blue + ": image ./gpl.png does not have an alt attribute")
   end
 
-  it "fails for image without a dir prefix" do
-    missingImageDirPrefixFilepath = "#{FIXTURES_DIR}/missingImageDirPrefix.html"
-    @imageCheck = Images.new(missingImageDirPrefixFilepath, HTML::Proofer.create_nokogiri(missingImageDirPrefixFilepath))
-    @imageCheck.run
-    @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/missingImageDirPrefix.html".blue + ": internal image /gpl.png does not exist")
-  end
-
   it "fails for missing external images" do
     externalImageFilepath = "#{FIXTURES_DIR}/missingImageExternal.html"
-    @imageCheck = Images.new(externalImageFilepath, HTML::Proofer.create_nokogiri(externalImageFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", externalImageFilepath, HTML::Proofer.create_nokogiri(externalImageFilepath))
     @imageCheck.run
     @imageCheck.hydra.run
     @imageCheck.issues[0].sub!(/ #<Typhoeus::Response:[\w]+>/, "")
@@ -40,21 +33,21 @@ describe "Image tests" do
 
   it "fails for missing internal images" do
     internalImageFilepath = "#{FIXTURES_DIR}/missingImageInternal.html"
-    @imageCheck = Images.new(internalImageFilepath, HTML::Proofer.create_nokogiri(internalImageFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", internalImageFilepath, HTML::Proofer.create_nokogiri(internalImageFilepath))
     @imageCheck.run
     @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/missingImageInternal.html".blue + ": internal image ./doesnotexist.png does not exist")
   end
 
   it "fails for image with no src" do
     imageSrcFilepath = "#{FIXTURES_DIR}/missingImageSrc.html"
-    @imageCheck = Images.new(imageSrcFilepath, HTML::Proofer.create_nokogiri(imageSrcFilepath))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", imageSrcFilepath, HTML::Proofer.create_nokogiri(imageSrcFilepath))
     @imageCheck.run
     @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/missingImageSrc.html".blue + ": image has no src attribute")
   end
 
   it "fails for image with default mac filename" do
     terribleImageName = "#{FIXTURES_DIR}/terribleImageName.html"
-    @imageCheck = Images.new(terribleImageName, HTML::Proofer.create_nokogiri(terribleImageName))
+    @imageCheck = Images.new("#{FIXTURES_DIR}", terribleImageName, HTML::Proofer.create_nokogiri(terribleImageName))
     @imageCheck.run
     @imageCheck.issues[0].should eq("spec/html/proofer/fixtures/terribleImageName.html".blue + ": image has a terrible filename (./Screen Shot 2012-08-09 at 7.51.18 AM.png)")
   end

data/spec/html/proofer/links_spec.rb

@@ -4,21 +4,21 @@ describe "Links tests" do
 
   it "fails for broken external hash (even if the file exists)" do
     brokenHashExternalFilepath = "#{FIXTURES_DIR}/brokenHashExternal.html"
-    @linkCheck = Links.new(brokenHashExternalFilepath, HTML::Proofer.create_nokogiri(brokenHashExternalFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", brokenHashExternalFilepath, HTML::Proofer.create_nokogiri(brokenHashExternalFilepath))
     @linkCheck.run
     @linkCheck.issues[1].should eq("spec/html/proofer/fixtures/brokenHashExternal.html".blue + ": linking to ./missingImageAlt.html#asdfasfdkafl, but asdfasfdkafl does not exist")
   end
 
   it "fails for broken internal hash" do
     brokenHashInternalFilepath = "#{FIXTURES_DIR}/brokenHashInternal.html"
-    @linkCheck = Links.new(brokenHashInternalFilepath, HTML::Proofer.create_nokogiri(brokenHashInternalFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", brokenHashInternalFilepath, HTML::Proofer.create_nokogiri(brokenHashInternalFilepath))
     @linkCheck.run
-    @linkCheck.issues[0].should eq("spec/html/proofer/fixtures/brokenHashInternal.html".blue + ": linking to an internal hash called noHash that does not exist")
+    @linkCheck.issues[0].should eq("spec/html/proofer/fixtures/brokenHashInternal.html".blue + ": linking to internal hash #noHash that does not exist")
   end
 
   it "fails for broken external links" do
     brokenLinkExternalFilepath = "#{FIXTURES_DIR}/brokenLinkExternal.html"
-    @linkCheck = Links.new(brokenLinkExternalFilepath, HTML::Proofer.create_nokogiri(brokenLinkExternalFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", brokenLinkExternalFilepath, HTML::Proofer.create_nokogiri(brokenLinkExternalFilepath))
     @linkCheck.run
     @linkCheck.hydra.run
     @linkCheck.issues[0].sub!(/ #<Typhoeus::Response:[\w]+>/, "")
@@ -27,49 +27,56 @@ describe "Links tests" do
 
   it "fails for broken internal links" do
     brokenLinkInternalFilepath = "#{FIXTURES_DIR}/brokenLinkInternal.html"
-    @linkCheck = Links.new(brokenLinkInternalFilepath, HTML::Proofer.create_nokogiri(brokenLinkInternalFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", brokenLinkInternalFilepath, HTML::Proofer.create_nokogiri(brokenLinkInternalFilepath))
     @linkCheck.run
-    @linkCheck.issues[0].should eq("spec/html/proofer/fixtures/brokenLinkInternal.html".blue + ": internally linking to spec/html/proofer/fixtures/./notreal.html, which does not exist")
+    @linkCheck.issues[0].should eq("spec/html/proofer/fixtures/brokenLinkInternal.html".blue + ": internally linking to ./notreal.html, which does not exist")
   end
 
   it "fails for link with no href" do
     missingLinkHrefFilepath = "#{FIXTURES_DIR}/missingLinkHref.html"
-    @linkCheck = Links.new(missingLinkHrefFilepath, HTML::Proofer.create_nokogiri(missingLinkHrefFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", missingLinkHrefFilepath, HTML::Proofer.create_nokogiri(missingLinkHrefFilepath))
     @linkCheck.run
     @linkCheck.issues[0].should eq("spec/html/proofer/fixtures/missingLinkHref.html".blue + ": link has no href attribute")
   end
 
   it "should follow redirects" do
     linkWithRedirectFilepath = "#{FIXTURES_DIR}/linkWithRedirect.html"
-    @linkCheck = Links.new(linkWithRedirectFilepath, HTML::Proofer.create_nokogiri(linkWithRedirectFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", linkWithRedirectFilepath, HTML::Proofer.create_nokogiri(linkWithRedirectFilepath))
     @linkCheck.run
     @linkCheck.issues[0].should eq(nil)
   end
 
   it "should understand https" do
     linkWithHttpsFilepath = "#{FIXTURES_DIR}/linkWithHttps.html"
-    @linkCheck = Links.new(linkWithHttpsFilepath, HTML::Proofer.create_nokogiri(linkWithHttpsFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", linkWithHttpsFilepath, HTML::Proofer.create_nokogiri(linkWithHttpsFilepath))
     @linkCheck.run
     @linkCheck.issues[0].should eq(nil)
   end
 
   it "fails for broken hash links with status code numbers" do
     brokenLinkWithNumberFilepath = "#{FIXTURES_DIR}/brokenLinkWithNumber.html"
-    @linkCheck = Links.new(brokenLinkWithNumberFilepath, HTML::Proofer.create_nokogiri(brokenLinkWithNumberFilepath))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", brokenLinkWithNumberFilepath, HTML::Proofer.create_nokogiri(brokenLinkWithNumberFilepath))
     @linkCheck.run
-    @linkCheck.issues[0].should eq(nil)
+    @linkCheck.issues[0].should eq("\e[34mspec/html/proofer/fixtures/brokenLinkWithNumber.html\e[0m: linking to internal hash #25-method-not-allowed that does not exist")
   end
 
   it 'properly resolves implicit /index.html in link paths' do
     linkToFolder = "#{FIXTURES_DIR}/linkToFolder.html"
-    @linkCheck = Links.new(linkToFolder, HTML::Proofer.create_nokogiri(linkToFolder))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", linkToFolder, HTML::Proofer.create_nokogiri(linkToFolder))
     @linkCheck.run
     @linkCheck.issues[0].should eq(nil)
   end
 
   it 'properly checks links to root' do
     rootLink = "#{FIXTURES_DIR}/rootLink.html"
-    @linkCheck = Links.new(rootLink, HTML::Proofer.create_nokogiri(rootLink))
+    @linkCheck = Links.new("#{FIXTURES_DIR}", rootLink, HTML::Proofer.create_nokogiri(rootLink))
+    @linkCheck.run
+    @linkCheck.issues[0].should eq(nil)
+  end
+
+  it 'properly checks relative links' do
+    relativeLinks = "#{FIXTURES_DIR}/relativeLinks.html"
+    @linkCheck = Links.new("#{FIXTURES_DIR}", relativeLinks, HTML::Proofer.create_nokogiri(relativeLinks))
     @linkCheck.run
     @linkCheck.issues[0].should eq(nil)
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.1.0
 platform: ruby
 authors:
 - Garen Torikian
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-09 00:00:00.000000000 Z
+date: 2013-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -112,6 +112,7 @@ files:
 - lib/html-proofer.rb
 - lib/html/proofer.rb
 - lib/html/proofer/check.rb
+- lib/html/proofer/checkable.rb
 - lib/html/proofer/checks.rb
 - lib/html/proofer/checks/images.rb
 - lib/html/proofer/checks/links.rb
@@ -119,10 +120,12 @@ files:
 - spec/html/proofer/fixtures/Screen Shot 2012-08-09 at 7.51.18 AM.png
 - spec/html/proofer/fixtures/brokenHashExternal.html
 - spec/html/proofer/fixtures/brokenHashInternal.html
+- spec/html/proofer/fixtures/brokenInternalLink.html
 - spec/html/proofer/fixtures/brokenLinkExternal.html
 - spec/html/proofer/fixtures/brokenLinkInternal.html
 - spec/html/proofer/fixtures/brokenLinkWithNumber.html
 - spec/html/proofer/fixtures/existingImageExternal.html
+- spec/html/proofer/fixtures/folder/anchorLink.html
 - spec/html/proofer/fixtures/folder/index.html
 - spec/html/proofer/fixtures/gpl.png
 - spec/html/proofer/fixtures/index.html
@@ -131,11 +134,12 @@ files:
 - spec/html/proofer/fixtures/linkWithRedirect.html
 - spec/html/proofer/fixtures/missingImageAlt.html
 - spec/html/proofer/fixtures/missingImageAltText.html
-- spec/html/proofer/fixtures/missingImageDirPrefix.html
 - spec/html/proofer/fixtures/missingImageExternal.html
 - spec/html/proofer/fixtures/missingImageInternal.html
 - spec/html/proofer/fixtures/missingImageSrc.html
 - spec/html/proofer/fixtures/missingLinkHref.html
+- spec/html/proofer/fixtures/notarealhash.html
+- spec/html/proofer/fixtures/relativeLinks.html
 - spec/html/proofer/fixtures/rootLink.html
 - spec/html/proofer/fixtures/terribleImageName.html
 - spec/html/proofer/images_spec.rb
@@ -172,10 +176,12 @@ test_files:
 - spec/html/proofer/fixtures/Screen Shot 2012-08-09 at 7.51.18 AM.png
 - spec/html/proofer/fixtures/brokenHashExternal.html
 - spec/html/proofer/fixtures/brokenHashInternal.html
+- spec/html/proofer/fixtures/brokenInternalLink.html
 - spec/html/proofer/fixtures/brokenLinkExternal.html
 - spec/html/proofer/fixtures/brokenLinkInternal.html
 - spec/html/proofer/fixtures/brokenLinkWithNumber.html
 - spec/html/proofer/fixtures/existingImageExternal.html
+- spec/html/proofer/fixtures/folder/anchorLink.html
 - spec/html/proofer/fixtures/folder/index.html
 - spec/html/proofer/fixtures/gpl.png
 - spec/html/proofer/fixtures/index.html
@@ -184,11 +190,12 @@ test_files:
 - spec/html/proofer/fixtures/linkWithRedirect.html
 - spec/html/proofer/fixtures/missingImageAlt.html
 - spec/html/proofer/fixtures/missingImageAltText.html
-- spec/html/proofer/fixtures/missingImageDirPrefix.html
 - spec/html/proofer/fixtures/missingImageExternal.html
 - spec/html/proofer/fixtures/missingImageInternal.html
 - spec/html/proofer/fixtures/missingImageSrc.html
 - spec/html/proofer/fixtures/missingLinkHref.html
+- spec/html/proofer/fixtures/notarealhash.html
+- spec/html/proofer/fixtures/relativeLinks.html
 - spec/html/proofer/fixtures/rootLink.html
 - spec/html/proofer/fixtures/terribleImageName.html
 - spec/html/proofer/images_spec.rb

data/spec/html/proofer/fixtures/missingImageDirPrefix.html

@@ -1,9 +0,0 @@
-<html>
-
-<body>
-
-	<p>Blah blah blah. <img alt="A broken image, due to dir prefix" src="/gpl.png" /> </p>
-
-</body>
-
-</html>