bunto-redirect-from 5.0.0 → 6.0.0

data/spec/bunto_redirect_from/redirector_spec.rb

@@ -1,116 +1,149 @@
-require "spec_helper"
-
-describe BuntoRedirectFrom::Redirector do
-  let(:redirector)                  { described_class.new }
-  let(:post_to_redirect)            { setup_post("2014-01-03-redirect-me-plz.md") }
-  let(:doc_to_redirect_from)        { setup_doc("redirect-me-plz.md") }
-  let(:doc_to_redirect_to)          { setup_doc("redirect-somewhere-else-plz.html") }
-  let(:page_with_one)               { setup_page("one_redirect_url.md") }
-  let(:page_with_many)              { setup_page("multiple_redirect_urls.md") }
-  let(:page_with_one_redirect_to)   { setup_page("one_redirect_to.md") }
-  let(:page_with_many_redirect_to)  { setup_page("multiple_redirect_tos.md") }
-
-  it "knows if a page or post is requesting a redirect page" do
-    if Bunto::VERSION < '3.0.0'
-      expect(redirector.has_alt_urls?(post_to_redirect)).to be_truthy
-    else
-      skip "Don't need to test posts in Bunto 3"
-    end
-  end
-
-  it "knows if a document is requesting a redirect page" do
-    expect(redirector.has_alt_urls?(doc_to_redirect_from)).to be_truthy
-  end
-
-  it "knows if a document is requesting a redirect away" do
-    expect(redirector.redirect_to_url(doc_to_redirect_to)).to eql(["http://www.zombo.com"])
-  end
-
-  it "handles one redirect path" do
-    expect(redirector.alt_urls(page_with_one)).to eql(["mencius/was/my/father"])
-  end
-
-  it "handles many redirect paths" do
-    expect(redirector.alt_urls(page_with_many)).to eql(["help", "contact", "let-there/be/light-he-said", "/geepers/mccreepin"])
-  end
-
-  it "handles a single redirect_to url" do
-    expect(redirector.redirect_to_url(page_with_one_redirect_to)).to eql(["https://www.github.com"])
-  end
-
-  it "handles a many redirect_to urls" do
-    expect(redirector.redirect_to_url(page_with_many_redirect_to)).to eql(["https://www.bunto.isc"])
-  end
-
-  it "does not include pages with a redirect in sitemap" do
-    expect(destination_sitemap).not_to include(%|one_redirect_to.html|)
-  end
-
-  context "refresh page generation" do
-    before(:each) do
-      described_class.new.generate(@site)
-    end
-
-    it "generates the refresh page for the post properly" do
-      expect(destination_file_exists?("posts/23128432159832/mary-had-a-little-lamb")).to be_truthy
-    end
-
-    it "generates the refresh pages for the page with multiple redirect_from urls" do
-      expect(destination_file_exists?("help")).to be_truthy
-      expect(destination_file_exists?("contact")).to be_truthy
-      expect(destination_file_exists?("let-there/be/light-he-said")).to be_truthy
-      expect(destination_file_exists?("/geepers/mccreepin")).to be_truthy
-    end
-
-    it "generates the refresh page for the page with one redirect_from url" do
-      expect(destination_file_exists?("mencius/was/my/father")).to be_truthy
-    end
-
-    it "generates the refresh page for the collection with one redirect_to url" do
-      expect(@dest.join("articles", "redirect-somewhere-else-plz.html")).to exist
-      expect(destination_doc_contents("articles", "redirect-somewhere-else-plz.html")).to include(%|<meta http-equiv="refresh" content="0; url=http://www.zombo.com">|)
-    end
-
-    it "generates the refresh page for the page with one redirect_to url" do
-      expect(destination_file_exists?("one_redirect_to.html")).to be_truthy
-      expect(destination_file_contents("one_redirect_to.html")).to include(%|<meta http-equiv="refresh" content="0; url=https://www.github.com">|)
-    end
-
-    it "generates the refresh page for the page with multiple redirect_to urls" do
-      expect(destination_file_exists?("multiple_redirect_tos.html")).to be_truthy
-      expect(destination_file_contents("multiple_redirect_tos.html")).to include(%|<meta http-equiv="refresh" content="0; url=https://www.bunto.isc">|)
-    end
-  end
-
-  context "prefix" do
-    it "uses site.github.url as the redirect prefix" do
-      @site.config['github'] = { "url" => "http://example.github.io/test" }
-      expect(redirector.redirect_url(@site, page_with_one)).to start_with("http://example.github.io/test")
-    end
-
-    it "converts non-string values in site.github.url to strings" do
-      @site.config['github'] = { "url" => TestStringContainer.new("http://example.github.io/test") }
-      expect(redirector.redirect_url(@site, page_with_one)).to start_with("http://example.github.io/test")
-    end
-
-    it "uses site.baseurl as the redirect prefix when site.github.url is not set" do
-      @site.config['baseurl'] = "/fancy/prefix"
-      expect(redirector.redirect_url(@site, page_with_one)).to start_with("/fancy/prefix")
-    end
-
-    it "prefers site.github.url over site.baseurl" do
-      @site.config['github'] = { "url" => "http://example.github.io/test" }
-      @site.config['baseurl'] = "/fancy/baseurl"
-      expect(redirector.redirect_url(@site, page_with_one)).to start_with("http://example.github.io/test")
-    end
-
-    it "no-ops when site.github.url and site.baseurl are not set" do
-      expect(redirector.redirect_url(@site, page_with_one)).to eql("/one_redirect_url.html")
-    end
-
-    it "no-ops when site.github is set but site.github.url is not" do
-      @site.config['github'] = "username"
-      expect(redirector.redirect_url(@site, page_with_one)).to eql("/one_redirect_url.html")
-    end
-  end
-end
+require "spec_helper"
+
+describe BuntoRedirectFrom::Redirector do
+  let(:redirector)                  { described_class.new }
+  let(:post_to_redirect)            { setup_post("2014-01-03-redirect-me-plz.md") }
+  let(:doc_to_redirect_from)        { setup_doc("redirect-me-plz.md") }
+  let(:doc_to_redirect_to)          { setup_doc("redirect-somewhere-else-plz.html") }
+  let(:doc_to_redirect_to_permalnk) { setup_doc("redirect-somewhere-else-im-a-permalink.html") }
+  let(:page_with_one)               { setup_page("one_redirect_url.md") }
+  let(:page_with_many)              { setup_page("multiple_redirect_urls.md") }
+  let(:page_with_one_redirect_to)   { setup_page("one_redirect_to.md") }
+  let(:page_with_many_redirect_to)  { setup_page("multiple_redirect_tos.md") }
+  let(:page_to_redirect_to_permlnk) { setup_page("tags/how we work.md") }
+
+  it "knows if a page or post is requesting a redirect page" do
+    if BuntoRedirectFrom.bunto_3?
+      skip "Don't need to test posts in Bunto 3"
+    else
+      expect(redirector.has_alt_urls?(post_to_redirect)).to be_truthy
+    end
+  end
+
+  it "knows if a document is requesting a redirect page" do
+    expect(redirector.has_alt_urls?(doc_to_redirect_from)).to be_truthy
+  end
+
+  it "knows if a document is requesting a redirect away" do
+    expect(redirector.redirect_to_url(doc_to_redirect_to)).to eql(["http://www.zombo.com"])
+  end
+
+  it "knows if a document is requesting a redirect away" do
+    expect(redirector.redirect_to_url(doc_to_redirect_to_permalnk)).to eql(["/tags/our-projects/"])
+  end
+
+  it "handles one redirect path" do
+    expect(redirector.alt_urls(page_with_one)).to eql(["mencius/was/my/father"])
+  end
+
+  it "handles many redirect paths" do
+    expect(redirector.alt_urls(page_with_many)).to eql(["help", "contact", "let-there/be/light-he-said", "/geepers/mccreepin"])
+  end
+
+  it "handles a single redirect_to url" do
+    expect(redirector.redirect_to_url(page_with_one_redirect_to)).to eql(["https://www.github.com"])
+  end
+
+  it "handles a many redirect_to urls" do
+    expect(redirector.redirect_to_url(page_with_many_redirect_to)).to eql(["https://www.buntorb.com"])
+  end
+
+  it "does not include pages with a redirect in sitemap" do
+    expect(destination_sitemap).not_to include(%|one_redirect_to.html|)
+  end
+
+  context "refresh page generation" do
+    before(:each) do
+      described_class.new.generate(@site)
+    end
+
+    it "generates the refresh page for the post properly" do
+      expect(dest_dir("posts/23128432159832/mary-had-a-little-lamb#{forced_output_ext}")).to exist
+    end
+
+    it "generates the refresh pages for the page with multiple redirect_from urls" do
+      expect(dest_dir("help")).to be_truthy
+      expect(dest_dir("contact")).to be_truthy
+      expect(dest_dir("let-there/be/light-he-said")).to be_truthy
+      expect(dest_dir("/geepers/mccreepin")).to be_truthy
+    end
+
+    it "generates the refresh page for the page with one redirect_from url" do
+      expect(dest_dir("mencius/was/my/father#{forced_output_ext}")).to exist
+    end
+
+    it "generates the refresh page for the collection with one redirect_to url" do
+      expect(dest_dir("articles", "redirect-somewhere-else-plz.html")).to exist
+      expect(dest_dir("articles", "redirect-somewhere-else-plz.html").read).to include(%|<meta http-equiv="refresh" content="0; url=http://www.zombo.com">|)
+    end
+
+    it "generates the refresh page for the collection with one redirect_to url and a permalink" do
+      expect(dest_dir("tags", "our projects", "index")).not_to exist
+      expect(dest_dir("tags", "our projects", "index.html")).to exist
+      expect(dest_dir("tags", "our projects", "index.html").read).to include(%|<meta http-equiv="refresh" content="0; url=/tags/our-projects/">|)
+    end
+
+    it "generates the refresh page for the page with one redirect_to url" do
+      expect(dest_dir("one_redirect_to.html")).to exist
+      expect(dest_dir("one_redirect_to.html").read).to include(%|<meta http-equiv="refresh" content="0; url=https://www.github.com">|)
+    end
+
+    it "generates the refresh page for the page with multiple redirect_to urls" do
+      expect(dest_dir("multiple_redirect_tos.html")).to exist
+      expect(dest_dir("multiple_redirect_tos.html").read).to include(%|<meta http-equiv="refresh" content="0; url=https://www.buntorb.com">|)
+    end
+
+    it "does not include any default layout" do
+      expect(dest_dir("multiple_redirect_tos.html")).to exist
+      expect(dest_dir("multiple_redirect_tos.html").read).not_to include('LAYOUT INCLUDED')
+    end
+
+    it "generates the refresh page for the page with one redirect_to url and a permalink" do
+      expect(dest_dir("tags", "how we work", "index")).not_to exist
+      expect(dest_dir("tags", "how we work", "index.html")).to exist
+      expect(dest_dir("tags", "how we work", "index.html").read).to include(%|<meta http-equiv="refresh" content="0; url=/tags/how-we-work/">|)
+    end
+  end
+
+  context "prefix" do
+    it "uses site.url as the redirect prefix when site.github.url is not set" do
+      @site.config['url'] = "http://notgithub.io"
+      @site.config['baseurl'] = nil
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("http://notgithub.io/one_redirect_url.html")
+    end
+
+    it "uses site.baseurl as the redirect prefix when site.github.url is not set" do
+      @site.config['url'] = nil
+      @site.config['baseurl'] = "/fancy/prefix"
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("/fancy/prefix/one_redirect_url.html")
+    end
+
+    it "uses site.url + site.baseurl as the redirect prefix when site.github.url is not set" do
+      @site.config['url'] = "http://notgithub.io"
+      @site.config['baseurl'] = "/fancy/prefix"
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("http://notgithub.io/fancy/prefix/one_redirect_url.html")
+    end
+
+    it "prefers site.github.url over site.url or site.baseurl" do
+      @site.config['url'] = "http://notgithub.io"
+      @site.config['baseurl'] = "/fancy/prefix"
+      @site.config['github'] = { "url" => "http://example.github.io/test" }
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("http://example.github.io/test/one_redirect_url.html")
+    end
+
+    it "converts non-string values in site.github.url to strings" do
+      @site.config['github'] = { "url" => TestStringContainer.new("http://example.github.io/test") }
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("http://example.github.io/test/one_redirect_url.html")
+    end
+
+    it "uses site.url when site.github is set but site.github.url is not" do
+      @site.config['github'] = "username"
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("http://bunto.github.io/one_redirect_url.html")
+    end
+
+    it "no-ops when site.github.url and site.baseurl and site.url are not set" do
+      @site.config['url'] = nil
+      expect(redirector.redirect_url(@site, page_with_one)).to eql("/one_redirect_url.html")
+    end
+  end
+end

data/spec/fixtures/_articles/redirect-me-plz.md

@@ -1,6 +1,6 @@
----
-title: Please redirect me, sir.
-redirect_from: /articles/23128432159832/mary-had-a-little-lamb
----
-
-Yay.
+---
+title: Please redirect me, sir.
+redirect_from: /articles/23128432159832/mary-had-a-little-lamb
+---
+
+Yay.

data/spec/fixtures/_articles/redirect-somewhere-else-im-a-permalink.html

@@ -0,0 +1,7 @@
+---
+title: Please redirect away from me and my permalink.
+permalink: /tags/our projects/
+redirect_to: /tags/our-projects/
+---
+
+Bye.

data/spec/fixtures/_articles/redirect-somewhere-else-plz.html

@@ -1,6 +1,6 @@
----
-title: Please redirect away from me, sir.
-redirect_to: "http://www.zombo.com"
----
-
-Boo.
+---
+title: Please redirect away from me, sir.
+redirect_to: "http://www.zombo.com"
+---
+
+Boo.

data/spec/fixtures/_authors/kansaichris.md

@@ -1,5 +1,5 @@
----
-redirect_from: /kansaichris/
----
-
-Hi.
+---
+redirect_from: /kansaichris/
+---
+
+Hi.

data/spec/fixtures/_config.yml

@@ -1,7 +1,7 @@
-url: http://bunto.isc
-gems:
-  - bunto-redirect-from
-  - bunto-sitemap
-collections:
-  articles:
-    output: true
+url: https://bunto.github.io
+gems:
+  - bunto-redirect-from
+  - bunto-sitemap
+collections:
+  articles:
+    output: true

data/spec/fixtures/_layouts/layout.html

@@ -0,0 +1,6 @@
+---
+---
+
+LAYOUT INCLUDED
+
+{{ content }}

data/spec/fixtures/_posts/2014-01-03-redirect-me-plz.md

@@ -1,6 +1,6 @@
----
-title: Please redirect me, sir.
-redirect_from: /posts/23128432159832/mary-had-a-little-lamb
----
-
-Yay.
+---
+title: Please redirect me, sir.
+redirect_from: /posts/23128432159832/mary-had-a-little-lamb
+---
+
+Yay.

data/spec/fixtures/multiple_redirect_tos.md

@@ -1,9 +1,9 @@
----
-title: I have lots of redirect to urls
-redirect_to:
-- https://www.bunto.isc
-- https://www.github.com
-- https://www.twitter.com
----
-
-Lots of redirect to urls.
+---
+title: I have lots of redirect to urls
+redirect_to:
+- https://bunto.github.io
+- https://www.github.com
+- https://www.twitter.com
+---
+
+Lots of redirect to urls.

data/spec/fixtures/multiple_redirect_urls.md

@@ -1,10 +1,10 @@
----
-title: I have lots of redirect urls
-redirect_from:
-- help
-- contact
-- let-there/be/light-he-said
-- /geepers/mccreepin
----
-
-Lots of redirect urls
+---
+title: I have lots of redirect urls
+redirect_from:
+- help
+- contact
+- let-there/be/light-he-said
+- /geepers/mccreepin
+---
+
+Lots of redirect urls

data/spec/fixtures/one_redirect_to.md

@@ -1,6 +1,6 @@
----
-title: I am going somewhere else.
-redirect_to: https://www.github.com
----
-
-Redirecting elsewhere.
+---
+title: I am going somewhere else.
+redirect_to: https://www.github.com
+---
+
+Redirecting elsewhere.

data/spec/fixtures/one_redirect_url.md

@@ -1,6 +1,6 @@
----
-title: I only have one redirect url.
-redirect_from: mencius/was/my/father
----
-
-One redirect url
+---
+title: I only have one redirect url.
+redirect_from: mencius/was/my/father
+---
+
+One redirect url

data/spec/fixtures/tags/how we work.md

@@ -0,0 +1,4 @@
+---
+redirect_to: "/tags/how-we-work/"
+permalink: "/tags/how we work/"
+---

data/spec/integrations_spec.rb

@@ -1,13 +1,13 @@
-require 'spec_helper'
-
-describe("Integration Tests") do
-  it "writes the redirect pages for collection items which are outputted" do
-    expect(@dest.join("articles", "redirect-me-plz.html")).to exist
-    expect(@dest.join("articles", "23128432159832", "mary-had-a-little-lamb")).to exist
-  end
-
-  it "doesn't write redirect pages for collection items which are not outputted" do
-    expect(@dest.join("authors")).not_to exist
-    expect(@dest.join("kansaichris")).not_to exist
-  end
-end
+require 'spec_helper'
+
+describe("Integration Tests") do
+  it "writes the redirect pages for collection items which are outputted" do
+    expect(dest_dir("articles", "redirect-me-plz.html")).to exist
+    expect(dest_dir("articles", "23128432159832", "mary-had-a-little-lamb#{forced_output_ext}")).to exist
+  end
+
+  it "doesn't write redirect pages for collection items which are not outputted" do
+    expect(dest_dir("authors")).not_to exist
+    expect(dest_dir("kansaichris")).not_to exist
+  end
+end