aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* typoRobin Houston2012-02-01-1/+1
|
* Revert small unintended changeRobin Houston2012-02-01-1/+1
| | | | | | Revert a small part of 33f8ef66084bbfb61bf0b4e0f53e1da5e7cc84dc, which was not intended, so that the behaviour is unchanged if a xapian database doesn’t exist, say.
* Close xapian db before opening it againRobin Houston2012-01-31-8/+14
| | | | | | This *ought* to fix the problem with the alert-tracks daemon opening more and more copies of the xapian db till it exhausts the available file descriptors.
* Update testRobin Houston2012-01-31-1/+1
| | | | Update test to take account of change 76fd5f9.
* Suppress stderr output from rake db:test:prepareRobin Houston2012-01-31-1/+1
|
* Keep message text if user is rate-limitedRobin Houston2012-01-31-5/+32
| | | | | | | If a user cannot make new requests because they are rate-limited, and they compose a request whilst logged out, include the text of the request in the message that explains about the rate limit so it is not lost.
* Use db:test:prepare to prepare the way for testsRobin Houston2012-01-31-2/+1
|
* Change (Banned) to (Account suspended)Robin Houston2012-01-31-15/+15
| | | | | | | | When a user account has been suspended, we write (Banned) after their user name. But sometimes we need to suspend accounts temporarily, such as pending the result of a discussion, and it is misleading to write (Banned) in such cases; so let’s change it to (Account suspended) instead.
* Load all fixtures for all testsRobin Houston2012-01-31-145/+1
| | | | | | | | | 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.
* Avoid need to be db superuser to run testsRobin Houston2012-01-31-0/+23
| | | | | Add a patch from louisecrow (https://github.com/mysociety/fixmytransport/blob/master/lib/patches/fixtures_constraint_disabling.rb) that makes it possible to run the tests without being a database superuser.
* fix shell syntax in scriptRobin Houston2012-01-31-2/+2
|
* Merge branch 'wdtk' into release/0.5Robin Houston2012-01-31-4/+4
|\ | | | | | | | | | | Conflicts: config/general.yml-example doc/CHANGES.md
| * Add new variable to general.yml-exampleRobin Houston2012-01-31-0/+3
| | | | | | | | | | | | Add MAX_REQUESTS_PER_USER_PER_DAY to general.yml-example, because the mySociety deployment system insists that the example file specify precisely the same variables as the deployed one.
| * Correct copy-pastoRobin Houston2012-01-31-2/+2
| | | | | | | | | | I am not sure why the tests anyway passed with this mistake, but clearly those trailing .should's ought not to have been there.
| * Add MAX_REQUESTS_PER_USER_PER_DAY to release notesRobin Houston2012-01-31-0/+1
| | | | | | | | Document MAX_REQUESTS_PER_USER_PER_DAY in the 0.5 release notes
* | Document additions to base Alaveteli theme made at ↵Seb Bacon2012-01-30-9/+44
| | | | | | | | 4899ae188f728bfd77b6db950fd5a1e3180c7be8
* | Fix test failure introduced in commit 24bbaa5afac5ce27c351e3b460be1b0182446ba1Seb Bacon2012-01-30-1/+7
| |
* | Make better wording at the top of the page listing subsets of public bodies. ↵Seb Bacon2012-01-30-9/+23
| | | | | | | | At the same time, cause the tests not to rely on WDTK data. Fixes #396
* | Add some tag fixtures, so that the default Alaveteli theme's public body ↵Seb Bacon2012-01-30-0/+36
| | | | | | | | categories have something to work with. Currently not used in any tests.
* | Prompt users to reindex xapian after loading sample dataSeb Bacon2012-01-30-1/+1
| |
* | Mention new rate-limiting feature in CHANGES.md and the example configSeb Bacon2012-01-30-0/+4
|/
* Rate limitingRobin Houston2012-01-29-8/+105
| | | | | | Add the capability to specify a limit to the number of requests a user can make per day, which can be turned off for specific users in the admin interface.
* Merge branch 'release/0.5' into wdtkRobin Houston2012-01-29-127/+410
|\
| * Test duplicate requests are coalescedRobin Houston2012-01-29-2/+185
| | | | | | | | Test that duplicate requests are coalesced on the front page.
| * Fail fasterRobin Houston2012-01-29-0/+3
| | | | | | | | | | | | Better error reporting when an incoming message has no raw_email. This is useful if you make a mistake when creating new test data, to pick an example application completely at random (not).
| * Change order of requests on front pageRobin Houston2012-01-29-12/+18
| | | | | | | | | | | | | | 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.)
| * Irrelevant changeRobin Houston2012-01-29-3/+1
| | | | | | | | | | So, well, I found this change when I ran git diff; and the truth is that I just prefer it aesthetically. Is that so sinful?
| * More test data, and fix testsRobin Houston2012-01-29-41/+108
| | | | | | | | | | | | 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-64/+81
| | | | | | | | | | | | | | | | | | | | | | 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 flexible input syntaxRobin Houston2012-01-29-1/+2
| | | | | | | | | | Accept test pairs on stdin even if they are not prefixed with the string "* FAILED: ".
| * Raise exception if $RAILS_ENV is unsetRobin Houston2012-01-29-1/+5
| | | | | | | | | | If $RAILS_ENV is unset, raise an exception rather than just blithely creating a directory called cache/attachments_.
| * Merge branch 'release/0.5' of github.com:sebbacon/alaveteli into release/0.5Seb Bacon2012-01-27-4/+8
| |\
| | * Merge branch 'wdtk' into release/0.5Robin Houston2012-01-27-166/+316
| | |\
| | * | More background on wkhtmltopdfSeb Bacon2012-01-26-4/+8
| | | |
* | | | Fix indentationRobin Houston2012-01-27-18/+17
| | | |
* | | | Small fixesRobin Houston2012-01-27-9/+9
|/ / / | | | | | | | | | | | | - There is no such thing as “cursor: hand;” - Opacity is not measured in pixels.
* | | clearing the test censor rules in an ensure blockRobin Houston2012-01-27-7/+9
| | | | | | | | | | | | | | | | | | Might as well be consistent about clearing the test censor rules in an ensure block. This would only mean that an exception in one test is less likely to cause unrelated failures in another.
* | | More test data and a new testRobin Houston2012-01-27-12/+122
| | | | | | | | | | | | | | | This is a test for what I thought issue #370 might be. However this test is passing, so it isn’t that.
* | | Remove stray puts in test codeRobin Houston2012-01-27-1/+0
| |/ |/| | | | | | | This was added for temporary debugging purposes, and ought to have been removed.
* | Merge branch 'wdtk' of github.com:sebbacon/alaveteli into wdtkRobin Houston2012-01-27-17/+35
|\ \
| * | Merge branch 'wdtk' of github.com:sebbacon/alaveteli into wdtkSeb Bacon2012-01-26-118/+140
| |\|
| * | Mainly accessibility related changes. Closes issue #367Seb Bacon2012-01-26-17/+35
| | |
* | | Add a trivial test for the list of recent requestsRobin Houston2012-01-27-10/+73
| | |
* | | Refactor test code so new test data can be addedRobin Houston2012-01-26-107/+176
| | | | | | | | | | | | | | | | | | | | | | | | Previously many of the tests made assumptions about the global structure of the test data set: the total number of requests, for example, or the names of all public bodies. This makes it difficult to add to the test data. This change is intended to make the test data easier to extend by eliminating such global assumptions.
* | | No tabs for indentationRobin Houston2012-01-26-1/+1
| | |
* | | indentationRobin Houston2012-01-26-31/+31
| |/ |/|
* | Fix #364 properly: third time lucky!Robin Houston2012-01-26-1/+1
| |
* | Fix #364 properly (?)Robin Houston2012-01-26-1/+1
| |
* | Fix #364Robin Houston2012-01-26-1/+1
| |
* | Clean up the indentationRobin Houston2012-01-26-95/+95
| |