aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers/admin_incoming_message_controller_spec.rb
Commit message (Collapse)AuthorAgeLines
* Move admin incoming message actions to use RESTful routes.Louise Crow2014-12-18-5/+5
| | | | | So data changing actions require a POST and can be protected against CSRF.
* Use RESTful routing for index and show actionsLouise Crow2014-12-12-1/+1
|
* Fix for an aborted transaction not being rolled-backMark Longair2013-10-07-0/+11
| | | | | | | | | | | | | | | | | | | | | We were seeing errors in xapian_create_job that indicated that the current transaction had been aborted but not rolled-back. It seems that this happened because sometimes xapian_create_job was called from within another transaction (e.g. from the transaction block in AdminIncomingMessageController#redeliver). The default behaviour of nested transactions in Rails is for them to behave as if all but the outermost transaction blocks had no effect; this means that the exception thrown in the inner transaction block in xapian_create_job wasn't causing a rollback, and it was then being caught before propagating up to the outermost transaction. Thus the transaction was never being rolled-back so any subsequent operation on this database connection would error. This commit fixes this problem by using the :requires_new => true option to transaction, which asks ActiveRecord to fake genuinely nested transcations using SAVEPOINT. Fixes #1124.
* Use earlier factory_girl syntaxLouise Crow2013-09-16-1/+1
|
* Fix spec descriptionsLouise Crow2013-09-16-2/+2
|
* Add the meat of the update action.Louise Crow2013-09-16-0/+70
|
* Add a simple edit action and templateLouise Crow2013-09-16-5/+28
|
* Slightly nicer action names.Louise Crow2013-09-16-5/+5
|
* Move incoming message admin to its own controller.Louise Crow2013-09-16-0/+51
Make specs that depend on multiple controllers and models interacting integration specs.