Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Add Extra role to ease use of {extra} field. | Hakim Cassimally | 2015-03-20 | -3/+6 |
| | | | | | | | | | Historically, the extra field has been used in two different ways by different cobrands, both as a list (e.g. Open311 category fields) and a hash (e.g. the Zurich cobrand). This commit consolidates usage, adding an API to make use of the field easier and always returning a hash for the code to use. Fixes #1018. | |||
* | Factor out test contact creation to function. | Matthew Somerville | 2015-03-13 | -7/+2 |
| | ||||
* | By default, use area-based alerts. | Matthew Somerville | 2014-12-12 | -21/+17 |
| | | | | | | | | | The body-based alert only works if the body ID matches the MapIt ID. This fixes #959. Further work needs to be done to enable proper body-based alerts that work properly in all circumstances. Consequently, factor out fixed body IDs from many tests. Also fix a couple of tests not overriding geocoder correctly. | |||
* | Fix test of postcode that has moved ever so slightly. | Matthew Somerville | 2014-05-30 | -5/+5 |
| | ||||
* | Add MAPIT_GENERATION variable, to pin whitelist. | Matthew Somerville | 2014-05-16 | -1/+3 |
| | | | | | | Otherwise, when MapIt Global is updated, the 'point' lookup, as it only returns the current generation by default, might no longer return the areas present in the whitelist. | |||
* | Add some overrides to zurich tests to stop failure | Struan Donald | 2014-04-08 | -0/+4 |
| | | | | Make sure mapit_id_whitelist set for tests that submit | |||
* | Fix test breakage | Hakim Cassimally | 2013-12-23 | -6/+23 |
| | | | | for Zurich tests, count how many reports already exist as a baseline. | |||
* | [Zurich] don't hardcode division ID in cleanup() | Hakim Cassimally | 2013-12-23 | -8/+9 |
| | ||||
* | [Zurich] admin template to respect closed_overdue | Hakim Cassimally | 2013-12-20 | -1/+5 |
| | | | | | | | | | | | | | This value is now cached, but the template calls the function, calculating it anew, so for old reports (closed within SLA) the problem_row.html template showed these as overdue. Instead, we point the template at a function which first checks the cached value. (Attempting to roll this functionality into the .overdue() method itself caused a fair amount of collateral test breakage, and it seems safest to extract the caching function for now.) Closes https://github.com/mysociety/FixMyStreet-Commercial/issues/462 | |||
* | [Zurich] stats export include 'hidden' | Hakim Cassimally | 2013-12-17 | -11/+19 |
| | | | | | | | | | | See https://github.com/mysociety/FixMyStreet-Commercial/issues/461 Again, we just need to substitute the %params line (which elides hidden reports) for the less restrictive %date_params. This commit includes very basic tests for the previously untested ?export=1 functionality. | |||
* | [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 | |||
* | 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 | -102/+171 |
| | ||||
* | [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 | |||
* | [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 | -59/+15 |
| | | | | You only need to pass in the values you wish to be different. | |||
* | [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 |
| | ||||
* | 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 |
|\ \ \ | |_|/ |/| | | ||||
| * | | [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 | -1/+1 |
| | | | | | | | | 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 |
| | | ||||
* | | [Zurich] Notes on steps required to get zurich tests to run | Edmund von der Burg | 2013-09-10 | -0/+21 |
|/ | ||||
* | Fix comment in tests | Chris Mytton | 2013-09-04 | -1/+1 |
| | ||||
* | [Zurich] Use get_ok for superuser adding bodies test | Chris Mytton | 2013-09-04 | -1/+1 |
| | ||||
* | [Zurich] Test for only superusers can add bodies | Chris Mytton | 2013-09-04 | -0/+6 |
| | ||||
* | [Zurich] Add test for superuser body editing | Chris Mytton | 2013-09-03 | -0/+7 |
| | ||||
* | Update test to match text change in June. | Matthew Somerville | 2013-08-30 | -2/+2 |
| | | | | Silly me. | |||
* | Update test for change in 73c7fa6. | Matthew Somerville | 2013-06-04 | -2/+2 |
| | ||||
* | Update state name from Erledigt to Beantwortet. | Matthew Somerville | 2013-04-19 | -6/+6 |
| |