Skip to content

V
V8
Commit 76b35ebe authored by Frank Tang's avatar Frank Tang Committed by Commit Bot
Browse files
Options

[Intl] Change fail tests on Android. 

Split the case of 'ar-SA' into
test/intl/regress-8432.js

Bug: v8:8432, v8:8413
Change-Id: I6a0e3040b132be6620649c7b7aa5fdd2e0f07053
Cq-Include-Trybots: luci.v8.try:v8_android_arm64_n5x_rel_ng_triggered
Reviewed-on: https://chromium-review.googlesource.com/c/1324574Reviewed-by: 's avatarJungshik Shin <jshin@chromium.org>
Reviewed-by: 's avatarAdam Klein <adamk@chromium.org>
Commit-Queue: Frank Tang <ftang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57372}
parent c6e09003
Hide whitespace changes
Inline Side-by-side
Showing with 63 additions and 19 deletions
test/intl/intl.status
View file @ 76b35ebe
......@@ -52,16 +52,9 @@
'relative-time-format/default-locale-fr-CA': [FAIL],
'relative-time-format/default-locale-pt-BR': [FAIL],
# These tests use some locales which are unsupported on Android.
'regress-8413-day': [FAIL],
'regress-8413-era': [FAIL],
'regress-8413-hour': [FAIL],
'regress-8413-minute': [FAIL],
'regress-8413-month': [FAIL],
'regress-8413-second': [FAIL],
'regress-8413-timeZoneName': [FAIL],
'regress-8413-weekday': [FAIL],
'regress-8413-year': [FAIL],
# https://bugs.chromium.org/p/v8/issues/detail?id=8432
'regress-8432': [FAIL],
}], # 'system == android'
['tsan', {
# Run for too long under some tsan configs.
......
test/intl/regress-8413-day.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-era.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-hour.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-minute.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-month.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-second.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-timeZoneName.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-weekday.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8413-year.js
View file @ 76b35ebe
......@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// "ar-SA" has problem on Android, split it to regress-8432.js
// All the known locales egrep by
// find third_party/icu/source/data/locales/*.txt|cut -d: -f 1| \
// sort -u|cut -d/ -f6|cut -d. -f1|tr "_" "-"
......@@ -9,7 +10,7 @@ let locales = [
"af-NA", "af", "af-ZA", "agq-CM", "agq", "ak-GH", "ak", "am-ET", "am", "an",
"ar-001", "ar-AE", "ar-BH", "ar-DJ", "ar-DZ", "ar-EG", "ar-EH", "ar-ER",
"ar-IL", "ar-IQ", "ar-JO", "ar-KM", "ar-KW", "ar-LB", "ar-LY", "ar-MA",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", "ar-SA", "ar-SD", "ar-SO", "ar-SS",
"ar-MR", "ar-OM", "ar-PS", "ar-QA", /* "ar-SA", */ "ar-SD", "ar-SO", "ar-SS",
"ars", "ar-SY", "ar-TD", "ar-TN", "ar", "ar-YE", "asa", "asa-TZ", "as-IN",
"ast-ES", "ast", "as", "az-AZ", "az-Cyrl-AZ", "az-Cyrl", "az-Latn-AZ",
"az-Latn", "az", "bas-CM", "bas", "be-BY", "bem", "bem-ZM", "be", "bez",
......
test/intl/regress-8432.js 0 → 100644
View file @ 76b35ebe
// Copyright 2018 the V8 project authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// Somehow only ar-SA fail on Android in regress-8413*.js
// Split it into this test just for ar-SA
// This is likely to be an ICU data trimming issue only apply to Android.
let locales = [ "ar-SA" ];
// "Table 5: Components of date and time formats" as in
// https://ecma-international.org/ecma-402/#sec-datetimeformat-abstracts
let table5 = [
["weekday", ["narrow", "short", "long"]],
["era", ["narrow", "short", "long"]],
["year", ["2-digit", "numeric"]],
["month", ["2-digit", "numeric", "narrow", "short", "long"]],
["day", ["2-digit", "numeric"]],
["hour", ["2-digit", "numeric"]],
["minute", ["2-digit", "numeric"]],
["second", ["2-digit", "numeric"]],
["timeZoneName", ["short", "long"]]
];
// Test each locale
for (let loc of locales) {
// Test each property in Table 5
for (let row of table5) {
let prop = row[0];
let values = row[1];
// Test each value of the property
for (let value of values) {
let opt = {};
opt[prop] = value;
let dft = new Intl.DateTimeFormat([loc], opt);
let result = dft.resolvedOptions();
assertTrue(values.indexOf(result[prop]) >= 0,
"Expect new Intl.DateTimeFormat([" + loc + "], {" + prop + ": '" +
value + "'}).resolvedOptions()['" + prop + "'] to return one of [" +
values + "] but got '" + result[prop] + "'");
}
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment