Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Mailers have their own home in Rails 3 | Henare Degan | 2013-02-25 | -871/+0 |
| | ||||
* | and_return with should_not_receive is deprecated. Fixes #826 | Henare Degan | 2013-02-25 | -2/+2 |
| | ||||
* | Mocking and stubbing work a bit differently, this allows the test to run | Henare Degan | 2013-02-08 | -1/+1 |
| | ||||
* | Add a test that exposes a problem we have with renaming attributes | Henare Degan | 2013-02-07 | -0/+8 |
| | ||||
* | Move over to rails 3 mail api because there appear to be problems with ↵ | Matthew Landauer | 2013-02-01 | -1/+1 |
| | | | | sending attachments | |||
* | Merge branch 'i18n_rails3_preparation' into rails-3-spike | Matthew Landauer | 2013-01-31 | -1/+1 |
|\ | | | | | | | | | | | | | | | Conflicts: Gemfile.lock app/controllers/general_controller.rb app/controllers/public_body_controller.rb spec/controllers/public_body_controller_spec.rb | |||
| * | Set locale with I18n rather than through globalize | Matthew Landauer | 2013-01-17 | -3/+3 |
| | | ||||
* | | 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 | |||
* | | Merge branch 'rails_xss' into rails-3-spike | Matthew Landauer | 2013-01-25 | -1/+19 |
|\| | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock config/environment.rb lib/i18n_fixes.rb | |||
| * | 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 |
| | | ||||
* | | Fixes to ApplicationMailer tests in Rails 3 | Matthew Landauer | 2013-01-23 | -15/+11 |
| | | ||||
* | | Fix deprecation warnings | Henare Degan | 2013-01-23 | -3/+3 |
| | | | | | | | | | | * Errors#on have been deprecated * Errors#[] now always returns an Array | |||
* | | Change syntax - this looks like a test to me | Henare Degan | 2013-01-23 | -1/+1 |
| | | ||||
* | | In 447e4af we started adding the error to each attribute so start testing each | Henare Degan | 2013-01-23 | -4/+10 |
| | | ||||
* | | Merge remote-tracking branch 'mysociety/develop' into rails-3-spike | Matthew Landauer | 2013-01-04 | -24/+32 |
|\| | | | | | | | | | | | | | | | | | | | | | 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 | |||
| * | Wrap specs on the extraction of RFC-822 headers in code that sets the ENV ↵ | Louise Crow | 2012-12-11 | -24/+32 |
| | | | | | | | | 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. | |||
* | | The run_callbacks API has changed | Henare Degan | 2012-12-12 | -1/+1 |
| | | | | | | | | https://rails.lighthouseapp.com/projects/8994/tickets/5472-unable-to-manually-run-activerecord-object-callbacks | |||
* | | Update to new RSpec module names | Henare Degan | 2012-12-12 | -1/+1 |
| | | ||||
* | | safe_mock_model doesn't seem to be needed, also tested in Ruby 1.9 | Henare Degan | 2012-12-12 | -12/+12 |
| | | ||||
* | | The with_locale has been removed in Globalize3 | Henare Degan | 2012-12-11 | -3/+3 |
| | | | | | | | | | | | | I think Globalize is supposed to pick up the locale from I18n anyway so I don't know if these are needed. I think I still haven't done the right thing but it's time to move on. | |||
* | | Switch to new method name | Henare Degan | 2012-12-11 | -3/+3 |
|/ | ||||
* | Convert url in comment to spec. | Louise Crow | 2012-12-06 | -0/+8 |
| | | | | | | Conflicts: lib/mail_handler/backends/tmail_backend.rb | |||
* | Rename _get_attachment_text_internal_one_file to ↵ | Louise Crow | 2012-12-06 | -2/+2 |
| | | | | 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 | -24/+3 |
| | ||||
* | Add spec for handling an RFC822 attachment. | Louise Crow | 2012-12-04 | -0/+18 |
| | | | | | | Conflicts: spec/models/incoming_message_spec.rb | |||
* | Add spec for the adding of headers to plain text bodies in attachments. | Louise Crow | 2012-12-04 | -0/+13 |
| | ||||
* | Rewrite and move spec so that it tests the mail handler method. | Louise Crow | 2012-12-04 | -5/+0 |
| | ||||
* | Rewrite spec to reflect delegation of low-level mail methods to mail ↵ | Louise Crow | 2012-12-04 | -23/+14 |
| | | | | handler, use mail handler methods. | |||
* | Standardize slug generation for external users so that it uses the same ↵ | Louise Crow | 2012-11-29 | -0/+16 |
| | | | | underlying method as other slug generation. Fixes #567. | |||
* | Use new function that copies existing xapian index in spec setup where a ↵ | Louise Crow | 2012-11-21 | -11/+11 |
| | | | | clean copy of the xapian index with fixtures loaded is required. | |||
* | Wrap address parsing in a address_from_string method in the mail handler. | Louise Crow | 2012-11-15 | -6/+41 |
| | ||||
* | Move address_from_name_and_email to mail handler. | Louise Crow | 2012-11-15 | -1/+1 |
| | ||||
* | Use mail handler in request mailer spec. | Louise Crow | 2012-11-15 | -1/+1 |
| | ||||
* | Move TMail extensions to mail handler. | Louise Crow | 2012-11-15 | -0/+20 |
| | ||||
* | Don't load raw emails data in specs that don't use it. | Louise Crow | 2012-11-15 | -18/+0 |
| | ||||
* | Factor out method for getting a mail object from a fixture file. | Louise Crow | 2012-11-15 | -10/+7 |
| | ||||
* | Use mailhandler method in incoming message specs. | Louise Crow | 2012-11-15 | -10/+5 |
| | ||||
* | Make error message expectation slightly less specific - message format is ↵ | Louise Crow | 2012-11-08 | -2/+4 |
| | | | | slightly different under ruby 1.9. | |||
* | use safe_mock_model | James McKinney | 2012-11-01 | -6/+6 |
| | ||||
* | Restore removed testing config param, remove spec for unset param - I think ↵ | Louise Crow | 2012-10-25 | -9/+0 |
| | | | | not really worth it. | |||
* | Given that MAX_REQUESTS_PER_USER_PER_DAY is documented as an optional ↵ | Louise Crow | 2012-10-25 | -49/+58 |
| | | | | variable, but that commonlib/rblib/config.rb does not allow nil defaults, replace nil value clause (which can never be reached) with blank value condition(blank now returned by default by lib/configuation.rb) | |||
* | Merge remote-tracking branch ↵ | Louise Crow | 2012-10-25 | -4/+5 |
|\ | | | | | | | 'openaustralia_github/follow-all-requests-description' into develop | |||
| * | Fix description of a search | Henare Degan | 2012-10-18 | -1/+2 |
| | | | | | | | | Used to say "comments or requests which are " when you did: View requests > All requests > Follow | |||
| * | Make test a bit more Rubyesque | Henare Degan | 2012-10-18 | -4/+4 |
| | | | | | | | | The purist would probably say separate these into 3 tests | |||
* | | Merge branch 'feature/ruby-19-compat-immediate' into develop | Louise Crow | 2012-10-24 | -0/+17 |
|\ \ | |/ |/| | | | | | Conflicts: Gemfile.lock | |||
| * | Convert example in comment to failing spec, update regex to make spec pass - ↵ | Louise Crow | 2012-10-18 | -0/+9 |
| | | | | | | | | again the use of converted HTML parts in emails means extra leading spaces. | |||
| * | Convert request cited in comment to failing spec, update regex to make it ↵ | Louise Crow | 2012-10-18 | -0/+8 |
| | | | | | | | | pass (we now use the HTML parts of emails by preference, so there are some leading spaces.) | |||
* | | Merge remote-tracking branch 'openaustralia_github/postfix_log_support' into ↵ | Louise Crow | 2012-10-12 | -0/+166 |
|\ \ | |/ |/| | | | | | | | | | | | develop Conflicts: Gemfile.lock app/models/exim_log.rb | |||
| * | Revert "Also allow arbitrary subdomain on email addresses picked up in the logs" | Matthew Landauer | 2012-10-12 | -5/+0 |
| | | | | | | | | This reverts commit 9ae51d5ff6a9f56fd1bd4ba4bdd84bfd43ac6d95. |