| 85e8e4c142e5299fccbe2b9b6f844290ff39c4d2 |
|
17-Feb-2014 |
Narayan Kamath <narayan@google.com> |
DO NOT MERGE Fix PropertiesTest.
A test case was using HashTable.get instead of
Properties.getProperty.
The problem is that Properties.propertyNames will
enumerate all properties from the object as well as
from the "default" Properties object it was constructed
with. The get method derived from HashTable isn't
aware of default properties though, so it will fail.
Using composition instead of inheritance would've
resulted in a cleaner design.
(cherry picked from commit 79395265fdd07388849b16695c437583415b7a6b)
Bug: 13763685
Bug: 13763900
Change-Id: I58323767e6872ed1722167d2a4a6d89a98d26ec7
|
| 79395265fdd07388849b16695c437583415b7a6b |
|
17-Feb-2014 |
Narayan Kamath <narayan@google.com> |
Fix PropertiesTest.
A test case was using HashTable.get instead of
Properties.getProperty.
The problem is that Properties.propertyNames will
enumerate all properties from the object as well as
from the "default" Properties object it was constructed
with. The get method derived from HashTable isn't
aware of default properties though, so it will fail.
Using composition instead of inheritance would've
resulted in a cleaner design.
Change-Id: I58323767e6872ed1722167d2a4a6d89a98d26ec7
|
| ab762bb740405d0fefcccf4a0899a234f995be13 |
|
15-Nov-2013 |
Narayan Kamath <narayan@google.com> |
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.
|
| d5e281743e55e1daa297fd8043b2d9b6e8e1bed2 |
|
15-Nov-2013 |
Narayan Kamath <narayan@google.com> |
Revert "Move tests from tests/api to harmony-tests."
This reverts commit 81bf28ad31131815d0a36a43a0eca3c29aefdfcd.
Breaks build
Change-Id: I3f1562921ffe8fdbec36971dd65db398c27c92db
|
| 81bf28ad31131815d0a36a43a0eca3c29aefdfcd |
|
15-Nov-2013 |
Narayan Kamath <narayan@google.com> |
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
|