Test that the action halts the lane when it aborts on violations

The suite stubs `UI.abort_with_message!` to a no-op so the other examples can
inspect the return value, which meant nothing proved the abort actually
terminates the lane — it was only asserted as "received". Add one example that
lets the real abort run (`and_call_original`) and asserts the action raises
rather than falling through to its return. Verified it bites: swapping the abort
for a non-halting `UI.error` makes this example fail.

Author: jkmassel

spec/ios_lint_localization_placeholder_changes_spec.rb

@@ -50,6 +50,17 @@ def lint_placeholder_changes(old:, new:, **extra_params)
     )
   end
 
+  it 'actually halts the lane (raises) when it aborts on violations, instead of returning' do
+    # The `before` block stubs `abort_with_message!` to a no-op so the other examples can inspect the
+    # return value. Here we let the real one run to prove the abort genuinely stops the lane: in
+    # production `UI.abort_with_message!` raises, so the action must not fall through to its `return`.
+    allow(FastlaneCore::UI).to receive(:abort_with_message!).and_call_original
+
+    expect do
+      lint_placeholder_changes(old: '"likes" = "%d likes";', new: '"likes" = "%@ likes";')
+    end.to raise_error(/changed their format placeholders incompatibly/)
+  end
+
   it 'ignores added and removed keys, only flagging changes to keys present in both' do
     result = lint_placeholder_changes(
       old: <<~OLD,