aboutsummaryrefslogtreecommitdiffstats
path: root/spec
Commit message (Collapse)AuthorAgeLines
...
| * | Handle a request with no user in the show request function0.16.0.5hotfix/0.16.0.5Louise Crow2014-01-28-0/+12
| | |
* | | Merge branch 'feature/1176-automate-authority-contacts' into rails-3-developLouise Crow2014-01-28-166/+578
|\ \ \ | | | | | | | | | | | | | | | | Conflicts: doc/CHANGES.md
| * | | Add admin urls to change request emailsLouise Crow2014-01-13-0/+4
| | | |
| * | | Add form for closing change request without actionLouise Crow2014-01-13-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | For cases where we don't want to make the change suggested. There doesn't seem to be any obvious default text to use in the response to the person who requested the change.
| * | | Populate admin public body forms based on a change request.Louise Crow2014-01-13-0/+147
| | | | | | | | | | | | | | | | | | | | Also add editable text for an email to be sent to the person requesting the change.
| * | | Reorganise specsLouise Crow2014-01-13-166/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to make it clearer where to add new specs, group specs by controller and action, merging specs relating to internationalisation with other specs as there seem to be no specific prerequisites in terms of set up for them, and removing some duplicate specs.
| * | | Form and models for asking for a public body change.Louise Crow2014-01-13-0/+205
| | | |
* | | | Merge branch 'feature/cache-request-lists' into rails-3-developLouise Crow2014-01-28-135/+96
|\ \ \ \
| * | | | Cache frequently accessed request lists.Louise Crow2013-12-19-135/+96
| |/ / / | | | | | | | | | | | | Make specs a bit more focused, remove view specs - they're not relevant to the new code in their current form and don't seem to merit updating.
* | | | Merge branch 'feature/1286-explain-anonymous-requests' into rails-3-developLouise Crow2014-01-28-0/+39
|\ \ \ \
| * | | | Link 'anonymous' users to help section with explanation.Louise Crow2014-01-14-0/+39
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Requests submitted using the write API may have an anonymous user. Add a section to the FAQ explaining this, and link to it from those requests. Note this commit removes some 'h' calls that are no longer needed in Rails 3. Contributes to https://github.com/mysociety/alaveteli/issues/1286
* | | | Merge branch 'feature/1269-improve-quoting-recognition' into rails-3-developLouise Crow2014-01-27-34/+112
|\ \ \ \
| * | | | Handle gmail-style quoting from the HTML part of a mail.Louise Crow2014-01-22-0/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already handled this type of quoting when coming directly from a text part of a mail, but when we choose the HTML part as the main part and then convert it to text, the 'On [date] <email> person said:' part can get split over two lines.
| * | | | Tighten up expectation, make expected files consistent.Louise Crow2014-01-17-34/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just expecting the parsed file to include the expected one would mean success in the case where nothing has been folded. Tighten up the expectation, and add quoting placeholders to expected files that didn't have them.
* | | | | Merge branch 'feature/fix-profile-photo-errors' into rails-3-developLouise Crow2014-01-27-7/+15
|\ \ \ \ \
| * | | | | Make custom error messages translated.Louise Crow2014-01-08-7/+15
| | |/ / / | |/| | |
* | | | | Merge branch 'trap-abnormal-exits' into rails-3-developLouise Crow2014-01-23-0/+30
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Rename tests to be clear as to what is being testedIan Chard2014-01-23-1/+1
| | | | |
| * | | | Tests for abnormal exit detectionIan Chard2014-01-14-0/+30
| | | | |
| * | | | Session keys are stored as strings in Rails 3.0.15.0.1hotfix/0.15.0.1Louise Crow2013-11-19-28/+28
| | | | | | | | | | | | | | | | | | | | Update our session-stripping code.
| * | | | Insert cookie stripping at correct point in middleware stack.Louise Crow2013-11-19-0/+12
| | | | |
| * | | | Use upstream globalize.Mark Longair2013-11-19-31/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're trying to switch to an upstream version of globalize, which now has Henare's change to allowed blank values without them being converted to nil - however, there is another intermediate change that was introduced which means that the name is only written back to the original table if the locale matches the default locale. This is described in more depth here: globalize/globalize#183 This change broke the make_single_language_example helper, but this commit fixes that by also setting the default locale before creating the example public bodies.
* | | | | Merge branch 'rails-3-develop' of ↵Louise Crow2014-01-14-0/+36
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | ssh://git.mysociety.org/data/git/public/alaveteli into rails-3-develop
| * | | | Make test fail as with issue #1001Louise Crow2014-01-14-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | This requires us to clear the I18n.fallbacks when changing the I18n.default_locale, otherwise the old default locale remains there.
| * | | | Add further tests for creation of the internal admin bodyMark Longair2014-01-09-0/+31
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | The internal admin body should be created automatically if it doesn't exist on calls to PublicBody.internal_admin_body; we've seen errors (#1001) where this fails after the default locale changes. (Although these tests don't actually replicate that problem.)
* | | | Merge branch 'feature/cleanup-popups' into release/0.16Louise Crow2014-01-14-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile.lock app/views/layouts/default.html.erb config/application.rb public/admin/stylesheets/admin.css
| * | | Cleanup popup notices.Louise Crow2014-01-14-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move HTML to view from controller, use same elements for other country popup and everypage - partly so they don't display on top of each other anymore. Don't position them over existing content, position them at the top of the page. Use consistent styling, and keep the javascript unobtrusive.
| * | | Treat any document/pdf attachment as if it were application/pdfMark Longair2013-12-05-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a fix for issue #1232. Richard Taylor pointed out that some PDF attachments had the non-standard content-type document/pdf, and that these weren't being treated as PDFs. (Ganesh Sittampalam discovered that all of these PDFs were generated by a Lexmark X945e, according to the PDF metadata.) This commit adds an extra case to normalise_content_type to map document/pdf to application/pdf. In fact, since the upgrade of the Mail gem in ccebe3c3d6d4dc5f81 the behaviour when handling the non-standard content-type document/pdf was much better, but this commit also means that you get the right icon for the attachment, and can be cherry-picked onto older versions to fix #1232.
| * | | Merge branch 'feature/restore-caching' into wdtkLouise Crow2013-11-26-75/+95
| |\ \ \
| * \ \ \ Merge branch 'release/0.15' into wdtkMark Longair2013-11-15-102/+423
| |\ \ \ \ | | | |/ / | | |/| | | | | | | | | | | | Conflicts: lib/tasks/stats.rake
| * | | | Reduce the memory used to serve /body/all-authorities.csvMark Longair2013-11-14-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On WDTK, /body/all-authorities was using lots of memory - this commit reduces that by (a) fetching the public bodies in batches, rather than keeping them all in memory at one time and (b) writing the CSV to a file and then returning it with X-Sendfile (or equivalent), rather than returning the whole file from memory with send_data. There's a FIXME to do with the layout of download directories; if that's changed, the example nginx config, etc. will need to be updated too. This commit also adds a basic test for reasonable CSV being returned and switches from FasterCSV to CSV in order to fix this NotImplementedError under Ruby 1.9: Please switch to Ruby 1.9's standard CSV library. It's FasterCSV plus support for Ruby 1.9's m17n encoding engine. (The CSV version seems to still work fine under 1.8.7.)
| * | | | Insert cookie stripping at correct point in middleware stack.Louise Crow2013-11-13-0/+12
| | | | |
| * | | | Session keys are stored as strings in Rails 3.Louise Crow2013-11-13-28/+28
| | | | | | | | | | | | | | | | | | | | Update our session-stripping code.
| * | | | Merge branch 'hotfix/0.14.0.2' into wdtkLouise Crow2013-10-03-0/+40
| |\ \ \ \
| * \ \ \ \ Merge branch 'hotfix/0.14.0.1' into wdtkLouise Crow2013-09-30-0/+10
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'release/0.14' into wdtkLouise Crow2013-09-24-440/+2398
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: app/views/request/show.html.erb
| * \ \ \ \ \ \ Merge branch 'hotfix/0.13.0.3' into wdtkLouise Crow2013-09-18-0/+37
| |\ \ \ \ \ \ \
| * \ \ \ \ \ \ \ Merge branch 'release/0.13' into wdtkLouise Crow2013-08-20-173/+506
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/he_IL/app.po locale/hr_HR/app.po locale/nb_NO/app.po locale/uk/app.po
| * \ \ \ \ \ \ \ \ Merge branch 'release/0.12' into wdtkLouise Crow2013-06-26-6/+0
| |\ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ Merge branch 'release/0.12' into wdtkLouise Crow2013-06-26-261/+524
| |\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/he_IL/app.po locale/hr_HR/app.po locale/nb_NO/app.po locale/uk/app.po spec/models/info_request_spec.rb
| * | | | | | | | | | | Respond to a (currently unsupported) json request for a public body list ↵Louise Crow2013-06-20-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with a 406, not a 500 caused by a missing template.
| * | | | | | | | | | | Merge branch 'hotfix/0.11.0.14' into wdtkLouise Crow2013-06-19-0/+10
| |\ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.13' into wdtkLouise Crow2013-06-17-4/+29
| |\ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.9' into wdtkLouise Crow2013-06-11-3/+3
| |\ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'wdtk' of ssh://git.mysociety.org/data/git/public/alaveteli ↵Louise Crow2013-06-10-0/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into wdtk Conflicts: spec/models/info_request_spec.rb
| | * | | | | | | | | | | | | | Cope with replying to a message with a missing or empty Subjecthotfix/0.11.0.5Mark Longair2013-06-06-0/+12
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Merge branch 'hotfix/0.11.0.7' into wdtkLouise Crow2013-06-10-0/+39
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: lib/tasks/temp.rake spec/models/info_request_spec.rb
| * | | | | | | | | | | | | | | Merge branch 'hotfix/0.11.0.4' into wdtkLouise Crow2013-06-05-0/+12
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.1' into wdtkLouise Crow2013-06-03-7/+88
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'release/0.11' into wdtkLouise Crow2013-06-03-856/+1684
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \