Closed Bug 938820 Opened 11 years ago Closed 11 years ago

Remove reflection from BaseTest

Categories

(Firefox for Android Graveyard :: Testing, defect)

All
Android
defect
Not set
normal

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).
sLauncherActivityClass should be made a member variable too (if possible).
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
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.