Use ICU case conversion/transliterator for case conversion behind a flag

test/intl/testcfg.py

Index: test/intl/testcfg.py
diff --git a/test/intl/testcfg.py b/test/intl/testcfg.py
index 51fa1e1a88d2fe2b350be8b7b16f1db62b9bed88..5c8e27101d97816db53b11ca514e42dd8579625f 100644
--- a/test/intl/testcfg.py
+++ b/test/intl/testcfg.py
@@ -54,7 +54,8 @@ class IntlTestSuite(testsuite.TestSuite):
     return tests
 
   def GetFlagsForTestCase(self, testcase, context):
-    flags = ["--allow-natives-syntax"] + context.mode_flags
+    flags = ["--allow-natives-syntax", "--icu-case-mapping"] + \

[Dan Ehrenberg, 2016/03/17 20:38:36]

BTW other testcfg.py's have support for a // Flags: line at the beginning of the
file, which is probably the better way to do this.

[jungshik at Google, 2016/04/08 18:09:33]

I'm not sure what you meant.  

$  find . -name testcfg.py | xargs grep Flags

does not turn up anything like that. 

[Dan Ehrenberg, 2016/04/08 21:13:46]

No, Flags is used in test cases, but implemented in testcfg.py. An example of a
test using flags is

test/mjsunit/es6/arguments-iterator.js

which has the line 

// Flags: --allow-natives-syntax 

In test/mjsunit/testcfg.py, this is supported by the code in
GetFlagsForTestCase. This could be copy-pasted into intl/testcfg.py, or the
tests could be moved to mjsunit (why are they separated out in the first place?
is it just historical?).

+        context.mode_flags
 
     files = []
     files.append(os.path.join(self.root, "assert.js"))