| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- First, the test was broken pretty badly. It would pass a
Jar style URL to a FileURLConnection and expect it to
work, this is out of spec and seems wrong in general
(file:///foo.apk!resources/foo.txt).
- The RI supports various "headers" on file URL connections
(sigh). As if the getHeaderField(int) and getHeaderFieldKey(int)
APIs weren't bad enough. This has now been implemented with
a naive implementation.
- Fixes FileURLConnection to populate a header map for
information it provides (content length, content type
etc.)
bug: 11664881
Change-Id: I7a3e9aaa79bf125abbcfe8367574115ce54718e3
|
|\ \ \ \
| |/ / /
| | | |
| | | | |
Change-Id: Ie6b363f2fcee69aa71f96fbab2d4179633f6f683
|
| |\ \ \ |
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
org.apache.harmony.tests.java.util.ControlTest
The test is probably broken, but the code under test also looks
broken. Unpicking it all and fixing it all is low priority.
Marking it as known-broken until it can be given the attention it
deserves.
Change-Id: I0e2784aa505ace4ab86149741c2faf94d8432df9
Bug: 13747957
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This fails on the RI and Android now. Add it to the known failures since
we don't deal with CRLs anyway.
Bug: 12905370
Change-Id: Id21af237aaa2709d71ffd3268a730fc4cf102d92
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
behavior to stay in spec"
* commit '5c3502f3d13ef2bebdeba86b9d7839ad834ce583':
Change DecimalFormat.setRoundingMode() behavior to stay in spec
|
| |\ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The serialization tests revealed that some DecimalFormat instances
could not be round-tripped. The reason is that the setRoundingMode()
method was setting a non-zero rounding increment, which cannot
be persisted, and also creates patterns that are outside of the
DecimalFormat spec.
Addressed the fact that ICU4C appears to support
RoundingMode.UNNECESSARY now by deleting special-case code.
Changed SerializationStressTest to use canonical DecimalFormat
patterns:
The ones chosen had a couple things that were not ideal:
1) Putting '-' in quotes causes ICU (for non-obvious reasons)
treat the negative pattern prefix as not being default. Default
in this case means "the positive prefix with a preceding '-'.
Removing the quotes meant it would recognize the pattern as
not needing everything after ;., so the pattern would be shrunk.
2) The positive pattern was also not one ICU could deal with
fully. ICU changed "#.#" to "#0.#" at parse time (which can be
corrected with a setMinimumIntegerDigits(0)). It's not clear why
ICU cannot deal with parsing "#.#" when it can produce it, but
there is a long comment in decimfmt.cc suggesting it is
intentional and to avoid format producing empty strings.
The pattern has been changed to one that ICU can handle and would
choose when the various serialized fields are read back. This is
necessary because NativeDecimalFormat compares the patterns and
not just the serialized fields. Leaving the pattern check part of
equals() means that two non-spec patterns will not be considered
equal(), which is important while Android allows non-spec patterns.
Restricting support in DecimalFormat to the documented subset of
characters is not worth the effort and may cause problems
elsewhere.
Change-Id: I2de588458b86619733c0dc1692d526f179059910
Bug: 11668227
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | |
| | | | |
* commit 'c501440e3fc82cbb427a441e81cfb534c137d9e6':
Update SSLEngineTest for OpenSSL
Update SSLEngineTest for RI
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Our new OpenSSL-based SSLEngine supports all the new stuff and no longer
fails tests.
Change-Id: I7db8e5134ca36ebd963c7081cd7ba79d91b3e5e2
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | | |
* commit '1e527edc0c3e9483104a76f4917aafc379d5b5c6':
Fix for ClassLoaderTest under CTS
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The tests could not work given the arrangement of classloaders
in Android apps. This also fixes a known failure case in
ClassLoaderTest which was due to missing resources. Also updated
ClassTest.
Change-Id: I114aa9b1a177911aed0aef87819d93a260f0b8d1
Bug: 12491655
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
Use expectations files instead.
Bug: 12924356
Change-Id: I0ea7917384adf7849404b6da5b7704e6f0c0c1c7
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change introduces NetworkChannel, but only bind() and
getLocalAddress() methods. To avoid breaking existing Android
applications that extend ServerSocketChannel, DatagramChannel and
SocketChannel the methods have been added with concrete
implementations that throw exceptions rather than leaving them
abstract.
In channel tests, usages of channel.socket().bind() and
channel.socket().getLocalSocketAddress() have been changed to
channel.bind() and channel.getLocalAddress(), since the behavior is
close enough and the tests should be written against the channel APIs
as much as possible. Tests have been added for new methods.
Removed further overriding in
DatagramChannelImpl.DatagramSocketAdapter and
SocketChannelImpl.SocketAdapter which revealed some bugs and lack of
clarity in the docs for the socket methods:
Improved the documentation for DatagramSocket.getLocalAddress(),
DatagramSocket.getLocalSocketAddress(),
ServerSocket.getInetAddress(), ServerSocket.getLocalPort(),
ServerSocket.getLocalSocketAddress(), Socket.getLocalAddress(),
Socket.getLocalPort(), Socket.getLocalSocketAddress(). These methods
treat special cases differently.
Fixed a bug in DatagramSocket.getLocalSocketAddress() where it would
incorrect throw an exception if the socket has been closed, which
contradicts the (updated) documentation and the RI. It now returns
null. Added tests.
Fixed a bug in Socket.close(): a closed socket would still report as
being connected. Added tests.
Fixed a bug in Socket.startupSocket() - the socket was recording it
was being bound, but was not updating the cached local address. This
method is called during ServerSocketChannel.accept() and would cause
the socket to report its local address as being the wildcard address,
when it is not. Added a test.
Change-Id: Ibec8527e1c72597e268d23e6c1f03eb16e46cdc4
Bug: 12464155
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Support_PortManager didn't really work, so rewrite all tests
that use it.
Also, dedup OldDatagramSocketTest with the harmony DatagramSocketTest.
Fix various broken test cases and terribly written tests.
bug: 11689863
Change-Id: I4efb9e02eb88517273fff50a0dec1d0262feafb2
|
|
|
|
|
|
|
|
|
|
| |
We explicitly disallow setLevel or setStrategy after
setInput. (It's clearly documented in the method docs).
Presumably, this is to prevent users from shooting
themselves in the foot & causing issues by changing
the strategy / level after we've started deflating.
Change-Id: Ifd65728ff0ec7e47aeec716aeb6f44242573c491
|
|
|
|
| |
Change-Id: I15c25ff373b1ca4acb5e530f75d028b971779549
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- We now guarantee that available() returns 0 iff.
deflater.finished() == true. Note that this is still
inconsistent with the specification of InputStream#available.
- Remove an unnecessary array copy in DeflaterInputStream#read.
- Remove tests that expect an IllegalArgumentException for
negative skip count values. The InputStream spec doesn't mandate
that we must throw here, and it's better to be consistent with
every othem InputStream specialization in the code.
- Remove obsolete failures from brokentests.txt.
bug: 12189307
Change-Id: Ie9b1ad5d7c050b005b5f6f44a5a283bfdaeb1e81
|
|
|
|
|
|
|
| |
Also moves a couple of tests from luni/src/test to
harmony-tests/ where they belong.
Change-Id: I0b441b1fcbd355bd3c66551d16732a671fb64f54
|
|
|
|
|
|
| |
Several failures have already been fixed.
Change-Id: I2f372ec27c8b283ac5afaf3dcdcdc23ade07b4c7
|
|\
| |
| |
| |
| |
| |
| | |
(Take 2)"
* commit 'cfeab7b3523347cdfa1ce1f7fffc922f2cc0ab65':
Move tests from tests/api to harmony-tests. (Take 2)
|
| |
| |
| |
| |
| |
| |
| | |
This reverts commit d5e281743e55e1daa297fd8043b2d9b6e8e1bed2.
The makefile issue that caused the original build break
has been resolved.
|
|\ \
| |/
| |
| |
| |
| |
| | |
harmony-tests.""
* commit '24053344dd64bef9f2ed1c1e7577b48352396688':
Revert "Move tests from tests/api to harmony-tests."
|
| |
| |
| |
| |
| |
| |
| |
| | |
This reverts commit 81bf28ad31131815d0a36a43a0eca3c29aefdfcd.
Breaks build
Change-Id: I3f1562921ffe8fdbec36971dd65db398c27c92db
|
|\ \
| |/
| |
| |
| | |
* commit '15e00b6ef3092d5fe6c94e328aa6da210c941d1d':
Move tests from tests/api to harmony-tests.
|
| |
| |
| |
| |
| |
| |
| | |
The vast majority of cleaned up harmony tests
are now in harmony-tests/src/test/org/apache/harmony/tests/
Change-Id: I56c2e484ff434b5618cf6751d602ae9f0db96b30
|
|\ \
| |/
| |
| |
| |
| |
| | |
tests."
* commit '643f400d45af3ea66a5481c20be33dc09c457999':
Use a consistent package path for harmony tests.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Harmony was never consistent about where they put their
tests. There's no reason we have to be the same.
Also, there's no need to have separate subtrees for nio,
nio_char etc. etc.
This change makes all existing apache harmony tests
subpackages of "org.apache.harmony.tests" and put them under
libcore/harmony-tests.
We somehow managed to end up with two vastly different
copies of CharsetDecoderTest & CharsetEncoderTest. I've
renamed the copies CharsetDecoder2Test & CharsetEncoder2Test
to avoid having to go in and resolve the complicated
diffs or speculate about the reasons for their
divergence.
Change-Id: Ic34e69c2faab8893edd54e05eccd7091f4a09abd
|
|\ \
| |/
| |
| |
| |
| |
| | |
failures list."
* commit 'ce055efbc8285b6ecfbbaee39f3b13fbb0b735bb':
Add remaining harmony failures to the known failures list.
|
| |
| |
| |
| |
| |
| |
| | |
I've erred on the side of caution and treated possibly
broken tests as failures.
Change-Id: Iebb49e5cee11f6e514da70e6c6bc7ac97964b0aa
|
|\ \
| |/
| |
| |
| |
| |
| | |
list."
* commit '3dd7d89465d0c21f4216e614d5eb75d7dba1d8dc':
Add a harmony test case to known failures list.
|
| |
| |
| |
| |
| |
| | |
bug: 11650799
Change-Id: If1df4f8de4cdbf827d6159aa9ba29fe19d207f66
|
|/
|
|
|
|
| |
(cherry picked from commit 1aab8425f2c1eafbbdb2ead2632177d7ddee658b)
Change-Id: I676705ab19fffa526473f3546583b0c4e1c5a9e0
|
|
|
|
|
|
|
|
| |
Allow them only before we've started parsing any
XML elements.
bug: 3452274
Change-Id: I7b146e4dae8c7aa23a0a227538ec02604f828400
|
|
|
|
|
|
| |
Bug: 10837242
Bug: 10729779
Change-Id: I014f8646b8782973c13a3a327b4fba8b90e8d0cb
|
|
|
|
|
|
|
| |
failures.
Bug: 10666742
Change-Id: I3f93546f814c5b493ad3738eebf8e792f661970c
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There were plenty of bad tests here, but there were some real bugs too.
* DecimalFormat was only handling RoundingMode.UNNECESSARY for double
formatting.
* DecimalFormat was not ensuring that it's superclass' fields were
being correctly updated.
* NumberFormat was throwing NPE for a null object because of an
improved detail message, despite being specified to throw IAE.
* We weren't mapping NumberFormat.Field instances to the corresponding icu4c
UNUM_x_FIELD constant, so we weren't actually setting FieldPosition
objects correctly.
* SimpleDateFormat was not formatting milliseconds correctly with 'S'.
* NativeDecimalFormat wasn't handling JNI NewString OOME correctly.
Bug: 2528220
Bug: 3056865
Bug: 3057080
Bug: 3057090
Change-Id: Iac11f902f2e9649e596e7e7b7bc501b13e956fca
|
|
|
|
| |
Change-Id: Ia9129ccf417ccef18cae468cf5e497fbe003fd66
|
|
|
|
|
| |
Bug: 10210999
Change-Id: Ie0686dae1e0b9dc016c1e99af8d9d4e146d73abd
|
|
|
|
|
|
|
| |
More obsolete stuff.
Bug: 10210892
Change-Id: I4a89a2439e38a173971a5b878430cb547d70871d
|
|
|
|
|
| |
Bug: 10127357
Change-Id: I202e77d0ea0c8b407aca364ad81c8ae1f9c617ad
|
|
|
|
|
|
|
| |
org.apache.harmony.luni.tests.java.util.ScannerParseLargeFileBenchmarkTest#testParseLargeFile.
Bug: 10127357
Change-Id: I559a69457f92e159f8779dfc50fd3a45381281f5
|
|
|
|
|
| |
Bug: 3032697
Change-Id: Idbf92cded4a7c55b97ee558b0fa844237d262206
|
|
|
|
| |
Change-Id: I7c90b08ee2b43c73e02512d0c6caaec098967e93
|
|
|
|
|
|
|
| |
Plus one real bug, in FileChannelImpl. The expectation I've removed was for a
test that doesn't seem to exist in either our tests or the harmony tests.
Change-Id: I1ea99042f3f8897f07ba8b4ad8e9a15f30ace79b
|
|
|
|
|
|
|
|
|
|
|
| |
Finding safe primes for the DH key pair generator can take up to 20
minutes sometimes. Use a pre-defined set of primes instead of skipping
the test altogether because of the extreme amount of time it takes.
This leaves us not testing one software path, but this situation is
strictly better than not running the test at all.
Change-Id: I03d457747e2ec1a2592c1c6350365cbe5aa33759
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Add hidden API to pass along the EC curve name in ECParameterSpec.
The lack of name passing made KeyFactory2Test fail because the
reconstructed ECDSA key had explicit curve parameters instead of an
OID naming the curve.
* Fix some mixing of PKCS8/X509EncodedKeySpec in DSA/ECDSA KeyFactory
implementations.
* Fix the KeyFactory2Test to output more useful error messages.
* Remove known failure which is no longer happening.
* Change EC_GROUP_get_curve_name to return the "shortName" string
which matches the EC_GROUP_new_by_curve_name
Bug: 3483365
Change-Id: I0a80be88bef728b2177f3593cc3421fa47b79470
|
|/
|
|
|
| |
Bug: 3387688
Change-Id: I620ae46fcf3681120c958dc847eabc6138144735
|
|
|
|
| |
Change-Id: I469766e0f8b9aecefa72a558afa9f5c226a99d33
|