Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Restore the download for hidden requests | Louise Crow | 2013-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 spec | Louise Crow | 2013-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 Crow | 2013-09-16 | -10/+6 |
| | ||||
* | Use expectation for hidden responses more generally. | Louise Crow | 2013-09-16 | -9/+9 |
| | ||||
* | Fix request in spec for hidden attachments not being viewed. | Louise Crow | 2013-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 Crow | 2013-09-16 | -1/+106 |
| | ||||
* | Refactor expectation into a method | Louise Crow | 2013-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 Crow | 2013-09-16 | -116/+74 |
| | ||||
* | Clarify specs. | Louise Crow | 2013-09-16 | -5/+5 |
| | | | | They refer to the case where the request itself is hidden. | |||
* | Add the meat of the update action. | Louise Crow | 2013-09-16 | -0/+70 |
| | ||||
* | Add a simple edit action and template | Louise Crow | 2013-09-16 | -5/+28 |
| | ||||
* | Slightly nicer action names. | Louise Crow | 2013-09-16 | -5/+5 |
| | ||||
* | Move incoming message admin to its own controller. | Louise Crow | 2013-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 Crow | 2013-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 Crow | 2013-07-29 | -5/+25 |
| | | | | suffixed with a number. | |||
* | Merge branch 'feature/localisation-fixes' into rails-3-develop | Louise Crow | 2013-07-29 | -138/+24 |
|\ | ||||
| * | Patch Globalize to compensate for the way gettext_i18n_rails patches ↵ | Louise Crow | 2013-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 Crow | 2013-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 Crow | 2013-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 request | Matthew Landauer | 2013-04-20 | -5/+6 |
| | | ||||
* | | Revert "Assume that format will be specified in the url requested, or ↵ | Louise Crow | 2013-06-26 | -6/+0 |
| | | | | | | | | | | | | default to HTML. Fixes #989." This reverts commit c1b3324922687778b316ee1a180582013bcf6d0e. | |||
* | | Don't call out to external urls during controller tests | Louise Crow | 2013-06-20 | -0/+12 |
| | | ||||
* | | Assume that format will be specified in the url requested, or default to ↵ | Louise Crow | 2013-06-20 | -0/+6 |
| | | | | | | | | HTML. Fixes #989. | |||
* | | Respond to a (currently unsupported) json request for a public body list ↵ | Louise Crow | 2013-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-develop | Louise Crow | 2013-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.13 | Louise Crow | 2013-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.4 | Louise Crow | 2013-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 Crow | 2013-06-10 | -0/+1 |
| | | | | | | | | | | | | loaded. | |||
* | | | Remove reference to incoming message from purge request test - it brings in ↵ | Louise Crow | 2013-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 Crow | 2013-06-07 | -0/+28 |
| | | | ||||
* | | | Add some context blocks to group together tests with the same status. | Louise Crow | 2013-06-07 | -57/+103 |
| | | | ||||
* | | | We're assuming feed urls without format suffixes should serve up atom/xml, ↵ | Louise Crow | 2013-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 Crow | 2013-06-05 | -1/+0 |
| | | | | | | | | | | | | version 0.5. Closes #972. | |||
* | | | Merge branch 'release/0.11' into rails-3-develop | Louise Crow | 2013-06-04 | -7/+32 |
|\| | | ||||
| * | | Handle json requests for hidden info requests. | Louise Crow | 2013-06-03 | -7/+32 |
| | | | ||||
* | | | Merge remote-tracking branch ↵ | Louise Crow | 2013-06-04 | -86/+134 |
|\ \ \ | | | | | | | | | | | | | 'openaustralia_github/more_info_on_report_request' into rails-3-develop | |||
| * | | | User needs to choose a reason | Matthew Landauer | 2013-03-22 | -5/+15 |
| | | | | ||||
| * | | | Rename parameter used in routes and controller | Matthew Landauer | 2013-03-22 | -9/+9 |
| | | | | ||||
| * | | | Rename controller actions | Matthew Landauer | 2013-03-22 | -11/+11 |
| | | | | ||||
| * | | | Move reporting actions to their own controller | Matthew Landauer | 2013-03-22 | -118/+124 |
| | | | | ||||
| * | | | Extract method | Matthew Landauer | 2013-03-22 | -1/+1 |
| | | | | ||||
| * | | | Only allow posts for reporting request. Don't try redirection when not logged in | Matthew Landauer | 2013-03-22 | -3/+4 |
| | | | | ||||
| * | | | Posted reason and message gets sent out in the email | Matthew Landauer | 2013-03-22 | -0/+7 |
| | | | | ||||
| * | | | Only allow reporting a request when logged in | Matthew Landauer | 2013-03-22 | -0/+24 |
| | | | | ||||
* | | | | Merge remote-tracking branch ↵ | Louise Crow | 2013-06-04 | -2/+4 |
|\ \ \ \ | |_|/ / |/| | | | | | | | 'openaustralia_github/inline_search_method_refactor' into rails-3-develop | |||
| * | | | Inline method InfoRequest.full_search | Matthew Landauer | 2013-03-25 | -2/+4 |
| |/ / | ||||
* | | | Make sure Atom feeds are served with content type application/atom+xml | Mark Longair | 2013-05-29 | -0/+1 |
| | | | | | | | | | | | | Fixes #961 | |||
* | | | Add a comment about the behaviour of the censor rules | Mark Longair | 2013-05-24 | -0/+2 |
| | | | ||||
* | | | The file_name on getting an attachment should be the display filename | Mark Longair | 2013-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 difficult | Mark Longair | 2013-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. |