aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers/request_controller_spec.rb
Commit message (Collapse)AuthorAgeLines
* Fix syntax of example email. The number of expected attachments was based on ↵Louise Crow2012-12-04-1/+1
| | | | the specific handling of a bad end boundary in tmail.
* Use new function that copies existing xapian index in spec setup where a ↵Louise Crow2012-11-21-1/+1
| | | | clean copy of the xapian index with fixtures loaded is required.
* Add spec - currently failing due to typo.Louise Crow2012-10-23-0/+42
|
* Add extra checks from ↵Louise Crow2012-10-23-2/+4
| | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb and some extra description.
* Adding new spec from ↵Louise Crow2012-10-23-2/+13
| | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb
* Add an option to disable comments on a requestRobin Houston2012-09-19-0/+16
| | | | Closes #30.
* Bugfix in spec.Louise Crow2012-09-11-1/+1
|
* Record a status update by someone other than the request owner as a request ↵Louise Crow2012-09-11-2/+12
| | | | classification.
* Update references to point to the mySociety repo.Louise Crow2012-08-29-1/+1
|
* Now that external request followups can't be made and status can't be ↵Louise Crow2012-08-23-12/+104
| | | | updated, don't show links or text that suggests they can.
* Don't allow status updates on external requests from the front end interface ↵Louise Crow2012-08-23-259/+331
| | | | | | (they can still be changed from the admin interface). Final part of fix for #562.
* Don't show 'requester only' actions for an external request.Louise Crow2012-08-23-0/+12
| | | | Part of the fix for #562.
* Don't error on a request to followup an external request.Louise Crow2012-08-23-0/+16
| | | | Partial solution to #562.
* Use user_name to supply the user name for an internal or external request in ↵Louise Crow2012-08-22-32/+42
| | | | | | the simple correspondence template used when a full request zip is downloaded. Fixes #561.
* Fix admin links in the case where an external request is viewed by an admin ↵Louise Crow2012-08-22-0/+12
| | | | user.
* Don't send comment alerts for external requests.Louise Crow2012-08-22-0/+10
|
* Re-enable testing of comment links in emails.Louise Crow2012-08-22-7/+2
|
* Fix comment typo.Louise Crow2012-08-22-2/+2
|
* Don't pass a page param of less than one through to Xapian, treat it as a ↵Louise Crow2012-08-21-1/+8
| | | | param of 1. Fixes #557.
* Merge branch 'develop' of github.com:sebbacon/alaveteli into developLouise Crow2012-08-20-0/+6
|\
| * Fail with NotFound if request slug doesn't existRobin Houston2012-08-20-0/+6
| | | | | | | | Closes #554.
* | Use find_by_url_title! to raise ActiveRecordNotFound in upload response when ↵Louise Crow2012-08-20-0/+4
|/ | | | | | no record is found using url_title. Closes #553.
* When displaying attachments, only cache them out to files if they are ↵Louise Crow2012-08-16-198/+219
| | | | visible to everyone, so can be served up from a file cache without authentication.
* Merge branch 'develop' of github.com:sebbacon/alaveteli into developRobin Houston2012-06-23-24/+44
|\ | | | | | | | | | | | | | | Conflicts: Gemfile.lock script/handle-mail-replies script/handle-mail-replies.rb spec/controllers/request_controller_spec.rb
| * Only invalidate a user's cached requests when their *name* changes. ↵Seb Bacon2012-06-20-1/+7
| | | | | | | | Otherwise we issue PURGEs every time, for example, we send out email alerts.
| * Fix test to match current wordingSeb Bacon2012-06-19-1/+1
| |
| * Correctly name the person who has requested something for admin attentionSeb Bacon2012-06-19-0/+1
| |
| * Revert "Set a "Content-Disposition: attachment" header when downloading ↵Seb Bacon2012-06-15-9/+1
| | | | | | | | | | | | | | | | files. Closes #428" This reverts commit 506af7a640f63b17000ccfc5e1344bbc3039c913. See issue #428 for a discussion
| * Use POST rather than GET for tests (this matches better what the UI does)Seb Bacon2012-06-12-4/+4
| |
| * Ensure attention-requesting tests pass by faking a logged in userSeb Bacon2012-06-12-0/+5
| |
| * Make emails requesting admin attention appear to come from the user who ↵Seb Bacon2012-06-12-0/+10
| | | | | | | | requested the attention, not the user who made the original request.
* | Update tests to reflect code changesRobin Houston2012-06-07-0/+23
| | | | | | | | | | Since users must now be logged-in to report requests, the tests need to reflect this.
* | Update test for e3cfe550fe835c68b25d693abd40aedd954913b3Robin Houston2012-06-07-1/+1
|/ | | | | The wording that was changed by e3cfe550fe835c68b25d693abd40aedd954913b3 turns out to be relied on by one of the tests.
* Test for reporting functionalitySeb Bacon2012-05-24-0/+41
|
* Fix typo (introduced in commit cc023c67) that broke a testSeb Bacon2012-05-11-1/+1
|
* Don't cache results from pages with different locales using the same key. ↵Seb Bacon2012-05-04-1/+9
| | | | Fixes #454
* Don't ever create more than one entry for each URL that we want Varnish to ↵Seb Bacon2012-05-04-0/+10
| | | | purge.
* Modify InfoRequest stubs so they work with tests that iterate over the ↵Seb Bacon2012-05-02-0/+4
| | | | collection (e.g. `@user.info_requests.each`)
* More changes and refactoring to make purges work.Seb Bacon2012-04-20-13/+8
|
* Merge branch 'develop' into purge-requestsSeb Bacon2012-04-19-11/+10
|\ | | | | | | | | Conflicts: spec/controllers/request_controller_spec.rb
| * Merge pull request #464 from jpmckinney/small-fixesRobin Houston2012-04-03-1/+1
| |\ | | | | | | Small fixes
| | * use Rails.root, not RAILS_ROOTJames McKinney2012-04-02-1/+1
| | |
| * | Remove cache/zips/ before running tests that write to thereMark Longair2012-04-02-0/+1
| |/
| * Fix broken testRobin Houston2012-03-13-5/+5
| |
| * Correct bad nestingRobin Houston2012-03-13-3/+2
| |
| * Admin users clicking confirmation linksRobin Houston2012-03-13-1/+1
| | | | | | | | Fixes #446. See issue for details.
| * Merge jpmckinney/bundlerSeb Bacon2012-02-15-2/+0
| |
| * Set a "Content-Disposition: attachment" header when downloading files. ↵Seb Bacon2012-02-15-1/+9
| | | | | | | | Closes #428
| * Make /similar/request/NONESUCH a 404 not 500 errorRobin Houston2012-02-10-0/+24
| | | | | | | | Fixes #421.
* | first stab at sending PURGE requests to upstream varnish for request pages. ↵Seb Bacon2012-03-13-1/+57
| | | | | | | | Next step: making it asynchronous, e.g. with a queue of things to purge via a cron job.