aboutsummaryrefslogtreecommitdiffstats
path: root/t
Commit message (Collapse)AuthorAgeLines
...
* Add test for de47a7f.Matthew Somerville2014-03-12-15/+63
| | | | | | This tests that the default string used by the old-style Open311 update code if there is no incoming string and the report is marked fixed, is translated.
* Merge remote branch 'origin/sweden-rebase'Matthew Somerville2014-03-11-0/+200
|\ | | | | | | | | | | | | Conflicts: bin/update-schema locale/sv_SE.UTF-8/LC_MESSAGES/FixMyStreet.po perllib/FixMyStreet/DB/Result/Body.pm
| * Alter Open311 description to only include English on FixMyStreet.Jon Kristensen2014-03-11-0/+2
| | | | | | | | This could be made cobrand-customisable in future.
| * Bring back open311-update-reports, GetUpdates.pm and getupdates.tJon Kristensen2014-03-11-0/+198
| | | | | | | | | | | | This commit reverses the "Remove unused Open311 GetUpdates code." commit. FixaMinGata, the Swedish FixMyStreet cobrand, is still using open311-update-reports.
* | Export reports as CSV on /dashboardHakim Cassimally2014-03-10-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide a button to download the details and text of reports at the second "Look up" button on /dashboard. Provides acknowledged, fixed, closed dates by parsing updates. Fixes #692 (and mysociety/FixMyStreet-Commercial#399) Email is not shown, and name honours anonymous flag: as /dashboard is accessible to people who are from_body (who still do not see anon-users name and email in the FMS web app.) A basic test is provided as per Zurich's similar report.
* | Re-enable dashboard testsHakim Cassimally2014-03-10-519/+521
| | | | | | | | | | | | | | | | The tests have been commented out due to long-running failure. It looks like the failures happen if working copy MAPIT_URL is not pointing at live site. Have updated with an override_config setting this to live site (which is obviously suboptimal, but seems to be widespread practice across t/ at the moment.)
* | [Hart] basic Hart cobrandingHakim Cassimally2014-03-05-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally copied from template of Bromley code, substituting logo, council ID, geolocation data, and renaming .scss variables etc. (NB: this was a bad idea. As per notes/cobranding.txt, should have started with less copying of files, as this required more cleanup. Bromley's code has some particularities, like smaller map, split First/Last name fields and so on. The current branch has been rebased to present a slightly idealised picture of the process.) (Includes contributions by @davea and @stevenday) * Some code cleanup and added is_two_tier * Add Hart-specific links to footer * Shift FMS nav out of the main header * Update styling to follow design more closely * Flip nav colours to better indicate links * Make header banner entire width of the page * Stretch footer bg to entire width of page * Add top-level nav links * Correct header height on mobile * Correct wording of top menu item * Remove FMS logo from top of page * Shift top level nav to correct position * Invert media query so IE8 displays header properly * hide folding triangle, as makes no sense on this cobrand * Some more renaming of Bromley to Hart * Treat Hart the same as Bromley in the JS * Whitespace/formatting fixes * Basic test for cobrand * Javascript #map_box fixes * Make sure #map_box is not in desk-only * ensure there is only single map (one .wrapper only) * colour tweaks * Fix example places (postcode, spelling) * Basic FAQ text * Set Zoom level 3 * CSS/HTML/JS tweaks removing Bromley customizations * Switch templates to put navigation in footer, as FMS.com does * remaining HTML/CSS changes for large map * replace favicon * move site-logo scss to one place * make small logo transparent
* | Merge branch 'issues/677-open311-errors-on-comment-processing'Matthew Somerville2014-02-27-9/+25
|\ \
| * | Fix uncommon Open311 errors on processing updates.Hakim Cassimally2014-02-27-9/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #677. The query to fetch the related alert sometimes: * retrieves no result, causing an error on $alert->id * retrieves multiple results, causing deprecation warnings (e.g. which may at some point become future errors in DBIC) This commit turns the resultset('DB::Alert')->find(...) into a search(...) and loops over the results, to suppress them.
* | | Make sure language is set early enough in email alert loop.Matthew Somerville2014-02-26-10/+28
| | | | | | | | | | | | | | | The call to set_lang_and_domain needs to happen before the first string is translated (the report state).
* | | Make sure bodies returned in consistent order.Matthew Somerville2014-02-25-2/+2
|/ /
* | Make alert_type.t test more robust.Matthew Somerville2014-02-24-4/+8
| | | | | | | | | | | | Put alerts in default cobrand, so changes to other cobrand email templates don't affect the test; and clean up at the end of the test so that it can be rerun without issue.
* | Correct subtest variable name, fix uninitialized value warning.Matthew Somerville2014-01-30-2/+2
| |
* | Don't update problem state from service request if currently hidden.Matthew Somerville2014-01-30-18/+13
| |
* | Fix test breakageHakim Cassimally2013-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 Cassimally2013-12-23-8/+9
| |
* | [Zurich] admin template to respect closed_overdueHakim Cassimally2013-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
* | Merge branch 'issues/commercial/461-zurich-export-hidden-stats'Hakim Cassimally2013-12-17-11/+19
|\ \
| * | [Zurich] stats export include 'hidden'Hakim Cassimally2013-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.
* | | Test for admin_base_url in RS::ProblemHakim Cassimally2013-12-17-0/+11
|/ / | | | | | | | | Test that the slash is added in correct place (the base URL should not have its own trailing slash).
* | [Zurich] testing - create superuser earlierHakim Cassimally2013-12-10-2/+7
| | | | | | | | So that tests pass on clean dataset such as travis...
* | [Zurich] also allow hidden reports for closed_overdueHakim Cassimally2013-12-10-2/+2
| |
* | [Zurich] fix moderated statistics to include hiddenHakim Cassimally2013-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 changesHakim Cassimally2013-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 stanzaHakim Cassimally2013-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() routineHakim Cassimally2013-12-09-4/+9
| |
* | [Zurich] test /admin/statsHakim Cassimally2013-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_overdueHakim Cassimally2013-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 bodiesChris Mytton2013-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 Somerville2013-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 Somerville2013-11-07-6/+6
| | | | | | | | Use a relative URL in the CSS in case it's proxied.
* | Merge branch 'multiple_emails_per_contact'Struan Donald2013-11-04-0/+46
|\ \
| * | Tests for contact with comma seperated multiple emailsStruan Donald2013-11-04-0/+46
| | |
* | | Merge branch '389-report-search-ref' of ↵Matthew Somerville2013-11-04-0/+7
|\ \ \ | |/ / |/| | | | | https://github.com/andylolz/fixmystreet into andylolz-389-report-search-ref
| * | One more external id-related testAndy Lulham2013-10-26-0/+3
| | |
| * | Add a test for the external_id admin report searchAndy Lulham2013-10-26-0/+4
| | |
* | | Set confirmed timestamp on any admin unconfirmed state change.Matthew Somerville2013-10-25-1/+10
| | | | | | | | | | | | This fixes #527.
* | | More accurate config overriding for Zurich.Matthew Somerville2013-10-24-2/+6
| | |
* | | Fix test suite to run regardless of config setup.Matthew Somerville2013-10-24-607/+904
| | |
* | | [Zurich] Remove phone number requirement for mobile app reportsChris Mytton2013-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 byStruan Donald2013-10-02-4/+4
| | | | | | | | Fixes #586
* | Fix Bromley use of 'no further action'.Matthew Somerville2013-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 emailsChris Mytton2013-09-20-1/+1
| | | | | | | | Closes mysociety/FixMyStreet-Commercial#410
* | Have a central config-override function.Matthew Somerville2013-09-13-60/+20
| | | | | | | | You only need to pass in the values you wish to be different.
* | switch to Sub::Override from Test::MockModuleStruan Donald2013-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 casesChris Mytton2013-09-12-39/+44
| |
* | [Zurich] Put the phone number test into a subtest blockChris Mytton2013-09-12-29/+30
| |
* | [Zurich] Make sure mapit url is set properlyChris Mytton2013-09-12-0/+1
| |
* | [Zurich] Try to override config in testsChris Mytton2013-09-12-1/+25
| |
* | explicitly set bodies_str for abuse and app store testsStruan Donald2013-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
generated by cgit v1.2.3 (git 2.25.1) at 2025-07-23 17:26:03 +0000