aboutsummaryrefslogtreecommitdiffstats
path: root/spec/controllers/general_controller_spec.rb
Commit message (Collapse)AuthorAgeLines
* Don't redirect to signin on expiration of a non remember-me sessionLouise Crow2015-04-23-1/+0
| | | | In practice, it's just confusing if you've been away from the site.
* Add an autodetect feed for the frontpage list of requestsLouise Crow2015-03-19-0/+8
| | | | Closes Â#434.
* Merge branch 'hotfix/0.20.0.7' into rails-3-developLouise Crow2015-02-05-0/+6
|\ | | | | | | | | Conflicts: config/initializers/alaveteli.rb
| * Do not allow requests for search results after the first 500Louise Crow2015-02-03-0/+6
| | | | | | | | | | | | | | The pages for these search results are extremely slow to load. This is not an ideal solution by any means. Really we want to dig into why high offsets are so slow, and whether there's anything we can do to fix that.
* | Merge branch 'hotfix/0.20.0.2' into rails-3-developLouise Crow2015-01-07-0/+12
|\|
| * Sanitize the blog contents0.20.0.2Louise Crow2014-12-22-0/+12
| |
* | Enforce a lifetime on session cookiesLouise Crow2014-12-12-0/+29
|/ | | | | | Problem described in http://seclists.org/fulldisclosure/2013/Sep/145 Pattern taken from https://www.coffeepowered.net/2013/09/26/rails-session-cookies/
* Handle unhelpful stemmingGareth Rees2014-06-25-1/+1
| | | | | | | Stemming returns 'bore' as the word to highlight which can't be matched in the original phrase. Also removes duplicates from the results
* Use regex based highlightingGareth Rees2014-06-25-1/+1
|
* Only show the blog if a BLOG_FEED is configured.Louise Crow2013-12-09-0/+11
|
* Move getting recent requests into a helper method.Louise Crow2013-11-25-42/+0
|
* Refactor some tests to use with_default_localeMark Longair2013-11-19-10/+8
|
* Show 'browse all' and 'ask us to add' one for authority only search.Louise Crow2013-11-07-0/+5
| | | | | This is for the case where there are no results and the user was explicitly searching for authorities.
* Show highlights and correction regardless of search type.Louise Crow2013-11-07-0/+11
| | | | Contributes to #1118.
* Switch routing-filter (which takes locale out of the params and puts it in ↵Louise Crow2013-07-18-85/+3
| | | | the URL) off by default in model, controller, and helper tests. This means we can supply the locale as a param. Turn it on specifically for a couple of controller tests that test routing, and change other url localization tests into integration tests.
* Revert "Assume that format will be specified in the url requested, or ↵Louise Crow2013-06-26-6/+0
| | | | | | default to HTML. Fixes #989." This reverts commit c1b3324922687778b316ee1a180582013bcf6d0e.
* Don't call out to external urls during controller testsLouise Crow2013-06-20-0/+2
|
* Assume that format will be specified in the url requested, or default to ↵Louise Crow2013-06-20-0/+6
| | | | HTML. Fixes #989.
* Remove support for theme stylesheet inclusion via template deprecated in ↵Louise Crow2013-06-05-1/+0
| | | | version 0.5. Closes #972.
* Remove debian-specific require clause and use Gemfile to specify ↵Louise Crow2013-04-30-1/+10
| | | | non-standard require - bundler should be supplying the gem now.
* Rename Configuration class to avoid conflict with ActiveSupport::ConfigurableHenare Degan2013-03-03-4/+4
|
* Handling of globbed parameters in routes has changedMatthew Landauer2013-02-01-9/+9
|
* Replace use of have_text with contain from webrat or matchMatthew Landauer2013-01-29-9/+9
|
* Replace use of have_tag with have_selector from webratMatthew Landauer2013-01-28-4/+4
|
* response.status now returns an integerMatthew Landauer2013-01-04-1/+1
|
* integrate_views is render_views RSpec 2Henare Degan2012-12-11-2/+2
|
* 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-3/+3
| | | | clean copy of the xapian index with fixtures loaded is required.
* 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
|
* Handle the case where the blog feed already has querystring parameters.Louise Crow2012-10-11-0/+16
|
* Update references to point to the mySociety repo.Louise Crow2012-08-29-7/+7
|
* Ensure we generate URLs containing the current locale, even if the locale ↵Seb Bacon2012-06-14-1/+18
| | | | | | contains an underscore. Includes a bonus test to ensure that locales with unknown territories fall back to known languages.
* Use PO files from `spec/fixtures/locale` rather than the application - makes ↵Seb Bacon2012-06-14-0/+5
| | | | it less brittle.
* Always show tracking links even for things that currently have no resultsSeb Bacon2012-05-11-0/+5
|
* Don’t index unconfirmed usersRobin Houston2012-02-06-0/+16
| | | | Closes #415.
* Load all fixtures for all testsRobin Houston2012-01-31-13/+0
| | | | | | | | | The ad hoc specification of fixtures has been an ongoing source of bugs in the tests. The straw that broke the camel’s back is that 7c6eb09 requires the fixtures to be loaded in order (i.e. children before their parents), and it would have been a painful process to reorder all the dozens of different fixture lists, but the test system ought to be more reliable this way.
* Test duplicate requests are coalescedRobin Houston2012-01-29-0/+5
| | | | Test that duplicate requests are coalesced on the front page.
* Change order of requests on front pageRobin Houston2012-01-29-1/+6
| | | | | | | Put the most-recently-answered first, rather than the most-recently-reclassified. This is inspired by issue #370, though it does not fix all the issues raised in that ticket. (The wording is still outright misleading in some cases.)
* More test data, and fix testsRobin Houston2012-01-29-6/+16
| | | | | | Add some more test data, and fix the tests to accommodate both this new test data and the fact that raw emails are now correctly loaded into the test environment.
* Load all raw emails for testingRobin Houston2012-01-29-3/+3
| | | | | | | | | | | Previously there was just one raw email, and the test code relied on that fact. Generalise it to handle multiple raw emails. This change causes a number of tests to fail, because it exposes failures that should have happened when the second raw email was added but were masked by the fact that the text of this second raw email was never loaded. These failures will be fixed in the next commit.
* More test data and a new testRobin Houston2012-01-27-2/+4
| | | | | This is a test for what I thought issue #370 might be. However this test is passing, so it isn’t that.
* Add a trivial test for the list of recent requestsRobin Houston2012-01-27-0/+12
|
* Improve test isolationRobin Houston2012-01-24-0/+2
| | | | | Towards #361. Fixes: spec/models/xapian_spec.rb spec/controllers/general_controller_spec.rb
* Fix a bunch of order-dependent failuresRobin Houston2012-01-24-1/+1
| | | | This fixes a lot of the failures in #361.
* Remove assumptionRobin Houston2012-01-20-6/+5
| | | | | | | I feel uncomfortable about the assumption that FastGettext.default_available_locales is always equal to I18n.available_locales. Even if that’s true now, it might not always be -- and if a bug causes them to become erroneously different, the previous code would have masked that.
* Make sure the FakeWeb registry is kept cleanRobin Houston2012-01-20-0/+7
| | | | | | | To be fair this change does not seem to make any tests pass that didn’t pass before, but (based on my limited understanding) it seems like a good idea. It also conforms to how FakeWeb is used in application_controller_spec.rb.