Commit message (Collapse) | Author | Age | Lines | ||
---|---|---|---|---|---|
... | |||||
* | | 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 | |
| | | |||||
* | | 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/+30 | |
|\ \ \ | | | | | | | | | | | | | 'openaustralia_github/more_info_on_report_request' into rails-3-develop | ||||
| * | | | Move reporting actions to their own controller | Matthew Landauer | 2013-03-22 | -118/+30 | |
| | | | | |||||
| * | | | 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 | |
| |/ / | |||||
* | | | 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. | ||||
* | | | Check that display_filename matches URL part number or fallback | Mark Longair | 2013-05-24 | -15/+35 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the display_filename of the attachment found from the URL part number doesn't match the passed in display_filename then the email may have been reparsed, causing a reordering. In that case, look to see if there is another attachment that uniquely matches that filename, and, if so, return that other attachment. If no matching uniquely matching filename is found, redirect to the incoming message, rather than returning a 404. | ||||
* | | | All the tests in this group rely on having the xapian db - make it part of ↵ | Louise Crow | 2013-05-02 | -1/+1 | |
| | | | | | | | | | | | | the setup. | ||||
* | | | The test "should work" relies on the xapian index existing - make sure this ↵ | Louise Crow | 2013-05-01 | -2/+4 | |
| |/ |/| | | | | | is true for all tests in this group. | ||||
* | | Fixup test issue found with config.order = 'random:5549' | Matthew Landauer | 2013-03-26 | -0/+2 | |
|/ | |||||
* | Fix another order dependent test issue | Matthew Landauer | 2013-03-19 | -0/+4 | |
| | |||||
* | Another order dependent test issue fixed | Matthew Landauer | 2013-03-19 | -0/+2 | |
| | |||||
* | Reenable test now that we don't do HTML validation in the test suite | Henare Degan | 2013-03-16 | -9/+6 | |
| | |||||
* | Changed in 166b6d90bc2dc79536d8d2d145281fa340075735 | Henare Degan | 2013-03-15 | -1/+1 | |
| | |||||
* | Update to current matcher API | Henare Degan | 2013-03-15 | -2/+2 | |
| | |||||
* | Merge remote-tracking branch 'mysociety/develop' into rails-3-develop | Henare Degan | 2013-03-14 | -24/+81 | |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock app/controllers/admin_request_controller.rb app/controllers/admin_track_controller.rb app/controllers/request_controller.rb app/controllers/services_controller.rb app/helpers/link_to_helper.rb app/mailers/request_mailer.rb app/models/application_mailer.rb app/models/info_request.rb app/views/admin_censor_rule/edit.html.erb app/views/admin_censor_rule/new.html.erb app/views/admin_public_body/_form.html.erb app/views/admin_public_body/_locale_selector.html.erb app/views/admin_public_body/_one_list.html.erb app/views/admin_public_body/edit.html.erb app/views/admin_public_body/list.html.erb app/views/admin_public_body/new.html.erb app/views/admin_request/_incoming_message_actions.html.erb app/views/admin_request/edit.html.erb app/views/admin_request/edit_comment.html.erb app/views/admin_request/edit_outgoing.html.erb app/views/admin_request/list.html.erb app/views/admin_request/list_old_unclassified.html.erb app/views/admin_request/show.html.erb app/views/admin_track/_some_tracks.html.erb app/views/admin_track/list.html.erb app/views/admin_user/edit.html.erb app/views/admin_user/list.html.erb app/views/admin_user/show.html.erb app/views/general/_footer.html.erb app/views/general/exception_caught.html.erb app/views/help/contact.html.erb app/views/layouts/default.html.erb app/views/public_body/_alphabet.html.erb app/views/request/_request_listing_single.html.erb app/views/request/_sidebar.html.erb app/views/request/new.html.erb app/views/request/show.html.erb app/views/request_mailer/external_response.rhtml app/views/request_mailer/fake_response.rhtml config/environment.rb config/environments/production.rb config/routes.rb spec/controllers/admin_censor_rule_controller_spec.rb spec/controllers/request_controller_spec.rb spec/controllers/track_controller_spec.rb spec/helpers/link_to_helper_spec.rb spec/mailers/request_mailer_spec.rb spec/models/info_request_spec.rb spec/spec_helper.rb spec/views/public_body/show.html.erb_spec.rb spec/views/request/show.html.erb_spec.rb vendor/plugins/rails_xss/lib/rails_xss/erubis.rb | ||||
| * | Update URLs in spec now that custom url methods have been removed. | Louise Crow | 2013-03-13 | -3/+3 | |
| | | |||||
| * | 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 | |
| | | |||||
| * | 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. | ||||
| * | modernise redirect tests by converting to using url helpers | Matthew Landauer | 2013-01-29 | -7/+7 | |
| | |