Commit message (Collapse) | Author | Age | Lines | ||
---|---|---|---|---|---|
... | |||||
| * | Remove interlock as it's not compatible with Rails 3 and is unmaintained | Henare Degan | 2013-01-18 | -13/+0 | |
| | | |||||
* | | Surely we expect the modified body here | Henare Degan | 2013-03-04 | -1/+1 | |
| | | |||||
* | | Reload the object after reparsing (Rails 3.1 caching) | Henare Degan | 2013-03-04 | -0/+1 | |
| | | |||||
* | | response.redirected_to is deprecated | Henare Degan | 2013-03-03 | -1/+1 | |
| | | |||||
* | | Rename ALL THE TEMPLATES!!1!!!one!!1!! | Henare Degan | 2013-02-27 | -1/+1 | |
| | | | | | | | | .rhtml is deprecated in favour of .erb in Rails 3 | ||||
* | | file_name is passed as a string not an array | Henare Degan | 2013-02-26 | -22/+21 | |
| | | |||||
* | | Update to new mail sending API | Henare Degan | 2013-02-25 | -5/+9 | |
| | | |||||
* | | Mark this test as pending until it's fixed. See #819 | Henare Degan | 2013-02-20 | -6/+9 | |
| | | |||||
* | | Clear out ActionMailer::Base.deliveries before each test. Rspec should be ↵ | Matthew Landauer | 2013-01-29 | -33/+0 | |
| | | | | | | | | doing this but isn't for controller tests | ||||
* | | modernise redirect tests by converting to using url helpers | Matthew Landauer | 2013-01-29 | -7/+7 | |
| | | |||||
* | | When matching the body of an email to a regex cast it to a string | Matthew Landauer | 2013-01-29 | -6/+6 | |
| | | |||||
* | | Email address returned from mail doesn't have the name part anymore | Matthew Landauer | 2013-01-29 | -9/+9 | |
| | | |||||
* | | Pass string to paremeter in test | Matthew Landauer | 2013-01-29 | -1/+1 | |
| | | |||||
* | | When mocking ActsAsXapian can't use mock_model because it's not a model | Matthew Landauer | 2013-01-29 | -2/+2 | |
| | | |||||
* | | Reset Actionmailer deliveries on controller tests. Apparently rspec isn't ↵ | Matthew Landauer | 2013-01-29 | -0/+26 | |
| | | | | | | | | now doing this automatically | ||||
* | | Replace use of have_text with contain from webrat or match | Matthew Landauer | 2013-01-29 | -22/+22 | |
| | | |||||
* | | Correct use of have_selector in b37586c7165afb7b3447dfe2780169a88cb0a942. | Matthew Landauer | 2013-01-29 | -9/+17 | |
| | | |||||
* | | Replace use of have_tag with have_selector from webrat | Matthew Landauer | 2013-01-28 | -21/+21 | |
| | | |||||
* | | Use preceding slash on paths to fixtures files | Matthew Landauer | 2013-01-28 | -2/+2 | |
| | | |||||
* | | Disable routing filter in tests by using RoutingFilter.active rather than ↵ | Matthew Landauer | 2013-01-25 | -3/+2 | |
| | | | | | | | | writing to ActionController::Routing::Routes.filters | ||||
* | | Merge remote-tracking branch 'mysociety/develop' into rails-3-spike | Matthew Landauer | 2013-01-04 | -3/+86 | |
|\| | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile.lock Rakefile app/controllers/request_controller.rb lib/tasks/rspec.rake spec/controllers/services_controller_spec.rb spec/spec_helper.rb spec/views/request/_after_actions.rhtml_spec.rb | ||||
| * | 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 | -0/+27 | |
| | | | | | | | | 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. | ||||
* | | | No need to specify extension when testing particular template is rendered | Matthew Landauer | 2013-01-04 | -5/+5 | |
| | | | |||||
* | | | integrate_views is render_views RSpec 2 | Henare Degan | 2012-12-11 | -15/+15 | |
| | | | |||||
* | | | Remove interlock as it's not compatible with Rails 3 and is unmaintained | Henare Degan | 2012-12-11 | -13/+0 | |
|/ / | |||||
* | | 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. | ||||
* | | Use new function that copies existing xapian index in spec setup where a ↵ | Louise Crow | 2012-11-21 | -1/+1 | |
|/ | | | | clean copy of the xapian index with fixtures loaded is required. | ||||
* | Add spec - currently failing due to typo. | Louise Crow | 2012-10-23 | -0/+42 | |
| | |||||
* | Add extra checks from ↵ | Louise Crow | 2012-10-23 | -2/+4 | |
| | | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb and some extra description. | ||||
* | Adding new spec from ↵ | Louise Crow | 2012-10-23 | -2/+13 | |
| | | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb | ||||
* | Add an option to disable comments on a request | Robin Houston | 2012-09-19 | -0/+16 | |
| | | | | Closes #30. | ||||
* | Bugfix in spec. | Louise Crow | 2012-09-11 | -1/+1 | |
| | |||||
* | Record a status update by someone other than the request owner as a request ↵ | Louise Crow | 2012-09-11 | -2/+12 | |
| | | | | classification. | ||||
* | Update references to point to the mySociety repo. | Louise Crow | 2012-08-29 | -1/+1 | |
| | |||||
* | Now that external request followups can't be made and status can't be ↵ | Louise Crow | 2012-08-23 | -12/+104 | |
| | | | | updated, don't show links or text that suggests they can. | ||||
* | Don't allow status updates on external requests from the front end interface ↵ | Louise Crow | 2012-08-23 | -259/+331 | |
| | | | | | | (they can still be changed from the admin interface). Final part of fix for #562. | ||||
* | Don't show 'requester only' actions for an external request. | Louise Crow | 2012-08-23 | -0/+12 | |
| | | | | Part of the fix for #562. | ||||
* | Don't error on a request to followup an external request. | Louise Crow | 2012-08-23 | -0/+16 | |
| | | | | Partial solution to #562. | ||||
* | Use user_name to supply the user name for an internal or external request in ↵ | Louise Crow | 2012-08-22 | -32/+42 | |
| | | | | | | the simple correspondence template used when a full request zip is downloaded. Fixes #561. | ||||
* | Fix admin links in the case where an external request is viewed by an admin ↵ | Louise Crow | 2012-08-22 | -0/+12 | |
| | | | | user. | ||||
* | Don't send comment alerts for external requests. | Louise Crow | 2012-08-22 | -0/+10 | |
| | |||||
* | Re-enable testing of comment links in emails. | Louise Crow | 2012-08-22 | -7/+2 | |
| |