Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Rework the temporary storing of a csv file of public body info between a dry ↵ | Louise Crow | 2012-08-20 | -18/+98 |
| | | | | run and real load. Use a temp file, not the session for the data. Also fixes #503. | |||
* | Add warning about regex censor rules. | Louise Crow | 2012-08-14 | -1/+1 |
| | | | | Fix spec to accommodate validation of text presence. | |||
* | Validate presence of text to replace in censor rules. | Louise Crow | 2012-08-14 | -0/+6 |
| | ||||
* | Add basic validation for regexp censor rules that a valid regexp can be ↵ | Louise Crow | 2012-08-14 | -1/+35 |
| | | | | created with the text of the rule. | |||
* | Make global validation work correctly for regex censor rules. | Louise Crow | 2012-08-14 | -2/+9 |
| | ||||
* | Apply global and public body censor rules to binary files. | Louise Crow | 2012-08-14 | -23/+60 |
| | ||||
* | Add case of a regex censor rule to test of binary masking. | Louise Crow | 2012-08-14 | -13/+20 |
| | ||||
* | Handle regexp rules when running censor rules on binary files. | Louise Crow | 2012-08-14 | -6/+18 |
| | ||||
* | Add association between public bodies and censor rules, have info_request ↵ | Louise Crow | 2012-08-14 | -0/+8 |
| | | | | apply rules from its public body. | |||
* | Add concept of global censor rules as orthogonal to regex censor rules. ↵ | Louise Crow | 2012-08-14 | -144/+235 |
| | | | | Apply global rules to every request, not regex rules. | |||
* | Add before and after methods to isolate any effect these tests have on locale. | Louise Crow | 2012-08-13 | -2/+12 |
| | ||||
* | Merge branch 'feature/public-body-api-2a' into develop | Robin Houston | 2012-07-16 | -1/+10 |
|\ | ||||
| * | Beef up mocks | Robin Houston | 2012-07-16 | -1/+10 |
| | | | | | | | | | | Add the user_name and is_external? properties to the InfoRequest mock objects used for view testing, where necessary. | |||
* | | Misc fixes to get tests to pass - notably a MonkeyPatch to work around ↵ | Seb Bacon | 2012-07-13 | -0/+14 |
| | | | | | | | | problem testing template results | |||
* | | Support regular expressions in CensorRules; also support 'global' ↵ | Seb Bacon | 2012-07-13 | -2/+61 |
|/ | | | | CensorRules that aren't attached to a User or Request or Public Body (but don't expose this in the admin UI). Fixes #33 | |||
* | JSON new events feed | Robin Houston | 2012-07-04 | -1/+39 |
| | ||||
* | Atom feed of request events | Robin Houston | 2012-07-04 | -0/+16 |
| | | | | | | | | | | We need not only new requests, but new outgoing correspondence of any sort. The idea is that this feed will contain any event that would have triggered an email to be sent to the public body, so can be used as an alternative, equivalent way to stay up-to-date with happenings on WDTK (or the Alaveteli installation of choice). | |||
* | Don't allow non-superusers to access admin interface (eek!) Fixes #515 | Seb Bacon | 2012-06-27 | -1/+9 |
| | ||||
* | Merge branch 'develop' of github.com:sebbacon/alaveteli into develop | Robin Houston | 2012-06-23 | -26/+13149 |
|\ | | | | | | | | | | | | | | | Conflicts: Gemfile.lock script/handle-mail-replies script/handle-mail-replies.rb spec/controllers/request_controller_spec.rb | |||
| * | Only invalidate a user's cached requests when their *name* changes. ↵ | Seb Bacon | 2012-06-20 | -1/+7 |
| | | | | | | | | Otherwise we issue PURGEs every time, for example, we send out email alerts. | |||
| * | Fix test to match current wording | Seb Bacon | 2012-06-19 | -1/+1 |
| | | ||||
| * | Correctly name the person who has requested something for admin attention | Seb Bacon | 2012-06-19 | -0/+1 |
| | | ||||
| * | Revert "Set a "Content-Disposition: attachment" header when downloading ↵ | Seb Bacon | 2012-06-15 | -9/+1 |
| | | | | | | | | | | | | | | | | files. Closes #428" This reverts commit 506af7a640f63b17000ccfc5e1344bbc3039c913. See issue #428 for a discussion | |||
| * | Ensure we generate URLs containing the current locale, even if the locale ↵ | Seb Bacon | 2012-06-14 | -1/+4277 |
| | | | | | | | | | | | | 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 Bacon | 2012-06-14 | -0/+8821 |
| | | | | | | | | it less brittle. | |||
| * | Fix bug that was preventing Rails from loading when *not* in test mode ↵ | Seb Bacon | 2012-06-14 | -0/+6 |
| | | | | | | | | (exposed by the bug fixed in 7c846594 that was introduced in 775e122c) | |||
| * | Fix test breakage (really we should use a test app.po rather than the real ↵ | Seb Bacon | 2012-06-14 | -1/+1 |
| | | | | | | | | one for testing!) | |||
| * | Use POST rather than GET for tests (this matches better what the UI does) | Seb Bacon | 2012-06-12 | -4/+4 |
| | | ||||
| * | Ensure attention-requesting tests pass by faking a logged in user | Seb Bacon | 2012-06-12 | -0/+5 |
| | | ||||
| * | Make emails requesting admin attention appear to come from the user who ↵ | Seb Bacon | 2012-06-12 | -0/+10 |
| | | | | | | | | requested the attention, not the user who made the original request. | |||
* | | API controller tests all pass | Robin Houston | 2012-06-18 | -3/+15 |
| | | ||||
* | | Fix test for updated test data | Robin Houston | 2012-06-07 | -1/+1 |
| | | | | | | | | | | Replace another constant with a computed value, so we can accommodate new test data without spuriously failing. | |||
* | | Beef up mocks with new method | Robin Houston | 2012-06-07 | -1/+3 |
| | | ||||
* | | Make test more flexible | Robin Houston | 2012-06-07 | -3/+3 |
| | | | | | | | | | | | | | | Remove another dependency of a test on the aggregate set of test data, part of a long-term programme to make it easier to add test data (and required here because recently-added test data was causing these tests to fail!) | |||
* | | Need another stub now | Robin Houston | 2012-06-07 | -0/+1 |
| | | ||||
* | | Update tests to reflect code changes | Robin Houston | 2012-06-07 | -0/+23 |
| | | | | | | | | | | Since users must now be logged-in to report requests, the tests need to reflect this. | |||
* | | Update test for e3cfe550fe835c68b25d693abd40aedd954913b3 | Robin Houston | 2012-06-07 | -1/+1 |
| | | | | | | | | | | The wording that was changed by e3cfe550fe835c68b25d693abd40aedd954913b3 turns out to be relied on by one of the tests. | |||
* | | WIP attachment uploading via API | Robin Houston | 2012-06-06 | -4/+20 |
| | | ||||
* | | Test for attaching a file via the API | Robin Houston | 2012-06-06 | -0/+36 |
| | | | | | | | | Not yet implemented, so the test fails. | |||
* | | Correct comment\ | Robin Houston | 2012-06-06 | -1/+1 |
| | | ||||
* | | Use a fixture for the external request | Robin Houston | 2012-06-06 | -4/+39 |
| | | ||||
* | | Fix typo in fixture yaml | Robin Houston | 2012-06-06 | -2/+2 |
| | | | | | | | | | | I’m sure created_at should not have been there twice. Presumably the second one was supposed to be updated_at. | |||
* | | API: Some testing of show_request | Robin Houston | 2012-06-06 | -0/+13 |
| | | ||||
* | | API: test also for refusal conditions | Robin Houston | 2012-06-06 | -0/+39 |
| | | | | | | | | | | | | The API must not allow people to update requests that they shouldn’t, i.e. only requests that were created by the same public body, using the API, can be added to using the API. | |||
* | | API: add followup to request | Robin Houston | 2012-06-06 | -13/+61 |
| | | | | | | | | You can add a followup to a request using the API. | |||
* | | API: we can add a response to a request | Robin Houston | 2012-06-06 | -0/+60 |
| | | ||||
* | | API create_request method is working | Robin Houston | 2012-06-06 | -3/+4 |
| | | | | | | | | | | | | "when using the API", it "should create a new request from a POST", AND IT DOES! | |||
* | | Fix test and fixtures so they only fail due to missing business logic ↵ | Seb Bacon | 2012-06-06 | -4/+11 |
| | | | | | | | | (corrected typo, added missing fixture data, etc) | |||
* | | Test for creating a new request via the API | Robin Houston | 2012-06-06 | -0/+33 |
|/ | | | | This is not yet implemented. Test first! | |||
* | File.dirname(__FILE__) is not necessarily absolute | Robin Houston | 2012-06-03 | -1/+1 |
| | | | | | | | It’s weird that there’s so much code here that implicitly assumes File.dirname(__FILE__) is an absolute path, because really in general it very much is not! This assumption was invalidated by the recent bundler-awareness changes. |