Add a test to make sure that a die in a Controller's end sub is propogated as a 500 error

Unbreak the pod-coverage tests

Without removing those imports, it expects us to document them.

Yes, it is

Remove special-casing of '*' for get_action_methods

Instead, pull '*' out of _controller_actions early on. The part where they're
applied to actions is still gross.

Pull a bunch of gross hacks to get actions => { '*' => ... } working again

Add a test for configuring attributes for all actions

Add a test view for dumping actions

Make this error message more readable

Make this error message more readable

Agreed

Merge branch 'master' into psgi

* master:
  A failing reverse lookup doesn't imply it's a local address
  Tiny tidyups to copyright and license

Conflicts:
lib/Catalyst/Engine/HTTP.pm

A failing reverse lookup doesn't imply it's a local address

Fix a syntax error introduced in d367082

Use the Plack LighttpdScriptNameFix middleware.

This replaces our home-baked solution with something which works better
and in more cases

Update TODO

Clean up FIXMEs

Fix a warning under aggregation

Pass script options into auto loader

Plack::Handler::XXX therefore gets the options even when auto-detecting.

Retrieve args for old scripts so FCGI --listen works

Force FCGI detection for FastCGI scripts.

Do not commit conflicts, idiot

Fix bug stopping fastcgi script working

Update todo

Update todo

Fix the breakage the last commit introduced

make ->setup_engine($engine) supported API again

It should always take preference over CATALYST_ENGINE. The compat hack is really
just PSGI specific.

No, we don't want this

If we'd check it, we'd have to check it on every request. That's entirely not
worth it as I'm not even aware of any server that doesn't support psgi.streaming
right now, let alone a server people actually use.

If people complain, documentation suggesting to load the BufferedWrite
middleware needs to be written.

Document middlewares applied automatically

Explicitly document the wrong Engine::PSGI thing.

I did it accidentally, and it took rafl half an hour to chase down in d5024dd882,
so someone as dumb as me is likely to do it (just substituting the method name,
rather than removing the additional closure) when upgrading from ::Engine::PSGI.

Update TODO

Partially unify local and remote request codepaths

Use Plack::Test::ExternalServer in Catalyst::Test remote requests

We already have a warning for this

Fix the test psgi file

psgi_app is just a getter

Don't assume the app gets loaded before the first request

First stab documenting writing a .psgi file

Legacy .psgi files are never loaded

Ignore old-style .psgi files

De-uglify a comment

Document status of the misc engines.

Both SCGI and XMPP appear to be bork

Add a test showing how having an Engine::PSGI .psgi file breaks.

Notably, this .psgi file is in the app root, whereas ::Engine::PSGI's docs and helper suggest that
you create it in script/. So I don't have a problem with not supporting the .psgi working for tests
until you've converted it if you move it into the app root really - however it would be nice to be
able to detect it was broken, rather than just hanging.

Test showing how having a .psgi file will ruin your day

local_request is now tested

(and deprecated!)

Add tests for the deprecated local_request

strictures and warnings for live_catalyst_test.t

This was fixed already

Fix the conditional here.

We do always need a trailing slash if we have a server path and either
just a /, or no uri at all (e.g. http://localhost). This matters when
we're mounted at a non-root path, as cataltst needs to see
http://localhost/non_root_path/, rather than http://localhost/non_root_path,
otherwise we'll try to resolve /non_root path as an action.

Trim hatespace

Tiny tidyups to copyright and license

The IIS 6 script name fix is now part of Plack

Depend on it and use it.

Merge branch 'master' into psgi

* master:
  Fix Path actions debug screen to display number of arguments

Fix Path actions debug screen to display number of arguments

Merge branch 'master' into psgi

* master:
  Make default test names reflect reality more and fix a buglet, rafl++
  Give action_ok, action_redirect and action_notfound default test names

Make default test names reflect reality more and fix a buglet, rafl++

Give action_ok, action_redirect and action_notfound default test names

Merge branch 'master' into psgi

* master:
  Avoid super long command lines when not aggregating
  Improve the documentation for various functions in Catalyst::Test

Avoid super long command lines when not aggregating

This probably also fixes prove -lr t/, which people have been complaining about
every now and then.

Improve the documentation for various functions in Catalyst::Test

Deprecate local_request and remote_request

Move out building of the horrible ctx_request closure

Also move out the rest, while we're at it. This is intended to aid a subsequent
refactoring.

Remove some kane-isms that hurt my eyes

Run http-server.t tests on something other than the server root

Remove testsuite's assumptions about where it's mounted on a web server

Make http-server.t use Test::TCP

No dep, as it's author-only and pulled in by Plack already anyway.

Don't use the server.pl script in the live tests

Instead, just spin up a server with Plack::Loader. The script classes are
already tested separately, so we only need this test to make sure live tests
work.

This is intended to aid a subsequent refactoring.

.psgi stuff is now sorted out

::Wx is declared dead

lighttpd and iis6 now work

Reduce code repetition

Add legacy workaround for nginx PATH_INFO

Make legacy IIS env fixing work again

Make legacy lighttpd env fixing work again

Load middlewares where they're actually used

Make _wrapped_legacy_psgi_app do wrapping only

We now pass in the psgi app to wrap, which should make it quite a bit easier to
test. We'll need that to check if the various quirk fixes for different web
servers actually work.

Version 5.89002

The HTTP engine is no more, and was also never required for this test

Avoid loading actual engines in script tests

This way we can run these tests without having FCGI.pm installed.

PSGI, not Plack

Update Upgrading docs

Make getting to a PSGI app saner

All the user should ever need to call is ->psgi_app (previously raw_psgi_app),
to get an unwrapped psgi app as provided by the engine. This is what should be
used when writing .psgi files.

For all other ways of running catalyst apps there's no need to be able to get to
the psgi application that'll actually run, so here we're making everything
that's responsible for reading .psgi files to run them with Catalyst::Script::*
or for wrapping the plain psgi code ref with back-compat middleware private.

Unfortunately this removes some of the docs about the automatic .psgi
loading. These should be added again elsewhere. Also the Upgrading docs need
updating after this.

Version 5.89001

Stop psgi_file.t from recursing

psgi_app is *not* to be used in .psgi files. It's the compat layer that'll give
you either an app loaded from from a .psgi file, or the default PSGI app with
the reverse proxy middleware and stuff.

In .psgi files only raw_psgi_app should ever be used, as that'll return the bare
catalyst psgi app to then be wrapped in middlewares manually.

I see how the naming is shit and confusing though. Suggestions welcome.

Rename Engine::Loader to EngineLoader

It's not an engine, as those exist no more, so move it out of the ::Engine::
namespace.

Changelog updates

Merge branch 'master' into psgi

* master: (22 commits)
  Change repos metadata to git
  Version 5.80032
  Pass the extra restart options
  Fix test
  Changelog
  added myself (dd070) in contributors list
  removed duplicate parameters in return value of _restarter_args
  added more parameters in return value of _restarter_args
  Add a few tests
  Patch to make restarter class configurable / settable in prefs
  Apply more correct fix than 8df53b (I hope)
  Rename test
  Make tests more clear
  Merge revert 8df53bed
  Un-TODO abraxxa's tests
  This scares me, but it fixes stuf work work
  Fix 5.80 bug which causes slurp to fail if called multiple times
  Fix body predicate bug/back compat issue
  Version 5.80031
  added test for chained dispatcher fail on multiple CaptureArgs(0) parts introduced in 5.80030
  ...

Conflicts:
Changes
Makefile.PL
lib/Catalyst.pm
lib/Catalyst/Engine/FastCGI.pm
lib/Catalyst/Runtime.pm

Properly skip known broken tests

Faked passing tests are not cool, and nor are __END__ sections, as those will
break aggregation.

If we actually want to prevent the body of the tests from being compiled,
putting the C<< plan skip_all => ... >> in a BEGIN block is the right thing to
do.

Change repos metadata to git

Version 5.80032

Pass the extra restart options

Fix test

Changelog

Merge branch 'restarter_changeclass' into 5.80032_dev

* restarter_changeclass:
  added myself (dd070) in contributors list
  removed duplicate parameters in return value of _restarter_args
  added more parameters in return value of _restarter_args
  Add a few tests
  Patch to make restarter class configurable / settable in prefs

added myself (dd070) in contributors list

removed duplicate parameters in return value of _restarter_args

added more parameters in return value of _restarter_args

Add a few tests

Patch to make restarter class configurable / settable in prefs

Merge branch 'chained_fix' into 5.80032_dev

* chained_fix:
  Apply more correct fix than 8df53b (I hope)
  Rename test
  Make tests more clear
  Merge revert 8df53bed
  Un-TODO abraxxa's tests

Apply more correct fix than 8df53b (I hope)