[TASK] Switch to new unit testing strategy #752
Conversation
| ->loadConfigurationAndInitialize(FALSE) | ||
| ->loadTypo3LoadedExtAndExtLocalconf(FALSE) | ||
| ->initializeLanguageObject() | ||
| ; |
There was a problem hiding this comment.
Space found before semicolon; expected ");" but found ")
;"
|
Note: failed because of bad timezone setting on build server. The warnings that are dispatched should be okay to ignore; when merged to |
|
Other note: a few |
This is an automated comment based on an automated formal commit review.Your pull request contains formal errors. Comments have been assigned to each commit in your pull request - please review and adjust. Feel free to ask for help if you need it! |
NamelessCoder
force-pushed
the
workbench/unittestdecouple
branch
from
6e0ce39
to
4f726fc
Compare
| ->loadConfigurationAndInitialize(FALSE) | ||
| ->loadTypo3LoadedExtAndExtLocalconf(FALSE) | ||
| ->initializeLanguageObject() | ||
| ; |
There was a problem hiding this comment.
Space found before semicolon; expected ");" but found ")
;"
This is an automated comment based on an automated formal commit review.Your pull request contains formal errors. Comments have been assigned to each commit in your pull request - please review and adjust. Feel free to ask for help if you need it! |
| @@ -55,6 +57,7 @@ public function getRenderTestValues() { | |||
| array(array('content' => '', 'alternative' => 'alternative'), 'alternative'), | |||
| array(array('content' => NULL, 'alternative' => 'alternative'), 'alternative'), | |||
| array(array('content' => 0, 'alternative' => 'alternative'), 'alternative'), | |||
| /* | |||
There was a problem hiding this comment.
Why comment out something instead of removing it.
[TASK] Switch to new unit testing strategy
You do not need to review the entire thing. Just make sure that our metadata files are okay and that this patch works on your machine ;)