append .HTML to extensionlist redirects

Author: benbalter

lib/jekyll-redirect-from/redirect_page.rb

@@ -52,15 +52,6 @@ def set_paths(from, to)
       })
     end
 
-    # Redirects without a trailing slash should produce extentionless files
-    def output_ext
-      if File.extname(permalink).empty? && !permalink.end_with?("/")
-        ""
-      else
-        _renderer.output_ext
-      end
-    end
-
     private
 
     def context

spec/integrations_spec.rb

@@ -6,7 +6,7 @@
 
   context "pages" do
     context "single redirect from" do
-      let(:relative_path) { "some/other/path" }
+      let(:relative_path) { "some/other/path.html" }
 
       it "exists in the built site" do
         expect(path).to exist
@@ -17,7 +17,7 @@
     context "multiple redirect froms" do
       %w(help contact let-there/be/light-he-said geepers/mccreepin).each do |redirect|
         context "the #{redirect} redirect" do
-          let(:relative_path) { redirect }
+          let(:relative_path) { "#{redirect}.html" }
 
           it "exists in the built site" do
             expect(path).to exist
@@ -48,7 +48,7 @@
 
   context "documents" do
     context "a single redirect from" do
-      let(:relative_path) { "articles/23128432159832/mary-had-a-little-lamb" }
+      let(:relative_path) { "articles/23128432159832/mary-had-a-little-lamb.html" }
 
       it "exists in the built site" do
         expect(path).to exist

spec/jekyll_redirect_from/redirect_page_spec.rb

@@ -137,7 +137,7 @@
       let(:from) { "/foo" }
 
       it "adds .html" do
-        expected = File.expand_path "foo", site.dest
+        expected = File.expand_path "foo.html", site.dest
         expect(subject.destination("/")).to eql(expected)
       end
     end
@@ -154,7 +154,7 @@
     context "when redirect from is an HTML file" do
       let(:from) { "/foo.html" }
 
-      it "doesn't add .html" do
+      it "adds .html" do
         expected = File.expand_path "foo.html", site.dest
         expect(subject.destination("/")).to eql(expected)
       end
@@ -173,7 +173,7 @@
       let(:from) { "foo" }
 
       it "adds the slash" do
-        expected = File.expand_path "foo", site.dest
+        expected = File.expand_path "foo.html", site.dest
         expect(subject.destination("/")).to eql(expected)
       end
     end
@@ -199,8 +199,8 @@
     context "with no slash" do
       let(:from) { "foo" }
 
-      it "uses no extension" do
-        expect(subject.output_ext).to eql("")
+      it "uses HTML" do
+        expect(subject.output_ext).to eql(".html")
       end
     end
   end