Trim stray trailing whitespace in pl_PL provider data

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Refreshed ISO currencies (#919)
 - Improved italian phone number formats
 - Replaced `via.placeholder.com` with `placehold.co` in Image provider (#1013)
+- Removed stray trailing whitespace from the `Staffa` street name and two bank names in the `pl_PL` provider
 
 ## [2024-11-09, v1.24.0](https://github.com/FakerPHP/Faker/compare/v1.23.1..v1.24.0)
 

src/Faker/Provider/pl_PL/Address.php

@@ -169,7 +169,7 @@ class Address extends \Faker\Provider\Address
         'Skośna', 'Skrajna', 'Słoneczna', 'Słonecznikowa', 'Słowackiego', 'Słowiańska',
         'Słowicza', 'Sobieskiego', 'Jana III Sobieskiego', 'Sokola', 'Al. Solidarności',
         'Solna', 'Solskiego', 'Sosnowa', 'Sowia', 'Sowińskiego', 'Spacerowa',
-        'Spokojna', 'Sportowa', 'Spółdzielcza', 'Srebrna', 'Staffa ', 'Stalowa', 'Staromiejska',
+        'Spokojna', 'Sportowa', 'Spółdzielcza', 'Srebrna', 'Staffa', 'Stalowa', 'Staromiejska',
         'Starowiejska', 'Staszica', 'Stawowa', 'Stolarska', 'Strażacka', 'Stroma',
         'Struga', 'Strumykowa', 'Strzelecka', 'Studzienna', 'Wita Stwosza', 'Sucha',
         'Sucharskiego', 'Szafirowa', 'Szarych Szeregów', 'Szczecińska', 'Szczęśliwa',

src/Faker/Provider/pl_PL/Payment.php

@@ -38,7 +38,7 @@ class Payment extends \Faker\Provider\Payment
         '219' => 'DNB Bank Polska Spółka Akcyjna',
         '224' => 'Banque PSA Finance Spółka Akcyjna Oddział w Polsce',
         '225' => 'Svenska Handelsbanken AB Spółka Akcyjna Oddział w Polsce',
-        '235' => 'BNP Paribas S.A. Oddział w Polsce ',
+        '235' => 'BNP Paribas S.A. Oddział w Polsce',
         '236' => 'Danske Bank A/S Spółka Akcyjna Oddział w Polsce',
         '237' => 'Skandinaviska Enskilda Banken AB (Spółka Akcyjna) - Oddział w Polsce',
         '239' => 'CAIXABANK, S.A. (SPÓŁKA AKCYJNA) ODDZIAŁ W POLSCE',
@@ -64,7 +64,7 @@ class Payment extends \Faker\Provider\Payment
         '272' => 'AS Inbank Spółka Akcyjna - Oddział w Polsce',
         '273' => 'China Construction Bank (Europe) S.A. (Spółka Akcyjna) Oddział w Polsce',
         '274' => 'MUFG Bank (Europe) N.V. S.A. Oddział w Polsce',
-        '275' => 'John Deere Bank S.A. Spółka Akcyjna Oddział w Polsce ',
+        '275' => 'John Deere Bank S.A. Spółka Akcyjna Oddział w Polsce',
         '277' => 'Volkswagen Bank GmbH Spółka z ograniczoną odpowiedzialnością Oddział w Polsce',
         '278' => 'ING Bank Hipoteczny Spółka Akcyjna',
         '279' => 'Raiffeisen Bank International AG (Spółka Akcyjna) Oddział w Polsce',

test/Faker/Provider/pl_PL/AddressTest.php

@@ -18,6 +18,17 @@ public function testState(): void
         self::assertMatchesRegularExpression('/[a-z]+/', $state);
     }
 
+    public function testStreetNamesAreNotPaddedWithWhitespace(): void
+    {
+        $reflection = new \ReflectionClass(Address::class);
+        $streets = $reflection->getProperty('street');
+        $streets->setAccessible(true);
+
+        foreach ($streets->getValue() as $street) {
+            self::assertSame(trim($street), $street, sprintf('Street name "%s" is padded with whitespace', $street));
+        }
+    }
+
     protected function getProviders(): iterable
     {
         yield new Address($this->faker);

test/Faker/Provider/pl_PL/PaymentTest.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace Faker\Test\Provider\pl_PL;
+
+use Faker\Provider\pl_PL\Payment;
+use Faker\Test\TestCase;
+
+/**
+ * @group legacy
+ */
+final class PaymentTest extends TestCase
+{
+    public function testBankNamesAreNotPaddedWithWhitespace(): void
+    {
+        $reflection = new \ReflectionClass(Payment::class);
+        $banks = $reflection->getProperty('banks');
+        $banks->setAccessible(true);
+
+        foreach ($banks->getValue() as $bank) {
+            self::assertSame(trim($bank), $bank, sprintf('Bank name "%s" is padded with whitespace', $bank));
+        }
+    }
+
+    protected function getProviders(): iterable
+    {
+        yield new Payment($this->faker);
+    }
+}