aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers
Commit message (Collapse)AuthorAgeLines
...
* | params_from doesn't exist anymoreMatthew Landauer2013-01-29-2/+2
| |
* | When matching mail.body with regex cast to string firstMatthew Landauer2013-01-29-3/+3
| |
* | Replace use of response.should send_email with checking ↵Matthew Landauer2013-01-29-4/+4
| | | | | | | | ActionMailer::Base.deliveries
* | 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
| |
* | When mocking ActsAsXapian can't use mock_model because it's not a modelMatthew Landauer2013-01-29-2/+2
| |
* | Reset Actionmailer deliveries on controller tests. Apparently rspec isn't ↵Matthew Landauer2013-01-29-0/+26
| | | | | | | | now doing this automatically
* | Replace use of have_text with contain from webrat or matchMatthew Landauer2013-01-29-33/+33
| |
* | Correct use of have_selector in b37586c7165afb7b3447dfe2780169a88cb0a942.Matthew Landauer2013-01-29-9/+17
| |
* | Replace use of have_tag with have_selector from webratMatthew Landauer2013-01-28-26/+26
| |
* | Use preceding slash on paths to fixtures filesMatthew Landauer2013-01-28-4/+4
| |
* | Disable routing filter in tests by using RoutingFilter.active rather than ↵Matthew Landauer2013-01-25-24/+16
| | | | | | | | writing to ActionController::Routing::Routes.filters
* | Change method name as it conflicts with globalize3 method nameHenare Degan2013-01-24-1/+1
| |
* | Merge remote-tracking branch 'mysociety/develop' into rails-3-spikeMatthew Landauer2013-01-04-58/+129
|\| | | | | | | | | | | | | | | | | | | | | 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 Crow2012-12-17-0/+8
| |
| * 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-1/+11
| |\
| | * Check that a request is publicly visible before generating a download link.Louise Crow2012-12-13-1/+11
| | |
| * | 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.
* | | No need to specify extension when testing particular template is renderedMatthew Landauer2013-01-04-7/+7
| | |
* | | response.status now returns an integerMatthew Landauer2013-01-04-6/+6
| | |
* | | Get rid of resetting of ActionController::Routing::Routes.filtersMatthew Landauer2013-01-04-28/+0
| | |
* | | integrate_views is render_views RSpec 2Henare Degan2012-12-11-47/+47
| | |
* | | The with_locale has been removed in Globalize3Henare Degan2012-12-11-9/+9
| | | | | | | | | | | | | | | | | | 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.
* | | Remove interlock as it's not compatible with Rails 3 and is unmaintainedHenare Degan2012-12-11-13/+0
| | |
* | | Switch to new method nameHenare Degan2012-12-11-9/+9
|/ /
* | Fix syntax of example email. The number of expected attachments was based on ↵Louise Crow2012-12-04-1/+1
| | | | | | | | the specific handling of a bad end boundary in tmail.
* | Add option to load_file_fixture to specify that the file contents should ↵Louise Crow2012-12-04-2/+1
| | | | | | | | just be loaded as binary.
* | The absence of an API key in an API request should be a permission denied ↵Louise Crow2012-12-03-6/+19
| | | | | | | | error, so that notification emails don't get sent.
* | Make regex for space normalization more unambiguous - it's the spaces at the ↵Louise Crow2012-11-29-55/+55
| | | | | | | | beginning and end of the whole string that we want to strip entirely, not at the beginning and end of each line (interpretation of ^ and $ is subject to default multiline behaviour of the regexp interpreter)
* | Give the implicit default locale in a URL without locale precedence over the ↵Louise Crow2012-11-28-0/+50
| | | | | | | | session in the case where the default locale is not being included in URLs. This allows the user to return to the default locale.
* | Use new function that copies existing xapian index in spec setup where a ↵Louise Crow2012-11-21-55/+55
| | | | | | | | clean copy of the xapian index with fixtures loaded is required.
* | Merge branch 'release/0.6.8' into developLouise Crow2012-11-12-11/+11
|\| | | | | | | | | Conflicts: lib/world_foi_websites.rb
| * Stub configuration in order to setup test cases. Previous direct setting of ↵Louise Crow2012-11-12-11/+11
| | | | | | | | config values was not isolated to particular tests.
* | Pass a hash of params to foi_fragment_cache_path in spec, as is passed in ↵Louise Crow2012-11-08-1/+8
|/ | | | actual usage.
* Add some specs for admin_current_userLouise Crow2012-10-30-0/+27
|
* Pull out a bit of shared setup code.Louise Crow2012-10-30-13/+11
|
* Don't load raw emails before the frontpage specs - they don't rely on the ↵Louise Crow2012-10-29-1/+0
| | | | raw email table being populated, and it's slow.
* Make spec a bit more generic in terms of what we're expecting. Really what ↵Louise Crow2012-10-29-12/+28
| | | | we want is reverse date order on the events, and only successful responses or initial sending.
* Mock xapian call for specs that are really for orthogonal things like ↵Louise Crow2012-10-29-0/+16
| | | | localization.
* Move search spec to search group.Louise Crow2012-10-29-10/+7
|
* Split up specs for frontpage and search.Louise Crow2012-10-29-25/+27
|
* Add spec - currently failing due to typo.Louise Crow2012-10-23-0/+42
|