aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers/request_controller_spec.rb
Commit message (Collapse)AuthorAgeLines
* Handle json requests for hidden info requests.Louise Crow2013-06-03-7/+32
|
* 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.
* Check that display_filename matches URL part number or fallbackMark Longair2013-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 Crow2013-05-02-1/+1
| | | | the setup.
* The test "should work" relies on the xapian index existing - make sure this ↵Louise Crow2013-05-01-2/+4
| | | | is true for all tests in this group.
* Fixup test issue found with config.order = 'random:5549'Matthew Landauer2013-03-26-0/+2
|
* Fix another order dependent test issueMatthew Landauer2013-03-19-0/+4
|
* Another order dependent test issue fixedMatthew Landauer2013-03-19-0/+2
|
* Reenable test now that we don't do HTML validation in the test suiteHenare Degan2013-03-16-9/+6
|
* Changed in 166b6d90bc2dc79536d8d2d145281fa340075735Henare Degan2013-03-15-1/+1
|
* Update to current matcher APIHenare Degan2013-03-15-2/+2
|
* Merge remote-tracking branch 'mysociety/develop' into rails-3-developHenare Degan2013-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 Crow2013-03-13-3/+3
| |
| * Now direct changes of state to error_message and requires_admin to a new ↵Matthew Landauer2013-03-06-15/+19
| | | | | | | | page asking for more info
| * Can also record a message when posting to RequestController#describe_stateMatthew Landauer2013-03-05-3/+21
| |
| * Remove code added previously because on the wrong trackMatthew Landauer2013-03-05-62/+0
| |
| * Don't allow external requests to have their state changedMatthew Landauer2013-03-04-0/+14
| |
| * Record who changes the stateMatthew Landauer2013-03-04-1/+1
| |
| * Fix bug when user not owner of request changing state to requires_admin ↵Matthew Landauer2013-03-04-3/+12
| | | | | | | | email should come from user making the change
| * Inline temporary variablesMatthew Landauer2013-03-04-0/+1
| |
| * Simplify controller test by mock behaviour in AbilityMatthew Landauer2013-03-03-45/+13
| |
| * Can update status when request is old and unclassifiedMatthew Landauer2013-03-03-0/+24
| |
| * admin user can change statusMatthew Landauer2013-03-03-2/+16
| |
| * Refactor tests a littleMatthew Landauer2013-03-03-19/+21
| |
| * only can make the change as the owner of a requestMatthew Landauer2013-03-03-0/+14
| |
| * Add basic authentication to new actionMatthew Landauer2013-03-03-16/+26
| |
| * Switch to pretty urlsMatthew Landauer2013-03-02-1/+1
| |
| * New request controller action for requires_admin state with messageMatthew Landauer2013-03-02-0/+16
| |
| * Use routes to only allow post to RequestController#describe_stateMatthew Landauer2013-03-02-9/+5
| |
| * Test for what happens when user is playing the classification gameMatthew Landauer2013-03-02-0/+11
| |
| * Test for situation where request changes while user is looking at it and ↵Matthew Landauer2013-03-02-0/+10
| | | | | | | | they change the status
| * Test for forgetting to select the statusMatthew Landauer2013-03-02-0/+8
| |
| * Don't integrate views here - we're not looking at a controller.Louise Crow2013-02-16-1/+0
| |
| * Clear the purge requests in the setup for these tests so they are isolated ↵Louise Crow2013-02-16-0/+3
| | | | | | | | from the behaviour of other tests.
| * modernise redirect tests by converting to using url helpersMatthew Landauer2013-01-29-7/+7
| |
| * Remove interlock as it's not compatible with Rails 3 and is unmaintainedHenare Degan2013-01-18-13/+0
| |
* | Surely we expect the modified body hereHenare Degan2013-03-04-1/+1
| |
* | Reload the object after reparsing (Rails 3.1 caching)Henare Degan2013-03-04-0/+1
| |
* | response.redirected_to is deprecatedHenare Degan2013-03-03-1/+1
| |
* | Rename ALL THE TEMPLATES!!1!!!one!!1!!Henare Degan2013-02-27-1/+1
| | | | | | | | .rhtml is deprecated in favour of .erb in Rails 3
* | file_name is passed as a string not an arrayHenare Degan2013-02-26-22/+21
| |
* | Update to new mail sending APIHenare Degan2013-02-25-5/+9
| |
* | Mark this test as pending until it's fixed. See #819Henare Degan2013-02-20-6/+9
| |
* | Clear out ActionMailer::Base.deliveries before each test. Rspec should be ↵Matthew Landauer2013-01-29-33/+0
| | | | | | | | doing this but isn't for controller tests
* | modernise redirect tests by converting to using url helpersMatthew Landauer2013-01-29-7/+7
| |
* | When matching the body of an email to a regex cast it to a stringMatthew Landauer2013-01-29-6/+6
| |
* | Email address returned from mail doesn't have the name part anymoreMatthew Landauer2013-01-29-9/+9
| |
* | Pass string to paremeter in testMatthew Landauer2013-01-29-1/+1
| |
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-20 10:07:06 +0000