| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | | |
* commit '1ff5465107e518d45cadeebbe70d23e2a8f8c56e':
Change the DateFormatSymbols serialized form
|
| |\ \ \ |
|
| | | |/
| | |/|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Adding locale to the serialized form of
DateFormatSymbols.
Some duplicated code around the generation of
numeric / offset timezone strings (e.g. GMT+08:00) has
been removed. A new hidden method has been added
to TimeZone to create this string.
The timezone string lookup has been moved into
LocaleData and it now has a single path rather
than treating the customZoneStrings path as special.
customZoneStrings has been removed from
DateFormatSymbols because it would potentially
have an incorrect value after serialization and
it is no longer required.
Bug: 16502916
Change-Id: I2b317e34ba4772beb372a75dd08c95113408b9cc
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | | |
* commit '9ae2eb2e586d43f92c6339e9d7072ec1eef3f0ed':
Fix Harmony-707 test for CTS
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Change the way Harmony-707 test works.
We cannot check timeouts equality, as timeout is approximated
by the Linux Kernel (see set_sock_timeout() in net/core/sock.c).
Previously, testing with the value provided to setSoTimout()
could fail, depending on the Kernel configuration:
- On a device running a kernel compiled with CONFIG_HZ=100,
it would work
- On a device running a kernel compiled with CONFIG_HZ=250,
it would not:
* Kernel stored a timeout of 3 (3.49975 cast to long)
*timeo_p = tv.tv_sec*HZ + (tv.tv_usec+(1000000/HZ-1))/(1000000/HZ)
with tv.tv_sec=0, tv.tv_usec=10000 (10ms), and HZ=250
* Kernel returned a timeout of 12ms
v.tm.tv_usec = ((sk->sk_rcvtimeo % HZ) * 1000000) / HZ
with sk->sk_rcvtimeo=3 and HZ=250
Instead, we have to ensure that the timeout is not reset to the
internal default timeout after calling accept():
- set a timeout
- immediately retrieve it and store it (this value may differ
from the previously set value)
- call accept()
- retrieve and check the timeout with the previously stored
timeout
Change-Id: I7f05c6843d9e48e6d696dca17173fce028a45a68
Signed-off-by: Samuel Rats <samuel.rats@gmail.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
For locales whose language code is "und" we use Locale.ROOT
instead. This also fixes two other corner cases :
- We were using the wrong locale to fetch timezone strings
when the input locale was null. we now use the same locale
throughout by making sure we don't perform any subsititutions
in LocaleData.get.
- Adds a clearer comment about the broken serialization
behaviour.
bug: 15849709
Change-Id: I95e7eb0ccb7458711038ce9b1c76b3279acda9d6
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Use the modern "es-u-co-trad" instead of "es-TRADITIONAL".
The latter is an invalid language tag (TRADITIONAL is an invalid
variant) so we will not support it any longer.
bug: 15849709
Change-Id: Ia4172879041f42a3b5ff8b027b44c48b3c336f2c
|
| |\ \ |
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
toArray() was returning entries in reverse order.
toArray(T[]) was correct.
Thanks to Chris Povirk for discovering.
The fix is simple.
Change includes additional tests to avoid regressions.
Test & support code has been cleaned up a little to
improve coverage and make it (slightly) less horrible.
Bug: https://code.google.com/p/android/issues/detail?id=72073
Change-Id: I42ff90e0f592482289cd0cd9fdbdaabf0c17ad93
|
| | |
| | |
| | |
| | |
| | |
| | | |
(cherry-pick of f39b5ca3c6378c3c36c63889577004b9693ea9c6.)
Change-Id: Idaf030cd369e728ec37df7107cd30062db899b7c
|
| | |
| | |
| | |
| | |
| | |
| | | |
(cherry-pick of 63744c884dd4b4f4307f2b021fb894af164972af.)
Change-Id: Ibf79a402e1bad98a262e380fcee3d35c127ae6d5
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
For locales whose language code is "und" we use Locale.ROOT
instead. This also fixes two other corner cases :
- We were using the wrong locale to fetch timezone strings
when the input locale was null. we now use the same locale
throughout by making sure we don't perform any subsititutions
in LocaleData.get.
- Adds a clearer comment about the broken serialization
behaviour.
bug: 15849709
(cherry picked from commit 043a1424a4e3bbb5abc9d9e11c9c088b20f4ca7d)
Change-Id: I716fb421fb8643dedebb3a7797a76ed1dd86c548
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Use the modern "es-u-co-trad" instead of "es-TRADITIONAL".
The latter is an invalid language tag (TRADITIONAL is an invalid
variant) so we will not support it any longer.
bug: 15849709
(cherry picked from commit ee7d0c5bfcc6891034eaea0fa4a9571ffde5d2ae)
Change-Id: I560837f07fa67cc556381e0644dd49e6c9fdc59c
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
toArray() was returning entries in reverse order.
toArray(T[]) was correct.
Thanks to Chris Povirk for discovering.
The fix is simple.
Change includes additional tests to avoid regressions.
Test & support code has been cleaned up a little to
improve coverage and make it (slightly) less horrible.
Bug: https://code.google.com/p/android/issues/detail?id=72073
(cherry picked from commit 244e90adfe67264364de88df0fd741db0eb12b7f)
Change-Id: Ief017ca70863b799242ccfd04b8827e548548ea4
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
* commit '47c61891491acce74675622abf3132ff03464ae5':
Fix race condition in WeakHashMapTest.
|
| |\ \ \
| | |/ /
| | | |
| | | |
| | | | |
* commit '31e8ef04eb3bc2029eb94393c3ce87245dd75da2':
Fix race condition in WeakHashMapTest.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
There was a race condition where the reference queue daemon wouldn't
necessarily have processed the cleared references by the time we
called WeakHashMap.size(). The race is fixed by doing a busy wait
with a 5 second timeout. This should't increase the test time in the
average case since the cleared weak references should be processed
shortly after induceFinalization finishes.
Bug: 15775475
Change-Id: I66350b6c87252e98bfbd945fe1f96bef58ec24a9
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | | |
test"
* commit '3568b5622585cdde1f15503784971653710fc2aa':
Unimplement Console.readPassword and remove test
|
| |\ \ \
| | |/ /
| | | |
| | | |
| | | | |
* commit 'c477bf9eaf4553ed55b9d26c6bd65be3cc5665c6':
Unimplement Console.readPassword and remove test
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
ConsoleTest.test_readPassword* fails CTS tests.
Context: Under standard Android applications the
System.console() method will return null.
The Console.readPassword() method must turn off echo.
Under automated tests the ConsoleTest is exercising
a Console object with fake stdout / stdin streams to
get around the absence of a System.console().
setEcho() is hardcoded to use stdin / stdout in
the native code. Under the test runners stdin / stdout
are not connected to a TTY leading to a ENOTTY.
Under Android readPassword is unimportant. Rather than leaving
untested security-related code the password methods are being
unimplemented. It is unlikely this code has ever been used on
Android given the absence of a System Console instance.
Bug: 12491103
Change-Id: I755c014e6b53236f5bef0535be137dd721918b44
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
variants"
* commit '8c83d2aa1fd7b7e3f9ca7415f9dfdf94de1314ec':
Fix CurrencyTest.test_getSymbol for unknown variants
|
| |\ \
| | |/
| | |
| | |
| | | |
* commit '2eb56b69de339978a29d94531759b465742f027f':
Fix CurrencyTest.test_getSymbol for unknown variants
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The behavior of CurrencyTest.getSymbol has changed since
K for unrecoginzed locales.
Commit de0eb683370d789e7bb25673b350b8dbf2ba5d69 now means
the ICU default locale is set when the Java default locale
is changed. Previously it would have been left unset and
would have fallen back to using the system settings (most
likely US or UK English during tests).
The test is setting the Java default locale to "kr-KR"
(a locale unrecognized by ICU) and requesting the currency
symbol for USD for locale "kr-KR" (still unrecognized by
ICU).
Before the commit: Because the kr-KR argument is unrecognized
ICU tries the ICU default (which was unset and so picked
system setting of en). It returned "$".
After the commit: As above, but now the ICU default is set to
kr-KR. Since there is no entry for USD in the default it falls
back to "root", and returns the value found there: "US$".
Behavior when the default locale is unrecognized is edge-case
and arguably it is no more / less correct before or after
this commit. The test has been updated to reflect
the new value + some tidy up of the test generally.
This change also contains a more focused test that demonstrates
the behavior in ICU and adds documentation to make ICU
slightly clearer.
Arguably neither the new ICUTest nor the existing
CurrencyTest code should exist because they assert edge-case behavior
that Android may wish to change in future.
Bug: 14821275
Change-Id: I8beb6004b0c37d2fe59855d13d2c8e2e6fe1c530
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
* commit '4aadbe01b75956c4a1c5af56723e5e6e3da25c47':
Remove more.
|
| |\ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Idaf030cd369e728ec37df7107cd30062db899b7c
|
|\ \ \ \ \
| |/ / / /
| | | | |
| | | | | |
Change-Id: I2dbbda225d0fbe61da34f93abbb466ed490ba405
|
| |/ / /
| | | |
| | | |
| | | | |
Change-Id: Ibf79a402e1bad98a262e380fcee3d35c127ae6d5
|
|\ \ \ \
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | | |
execution on hammerhead"
* commit '4a35ec7e9eacc403a904d83df09a7bd762ebec0f':
Tidy up MulticastSocketTest and fix execution on hammerhead
|
| |\ \ \
| | | |/
| | |/|
| | | |
| | | | |
* commit '2b9311770bc531f15edb43f382696f3954a7c4e8':
Tidy up MulticastSocketTest and fix execution on hammerhead
|
| | |\ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Similar to the fixes for DatagramChannelMulticastTest
in 6d7b081971d12b25426c46e4989e7df3a5995eae but
for java.net.MulticastSocket.
Because MulticastSocketTest was the genesis of
DatagramChannelMulticastTest it had similar issues to that one,
but also contained other unnecessary or badly formatted code.
The main changes here are:
1) Fix the tests that were implicitly assuming that the
IPv6NetworkInterface1 was the same interface as the IPv4
version (networkInterface1).
2) Renamed variables / moved setUp to the top of the class /
closed sockets after use / fixed try/catch blocks to only wrap
expected throwing code / pulled out common code into new methods
and generally tidied up.
3) Removed the use of the "MulticastServer" class entirely.
This "server" would listen to socket on a separate thread and
was the source of multiple Thread.sleep(1000) calls in the test.
The tests rely exclusively on multicast loopback mode anyway
(MulticastSocket.setLoopbackMode()) and
MulticastSocket.receive() is blocking, which means it either
gets an immediate result or the test can rely on the
setSoTimeout() method to limit the time spent waiting. The use
of multiple threads was therefore unnecessary.
Bug: 15509450
Change-Id: Ie448cc3b2d7432012957db34c72949a7b4da359f
|
|\ \ \ \ \
| |/ / / /
| | | | /
| |_|_|/
|/| | | |
* commit '3dc2e67e393d89902fac3deee351ee20cde3d2e3':
Tests for Runtime.*Memory()
|
| |\ \ \
| | |/ /
| | | |
| | | |
| | | | |
* commit '995dc1746d6723734892b348e8faf627d830695e':
Tests for Runtime.*Memory()
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The docs describe a condition we do not test for.
The freeMemory / totalMemory tests have been consolidated
into a single test and joined by an additional assertion for
maxMemory().
The test for freeMemory from OldRuntimeTest has been tidied
up and moved to RuntimeTest.
Bug: 15507122
Change-Id: I4343215fb7f6b441613e25d3844d67812f5b5e94
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
InetAddressTest#test_isReachableLjava_net_NetworkInterfaceII" into klp-modular-dev
* commit '35f521cdcd0b877b3a3c4a7df631529a31cee45b':
DO NOT MERGE : Fix for InetAddressTest#test_isReachableLjava_net_NetworkInterfaceII
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
InetAddressTest#test_isReachableLjava_net_NetworkInterfaceII
This is speculative because the test passes for most devices
making it difficult to test.
The test was expecting IP 1.1.1.1 to be "unreachable". 1.1.1.1
may have special significance for the devices the test fails
for or the test environment they are using. 1.1.1.1 is not a
good choice for an "unreachable IP" since it is not only a
valid IP, but it was assigned to Google. It is also like to be
picked for (non-standard compliant) local networks due to its
ease of typing.
Switching the test to an IP address that should truly be
unreachable should not break anything and may fix the test.
This patch will also be applied to AOSP/master separately.
Bug: 15453368
Change-Id: I4f7e5b95d72b4952716581a783f520f6a8d7392d
|
| | |\ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The fix is in test_isJavaIdentifierPartC() and reverses
the assertion for Character.isJavaIdentifierPart('\u200b')
from false to true. The test method below it
(isIdentifierPart(int)) was already asserting this and
the behavior has been confirmed against the RI.
Other changes made:
Uncommented some assertions that now pass, removed unnecessary /
incorrect comments, added missing braces, some reformatting to 100
chars to improve readability, removed empty lines at the beginning of
methods . Converted all assertTrue(msg, !condition); to
assertFalse(msg, condition).
Bug: 13748395
(cherry picked from commit 2a025c1955e9fccac12587c019f3ae82ff5505c3)
Change-Id: I18ac7d2b0b8f87d4a13cb32113bbea1abe691a4a
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Most failures were because tests were trying to
open files relative to the current working directory,
instead of in a temp directory.
FileTest was failing because it didn't construct
paths correctly. It would add an additional separator
char even if the path already ended with one.
bug:14895550
bug:14893836
bug:14893333
(cherry picked from commit e10c61bb13373b95ba857cc3af60f94b9941ab9e)
Change-Id: I083f6d5713fd55e44b7e14a9bef7ec136fac3ce4
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The test assumed that the Enumeration returned from
element() and keys() could be cast to Iterator. Even if it was
previously ok, it isn't now and it is not part of the public
API.
Manual merge from aosp/master for klp-modular-dev.
(cherry-pick of commit 4df29e1e1cf169ff6c77b8dd247de8c79b81d552
from aosp/master)
Bug: 13747624
Change-Id: Ifc3acc0e70bad561f8d9aa443494b40ba0e37346
|
| | |\ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Remove negative tests for blocks that weren't assigned in
Java6 but are currently assigned. Those tests weren't very
useful anyway, since this class must serve as a test of our
ICU glue and not of the underlying ICU data.
Also removes a test for the deprecated SURROGATES_AREA member
which we don't support to remain consistent with ICU's treatment
of blocks in patterns.
bug: 12491302
(cherry picked from commit 7ed285657c33638ed939da5b2751e02f8e42648e)
Change-Id: I30313c7c63df1832b8fdc227bfe9cab46434d81d
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Also adds an explanatory comment and makes the
intent of the implementation clearer without changing
any of the functionality.
TL;DR - there doesn't appear to be any sensible way
to provide any sort of consisent semantics if we plan
on supporting these fields.
Partial cherry-pick from commit
5eea675931db0bf976c2347f4e3b7e4703d337a4.
The implementation is left unmodified.
bug: 12778197
Change-Id: I0f161967412a42c582ba60da1c96d0ce37a90bcf
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The issues here relate to how we deal with ambiguous wall
clock times, i.e, wall clock values that we "skip" when we
move the clock forward when transitioning into daylight
savings, and wall clock times that occur twice when we
move the clock backward
This isn't a regression. We've never handled this correctly.
bug: 12778197
(cherry picked from commit 3f2128b3408255e317f66c10c6dd5edbff53b3ee)
Change-Id: If0fd5b5e0cb79363ffd3759c7cb6b2c63aabee82
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We're going through the trouble of accounting for
DST changes in GregorianCalendar.add(int, int) (as per
the spec), so adjust our tests to agree.
bug: 12778197
(cherry picked from commit 1326a80ee854ad7df3ac183874734e418194572c)
Change-Id: I92b072e2385c7fbd6fcac9de25eec7542d53e2d6
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
ICU4C data (see icu4c/data/misc/supplementalData.txt)
as well as the RI agree that the minimum number of days
in the first week of the year should be 1 in canada.
(I'm not a 100% convinced that this is correct, the
canadian standards board claims to have implemented
ISO 8601, which requires a 4 day minimum. However, that
site links to several documents such as CSA Z234.5 that
are not publicly accessible).
(cherry picked from commit ce54057889055ab8c4cb45c51df0ca9ee09ebb43)
Change-Id: I7d152ba1f157bfaffeac6c38c37828aaf0004cb2
|
| |\ \ \
| | | |/
| | |/|
| | | | |
Change-Id: I54e57263e968e3f28ae07949d74d47f0907ab175
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use expectations files instead.
(cherry picked from commit 6a6b612286976cc185c898803fe51e4e062bd9eb)
Bug: 12924356
Change-Id: I9b7e71805a80176c873cffe46bed65f81de1903d
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | | |
ReferenceQueue ordering semantics."
* commit '100e8709eff57f7803a6337d278a5c3b2cf12bb4':
Fix tests that relied on ReferenceQueue ordering semantics.
|