From c3a2b8c10865c1a132824af41ee3295a2597e787 Mon Sep 17 00:00:00 2001 From: Aleksander Szut Date: Tue, 1 Sep 2026 11:06:55 +0200 Subject: [PATCH] Trim stray trailing whitespace in pl_PL provider data MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three entries in the pl_PL provider carried an accidental trailing space: the street name "Staffa " and the bank names "BNP Paribas S.A. Oddział w Polsce " and "John Deere Bank S.A. Spółka Akcyjna Oddział w Polsce ". The padded street name in particular breaks round-trip assertions in consumer test suites: a framework that trims request input (for example Laravel's TrimStrings middleware) stores "Staffa" while the test still compares against the raw "Staffa " it fed in. No entries are added or removed, so seeded generators keep producing the same sequence. Co-Authored-By: Claude Opus 5 (1M context) --- CHANGELOG.md | 1 + src/Faker/Provider/pl_PL/Address.php | 2 +- src/Faker/Provider/pl_PL/Payment.php | 4 ++-- test/Faker/Provider/pl_PL/AddressTest.php | 11 +++++++++ test/Faker/Provider/pl_PL/PaymentTest.php | 28 +++++++++++++++++++++++ 5 files changed, 43 insertions(+), 3 deletions(-) create mode 100644 test/Faker/Provider/pl_PL/PaymentTest.php diff --git a/CHANGELOG.md b/CHANGELOG.md index eba2c05d41..9488d39efa 100644 --- a/CHANGELOG.md +++ b/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) diff --git a/src/Faker/Provider/pl_PL/Address.php b/src/Faker/Provider/pl_PL/Address.php index a7b01bbb1b..4d616d6606 100644 --- a/src/Faker/Provider/pl_PL/Address.php +++ b/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', diff --git a/src/Faker/Provider/pl_PL/Payment.php b/src/Faker/Provider/pl_PL/Payment.php index f2a6030746..cfc3b3a82c 100644 --- a/src/Faker/Provider/pl_PL/Payment.php +++ b/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', diff --git a/test/Faker/Provider/pl_PL/AddressTest.php b/test/Faker/Provider/pl_PL/AddressTest.php index e69cbb9160..8bbc464687 100644 --- a/test/Faker/Provider/pl_PL/AddressTest.php +++ b/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); diff --git a/test/Faker/Provider/pl_PL/PaymentTest.php b/test/Faker/Provider/pl_PL/PaymentTest.php new file mode 100644 index 0000000000..428ea90b7e --- /dev/null +++ b/test/Faker/Provider/pl_PL/PaymentTest.php @@ -0,0 +1,28 @@ +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); + } +}