Closed
Bug 938820
Opened 11 years ago
Closed 11 years ago
Remove reflection from BaseTest
Categories
(Firefox for Android Graveyard :: Testing, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 969922
People
(Reporter: mcomella, Unassigned)
References
Details
Reflection is used for the super class constructor of BaseTest (ActivityInstrumentationTestCase2).
Reporter | ||
Updated•11 years ago
|
Component: General → Testing
Reporter | ||
Comment 1•11 years ago
|
||
This also applies to UITest once bug 910859 lands.
Reporter | ||
Comment 2•11 years ago
|
||
sLauncherActivityClass should be made a member variable too (if possible).
Comment 3•11 years ago
|
||
Here's a try build that might achieve this:
https://tbpl.mozilla.org/?tree=Try&rev=48c4ace2b5ab
Reporter | ||
Comment 4•11 years ago
|
||
The patches in bug 969922 (which are contained in the try build in comment 3) has resolved this issue.
Nice work, Nick. :)
There is still the reflection to obtain an instance of the class we're testing (i.e. `Class.forName(PreproccesedConstant)`), but considering we did not remove this from UITest, I take it it's a challenge. If you think it's worth trying, file a new bug for its removal from both BaseTest and UITest.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•