Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Use earlier factory_girl syntax | Louise Crow | 2013-09-16 | -5/+7 |
| | ||||
* | Add public criteria for message event access methods | Louise Crow | 2013-09-16 | -9/+10 |
| | | | | | | | get_last_response_event and get_last_outgoing_event are used in various places to determine which events to link to, use in queries etc. Restrict them to refer to the last publicly visible event of the relevant type, and rename them to make that clear. | |||
* | Fix spec descriptions | Louise Crow | 2013-09-16 | -2/+2 |
| | ||||
* | Add prominence fields and cache expiry to outgoing admin | Louise Crow | 2013-09-16 | -13/+95 |
| | ||||
* | 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 |
| |/ / |