Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | [Zurich] testing - create superuser earlier | Hakim Cassimally | 2013-12-10 | -2/+7 |
| | | | | So that tests pass on clean dataset such as travis... | |||
* | [Zurich] also allow hidden reports for closed_overdue | Hakim Cassimally | 2013-12-10 | -2/+2 |
| | ||||
* | [Zurich] fix moderated statistics to include hidden | Hakim Cassimally | 2013-12-10 | -53/+127 |
| | | | | | | As per comments after testing mysociety/FixMyStreet-Commercial#459, the business logic of the actual state changes is fine, but the stats do also then need to include hidden records. | |||
* | [Zurich] log admin changes | Hakim Cassimally | 2013-12-09 | -0/+10 |
| | | | | | To allow us to infer additional stats changes requirements such as in mysociety/FixMyStreet-Commercial#459 better in future. | |||
* | [Zurich] move mech test into override_config stanza | Hakim Cassimally | 2013-12-09 | -2/+2 |
| | | | | | This was causing previous test failures when config != Zurich, due to expected logic not getting triggered. | |||
* | [Zurich] test factor out cleanup() routine | Hakim Cassimally | 2013-12-09 | -4/+9 |
| | ||||
* | [Zurich] test /admin/stats | Hakim Cassimally | 2013-12-09 | -0/+39 |
| | | | | | | | First test that this page is only available to superuser. Then test that the moderated and fixed stats are doing something that is (hopefully) the spec in mysociety/FixMyStreet-Commercial#459 | |||
* | [Zurich] test moderated_overdue | Hakim Cassimally | 2013-12-09 | -0/+43 |
| | | | | | | - is set to 0 on state change within time-limit - is set to 1 on state change after time-limit - only marks overdue on first time | |||
* | Don't display categories for deleted bodies | Chris Mytton | 2013-11-11 | -0/+18 |
| | | | | | | | | | | | | When a body has been marked as deleted (currently only exposed in the Zurich cobrand's admin) the categories for that body should not be available on the report_new form. Prior to this change there was an error running the 'zurich-overdue-alert' script, because it wasn't expecting the 'bodies_str' field to have any entries in the format 'x|y,z', this prevents this problem from happening in the future (though it doesn't fix any existing entries with this issue). | |||
* | If only one cobrand given, always use it. | Matthew Somerville | 2013-11-07 | -32/+73 |
| | | | | | | | | | If the ALLOWED_COBRANDS configuration variable only contains one entry (and also work if it's a string rather than a list), always use that cobrand, no matter what the hostname is. The example Vagrantfile no longer needs the configuration changes at all. | |||
* | Add more admin external link icons. | Matthew Somerville | 2013-11-07 | -6/+6 |
| | | | | Use a relative URL in the CSS in case it's proxied. | |||
* | Merge branch 'multiple_emails_per_contact' | Struan Donald | 2013-11-04 | -0/+46 |
|\ | ||||
| * | Tests for contact with comma seperated multiple emails | Struan Donald | 2013-11-04 | -0/+46 |
| | | ||||
* | | Merge branch '389-report-search-ref' of ↵ | Matthew Somerville | 2013-11-04 | -0/+7 |
|\ \ | |/ |/| | | | https://github.com/andylolz/fixmystreet into andylolz-389-report-search-ref | |||
| * | One more external id-related test | Andy Lulham | 2013-10-26 | -0/+3 |
| | | ||||
| * | Add a test for the external_id admin report search | Andy Lulham | 2013-10-26 | -0/+4 |
| | | ||||
* | | Set confirmed timestamp on any admin unconfirmed state change. | Matthew Somerville | 2013-10-25 | -1/+10 |
| | | | | | | | | This fixes #527. | |||
* | | More accurate config overriding for Zurich. | Matthew Somerville | 2013-10-24 | -2/+6 |
| | | ||||
* | | Fix test suite to run regardless of config setup. | Matthew Somerville | 2013-10-24 | -607/+904 |
| | | ||||
* | | [Zurich] Remove phone number requirement for mobile app reports | Chris Mytton | 2013-10-21 | -0/+23 |
|/ | | | | | | | | | | | | | | | | | The phone number was made mandatory in #541. To make this change compatible with older versions of the mobile app I've had to drop server side validation of the phone number for requests that come in from mobile apps. To determine if it's a mobile request I'm checking the 'service' parameter in the request, which is only set in requests from the mobile apps [1]. The phone validation will still happen client side for the newer versions of the app, and the web app will continue to work with the server side validations. [1] https://github.com/mysociety/zurich_mobile/blob/1fec2bc21dad9326819105cdbf5d47183e7503ba/www/js/models.js#L75 | |||
* | Reported via mobile device rather than by | Struan Donald | 2013-10-02 | -4/+4 |
| | | | | Fixes #586 | |||
* | Fix Bromley use of 'no further action'. | Matthew Somerville | 2013-09-25 | -0/+45 |
| | | | | | If SET isn't given, the line means [% foo = ('bar' IF baz) %] rather than [% ([SET] foo = 'bar') IF baz %]. | |||
* | [Zurich] Default to not sending rejected emails | Chris Mytton | 2013-09-20 | -1/+1 |
| | | | | Closes mysociety/FixMyStreet-Commercial#410 | |||
* | Have a central config-override function. | Matthew Somerville | 2013-09-13 | -60/+20 |
| | | | | You only need to pass in the values you wish to be different. | |||
* | switch to Sub::Override from Test::MockModule | Struan Donald | 2013-09-12 | -6/+7 |
| | | | | | This is for consistency as Sub::Override is used for another test and also so we minimise the number of modules installed just for testing | |||
* | [Zurich] Use subtest blocks to seperate test cases | Chris Mytton | 2013-09-12 | -39/+44 |
| | ||||
* | [Zurich] Put the phone number test into a subtest block | Chris Mytton | 2013-09-12 | -29/+30 |
| | ||||
* | [Zurich] Make sure mapit url is set properly | Chris Mytton | 2013-09-12 | -0/+1 |
| | ||||
* | [Zurich] Try to override config in tests | Chris Mytton | 2013-09-12 | -1/+25 |
| | ||||
* | explicitly set bodies_str for abuse and app store tests | Struan Donald | 2013-09-12 | -0/+2 |
| | | | | | | | | The first email count test in the abuse subtest was failing on Travis but not on my machine because the sent email alert test was running on my machine, as it has the mock module installed, and this was resetting the bodies_str to one with a confirmed contact. Setting bodies_str on both these tests stops this sort of thing happening again | |||
* | mechanism to stop app store tests being sent | Struan Donald | 2013-09-12 | -0/+29 |
| | | | | | | In order to allow the iOS app to be tested in the app store we need to allow the user to create a report but we don't want to send it. To allow this hide reports with a specific title during send_reports | |||
* | Test to check reports from abuse users not sent | Struan Donald | 2013-09-12 | -0/+47 |
| | | | | | | Reports that have been created by a user who is logged in but whose email is in the abuse table should be hidden and not sent by send_reports. | |||
* | Merge branch 'zurich-optional-problem-rejected-email' | Chris Mytton | 2013-09-12 | -0/+16 |
|\ | | | | | | | | | Conflicts: t/cobrand/zurich.t | |||
| * | [Zurich] Test rejected email are only sent when requested | Chris Mytton | 2013-09-11 | -0/+16 |
| | | ||||
* | | Merge branch 'zurich-407-generate-note-when-sdm-assigned' | Chris Mytton | 2013-09-12 | -1/+43 |
|\ \ | ||||
| * | | [Zurich] create internal note when the category is changed | Edmund von der Burg | 2013-09-11 | -3/+7 |
| | | | ||||
| * | | [Zurich] Add tests for changing categories in the admin interface | Edmund von der Burg | 2013-09-11 | -0/+38 |
| | | | ||||
* | | | Merge branch 'zurich-mark-external-addresses-deleted' | Chris Mytton | 2013-09-12 | -0/+14 |
|\ \ \ | | | | | | | | | | | | | | | | | Conflicts: t/cobrand/zurich.t | |||
| * | | | [Zurich] Test deleted bodies can't have problems assigned | Chris Mytton | 2013-09-11 | -0/+14 |
| | |/ | |/| | ||||
* | | | Merge branch 'master' into zurich-add-internal-notes | Chris Mytton | 2013-09-12 | -2/+9 |
|\ \ \ | |_|/ |/| | | ||||
| * | | Merge branch 'rabx-column-refactor' | Chris Mytton | 2013-09-12 | -1/+68 |
| |\ \ | ||||
| * \ \ | Merge branch 'zurich-mandatory-telephone-number' | Chris Mytton | 2013-09-12 | -2/+9 |
| |\ \ \ | | |_|/ | |/| | | ||||
| | * | | Merge branch 'master' into zurich-mandatory-telephone-number | Chris Mytton | 2013-09-05 | -14/+37 |
| | |\ \ | ||||
| | * | | | [Zurich] Add test for missing phone number | Chris Mytton | 2013-09-05 | -0/+7 |
| | | | | | ||||
| | * | | | [Zurich] Fix spacing in tests | Chris Mytton | 2013-09-05 | -2/+2 |
| | | | | | ||||
* | | | | | [Zurich] Save internal notes as hidden comments, rather than in the extra field | Edmund von der Burg | 2013-09-10 | -3/+5 |
| | | | | | ||||
* | | | | | Handle user already existing (as it may in dev) | Edmund von der Burg | 2013-09-10 | -0/+1 |
| | | | | | ||||
* | | | | | Don't use a domain name when testing that the default ALLOWED_COBRANDS will ↵ | Edmund von der Burg | 2013-09-10 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | match to the wrong cobrand. | |||
* | | | | | Override the config so that Zurich tests can be run with normal dev config | Edmund von der Burg | 2013-09-10 | -6/+38 |
| | | | | | ||||
* | | | | | [Zurich] remove debugging from test | Edmund von der Burg | 2013-09-10 | -3/+0 |
| | | | | |