set sp and sp name in quotes

cloudfoundry · Feb 13, 2025 · 7665514 · 7665514
1 parent 93828d8
commit 7665514
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/app/controllers/v3/service_instances_controller.rb b/app/controllers/v3/service_instances_controller.rb
@@ -438,13 +438,13 @@ def unprocessable_service_plan!
   end
 
   def unavailable_service_plan!(service_plan, service_plan_guid)
-    unprocessable!("Invalid service plan. The service plan #{service_plan} with guid #{service_plan_guid} has been removed from the service broker's catalog." \
+    unprocessable!("Invalid service plan. The service plan '#{service_plan}' with guid '#{service_plan_guid}' has been removed from the service broker's catalog." \
                    'It is not possible to create new service instances using this plan.')
   end
 
   def service_plan_not_visible_in_space!(service_plan, service_plan_guid, space_name, space_guid)
-    unprocessable!("Invalid service plan. Ensure that the service plan #{service_plan} with guid #{service_plan_guid} \
-is visible in your current space #{space_name} with guid #{space_guid}.")
+    unprocessable!("Invalid service plan. Ensure that the service plan '#{service_plan}' with guid '#{service_plan_guid}' \
+is visible in your current space '#{space_name}' with guid '#{space_guid}'.")
   end
 
   def invalid_service_plan_relation!

diff --git a/spec/request/service_instances_spec.rb b/spec/request/service_instances_spec.rb
@@ -1260,8 +1260,8 @@ def check_filtered_instances(*instances)
             expect(last_response).to have_status_code(422)
             expect(parsed_response['errors']).to include(
               include({
-                        'detail' => "Invalid service plan. Ensure that the service plan #{service_plan.name} with \
-guid #{service_plan_guid} is visible in your current space #{space.name} with guid #{space.guid}.",
+                        'detail' => "Invalid service plan. Ensure that the service plan '#{service_plan.name}' with \
+guid '#{service_plan_guid}' is visible in your current space '#{space.name}' with guid '#{space.guid}'.",
                         'title' => 'CF-UnprocessableEntity',
                         'code' => 10_008
                       })
@@ -1277,7 +1277,7 @@ def check_filtered_instances(*instances)
             expect(last_response).to have_status_code(422)
             expect(parsed_response['errors']).to include(
               include({
-                        'detail' => "Invalid service plan. The service plan #{service_plan.name} with guid #{service_plan.guid} " \
+                        'detail' => "Invalid service plan. The service plan '#{service_plan.name}' with guid '#{service_plan.guid}' " \
                                     "has been removed from the service broker's catalog." \
                                     'It is not possible to create new service instances using this plan.',
                         'title' => 'CF-UnprocessableEntity',
@@ -2416,7 +2416,7 @@ def check_filtered_instances(*instances)
             expect(last_response).to have_status_code(422)
             expect(parsed_response['errors']).to include(
               include({
-                        'detail' => "Invalid service plan. The service plan #{service_plan.name} with guid #{service_plan.guid} " \
+                        'detail' => "Invalid service plan. The service plan '#{service_plan.name}' with guid '#{service_plan.guid}' " \
                                     "has been removed from the service broker's catalog." \
                                     'It is not possible to create new service instances using this plan.',
                         'title' => 'CF-UnprocessableEntity',