Add test for #210 report

matsumotory · Sep 28, 2016 · 67403f3 · 67403f3
1 parent adffb4b
commit 67403f3
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 0 deletions.
diff --git a/test/conf/nginx.conf b/test/conf/nginx.conf
@@ -694,6 +694,14 @@ http {
         location /nginx_false_true {
           mruby_content_handler_code 'Nginx.rputs (Nginx::FALSE + Nginx::TRUE)';
         }
+        location /issue_210 {
+          mruby_post_read_handler_code 'Nginx::Request.new.headers_out["hoge"] = "fuga"';
+          mruby_content_handler_code 'Nginx.rputs "hello"';
+        }
+        location /issue_210_2 {
+          mruby_post_read_handler build/nginx/html/issue-210.rb cache;
+          mruby_content_handler_code 'Nginx.rputs "hello2"';
+        }
     }
 
     server {

diff --git a/test/html/issue-210.rb b/test/html/issue-210.rb
@@ -0,0 +1 @@
+Nginx::Request.new.headers_out["fuga"] = "hoge"
diff --git a/test/t/ngx_mruby.rb b/test/t/ngx_mruby.rb
@@ -231,6 +231,18 @@ def base_ssl(port)
   t.assert_equal "false", res["body"]
 end
 
+t.assert('ngx_mruby - bug; mruby_post_read_handler not running in 1.18.3+', 'location /issue_210') do
+  res = HttpRequest.new.get base + '/issue_210'
+  t.assert_equal "fuga", res["hoge"]
+  t.assert_equal "hello", res["body"]
+end
+
+t.assert('ngx_mruby - bug; mruby_post_read_handler not running in 1.18.3+ for file code', 'location /issue_210_2') do
+  res = HttpRequest.new.get base + '/issue_210_2'
+  t.assert_equal "hoge", res["fuga"]
+  t.assert_equal "hello2", res["body"]
+end
+
 p nginx_version
 
 if nginx_version.split(".")[1].to_i > 6
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Nginx::Request.new.headers_out["fuga"] = "hoge"