Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Set locale with I18n rather than through globalize | Matthew Landauer | 2013-04-02 | -7/+7 |
| | | | | | | Conflicts: app/controllers/general_controller.rb | |||
* | Use full url in test for redirect to ensure in correct locale | Matthew Landauer | 2013-04-02 | -1/+1 |
| | ||||
* | Set locale in controller test by passing parameter in get | Matthew Landauer | 2013-04-02 | -9/+5 |
| | | | | Also using I18n.locale to pass the current locale around. | |||
* | Allow the emergency user account to be disabled. | Louise Crow | 2013-03-19 | -0/+13 |
| | ||||
* | Update URLs in spec now that custom url methods have been removed. | Louise Crow | 2013-03-13 | -3/+3 |
| | ||||
* | Merge remote-tracking branch ↵ | Louise Crow | 2013-03-13 | -22/+79 |
|\ | | | | | | | | | | | | | | | | | | | | | 'openaustralia_github/one_support_email_on_admin_attention_state' into develop Conflicts: Gemfile.lock app/controllers/request_controller.rb app/models/request_mailer.rb app/views/admin_general/index.rhtml spec/models/request_mailer_spec.rb | |||
| * | Now direct changes of state to error_message and requires_admin to a new ↵ | Matthew Landauer | 2013-03-06 | -15/+19 |
| | | | | | | | | page asking for more info | |||
| * | Can also record a message when posting to RequestController#describe_state | Matthew Landauer | 2013-03-05 | -3/+21 |
| | | ||||
| * | Remove code added previously because on the wrong track | Matthew Landauer | 2013-03-05 | -62/+0 |
| | | ||||
| * | Don't allow external requests to have their state changed | Matthew Landauer | 2013-03-04 | -0/+14 |
| | | ||||
| * | Record who changes the state | Matthew Landauer | 2013-03-04 | -1/+1 |
| | | ||||
| * | Fix bug when user not owner of request changing state to requires_admin ↵ | Matthew Landauer | 2013-03-04 | -3/+12 |
| | | | | | | | | email should come from user making the change | |||
| * | Inline temporary variables | Matthew Landauer | 2013-03-04 | -0/+1 |
| | | ||||
| * | Simplify controller test by mock behaviour in Ability | Matthew Landauer | 2013-03-03 | -45/+13 |
| | | ||||
| * | Can update status when request is old and unclassified | Matthew Landauer | 2013-03-03 | -0/+24 |
| | | ||||
| * | admin user can change status | Matthew Landauer | 2013-03-03 | -2/+16 |
| | | ||||
| * | Refactor tests a little | Matthew Landauer | 2013-03-03 | -19/+21 |
| | | ||||
| * | only can make the change as the owner of a request | Matthew Landauer | 2013-03-03 | -0/+14 |
| | | ||||
| * | Add basic authentication to new action | Matthew Landauer | 2013-03-03 | -16/+26 |
| | | ||||
| * | Switch to pretty urls | Matthew Landauer | 2013-03-02 | -1/+1 |
| | | ||||
| * | New request controller action for requires_admin state with message | Matthew Landauer | 2013-03-02 | -0/+16 |
| | | ||||
| * | Use routes to only allow post to RequestController#describe_state | Matthew Landauer | 2013-03-02 | -9/+5 |
| | | ||||
| * | Test for what happens when user is playing the classification game | Matthew Landauer | 2013-03-02 | -0/+11 |
| | | ||||
| * | Test for situation where request changes while user is looking at it and ↵ | Matthew Landauer | 2013-03-02 | -0/+10 |
| | | | | | | | | they change the status | |||
| * | Test for forgetting to select the status | Matthew Landauer | 2013-03-02 | -0/+8 |
| | | ||||
* | | Merge remote-tracking branch 'openaustralia_github/make-links-relative' into ↵ | Louise Crow | 2013-03-06 | -2/+1 |
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | develop Conflicts: app/controllers/admin_request_controller.rb app/models/request_mailer.rb app/views/admin_general/index.rhtml app/views/admin_general/stats.rhtml app/views/admin_public_body/_one_list.rhtml app/views/admin_public_body/_tags.rhtml app/views/admin_public_body/show.rhtml app/views/admin_request/show.rhtml app/views/admin_user/show.rhtml app/views/layouts/admin.rhtml app/views/public_body/show.rhtml app/views/request/new.rhtml | |||
| * | Stop using main_url to generate absolute urls for the main site | Matthew Landauer | 2013-02-15 | -2/+1 |
| | | ||||
* | | Clear purge requests in the setup of tests that count them. | Louise Crow | 2013-02-16 | -3/+7 |
| | | ||||
* | | Don't integrate views here - we're not looking at a controller. | Louise Crow | 2013-02-16 | -1/+0 |
| | | ||||
* | | Clear the purge requests in the setup for these tests so they are isolated ↵ | Louise Crow | 2013-02-16 | -0/+3 |
| | | | | | | | | from the behaviour of other tests. | |||
* | | Retain filter settings when redirecting a public body to its canonical url | Matthew Landauer | 2013-02-12 | -0/+5 |
|/ | ||||
* | modernise redirect tests by converting to using url helpers | Matthew Landauer | 2013-01-29 | -7/+7 |
| | ||||
* | Remove interlock as it's not compatible with Rails 3 and is unmaintained | Henare Degan | 2013-01-18 | -13/+0 |
| | ||||
* | Only serve up 'similar' pages up to the offset we use for list. | Louise Crow | 2012-12-17 | -0/+8 |
| | ||||
* | Adding mocking of incoming message id. | Louise Crow | 2012-12-17 | -0/+1 |
| | ||||
* | Rewrite specs that were in spec/controller/application_controller as ↵ | Louise Crow | 2012-12-17 | -56/+71 |
| | | | | full-stack controller specs in the relevant controllers. It turns out that having spec blocks that reference the ApplicationController class directly i.e. "describe ApplicationController" can have unpredictable effects. actionpack's action_controller/test_case.rb rewrites rescue_action_without_handler on whatever it is included in, and if this is done on a controller class, and then directly on action controller, it can result in an infinite loop of recursive calls. This turns out to be the problem that was causing some tests in error_spec.rb to fail in Travis under Ruby 1.9. | |||
* | Merge branch 'hotfix/authenticate-and-expire-download-zips' into develop | Louise Crow | 2012-12-13 | -1/+11 |
|\ | ||||
| * | Check that a request is publicly visible before generating a download link. | Louise Crow | 2012-12-13 | -1/+11 |
| | | ||||
* | | Merge branch 'hotfix/no-html-conversion-for-hidden' into develop | Louise Crow | 2012-12-13 | -0/+15 |
|\| | ||||
| * | Don't offer or allow viewing of an HTML version of a response attachment if ↵ | Louise Crow | 2012-12-13 | -0/+15 |
| | | | | | | | | the request is hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently can produce image files that will then be publicly viewable directly from the webserver (see config/httpd.conf). If necessary we can revisit this code to enable admins and requesters to view the HTML version created by our own conversion without adding these files to a path that is served directly by the web server. | |||
* | | Reformat for line length, add the expected response code. | Louise Crow | 2012-12-13 | -2/+9 |
| | | ||||
* | | Don't offer or allow viewing of an HTML version of a request if it is ↵ | Louise Crow | 2012-12-13 | -0/+15 |
| | | | | | | | | hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently produces image files that will then be publicly viewable. If necessary we can revisit this code to enable admins and requesters to view the HTML version created by our own conversion without adding these files to a path that is served directly by the web server. | |||
* | | Fix syntax of example email. The number of expected attachments was based on ↵ | Louise Crow | 2012-12-04 | -1/+1 |
| | | | | | | | | the specific handling of a bad end boundary in tmail. | |||
* | | Add option to load_file_fixture to specify that the file contents should ↵ | Louise Crow | 2012-12-04 | -2/+1 |
| | | | | | | | | just be loaded as binary. | |||
* | | The absence of an API key in an API request should be a permission denied ↵ | Louise Crow | 2012-12-03 | -6/+19 |
| | | | | | | | | error, so that notification emails don't get sent. | |||
* | | Make regex for space normalization more unambiguous - it's the spaces at the ↵ | Louise Crow | 2012-11-29 | -55/+55 |
| | | | | | | | | beginning and end of the whole string that we want to strip entirely, not at the beginning and end of each line (interpretation of ^ and $ is subject to default multiline behaviour of the regexp interpreter) | |||
* | | Give the implicit default locale in a URL without locale precedence over the ↵ | Louise Crow | 2012-11-28 | -0/+50 |
| | | | | | | | | session in the case where the default locale is not being included in URLs. This allows the user to return to the default locale. | |||
* | | Use new function that copies existing xapian index in spec setup where a ↵ | Louise Crow | 2012-11-21 | -55/+55 |
| | | | | | | | | clean copy of the xapian index with fixtures loaded is required. | |||
* | | Merge branch 'release/0.6.8' into develop | Louise Crow | 2012-11-12 | -11/+11 |
|\| | | | | | | | | | Conflicts: lib/world_foi_websites.rb | |||
| * | Stub configuration in order to setup test cases. Previous direct setting of ↵ | Louise Crow | 2012-11-12 | -11/+11 |
| | | | | | | | | config values was not isolated to particular tests. |