diff options
author | Matthew Somerville <matthew-github@dracos.co.uk> | 2018-03-02 16:27:21 +0000 |
---|---|---|
committer | Matthew Somerville <matthew-github@dracos.co.uk> | 2018-03-02 16:27:21 +0000 |
commit | 9154199ddf8ddb373ca34758f96d009401e4dc81 (patch) | |
tree | 517912792aa075e4b4b33b2e8b58375a794081cb /t/cobrand | |
parent | ce1e33244a53db2000689a52e4b1eecd68105290 (diff) |
[Zurich] Do not reopen closed reports by mistake.
The code for moving a report to the intermediary state was running
even if the report was in a closed state and had not been changed.
Diffstat (limited to 't/cobrand')
-rw-r--r-- | t/cobrand/zurich.t | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/t/cobrand/zurich.t b/t/cobrand/zurich.t index 385f7f077..5c1ff9a0e 100644 --- a/t/cobrand/zurich.t +++ b/t/cobrand/zurich.t @@ -289,6 +289,9 @@ subtest "report_edit" => sub { is ( $report->state, 'hidden', 'Closing as hidden sets state to hidden'); is ( $report->get_extra_metadata('closure_status'), undef, 'Closing as hidden unsets closure_status'); + $mech->submit_form_ok( { with_fields => { new_internal_note => 'Initial internal note.' } } ); + $report->discard_changes; + is ( $report->state, 'hidden', 'Another internal note does not reopen'); reset_report_state($report); is ( $report->get_extra_metadata('moderated_overdue'), undef, 'Sanity check' ); |