CLDR-16857 add SLE to cov=80, add test (#3088)

- add test that checks modern currencies
unicode-org · Aug 15, 2023 · 13252bb · 13252bb
1 parent 9f668ed
commit 13252bb
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 1 deletion.
diff --git a/common/supplemental/coverageLevels.xml b/common/supplemental/coverageLevels.xml
@@ -68,7 +68,7 @@ For terms of use, see http://www.unicode.org/copyright.html
 		<coverageVariable key="%currency40" value="(BRL|CNY|EUR|GBP|INR|JPY|RUB|USD)"/>
 		<coverageVariable key="%currency60" value="(AUD|CAD|CHF|DKK|HKD|IDR|KRW|MXN|NOK|PLN|SAR|SEK|THB|TRY|TWD|ZAR)"/>
 		<coverageVariable key="%currency60_EU" value="(CZK|HUF)"/>
-		<coverageVariable key="%currency80" value="(AED|AFN|ALL|AMD|ANG|AOA|ARS|AWG|AZN|BAM|BBD|BDT|BGN|BHD|BIF|BMD|BND|BOB|BSD|BTN|BWP|BYN|BZD|CDF|CLP|CNH|COP|CRC|CUC|CUP|CVE|CZK|DJF|DOP|DZD|EGP|ERN|ETB|FJD|FKP|GEL|GHS|GIP|GMD|GNF|GTQ|GYD|HNL|HRK|HTG|HUF|ILS|IQD|IRR|ISK|JMD|JOD|KES|KGS|KHR|KMF|KPW|KWD|KYD|KZT|LAK|LBP|LKR|LRD|LSL|LYD|MAD|MDL|MGA|MKD|MMK|MNT|MOP|MRU|MUR|MVR|MWK|MYR|MZN|NAD|NGN|NIO|NPR|NZD|OMR|PAB|PEN|PGK|PHP|PKR|PYG|QAR|RON|RSD|RWF|SBD|SCR|SDG|SGD|SHP|SLL|SOS|SRD|SSP|STN|SYP|SZL|TJS|TMT|TND|TOP|TTD|TZS|UAH|UGX|UYU|UZS|VES|VND|VUV|WST|XCD|XAF|XOF|XPF|YER|ZMW)"/>
+		<coverageVariable key="%currency80" value="(AED|AFN|ALL|AMD|ANG|AOA|ARS|AWG|AZN|BAM|BBD|BDT|BGN|BHD|BIF|BMD|BND|BOB|BSD|BTN|BWP|BYN|BZD|CDF|CLP|CNH|COP|CRC|CUC|CUP|CVE|CZK|DJF|DOP|DZD|EGP|ERN|ETB|FJD|FKP|GEL|GHS|GIP|GMD|GNF|GTQ|GYD|HNL|HRK|HTG|HUF|ILS|IQD|IRR|ISK|JMD|JOD|KES|KGS|KHR|KMF|KPW|KWD|KYD|KZT|LAK|LBP|LKR|LRD|LSL|LYD|MAD|MDL|MGA|MKD|MMK|MNT|MOP|MRU|MUR|MVR|MWK|MYR|MZN|NAD|NGN|NIO|NPR|NZD|OMR|PAB|PEN|PGK|PHP|PKR|PYG|QAR|RON|RSD|RWF|SBD|SCR|SDG|SGD|SHP|SL[EL]|SOS|SRD|SSP|STN|SYP|SZL|TJS|TMT|TND|TOP|TTD|TZS|UAH|UGX|UYU|UZS|VES|VND|VUV|WST|XCD|XAF|XOF|XPF|YER|ZMW)"/>
 		<coverageVariable key="%currency100" value="(AFA|ADP|ALK|AO[KNR]|AR[ALMP]|ATS|AZM|BA[DN]|BE[CFL]|BG[LM]|BGO|BO[LPV]|BR[BCENRZ]|BUK|BY[BR]|CH[EW]|CL[EF]|CNX|COU|CS[DK]|CYP|DDM|DEM|EC[SV]|EEK|ES[ABP]|FIM|FRF|GEK|GHC|GNS|GQE|GRD|GW[EP]|HRD|IEP|IL[PR]|ISJ|ITL|KR[HO]|LT[LT]|LU[CFL]|LV[LR]|MAF|MCF|MDC|MGF|MKN|MLF|MRO|MT[LP]|MVP|MX[PV]|MZ[EM]|NIC|NLG|PE[IS]|PLZ|PTE|RHD|ROL|RUR|SD[DP]|SIT|SKK|SRG|STD|SUR|SVC|TJR|TMM|TPE|TRL|UAK|UGS|US[NS]|UY[IPW]|VE[BF]|VNN|XA[GU]|XB[ABCD]|XDR|XEU|XF[OU]|XP[DT]|XRE|XSU|XTS|XUA|YDD|YU[DMNR]|ZAL|ZMK|ZR[NZ]|ZW[DLR])"/>
 		<coverageVariable key="%cyclicNameTypes" value="([1-9]?[0-9])"/>
 		<coverageVariable key="%d0Types80" value="(ascii|fwidth|hwidth|lower|title|upper)"/>

diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/util/TestLevel.java b/tools/cldr-code/src/test/java/org/unicode/cldr/util/TestLevel.java
@@ -1,9 +1,17 @@
 package org.unicode.cldr.util;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.params.provider.Arguments.arguments;
 
+import java.util.Set;
+import java.util.TreeSet;
+import java.util.stream.Stream;
+import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.CsvSource;
+import org.junit.jupiter.params.provider.MethodSource;
 
 public class TestLevel {
     @ParameterizedTest(name = "{index}: min {0},{1}")
@@ -47,4 +55,49 @@ void testMax(final String a, final String b, final String c) {
                 actual,
                 () -> String.format("Expected Level.max(%s,%s) but was %s", aa, bb, actual));
     }
+
+    static SupplementalDataInfo sdi = null;
+
+    @BeforeAll
+    public static void setUp() throws Exception {
+        sdi = CLDRConfig.getInstance().getSupplementalDataInfo();
+    }
+
+    /** walk through all currencies looking for modern ones */
+    static final Stream<Arguments> modernCurrencies() {
+        final Set<String> all = new TreeSet<String>();
+
+        sdi.getCurrencyTerritories()
+                .forEach(
+                        (t) -> {
+                            sdi.getCurrencyDateInfo(t).stream()
+                                    // TODO: should we use RECENT_HISTORY? CLDR-16316
+                                    .filter(
+                                            di ->
+                                                    (di.isLegalTender()
+                                                            && (DateConstants.NOW.compareTo(
+                                                                            di.getStart())
+                                                                    >= 0)
+                                                            && (DateConstants.NOW.compareTo(
+                                                                            di.getEnd())
+                                                                    <= 0)))
+                                    .map(di -> di.getCurrency())
+                                    .forEach(c -> all.add(c));
+                        });
+        return all.stream().map(c -> arguments(c));
+    }
+
+    @ParameterizedTest()
+    @MethodSource("modernCurrencies")
+    public void testModernCurrencies(final String code) {
+        Level l =
+                sdi.getCoverageLevel(
+                        String.format(
+                                "//ldml/numbers/currencies/currency[@type=\"%s\"]/symbol", code),
+                        "und");
+        final Level expect = Level.MODERN;
+        assertTrue(
+                expect.isAtLeast(l),
+                () -> String.format("cov for %s is %s expected ≤ %s", code, l, expect));
+    }
 }