Skip to content

Commit 7f46037

Browse files
committedFeb 19, 2025
[Bristol][Alloy] Adds extra street cleansing categories
Moving from having all street cleansing categories under one group, to having street cleansing category in another group. Specifically, the bike category wants to fit into the main bike category.
1 parent 7a8a87b commit 7f46037

File tree

3 files changed

+37
-1
lines changed

3 files changed

+37
-1
lines changed
 

‎perllib/Open311/Endpoint/Integration/UK/Bristol/Alloy.pm

+6-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,12 @@ sub process_attributes {
3333

3434
my $attributes_values = $self->config->{'request_attribute_to_values'};
3535
my $code = $args->{service_code_alloy};
36-
if (grep { $code =~ /$_/ } keys %{$self->config->{service_whitelist}->{'SC-Street Cleansing'}}) {
36+
if (grep { $code =~ /$_/ } keys %{
37+
{
38+
%{$self->config->{service_whitelist}->{'SC-Street Cleansing'} || {} },
39+
%{$self->config->{service_whitelist}->{'Abandoned bikes/scooters'} || {} }
40+
}
41+
}) {
3742
my $value;
3843
if ($code eq 'Dead animal') {
3944
$value = [ $attributes_values->{TypeOfAnimal}->[ $args->{attributes}->{TypeOfAnimal} ] ];

‎t/open311/endpoint/bristol_alloy.t

+27
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,17 @@ subtest "check service group and category aliases" => sub {
110110
'type' => 'realtime',
111111
'service_code' => 'SC-Graffiti_Defect_1'
112112
},
113+
{
114+
"description" => "Abandoned private bike/scooter",
115+
"service_code" => "Abandoned_private_bike/scooter",
116+
"metadata" => "true",
117+
"service_name" => "Abandoned private bike/scooter",
118+
"groups" => [
119+
"Abandoned bikes/scooters"
120+
],
121+
"type" => "realtime",
122+
"keywords" => ""
123+
},
113124
{
114125
'service_name' => 'Bin overflowing',
115126
'metadata' => 'true',
@@ -292,6 +303,22 @@ for my $test (
292303
},
293304
expected_design => 'designs_bWCSCStreetCleansingDefect_5e21a98bca315003e0983035',
294305
},
306+
{
307+
title => "Bike report",
308+
extra_params => {
309+
'attribute[SizeOfIssue]' => '1',
310+
'attribute[category]' => 'Abandoned_private_bike/scooter',
311+
'service_code' => 'Abandoned_private_bike/scooter',
312+
},
313+
expected => {
314+
'attributes_bWCSCStreetCleansingDefectSizeOfIssue_5e21b5ccca31500d1c836be0' => ['5e1f1bf0ca31500c541f82cb'],
315+
'attributes_bWCSCStreetCleansingDefectJobType_5e21b5adca31500d1c836bc9' => ['5e2179a3ca315012d099999'],
316+
'attributes_bWCSCStreetCleansingDefectFullDetails_5e21b587ca31500cc0a2df3a' => "title\n\ndescription",
317+
'attributes_bWCSCStreetCleansingDefectLocality_5e6f97fe4cee260f90aca0d6' => ['5e16fa66ca314f0980300be5'],
318+
'attributes_bWCSCStreetCleansingDefectFixMyStreetID_67654e2a1e4316383b73050f' => '123',
319+
},
320+
expected_design => 'designs_bWCSCStreetCleansingDefect_5e21a98bca315003e0983035',
321+
},
295322
) {
296323
subtest $test->{title} => sub {
297324
my $res = $endpoint->run_test_request(

‎t/open311/endpoint/bristol_alloy.yml

+4
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,9 @@
1515
"SC-Fly-Tip Defect": { "alias": "Flytipping" },
1616
"SC-Graffiti Defect": { "alias": "Graffiti" },
1717
},
18+
"Abandoned bikes/scooters": {
19+
"Abandoned private bike/scooter": 1,
20+
},
1821
"SC-Street Cleansing": {
1922
"alias": "Street cleansing",
2023
"Bin overflowing": 1,
@@ -86,6 +89,7 @@
8689
'Bin overflowing': '5e217a6cca315012d0956696',
8790
'Blood': '5e21782dca315003e09804a2',
8891
'Dog fouling': '5e2179a3ca315012d0956667',
92+
'Abandoned private bike/scooter': '5e2179a3ca315012d099999',
8993
},
9094
},
9195

0 commit comments

Comments
 (0)