lanyon 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18793dd0415d539f17084ef5429627bc1035afac4ad154be2c813bff58865bd1
-  data.tar.gz: 23516e59289257ae2d483124c2493c01fc91e44bb9ceefdbc1b510769535794a
+  metadata.gz: 2334bd94271d53dfd592043f857b993de778bbae9ae2d37599626d6eb438f49b
+  data.tar.gz: 32604e26502c3bcd42d2b671ef2147fe3136a024ef17b7b46dd5870886b8a867
 SHA512:
-  metadata.gz: cedb0eaffd695e1ba9c7f492fdc0022e60d72d2c83deddb1a410803129fc9b180dceb075e365cd48b3b9d42e1bb74009753f49af1071115d0e9cdbba008c8bf2
-  data.tar.gz: 2049a769b9ce0dae8e424abbf3ed908ce1b892c2023042047903c3c4df6d5636fc9c03b7107fd3105fe11f0fab230578ad95877e5cba0bdf72e6ac73858b891f
+  metadata.gz: 767061c55f75506891ec7a3e47f4ca9fa844cc9b28b68cfcb4ade8b17b23e2ef45c453a6a4c55d0289014423d35fa8640150160d3d758b4c28eed929e729c52b
+  data.tar.gz: 4e8f14f744ee127c8afb7e158ba8e4f737e6d856896d27b923bef728b396baa28f4aced6a58ff74b38511b8a1158570b9a19edc1e56a8486c35caac340d2eba4

data/History.md

@@ -1,6 +1,11 @@
 Release History
 ===============
 
+## 0.4.2
+
+* Avoid deprecation warnings for minitest expectations
+* Improve code style
+
 ## 0.4.1
 
 * Drop pessimistic constraint on Jekyll version

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2015-2019 Marcus Stollsteimer
+Copyright (c) 2015-2020 Marcus Stollsteimer
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -56,7 +56,7 @@ or as Lanyon initialization options in `config.ru`.
 Example:
 
 ``` ruby
-run Lanyon.application(:destination => "mysite")
+run Lanyon.application(destination: "mysite")
 ```
 
 This will set a custom destination path, overriding the default (`_site`)
@@ -70,7 +70,7 @@ Additional Lanyon initialization options:
                     (default: false)
 
 Note that on read-only filesystems a site build will fail,
-so you must set `:skip_build => true` in these cases.
+so you must set `skip_build: true` in these cases.
 
 ### Custum 404 Page
 
@@ -107,7 +107,7 @@ Lanyon was inspired by [rack-jekyll][rack-jekyll] and written as a replacement.
 
 ## License
 
-Copyright © 2015-2019 Marcus Stollsteimer
+Copyright © 2015-2020 Marcus Stollsteimer
 
 Lanyon is licensed under the [MIT License][MIT].
 See also the included `LICENSE` file for more information.

data/Rakefile

@@ -7,7 +7,7 @@ def gemspec_file
 end
 
 
-task :default => [:test]
+task default: [:test]
 
 Rake::TestTask.new do |t|
   t.pattern = "test/**/test_*.rb"

data/lanyon.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |s|
 
   s.summary = "Lanyon serves your Jekyll site as a Rack application."
   s.description =
-    "Lanyon is a good friend of Jekyll, the static site generator, " +
+    "Lanyon is a good friend of Jekyll, the static site generator, " \
     "and transforms your website into a Rack application."
 
   s.authors = ["Marcus Stollsteimer"]
@@ -24,8 +24,8 @@ Gem::Specification.new do |s|
   s.add_dependency "jekyll", ">= 2.0"
   s.add_dependency "rack", ">= 1.6", "< 3.0"
 
-  s.add_development_dependency "rake", "~> 11.2"
   s.add_development_dependency "minitest", "~> 5.8"
+  s.add_development_dependency "rake", "~> 11.2"
 
   s.require_paths = ["lib"]
 
@@ -40,7 +40,7 @@ Gem::Specification.new do |s|
     ] +
     Dir.glob("lib/**/*") +
     Dir.glob("test/**/*") +
-    Dir.glob("demo/**/*").reject {|f| f =~ %r(\Ademo/_site/) }
+    Dir.glob("demo/**/*").reject {|f| f.start_with?("demo/_site/") }
 
   s.extra_rdoc_files = %w[README.md LICENSE History.md]
   s.rdoc_options = ["--charset=UTF-8", "--main=README.md"]

data/lib/lanyon.rb

@@ -70,7 +70,7 @@ module Lanyon
 
   # @private
   def self.default_options  # :nodoc:
-    { :skip_build => false }
+    { skip_build: false }
   end
 
   # @private

data/lib/lanyon/application.rb

@@ -44,7 +44,7 @@ module Lanyon
 
     def response(filename)  # :nodoc:
       response = Rack::Response.new(File.binread(filename))
-      response["Content-Type"]  = media_type(filename)
+      response["Content-Type"] = media_type(filename)
 
       response.finish
     end
@@ -63,7 +63,7 @@ module Lanyon
     end
 
     def html_wrap(title, content)  # :nodoc:
-      <<-document.gsub(/^ {6}/, "")
+      <<-DOCUMENT.gsub(/^ {6}/, "")
       <!DOCTYPE html>
       <html lang="en">
       <head>
@@ -73,7 +73,7 @@ module Lanyon
         #{content}
       </body>
       </html>
-      document
+      DOCUMENT
     end
 
     def default_404_body  # :nodoc:
@@ -97,7 +97,7 @@ module Lanyon
     end
 
     def redirect_body(to_path)  # :nodoc:
-      message = %Q{<p>Redirecting to <a href="#{to_path}">#{to_path}</a>.</p>}
+      message = %Q(<p>Redirecting to <a href="#{to_path}">#{to_path}</a>.</p>)
 
       html_wrap("Redirection", message)
     end
@@ -110,9 +110,9 @@ module Lanyon
 
       body = redirect_body(location)
       headers = {
-        "Location"      => location,
+        "Location" => location,
         "Cache-Control" => "max-age=#{cache_time}, must-revalidate",
-        "Expires"       => (Time.now + cache_time).httpdate
+        "Expires" => (Time.now + cache_time).httpdate
       }
 
       html_response(body, 301, headers)

data/lib/lanyon/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Lanyon
-  VERSION  = "0.4.1"
-  DATE     = "2019-11-03"
+  VERSION  = "0.4.2"
+  DATE     = "2020-01-19"
 end

data/test/helper.rb

@@ -8,7 +8,7 @@ require "rack/mock"
 require "lanyon"
 
 
-TEST_DIR = File.expand_path("..", __FILE__)
+TEST_DIR = File.expand_path(__dir__)
 
 SOURCE_DIR = File.join(TEST_DIR, "source")
 CACHE_DIR = File.join(SOURCE_DIR, ".jekyll-cache")
@@ -38,7 +38,6 @@ def sourcedir
   SOURCE_DIR
 end
 
-
 def silence_output
   original_stderr, original_stdout = $stderr, $stdout
   $stderr, $stdout = StringIO.new, StringIO.new
@@ -48,7 +47,6 @@ ensure
   $stderr, $stdout = original_stderr, original_stdout
 end
 
-
 def file_must_exist(filename)
   assert File.exist?(filename),
          "Expected file `#{filename}' to exist."

data/test/test_build.rb

@@ -11,7 +11,7 @@ describe "when creating a Lanyon application" do
 
     destdir = File.join(tempdir, "_site")
 
-    @dir_options = { :source => sourcedir, :destination => destdir }
+    @dir_options = { source: sourcedir, destination: destdir }
     @page = File.join(destdir, "index.html")
     @no_page = File.join(destdir, "not_a_page.html")
 
@@ -36,7 +36,7 @@ describe "when creating a Lanyon application" do
   end
 
   it "does not build the site when :skip_build option is set" do
-    options = {:skip_build => true}.merge(@dir_options)
+    options = { skip_build: true }.merge(@dir_options)
     silence_output do
       Lanyon.application(options)
     end
@@ -46,7 +46,7 @@ describe "when creating a Lanyon application" do
   end
 
   it "does always build the site with ::build" do
-    options = {:skip_build => true}.merge(@dir_options)
+    options = { skip_build: true }.merge(@dir_options)
     silence_output do
       Lanyon.build(options)
     end

data/test/test_configuration.rb

@@ -25,7 +25,7 @@ describe "when configuring site" do
 
     it "loads the correct default destination" do
       config = get_jekyll_config
-      config["destination"].must_equal File.join(Dir.pwd, "_site")
+      _(config["destination"]).must_equal File.join(Dir.pwd, "_site")
     end
   end
 
@@ -43,8 +43,8 @@ describe "when configuring site" do
 
     it "loads the configuration from file" do
       config = get_jekyll_config
-      config.must_include "config_file_opt"
-      config["config_file_opt"].must_equal "ok"
+      _(config).must_include "config_file_opt"
+      _(config["config_file_opt"]).must_equal "ok"
     end
   end
 
@@ -61,28 +61,28 @@ describe "when configuring site" do
     end
 
     it "loads the configuration from file" do
-      config = get_jekyll_config(:config => "_my_config.yml")
-      config.must_include "config_file_opt"
-      config["config_file_opt"].must_equal "ok"
+      config = get_jekyll_config(config: "_my_config.yml")
+      _(config).must_include "config_file_opt"
+      _(config["config_file_opt"]).must_equal "ok"
     end
   end
 
   describe "when initialization options are given" do
 
     it "has the initialization options" do
-      config = get_jekyll_config(:init_opt => "ok")
-      config.must_include "init_opt"
-      config["init_opt"].must_equal "ok"
+      config = get_jekyll_config(init_opt: "ok")
+      _(config).must_include "init_opt"
+      _(config["init_opt"]).must_equal "ok"
     end
 
     it "has the correct destination" do
-      config = get_jekyll_config(:destination => "/project/_site")
-      config["destination"].must_equal "/project/_site"
+      config = get_jekyll_config(destination: "/project/_site")
+      _(config["destination"]).must_equal "/project/_site"
     end
 
     it "does not pass :skip_build on to Jekyll" do
-      config = get_jekyll_config(:skip_build => "ok")
-      config.wont_include "skip_build"
+      config = get_jekyll_config(skip_build: "ok")
+      _(config).wont_include "skip_build"
     end
   end
 
@@ -101,17 +101,16 @@ describe "when configuring site" do
     end
 
     it "has all options and initialization options override file options" do
-      config = get_jekyll_config(:init_opt   => "ok",
-                                 :common_opt => "from init")
-      config.must_include "init_opt"
-      config.must_include "config_file_opt"
-      config.must_include "common_opt"
-      config["common_opt"].must_equal "from init"
+      config = get_jekyll_config(init_opt: "ok", common_opt: "from init")
+      _(config).must_include "init_opt"
+      _(config).must_include "config_file_opt"
+      _(config).must_include "common_opt"
+      _(config["common_opt"]).must_equal "from init"
     end
 
     it "has the correct destination" do
-      config = get_jekyll_config(:destination => "/project/_site_from_init")
-      config["destination"].must_equal "/project/_site_from_init"
+      config = get_jekyll_config(destination: "/project/_site_from_init")
+      _(config["destination"]).must_equal "/project/_site_from_init"
     end
   end
 end

data/test/test_integration.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: UTF-8
 
 require_relative "helper"
 
@@ -19,7 +18,7 @@ describe "when handling requests" do
 
     @destdir = File.join(tempdir, "_site")
 
-    app = get_app(:source => sourcedir, :destination => @destdir)
+    app = get_app(source: sourcedir, destination: @destdir)
     @request = Rack::MockRequest.new(app)
   end
 
@@ -36,23 +35,23 @@ describe "when handling requests" do
     end
 
     it "returns status 200" do
-      @response.status.must_equal 200
+      _(@response.status).must_equal 200
     end
 
     it "returns correct Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "17"
+      _(@response.original_headers["Content-Length"]).must_equal "17"
     end
 
     it "returns correct Content-Type header" do
-      @response.headers["Content-Type"].must_equal "text/html"
+      _(@response.headers["Content-Type"]).must_equal "text/html"
     end
 
     it "returns an ETag header" do
-      @response.headers["ETag"].wont_be_nil
+      _(@response.headers["ETag"]).wont_be_nil
     end
 
     it "returns correct body" do
-      @response.body.must_match %r{<p>Home Page</p>}
+      _(@response.body).must_match %r{<p>Home Page</p>}
     end
   end
 
@@ -64,20 +63,20 @@ describe "when handling requests" do
     end
 
     it "returns status 200" do
-      @response.status.must_equal 404
+      _(@response.status).must_equal 404
     end
 
     it "returns correct Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "142"
+      _(@response.original_headers["Content-Length"]).must_equal "142"
     end
 
     it "returns correct Content-Type header" do
-      @response.headers["Content-Type"].must_equal "text/html"
+      _(@response.headers["Content-Type"]).must_equal "text/html"
     end
 
     it "returns correct body" do
       expected = %r{<!DOCTYPE html>.*<p>404: Not Found</p>}m
-      @response.body.must_match expected
+      _(@response.body).must_match expected
     end
   end
 
@@ -88,7 +87,7 @@ describe "when handling requests" do
       @custom_404 = File.join(sourcedir, "404.html")
       File.open(@custom_404, "w") {|f| f.print "Custom 404" }
 
-      app = get_app(:source => sourcedir, :destination => @destdir)
+      app = get_app(source: sourcedir, destination: @destdir)
       request = Rack::MockRequest.new(app)
       @response = request.get("/not/a/page")
     end
@@ -98,15 +97,15 @@ describe "when handling requests" do
     end
 
     it "returns correct Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "10"
+      _(@response.original_headers["Content-Length"]).must_equal "10"
     end
 
     it "returns correct Content-Type header" do
-      @response.headers["Content-Type"].must_equal "text/html"
+      _(@response.headers["Content-Type"]).must_equal "text/html"
     end
 
     it "returns correct body" do
-      @response.body.must_equal "Custom 404"
+      _(@response.body).must_equal "Custom 404"
     end
   end
 
@@ -118,19 +117,19 @@ describe "when handling requests" do
     end
 
     it "returns status 200" do
-      @response.status.must_equal 200
+      _(@response.status).must_equal 200
     end
 
     it "returns correct Content-Type header" do
-      @response.headers["Content-Type"].must_equal "text/html"
+      _(@response.headers["Content-Type"]).must_equal "text/html"
     end
 
     it "returns correct Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "19"
+      _(@response.original_headers["Content-Length"]).must_equal "19"
     end
 
     it "returns correct body" do
-      @response.body.must_match %r{<p>A Blog Post</p>}
+      _(@response.body).must_match %r{<p>A Blog Post</p>}
     end
   end
 
@@ -142,12 +141,12 @@ describe "when handling requests" do
     end
 
     it "returns status 200" do
-      @response.status.must_equal 200
+      _(@response.status).must_equal 200
     end
 
     it "returns Content-Type 'application/octet-stream'" do
       type = @response.headers["Content-Type"]
-      type.must_equal "application/octet-stream"
+      _(type).must_equal "application/octet-stream"
     end
   end
 
@@ -156,12 +155,12 @@ describe "when handling requests" do
 
     it "returns correct Content-Type for *.css" do
       type = @request.get("/css/test.css").headers["Content-Type"]
-      type.must_equal "text/css"
+      _(type).must_equal "text/css"
     end
 
     it "returns correct Content-Type for *.min.js" do
       type = @request.get("/js/test.min.js").headers["Content-Type"]
-      type.must_equal "application/javascript"
+      _(type).must_equal "application/javascript"
     end
   end
 
@@ -169,11 +168,11 @@ describe "when handling requests" do
   describe "when asked for partially matching paths" do
 
     it "returns status 404 for path 1" do
-      @request.get("/2015/10/05/hello").status.must_equal 404
+      _(@request.get("/2015/10/05/hello").status).must_equal 404
     end
 
     it "returns status 404 for path 2" do
-      @request.get("/10/05/hello-world.html").status.must_equal 404
+      _(@request.get("/10/05/hello-world.html").status).must_equal 404
     end
   end
 
@@ -184,8 +183,8 @@ describe "when handling requests" do
       filename = File.join(@destdir, "/../_site/index.html")
       assert File.exist?(filename)
 
-      @response = @request.get("/../_site/index.html")
-      @response.status.must_equal 404
+      response = @request.get("/../_site/index.html")
+      _(response.status).must_equal 404
     end
   end
 
@@ -193,23 +192,24 @@ describe "when handling requests" do
   describe "when a directory is requested" do
 
     it "redirects to 'directory/' for 'directory' with index.html" do
-      @request.get("/dir-with-index").status.must_equal 301
+      _(@request.get("/dir-with-index").status).must_equal 301
     end
 
     it "returns status 200 for 'directory/' with index.html" do
-      @request.get("/dir-with-index/").status.must_equal 200
+      _(@request.get("/dir-with-index/").status).must_equal 200
     end
 
     it "returns correct body for 'directory/' with index.html" do
-      @request.get("/dir-with-index/").body.must_match %r{<p>Index of dir-with-index/</p>}
+      response_body = @request.get("/dir-with-index/").body
+      _(response_body).must_match %r{<p>Index of dir-with-index/</p>}
     end
 
     it "returns status 404 for 'directory' without index.html" do
-      @request.get("/dir-without-index").status.must_equal 404
+      _(@request.get("/dir-without-index").status).must_equal 404
     end
 
     it "returns status 404 for 'directory/' without index.html" do
-      @request.get("/dir-without-index/").status.must_equal 404
+      _(@request.get("/dir-without-index/").status).must_equal 404
     end
   end
 
@@ -221,20 +221,20 @@ describe "when handling requests" do
     end
 
     it "returns status 301" do
-      @response.status.must_equal 301
+      _(@response.status).must_equal 301
     end
 
     it "returns correct Location header" do
-      @response.headers["Location"].must_equal "/dir-with-index/"
+      _(@response.headers["Location"]).must_equal "/dir-with-index/"
     end
 
     it "returns a Cache-Control header" do
-      @response.headers["Cache-Control"].wont_be_nil
+      _(@response.headers["Cache-Control"]).wont_be_nil
     end
 
     it "returns correct body" do
       expected = %r{<!DOCTYPE html>.*<a href="/dir-with-index/">}m
-      @request.get("/dir-with-index").body.must_match expected
+      _(@request.get("/dir-with-index").body).must_match expected
     end
   end
 
@@ -247,11 +247,11 @@ describe "when handling requests" do
 
     it "returns correct body" do
       response_body = @response.body.force_encoding("UTF-8")
-      response_body.must_match %r{<p>¡Buenos días!</p>}
+      _(response_body).must_match %r{<p>¡Buenos días!</p>}
     end
 
     it "returns the bytesize as Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "23"
+      _(@response.original_headers["Content-Length"]).must_equal "23"
     end
   end
 
@@ -264,11 +264,11 @@ describe "when handling requests" do
 
     it "returns correct body" do
       expected = "File\r\nwith\r\nCRLF\r\nnewlines\r\n"
-      @response.body.must_equal expected
+      _(@response.body).must_equal expected
     end
 
     it "returns the bytesize as Content-Length header" do
-      @response.original_headers["Content-Length"].must_equal "28"
+      _(@response.original_headers["Content-Length"]).must_equal "28"
     end
   end
 
@@ -276,18 +276,18 @@ describe "when handling requests" do
   describe "when URL contains special characters" do
 
     it "returns status 200 for URL with escapes" do
-      @response = @request.get("%2F2015%2F11%2F05%2Fhello-world.html")
-      @response.status.must_equal 200
+      response = @request.get("%2F2015%2F11%2F05%2Fhello-world.html")
+      _(response.status).must_equal 200
     end
 
     it "returns status 200 for resource name with blank" do
-      @response = @request.get("with%20blank.html")
-      @response.status.must_equal 200
+      response = @request.get("with%20blank.html")
+      _(response.status).must_equal 200
     end
 
     it "returns status 200 for resource name with plus" do
-      @response = @request.get("with+plus.html")
-      @response.status.must_equal 200
+      response = @request.get("with+plus.html")
+      _(response.status).must_equal 200
     end
   end
 
@@ -305,20 +305,20 @@ describe "when handling requests" do
     end
 
     it "returns correct status code for unchanged '/'" do
-      @request.get("/", @correct_etag).status.must_equal 304
+      _(@request.get("/", @correct_etag).status).must_equal 304
     end
 
     it "does not return a Content-Length header for unchanged '/'" do
       response = @request.get("/", @correct_etag)
-      response.original_headers["Content-Length"].must_be_nil
+      _(response.original_headers["Content-Length"]).must_be_nil
     end
 
     it "returns correct status code for changed '/'" do
-      @request.get("/", @other_etag).status.must_equal 200
+      _(@request.get("/", @other_etag).status).must_equal 200
     end
 
     it "returns correct status code for 404" do
-      @request.get("/not/a/page", @other_etag).status.must_equal 404
+      _(@request.get("/not/a/page", @other_etag).status).must_equal 404
     end
   end
 
@@ -326,15 +326,15 @@ describe "when handling requests" do
   describe "when handling HEAD requests" do
 
     it "returns status 200 for '/'" do
-      @request.head("/").status.must_equal 200
+      _(@request.head("/").status).must_equal 200
     end
 
     it "returns correct Content-Length header for '/'" do
-      @request.head("/").original_headers["Content-Length"].must_equal "17"
+      _(@request.head("/").original_headers["Content-Length"]).must_equal "17"
     end
 
     it "does not return a body" do
-      @request.head("/").body.must_equal ""
+      _(@request.head("/").body).must_equal ""
     end
   end
 
@@ -342,20 +342,21 @@ describe "when handling requests" do
   describe "when handling OPTIONS requests" do
 
     it "returns status 200" do
-      @request.options("/").status.must_equal 200
+      _(@request.options("/").status).must_equal 200
     end
 
     it "returns correct Allow header" do
-      @request.options("/").original_headers["Allow"].must_equal "GET,HEAD,OPTIONS"
+      response_allow_header = @request.options("/").original_headers["Allow"]
+      _(response_allow_header).must_equal "GET,HEAD,OPTIONS"
     end
 
     it "does not return a body" do
-      @request.options("/").body.must_equal ""
+      _(@request.options("/").body).must_equal ""
     end
 
     it "returns 404 for nonexistent resource" do
-      @request.options("/not/a/page").status.must_equal 404
-      @request.options("/not/a/page").body.must_match %r{<p>404: Not Found</p>}
+      _(@request.options("/not/a/page").status).must_equal 404
+      _(@request.options("/not/a/page").body).must_match %r{<p>404: Not Found</p>}
     end
   end
 
@@ -363,15 +364,15 @@ describe "when handling requests" do
   describe "when handling POST, PUT, DELETE, and other requests" do
 
     it "returns status 405" do
-      @request.post("/").status.must_equal 405
-      @request.put("/").status.must_equal 405
-      @request.delete("/").status.must_equal 405
-      @request.request("OTHER", "/").status.must_equal 405
+      _(@request.post("/").status).must_equal 405
+      _(@request.put("/").status).must_equal 405
+      _(@request.delete("/").status).must_equal 405
+      _(@request.request("OTHER", "/").status).must_equal 405
     end
 
     it "returns correct body" do
       expected = %r{<!DOCTYPE html>.*<p>405: Method Not Allowed</p>}m
-      @request.post("/").body.must_match expected
+      _(@request.post("/").body).must_match expected
     end
   end
 end

data/test/test_router.rb

@@ -43,44 +43,44 @@ describe Lanyon::Router do
 
     it "returns path for '/'" do
       filename = File.join(@sitedir, "index.html")
-      @router.endpoint("/").must_equal filename
+      _(@router.endpoint("/")).must_equal filename
     end
 
     it "returns existing path" do
       filename = File.join(@sitedir, "page.html")
-      @router.endpoint("/page.html").must_equal filename
+      _(@router.endpoint("/page.html")).must_equal filename
     end
 
     it "returns existing path for resource without extension" do
       filename = File.join(@sitedir, "README")
-      @router.endpoint("/README").must_equal filename
+      _(@router.endpoint("/README")).must_equal filename
     end
 
     it "returns :not_found for non-existent path" do
-      @router.endpoint("/not-a-page.html").must_equal :not_found
+      _(@router.endpoint("/not-a-page.html")).must_equal :not_found
     end
 
     it "returns :not_found for partially matching paths" do
-      @router.endpoint("/dir1/dir2/").must_equal :not_found
-      @router.endpoint("/dir2/dir3").must_equal :not_found
-      @router.endpoint("ir1/di").must_equal :not_found
+      _(@router.endpoint("/dir1/dir2/")).must_equal :not_found
+      _(@router.endpoint("/dir2/dir3")).must_equal :not_found
+      _(@router.endpoint("ir1/di")).must_equal :not_found
     end
 
     it "returns path for '/path/to/dir/' with index" do
       filename = File.join(@sitedir, "dir-with-index/index.html")
-      @router.endpoint("/dir-with-index/").must_equal filename
+      _(@router.endpoint("/dir-with-index/")).must_equal filename
     end
 
     it "returns :must_redirect for '/path/to/dir' with index" do
-      @router.endpoint("/dir-with-index").must_equal :must_redirect
+      _(@router.endpoint("/dir-with-index")).must_equal :must_redirect
     end
 
     it "returns :not_found for '/path/to/dir/' without index" do
-      @router.endpoint("/dir-without-index/").must_equal :not_found
+      _(@router.endpoint("/dir-without-index/")).must_equal :not_found
     end
 
     it "returns :not_found for '/path/to/dir' without index" do
-      @router.endpoint("/dir-without-index").must_equal :not_found
+      _(@router.endpoint("/dir-without-index")).must_equal :not_found
     end
   end
 
@@ -89,31 +89,31 @@ describe Lanyon::Router do
 
     it "returns existing path" do
       filename = File.join(@sitedir, "page.html")
-      @router.endpoint("/page").must_equal filename
+      _(@router.endpoint("/page")).must_equal filename
     end
 
     describe "when both `foo' and `foo.html' exist" do
 
       it "returns `foo' and not `foo.html' when asked for `foo'" do
         filename = File.join(@sitedir, "foo")
-        @router.endpoint("/foo").must_equal filename
+        _(@router.endpoint("/foo")).must_equal filename
       end
 
       it "can also serve `foo.html'" do
         filename = File.join(@sitedir, "foo.html")
-        @router.endpoint("/foo.html").must_equal filename
+        _(@router.endpoint("/foo.html")).must_equal filename
       end
     end
 
     describe "when both `bar.html' and `bar/index.html' exist" do
 
       it "returns :must_redirect and not `bar.html' when asked for `bar'" do
-        @router.endpoint("/bar").must_equal :must_redirect
+        _(@router.endpoint("/bar")).must_equal :must_redirect
       end
 
       it "can also serve `bar.html'" do
         filename = File.join(@sitedir, "bar.html")
-        @router.endpoint("/bar.html").must_equal filename
+        _(@router.endpoint("/bar.html")).must_equal filename
       end
     end
   end
@@ -123,27 +123,27 @@ describe Lanyon::Router do
 
     it "discards leading '..' for existing path" do
       filename = File.join(@sitedir, "page.html")
-      @router.endpoint("/../../page.html").must_equal filename
+      _(@router.endpoint("/../../page.html")).must_equal filename
     end
 
     it "allows safe directory traversal" do
       filename = File.join(@sitedir, "index.html")
-      @router.endpoint("/dir1/../").must_equal filename
+      _(@router.endpoint("/dir1/../")).must_equal filename
     end
 
     it "returns :not_found for unsafe directory traversal 1" do
       filename = File.join(@sitedir, "/../_site/page.html")
       assert File.exist?(filename)
 
-      @router.endpoint("/../_site/page.html").must_equal :not_found
+      _(@router.endpoint("/../_site/page.html")).must_equal :not_found
     end
 
     it "returns :not_found for unsafe directory traversal 2" do
-      @router.endpoint("/%2E%2E/_site/").must_equal :not_found
+      _(@router.endpoint("/%2E%2E/_site/")).must_equal :not_found
     end
 
     it "returns :not_found for unsafe directory traversal 3" do
-      @router.endpoint("/dir1/../dir1/../../_site/").must_equal :not_found
+      _(@router.endpoint("/dir1/../dir1/../../_site/")).must_equal :not_found
     end
   end
 
@@ -153,7 +153,7 @@ describe Lanyon::Router do
     describe "when 404.html does not exist" do
 
       it "returns nil" do
-        @router.custom_404_body.must_be_nil
+        _(@router.custom_404_body).must_be_nil
       end
     end
 
@@ -169,7 +169,7 @@ describe Lanyon::Router do
       end
 
       it "returns correct body" do
-        @router.custom_404_body.must_equal "Custom 404"
+        _(@router.custom_404_body).must_equal "Custom 404"
       end
     end
   end
@@ -179,14 +179,14 @@ describe Lanyon::Router do
 
     it "strips trailing slash from root" do
       router = Lanyon::Router.new(@sitedir + "/")
-      router.root.must_equal @sitedir
+      _(router.root).must_equal @sitedir
     end
 
     it "does not append a trailing slash to root" do
       assert !@sitedir.end_with?("/")
 
       router = Lanyon::Router.new(@sitedir)
-      router.root.must_equal @sitedir
+      _(router.root).must_equal @sitedir
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lanyon
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Marcus Stollsteimer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-03 00:00:00.000000000 Z
+date: 2020-01-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -45,33 +45,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.2'
+        version: '5.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.2'
+        version: '5.8'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.8'
+        version: '11.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.8'
+        version: '11.2'
 description: Lanyon is a good friend of Jekyll, the static site generator, and transforms
   your website into a Rack application.
 email: sto.mar@web.de
@@ -140,7 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Lanyon serves your Jekyll site as a Rack application.