aboutsummaryrefslogtreecommitdiffstats
path: root/spec
Commit message (Collapse)AuthorAgeLines
* Add magic encoding line.Louise Crow2013-02-16-0/+1
|
* Don't use a from address that will affect other tests.Louise Crow2013-02-16-1/+1
|
* Clear purge requests in the setup of tests that count them.Louise Crow2013-02-16-7/+12
|
* 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.
* Direct the incoming mail to a fixture info_request so that it doesn't affect ↵Louise Crow2013-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 developLouise Crow2013-02-15-0/+21
|\
| * Fix for #808. SafeBuffer and ActionMailer::Quoting.quoted_printable don't ↵Louise Crow2013-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 developLouise Crow2013-02-14-0/+5
|\ \ | |/ |/|
| * Retain filter settings when redirecting a public body to its canonical urlMatthew Landauer2013-02-12-0/+5
| |
* | Rename spec file so that it's picked up by rake spec.Louise Crow2013-02-14-0/+0
|/
* Merge remote-tracking branch ↵Louise Crow2013-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 Landauer2013-01-25-13/+8
| | | | | | | | attributes_before_type_cast instead
* | Merge remote-tracking branch ↵Louise Crow2013-02-01-7/+7
|\ \ | | | | | | | | | 'openaustralia_github/use_url_helpers_in_redirect_tests' into develop
| * | modernise redirect tests by converting to using url helpersMatthew Landauer2013-01-29-7/+7
| |/
* | Merge remote-tracking branch ↵Louise Crow2013-02-01-2/+2
|\ \ | | | | | | | | | 'openaustralia_github/backport_rails_3_mailer_fix' into develop
| * | Rename mailer method to avoid naming conflict in Rails 3Matthew Landauer2013-01-25-2/+2
| |/
* / Calling TMail::Mail#base64_decode does not modify the mail object so doesn't ↵Matthew Landauer2013-01-28-1/+1
|/ | | | do anything as used here
* Remove interlock as it's not compatible with Rails 3 and is unmaintainedHenare Degan2013-01-18-15/+0
|
* Merge remote-tracking branch 'mysociety/develop' into rails_xssMatthew Landauer2013-01-15-212/+1486
|\
| * As we're validating filename with validates_presence_of, which doesn't allow ↵Louise Crow2013-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_nameLouise Crow2013-01-07-0/+7
| |
| * Only serve up 'similar' pages up to the offset we use for list.Louise Crow2012-12-17-0/+8
| |
| * Add a spec to test that mailin does not produce any output. This is ↵Louise Crow2012-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 Crow2012-12-17-0/+1
| |
| * Rewrite specs that were in spec/controller/application_controller as ↵Louise Crow2012-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 developLouise Crow2012-12-13-2/+28
| |\
| | * Check that a request is publicly visible before generating a download link.Louise Crow2012-12-13-1/+11
| | |
| | * Only offer download links on publicly visible requestsLouise Crow2012-12-13-1/+17
| | |
| * | Merge branch 'hotfix/no-html-conversion-for-hidden' into developLouise Crow2012-12-13-0/+15
| |\|
| | * Don't offer or allow viewing of an HTML version of a response attachment if ↵Louise Crow2012-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 Crow2012-12-13-2/+9
| | |
| * | Don't offer or allow viewing of an HTML version of a request if it is ↵Louise Crow2012-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 Crow2012-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 developLouise Crow2012-12-10-0/+14
| |\ \
| | * | Only trigger garbage collection every 4 seconds - speeds up the whole test ↵Henare Degan2012-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 Crow2012-12-06-0/+20
| | | |
| * | | Convert url in comment to spec.Louise Crow2012-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 Crow2012-12-06-0/+80
| | | | | | | | | | | | | | | | filenames, and within rfc822 subjects.
| * | | Convert example URL to spec.Louise Crow2012-12-06-0/+460
| | | |
| * | | Stub the close method on the object we're passing to the zip file extraction ↵Louise Crow2012-12-06-1/+3
| | | | | | | | | | | | | | | | function.
| * | | Rename _get_attachment_text_internal_one_file to ↵Louise Crow2012-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 Crow2012-12-06-25/+24
| | | |
| * | | Adding methods for getting the content type of a mail part, and getting ↵Louise Crow2012-12-04-0/+54
| | | | | | | | | | | | | | | | header strings from a mail.
| * | | Fix syntax of example email. The number of expected attachments was based on ↵Louise Crow2012-12-04-4/+3
| | | | | | | | | | | | | | | | the specific handling of a bad end boundary in tmail.
| * | | Add spec for handling an RFC822 attachment.Louise Crow2012-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 Crow2012-12-04-4/+8
| | | | | | | | | | | | | | | | just be loaded as binary.
| * | | Add spec for the adding of headers to plain text bodies in attachments.Louise Crow2012-12-04-0/+63
| | | |
| * | | Rewrite and move spec so that it tests the mail handler method.Louise Crow2012-12-04-5/+7
| | | |
| * | | Removing obsolete comment - doesn't apply to this spec.Louise Crow2012-12-04-3/+0
| | | |