Feeling Lucky
?
?
[#13412] - [Unit Test CS] Class instanciation
- Closed
- 31 Dec 2016
- Medium
- Build: staging
- # 13412
- Diff
- wilsonge:ut-class-instanciation
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This is the third of a set of PR's that's hopefully going to result in us using the code sniffer on the unit tests folder as well (so far ignored because it's a nightmare effort). This fixes issues where we have brackets for classes with no params
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Unit Tests |
jeckodevelopment
- | Milestone |
Added: |
||
wilsonge
- comment
- 31 Dec 2016
Go for it :)
wilsonge
- comment
- 31 Dec 2016
I'm closing this PR though in favour of another I'm about to do
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-31 11:56:06 |
| Closed_By | ⇒ | wilsonge |
Applying the code standards to the unit tests Sounds like a perfect test case for using the PHPCS 2.x auto fixers
joomla/coding-standards#109