From 355c11759173d77bb8ca48264923a6c83f5890fd Mon Sep 17 00:00:00 2001 From: Mark Davis Date: Wed, 12 Aug 2026 14:34:50 -0700 Subject: [PATCH 1/2] CLDR-19399 Check for available archive --- .../unicode/cldr/unittest/TestCLDRLocaleCoverage.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java index 2cd5f605a33..a55fc0cb34d 100644 --- a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java +++ b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java @@ -41,6 +41,7 @@ import org.unicode.cldr.util.SupplementalDataInfo; import org.unicode.cldr.util.SupplementalDataInfo.OfficialStatus; import org.unicode.cldr.util.SupplementalDataInfo.PopulationData; +import org.unicode.cldr.util.TestCLDRPaths; import org.unicode.cldr.util.TestCoverageLevel2; // test function import org.unicode.cldr.util.Validity; import org.unicode.cldr.util.Validity.Status; @@ -442,8 +443,16 @@ public void TestParentCoverage() { } } + // NOTE this could be sped up *a lot* if we had method to retrieve all the possible paths that + // could be Core or Basic + // Then we'd just make 60-70 queries against each CLDRFile instead of up to tens of thousands. + public void testBasicAgainstLastRelease() { String oldVersion = ToolConstants.CLDR_VERSIONS.get(ToolConstants.CLDR_VERSIONS.size() - 1); + if (!TestCLDRPaths.canUseArchiveDirectory()) { + warnln("Cannot access the archive directory"); + return; // print warning and exit if archive not present/usable + } String lastReleaseRuleFile = CLDRPaths.ARCHIVE_DIRECTORY + "/cldr-" From cbe8559198d7c78e8e2ff0153cd11e214e0c57e2 Mon Sep 17 00:00:00 2001 From: Mark Davis Date: Wed, 12 Aug 2026 14:54:35 -0700 Subject: [PATCH 2/2] CLDR-19399 Make terser and fix comment line --- .../org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java index a55fc0cb34d..f78f2f6105f 100644 --- a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java +++ b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java @@ -450,8 +450,8 @@ public void TestParentCoverage() { public void testBasicAgainstLastRelease() { String oldVersion = ToolConstants.CLDR_VERSIONS.get(ToolConstants.CLDR_VERSIONS.size() - 1); if (!TestCLDRPaths.canUseArchiveDirectory()) { - warnln("Cannot access the archive directory"); - return; // print warning and exit if archive not present/usable + // TestCLDRPaths prints warning if archive not present/usable, so just exit + return; } String lastReleaseRuleFile = CLDRPaths.ARCHIVE_DIRECTORY