Feeling Lucky
?
?
[#7569] - Add unit tests for JInstallerAdapter
- Closed
- 8 Sep 2015
- Medium
- Build: staging
- # 7569
- Diff
- wilsonge:installertests
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This gives us code coverage of ~75% of the lines in the class (15/23 methods in the class)
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Labels |
Added:
?
|
||
wilsonge
- comment
- 28 Jul 2015
Done everything I think
mbabker
- comment
- 28 Jul 2015
Well, the question was answered on the database thing. Guess it needs to extend TestCaseDatabase.
wilsonge
- comment
- 28 Jul 2015
I'm going to try mocking the JFactory::$database in the constructor. That might be enough. If not I'll revert it back to the way it was
brianteeman
- | Category | ⇒ | Unit Tests |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-09-08 23:33:38 |
| Closed_By | ⇒ | mbabker |
zero-24
- | Milestone |
Added: |
||
| Milestone | |||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone | |||
Can you put
@coverstags on all the test methods please?