about summary refs log tree commit diff
path: root/spec/requests
diff options
context:
space:
mode:
authorMatt Jankowski <mjankowski@thoughtbot.com>2017-04-09 08:39:41 -0400
committerEugen <eugen@zeonfederated.com>2017-04-09 14:39:41 +0200
commit71706f21c28f5ae623ee69810fe26a34fb79b446 (patch)
tree029ac8a62f15cbd28daa38e5711b46da094306e2 /spec/requests
parentb1881a3d481bccfba984d42380ab2f3780bd0845 (diff)
Ignore implied formats for catch all route requests (#1340)
A request to `/test` would show the custom 404 page, but a request to
`/test.test` would return a 404 with an empty body.

This change ignores the format on incoming catch all route requests, so that the
html 404 page is returned on these requests.
Diffstat (limited to 'spec/requests')
-rw-r--r--spec/requests/catch_all_route_request_spec.rb21
1 files changed, 21 insertions, 0 deletions
diff --git a/spec/requests/catch_all_route_request_spec.rb b/spec/requests/catch_all_route_request_spec.rb
new file mode 100644
index 000000000..22ce1cf59
--- /dev/null
+++ b/spec/requests/catch_all_route_request_spec.rb
@@ -0,0 +1,21 @@
+require "rails_helper"
+
+describe "The catch all route" do
+  describe "with a simple value" do
+    it "returns a 404 page as html" do
+      get "/test"
+
+      expect(response.status).to eq 404
+      expect(response.content_type).to eq "text/html"
+    end
+  end
+
+  describe "with an implied format" do
+    it "returns a 404 page as html" do
+      get "/test.test"
+
+      expect(response.status).to eq 404
+      expect(response.content_type).to eq "text/html"
+    end
+  end
+end