CLDR-8823 Correct pathheader where chinese was missing

- Also update a test err message for clarity and to make actionable
unicode-org · Jul 12, 2024 · 0f87b89 · 0f87b89
1 parent 1195c9b
commit 0f87b89
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/tools/cldr-code/src/main/resources/org/unicode/cldr/util/data/PathHeader.txt b/tools/cldr-code/src/main/resources/org/unicode/cldr/util/data/PathHeader.txt
@@ -24,7 +24,7 @@
 # We don't list roc, etc here because their months are hidden.
 %G = (gregorian)
 # all others use M## form months.
-%O = (coptic|dangi|ethiopic|hebrew|indian|islamic|persian)
+%O = (chinese|coptic|dangi|ethiopic|hebrew|indian|islamic|persian)
 %P = (future|past)
 %R = (gregorian|buddhist|coptic|ethiopic|ethiopic-amete-alem|hebrew|indian|islamic|japanese|persian|roc)
 %S = ([^/]*+)

diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestPathHeader.java b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestPathHeader.java
@@ -482,7 +482,9 @@ public void TestCoverage() {
             if (hidePathHeader != hideCoverage) {
                 String message = "PathHeader: " + status + ", Coverage: " + level + ": " + path;
                 if (hidePathHeader && !hideCoverage) {
-                    errln(message);
+                    errln(
+                            message
+                                    + " - PathHeader says to HIDE this, but it visible at <comprehensive coverage. Fix PathHeader to show, or fix coverage.");
                 } else if (!hidePathHeader && hideCoverage) {
                     logln(message);
                 }