refactor: revert back to an updated list of curerncies

Author: Jon Waldstein

src/Rules/Currency.php

@@ -10,18 +10,6 @@
 
 class Currency implements ValidationRule, ValidatesOnFrontEnd
 {
-    /**
-     * @var string[]
-     */
-    protected $currencyCodes;
-
-    /**
-     * @unreleased
-     */
-    public function __construct(?array $currencyCodes = null)
-    {
-        $this->currencyCodes = $currencyCodes ?? self::currencyCodes();
-    }
 
     /**
      * @inheritDoc
@@ -60,7 +48,7 @@ public function serializeOption()
      */
     public function __invoke($value, Closure $fail, string $key, array $values)
     {
-        if (!is_string($value) || !in_array(strtoupper($value), $this->currencyCodes, true)) {
+        if (!is_string($value) || !in_array(strtoupper($value), self::currencyCodes(), true)) {
             $fail(sprintf(__('%s must be a valid currency', '%TEXTDOMAIN%'), '{field}'));
         }
     }

tests/unit/Rules/CurrencyTest.php

@@ -1,4 +1,4 @@
-git <?php
+<?php
 
 declare(strict_types=1);
 
@@ -10,7 +10,7 @@
 class CurrencyTest extends TestCase
 {
     /**
-     * @since 1.1.0
+     * @unreleased
      * @dataProvider currencyProvider
      */
     public function testCurrencyValidations($currency, $shouldPass)
@@ -24,31 +24,12 @@ public function testCurrencyValidations($currency, $shouldPass)
         }
     }
 
-    /**
-     * Test custom currency codes override functionality.
-     *
-     * @since 1.1.0
-     */
-    public function testCustomCurrencyCodesOverride()
-    {
-        // Test with custom currency codes
-        $customCodes = ['XYZ', 'ABC', 'DEF'];
-        $rule = new Currency($customCodes);
-
-        // Custom codes should pass
-        self::assertValidationRulePassed($rule, 'XYZ');
-        self::assertValidationRulePassed($rule, 'ABC');
-        self::assertValidationRulePassed($rule, 'def'); // case insensitive
 
-        // Standard codes should fail when using custom list
-        self::assertValidationRuleFailed($rule, 'USD');
-        self::assertValidationRuleFailed($rule, 'EUR');
-    }
 
     /**
      * Test that obsolete currency codes no longer pass validation.
      *
-     * @since 1.1.0
+     * @unreleased
      * @dataProvider obsoleteCurrencyProvider
      */
     public function testObsoleteCurrencyCodesFail($currency)
@@ -60,7 +41,7 @@ public function testObsoleteCurrencyCodesFail($currency)
     /**
      * Test newly added currency codes pass validation.
      *
-     * @since 1.1.0
+     * @unreleased
      * @dataProvider newCurrencyProvider
      */
     public function testNewCurrencyCodesPass($currency)
@@ -72,7 +53,7 @@ public function testNewCurrencyCodesPass($currency)
     /**
      * Test case insensitivity with various currency codes.
      *
-     * @since 1.1.0
+     * @unreleased
      * @dataProvider caseInsensitiveProvider
      */
     public function testCaseInsensitivity($currency, $shouldPass)
@@ -87,7 +68,7 @@ public function testCaseInsensitivity($currency, $shouldPass)
     }
 
     /**
-     * @since 1.1.0
+     * @unreleased
      */
     public function currencyProvider(): array
     {
@@ -149,7 +130,7 @@ public function currencyProvider(): array
     /**
      * Currency codes that were removed in the 2024 update.
      *
-     * @since 1.1.0
+     * @unreleased
      */
     public function obsoleteCurrencyProvider(): array
     {
@@ -168,7 +149,7 @@ public function obsoleteCurrencyProvider(): array
     /**
      * New currency codes added in the 2024 update.
      *
-     * @since 1.1.0
+     * @unreleased
      */
     public function newCurrencyProvider(): array
     {
@@ -211,7 +192,6 @@ public function newCurrencyProvider(): array
             ['RWF'], // Rwandan Franc
             ['SDG'], // Sudanese Pound
             ['SLE'], // Sierra Leonean Leone (new)
-            ['SLL'], // Sierra Leonean Leone (old)
             ['SSP'], // South Sudanese Pound
             ['STN'], // São Tomé and Príncipe Dobra
             ['SZL'], // Swazi Lilangeni
@@ -237,7 +217,7 @@ public function newCurrencyProvider(): array
     /**
      * Test various case combinations.
      *
-     * @since 1.1.0
+     * @unreleased
      */
     public function caseInsensitiveProvider(): array
     {