Feeling Lucky
?
?
?
[#33260] - [4] Add unit test for ModuleAdapter::getElement
- Fixed in Code Base
- 23 Apr 2021
- Medium
- Build: 4.0-dev
- # 33260
- Diff
- PhilETaylor:unittest-ModuleAdapter-getElement
User tests: Successful: Unsuccessful:
Summary of Changes
Adds a unit test for ModuleAdapter::getElement that was useful when refactoring the code in #33182
Testing Instructions
The tests pass before refactoring
Apply PR: #33182
The tests pass after refactoring
Actual result BEFORE applying this Pull Request
The tests pass before refactoring
Expected result AFTER applying this Pull Request
The tests pass after refactoring
Documentation Changes Required
dgrammatiko
- comment
- 23 Apr 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33260.
| Category | ⇒ | Unit Tests |
| Status | New | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
richard67
- comment
- 23 Apr 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33260.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-23 19:29:16 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
wilsonge
- comment
- 23 Apr 2021
Thankyou!
I have tested this item✅ successfully on e49f156
New unit tests are run and pass when the other PR is applied and this one, too. I've run them on my local environment.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33260.