Skip to content

Commit

Permalink
Added env argument to all Rackup test fixtures
Browse files Browse the repository at this point in the history
The test suite wouldn't pass, and the individual fixtures wouldn't run in pow
without this required rack argument.
  • Loading branch information
rmm5t committed Apr 10, 2011
1 parent e87f56f commit 29355e7
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion test/fixtures/apps/env/config.ru
@@ -1,6 +1,6 @@
require "json"

run lambda {
run lambda { |env|
body = ENV.keys.grep(/^POW/).inject({}) { |e, k| e.merge(k => ENV[k]) }.to_json
[200, {'Content-Type' => 'text/plain', 'Content-Length' => body.length.to_s}, [body]]
}
2 changes: 1 addition & 1 deletion test/fixtures/apps/error/ok.ru
@@ -1 +1 @@
run lambda { [200, {'Content-Type' => 'text/plain'}, ['OK']] }
run lambda { |env| [200, {'Content-Type' => 'text/plain'}, ['OK']] }
2 changes: 1 addition & 1 deletion test/fixtures/apps/pid/config.ru
@@ -1,4 +1,4 @@
run lambda {
run lambda { |env|
body = Process.pid.to_s
[200, {'Content-Type' => 'text/plain', 'Content-Length' => body.length.to_s}, [body]]
}
2 changes: 1 addition & 1 deletion test/fixtures/apps/rc-error/config.ru
@@ -1,3 +1,3 @@
run lambda {
run lambda { |env|
[200, {'Content-Type' => 'text/plain', 'Content-Length' => '5'}, ['Hello']]
}

0 comments on commit 29355e7

Please sign in to comment.