Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Add magic encoding line. | Louise Crow | 2013-02-16 | -0/+1 |
| | ||||
* | Don't use a from address that will affect other tests. | Louise Crow | 2013-02-16 | -1/+1 |
| | ||||
* | Clear purge requests in the setup of tests that count them. | Louise Crow | 2013-02-16 | -7/+12 |
| | ||||
* | 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. | |||
* | Direct the incoming mail to a fixture info_request so that it doesn't affect ↵ | Louise Crow | 2013-02-16 | -2/+6 |
| | | | | the holding pen if run before other tests that count holding pen entries. | |||
* | Merge branch 'hotfix/0.7.0.1' into develop | Louise Crow | 2013-02-15 | -0/+21 |
|\ | ||||
| * | Fix for #808. SafeBuffer and ActionMailer::Quoting.quoted_printable don't ↵ | Louise Crow | 2013-02-15 | -0/+21 |
| | | | | | | | | play well together, so convert all subject lines to strings before passing them off to actionmailer. | |||
* | | Merge remote-tracking branch 'openaustralia_github/fix-issue-789' into develop | Louise Crow | 2013-02-14 | -0/+5 |
|\ \ | |/ |/| | ||||
| * | Retain filter settings when redirecting a public body to its canonical url | Matthew Landauer | 2013-02-12 | -0/+5 |
| | | ||||
* | | Rename spec file so that it's picked up by rake spec. | Louise Crow | 2013-02-14 | -0/+0 |
|/ | ||||
* | Merge remote-tracking branch ↵ | Louise Crow | 2013-02-01 | -13/+8 |
|\ | | | | | | | 'openaustralia_github/backport_timezone_spec_rails_3_fix' into develop | |||
| * | read_attribute does timezone conversion in rails 3. So using ↵ | Matthew Landauer | 2013-01-25 | -13/+8 |
| | | | | | | | | attributes_before_type_cast instead | |||
* | | Merge remote-tracking branch ↵ | Louise Crow | 2013-02-01 | -7/+7 |
|\ \ | | | | | | | | | | 'openaustralia_github/use_url_helpers_in_redirect_tests' into develop | |||
| * | | modernise redirect tests by converting to using url helpers | Matthew Landauer | 2013-01-29 | -7/+7 |
| |/ | ||||
* | | Merge remote-tracking branch ↵ | Louise Crow | 2013-02-01 | -2/+2 |
|\ \ | | | | | | | | | | 'openaustralia_github/backport_rails_3_mailer_fix' into develop | |||
| * | | Rename mailer method to avoid naming conflict in Rails 3 | Matthew Landauer | 2013-01-25 | -2/+2 |
| |/ | ||||
* / | Calling TMail::Mail#base64_decode does not modify the mail object so doesn't ↵ | Matthew Landauer | 2013-01-28 | -1/+1 |
|/ | | | | do anything as used here | |||
* | Remove interlock as it's not compatible with Rails 3 and is unmaintained | Henare Degan | 2013-01-18 | -15/+0 |
| | ||||
* | Merge remote-tracking branch 'mysociety/develop' into rails_xss | Matthew Landauer | 2013-01-15 | -212/+1486 |
|\ | ||||
| * | As we're validating filename with validates_presence_of, which doesn't allow ↵ | Louise Crow | 2013-01-14 | -1/+12 |
| | | | | | | | | blanks, ensure_filename! should populate a default filename on a blank filename, as well as on nil. | |||
| * | Handle case where info request doesn't have a user_name | Louise Crow | 2013-01-07 | -0/+7 |
| | | ||||
| * | Only serve up 'similar' pages up to the offset we use for list. | Louise Crow | 2012-12-17 | -0/+8 |
| | | ||||
| * | Add a spec to test that mailin does not produce any output. This is ↵ | Louise Crow | 2012-12-17 | -0/+21 |
| | | | | | | | | important as under some MTA configurations, any output will result in a bounce message being sent to the original mail sender. | |||
| * | 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 | -2/+28 |
| |\ | ||||
| | * | Check that a request is publicly visible before generating a download link. | Louise Crow | 2012-12-13 | -1/+11 |
| | | | ||||
| | * | Only offer download links on publicly visible requests | Louise Crow | 2012-12-13 | -1/+17 |
| | | | ||||
| * | | 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. | |||
| * | | Wrap specs on the extraction of RFC-822 headers in code that sets the ENV ↵ | Louise Crow | 2012-12-11 | -50/+61 |
| | | | | | | | | | | | | timezone. TMail renders headers using localtime, which is not ideal, but we're migrating away from it anyway, so I'm not sure it's worth delving into the internals of TMail to fix it. | |||
| * | | Merge remote-tracking branch 'henare_github/relax-gc-for-tests' into develop | Louise Crow | 2012-12-10 | -0/+14 |
| |\ \ | ||||
| | * | | Only trigger garbage collection every 4 seconds - speeds up the whole test ↵ | Henare Degan | 2012-12-08 | -0/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | suite by ~35% on my machine Via https://37signals.com/svn/posts/2742-the-road-to-faster-tests | |||
| * | | | Add specs for attachment attributes. | Louise Crow | 2012-12-06 | -0/+20 |
| | | | | ||||
| * | | | Convert url in comment to spec. | Louise Crow | 2012-12-06 | -0/+39 |
| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: lib/mail_handler/backends/tmail_backend.rb | |||
| * | | | Add a spec for some basic consistency in content types, url part numbers, ↵ | Louise Crow | 2012-12-06 | -0/+80 |
| | | | | | | | | | | | | | | | | filenames, and within rfc822 subjects. | |||
| * | | | Convert example URL to spec. | Louise Crow | 2012-12-06 | -0/+460 |
| | | | | ||||
| * | | | Stub the close method on the object we're passing to the zip file extraction ↵ | Louise Crow | 2012-12-06 | -1/+3 |
| | | | | | | | | | | | | | | | | function. | |||
| * | | | Rename _get_attachment_text_internal_one_file to ↵ | Louise Crow | 2012-12-06 | -5/+4 |
| | | | | | | | | | | | | | | | | get_attachment_text_one_file as it is now an externally-accessed method of the mail handler module. | |||
| * | | | Move methods for getting the text out of attachments to the mail handler module. | Louise Crow | 2012-12-06 | -25/+24 |
| | | | | ||||
| * | | | Adding methods for getting the content type of a mail part, and getting ↵ | Louise Crow | 2012-12-04 | -0/+54 |
| | | | | | | | | | | | | | | | | header strings from a mail. | |||
| * | | | Fix syntax of example email. The number of expected attachments was based on ↵ | Louise Crow | 2012-12-04 | -4/+3 |
| | | | | | | | | | | | | | | | | the specific handling of a bad end boundary in tmail. | |||
| * | | | Add spec for handling an RFC822 attachment. | Louise Crow | 2012-12-04 | -0/+165 |
| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: spec/models/incoming_message_spec.rb | |||
| * | | | Add option to load_file_fixture to specify that the file contents should ↵ | Louise Crow | 2012-12-04 | -4/+8 |
| | | | | | | | | | | | | | | | | just be loaded as binary. | |||
| * | | | Add spec for the adding of headers to plain text bodies in attachments. | Louise Crow | 2012-12-04 | -0/+63 |
| | | | | ||||
| * | | | Rewrite and move spec so that it tests the mail handler method. | Louise Crow | 2012-12-04 | -5/+7 |
| | | | | ||||
| * | | | Removing obsolete comment - doesn't apply to this spec. | Louise Crow | 2012-12-04 | -3/+0 |
| | | | |