aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers
Commit message (Collapse)AuthorAgeLines
* Restore the download for hidden requestsLouise Crow2013-09-16-6/+0
| | | | | | This was disabled for hidden requests as the download was by redirect, allowing people who have not been authenticated to conceivably access the download. We'll be moving to send_file instead, so can restore it.
* Rewrite download specLouise Crow2013-09-16-45/+3
| | | | | Make it an integration spec so we don't need to touch the internals so much.
* Replace fixture-based spec with factory-based equivalent.Louise Crow2013-09-16-10/+6
|
* Use expectation for hidden responses more generally.Louise Crow2013-09-16-9/+9
|
* Fix request in spec for hidden attachments not being viewed.Louise Crow2013-09-16-2/+15
| | | | | Add another spec for the same behaviour when an incoming message is hidden.
* Add new code and specs for hiding attachments.Louise Crow2013-09-16-1/+106
|
* Refactor expectation into a methodLouise Crow2013-09-16-6/+8
| | | | Remove expectation of pdf contents - we're looking at the raw file.
* Refactor prominence specs to use factories not fixtures.Louise Crow2013-09-16-116/+74
|
* Clarify specs.Louise Crow2013-09-16-5/+5
| | | | They refer to the case where the request itself is hidden.
* 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-74/+51
| | | | | Make specs that depend on multiple controllers and models interacting integration specs.
* Move outgoing message admin actions to their own controller.Louise Crow2013-09-16-12/+23
| | | | Eventually this should use standard RESTful routing.
* Handle the case of a name that hits the character limits and has been ↵Louise Crow2013-07-29-5/+25
| | | | suffixed with a number.
* Merge branch 'feature/localisation-fixes' into rails-3-developLouise Crow2013-07-29-138/+24
|\
| * Patch Globalize to compensate for the way gettext_i18n_rails patches ↵Louise Crow2013-07-23-0/+18
| | | | | | | | 118n.locale in the handling of locales with underscores. Fixes #999.
| * Switch routing-filter (which takes locale out of the params and puts it in ↵Louise Crow2013-07-18-139/+7
| | | | | | | | the URL) off by default in model, controller, and helper tests. This means we can supply the locale as a param. Turn it on specifically for a couple of controller tests that test routing, and change other url localization tests into integration tests.
* | Merge remote-tracking branch ↵Louise Crow2013-07-23-5/+6
|\ \ | |/ |/| | | | | | | | | 'openaustralia_github/tests_set_described_state' into rails-3-develop Conflicts: spec/models/info_request_spec.rb
| * Now record a state_update event *whenever* a user classifies a requestMatthew Landauer2013-04-20-5/+6
| |
* | Revert "Assume that format will be specified in the url requested, or ↵Louise Crow2013-06-26-6/+0
| | | | | | | | | | | | default to HTML. Fixes #989." This reverts commit c1b3324922687778b316ee1a180582013bcf6d0e.
* | Don't call out to external urls during controller testsLouise Crow2013-06-20-0/+12
| |
* | Assume that format will be specified in the url requested, or default to ↵Louise Crow2013-06-20-0/+6
| | | | | | | | HTML. Fixes #989.
* | Respond to a (currently unsupported) json request for a public body list ↵Louise Crow2013-06-20-1/+4
| | | | | | | | with a 406, not a 500 caused by a missing template.
* | Merge branch 'hotfix/0.11.0.13' into rails-3-developLouise Crow2013-06-17-2/+25
|\ \ | | | | | | | | | | | | Conflicts: spec/mailers/outgoing_mailer_spec.rb
| * | Explicitly set I18n.locale in set_gettext_locale in order to get Rails to ↵0.11.0.13hotfix/0.11.0.13Louise Crow2013-06-17-2/+25
| | | | | | | | | | | | expire the template cache and trigger a lookup.
| * | We're assuming feed urls without format suffixes should serve up atom/xml, ↵0.11.0.4hotfix/0.11.0.4Louise Crow2013-06-05-0/+12
| | | | | | | | | | | | so ignore any format from content negotiation in favour of that default.
* | | Better spec isolation - these specs require raw emails, so make sure they're ↵Louise Crow2013-06-10-0/+1
| | | | | | | | | | | | loaded.
* | | Remove reference to incoming message from purge request test - it brings in ↵Louise Crow2013-06-07-1/+1
| | | | | | | | | | | | a requirement to have loaded the raw emails, which is tangential to what we're testing.
* | | Allow the donation url to be configured. Closes #909.Louise Crow2013-06-07-0/+28
| | |
* | | Add some context blocks to group together tests with the same status.Louise Crow2013-06-07-57/+103
| | |
* | | We're assuming feed urls without format suffixes should serve up atom/xml, ↵Louise Crow2013-06-05-0/+12
| | | | | | | | | | | | so ignore any format from content negotiation in favour of that default.
* | | Remove support for theme stylesheet inclusion via template deprecated in ↵Louise Crow2013-06-05-1/+0
| | | | | | | | | | | | version 0.5. Closes #972.
* | | Merge branch 'release/0.11' into rails-3-developLouise Crow2013-06-04-7/+32
|\| |
| * | Handle json requests for hidden info requests.Louise Crow2013-06-03-7/+32
| | |
* | | Merge remote-tracking branch ↵Louise Crow2013-06-04-86/+134
|\ \ \ | | | | | | | | | | | | 'openaustralia_github/more_info_on_report_request' into rails-3-develop
| * | | User needs to choose a reasonMatthew Landauer2013-03-22-5/+15
| | | |
| * | | Rename parameter used in routes and controllerMatthew Landauer2013-03-22-9/+9
| | | |
| * | | Rename controller actionsMatthew Landauer2013-03-22-11/+11
| | | |
| * | | Move reporting actions to their own controllerMatthew Landauer2013-03-22-118/+124
| | | |
| * | | Extract methodMatthew Landauer2013-03-22-1/+1
| | | |
| * | | Only allow posts for reporting request. Don't try redirection when not logged inMatthew Landauer2013-03-22-3/+4
| | | |
| * | | Posted reason and message gets sent out in the emailMatthew Landauer2013-03-22-0/+7
| | | |
| * | | Only allow reporting a request when logged inMatthew Landauer2013-03-22-0/+24
| | | |
* | | | Merge remote-tracking branch ↵Louise Crow2013-06-04-2/+4
|\ \ \ \ | |_|/ / |/| | | | | | | 'openaustralia_github/inline_search_method_refactor' into rails-3-develop
| * | | Inline method InfoRequest.full_searchMatthew Landauer2013-03-25-2/+4
| |/ /
* | | Make sure Atom feeds are served with content type application/atom+xmlMark Longair2013-05-29-0/+1
| | | | | | | | | | | | Fixes #961
* | | Add a comment about the behaviour of the censor rulesMark Longair2013-05-24-0/+2
| | |
* | | The file_name on getting an attachment should be the display filenameMark Longair2013-05-24-1/+1
| | | | | | | | | | | | | | | Previously the behaviour would have been the same, since we weren't checking the filename so strictly.
* | | Make an attachment filename in test fixtures more difficultMark Longair2013-05-24-24/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes 'hello.txt' to 'hello-world.txt' in the incoming-request-two-same-name.email fixture. The reasoning for this change is that if there are no special characters in the filename then filename and display_filename will be the same and the tests won't pick up any confusion between the two. The test requests to :get_attachment and :get_attachment_as_html should get the display_filename rather than filename.