[TASK] Update localization rendering tests to newest Testing Framework 60/54060/2
authorTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 9 Sep 2017 10:20:18 +0000 (12:20 +0200)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 9 Sep 2017 12:19:05 +0000 (14:19 +0200)
addTypoScriptToTemplateRecord method was added to FunctionalTestCase.php
in Testing Framework.
So we can remove it from test cases.
Fix two other gremlins also (unneeded comment and unused variable).

Resolves: #82401
Releases: master, 8.7, 7.6
Change-Id: I8144bae5a807d60123ef5228a44eba579a7705c7
Reviewed-on: https://review.typo3.org/54060
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
typo3/sysext/core/Classes/Tests/FunctionalTestCase.php
typo3/sysext/extbase/Tests/Functional/Persistence/TranslatedContentTest.php
typo3/sysext/frontend/Tests/Functional/Rendering/LocalizedContentRenderingTest.php

index 4253fc0..ca69fb7 100644 (file)
@@ -594,6 +594,28 @@ abstract class FunctionalTestCase extends BaseTestCase
     }
 
     /**
+     * Adds TypoScript setup snippet to the existing template record
+     *
+     * @param int $pageId
+     * @param string $typoScript
+     */
+    protected function addTypoScriptToTemplateRecord($pageId, $typoScript)
+    {
+        $connection = $this->getDatabaseConnection();
+
+        $template = $connection->exec_SELECTgetSingleRow('*', 'sys_template', 'pid = '. $pageId . ' AND root = 1');
+        if (empty($template)) {
+            $this->fail('Cannot find root template on page with id: "' . $pageId . '"');
+        }
+        $updateFields['config'] = $template['config'] . LF . $typoScript;
+        $connection->exec_UPDATEquery(
+            'sys_template',
+            'uid = ' . $template['uid'],
+            $updateFields
+        );
+    }
+
+    /**
      * @param int $pageId
      * @param int $languageId
      * @param int $backendUserId
index ef0edd5..0db77ee 100644 (file)
@@ -15,9 +15,7 @@ namespace TYPO3\CMS\Extbase\Tests\Functional\Persistence;
  * The TYPO3 project - inspiring people to share!
  */
 
-use TYPO3\CMS\Core\Database\ConnectionPool;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
-use TYPO3\TestingFramework\Core\Functional\Framework\Frontend\Response;
 
 /**
  * This test is an Extbase version of the \TYPO3\CMS\Frontend\Tests\Functional\Rendering\LocalizedContentRenderingTest
@@ -878,7 +876,6 @@ class TranslatedContentTest extends \TYPO3\CMS\Core\Tests\Functional\DataHandlin
             [
                 'typoScript' => 'config.sys_language_overlay = hideNonTranslated
                                 config.sys_language_mode = content_fallback;1,0',
-//                'visibleRecordHeaders' => ['[Translate to Dansk:] Regular Element #1', '[Translate to Dansk:] Regular Element #3', 'Regular Element #2', '[DK] Without default language', '[DK] UnHidden Element #4'],
                 'visibleRecords' => [
                     297 => [
                         'header' => '[Translate to Dansk:] Regular Element #1',
@@ -1155,26 +1152,4 @@ class TranslatedContentTest extends \TYPO3\CMS\Core\Tests\Functional\DataHandlin
         ];
         return array_diff($allElements, $visibleTitles);
     }
-
-    /**
-     * Adds TypoScript setup snippet to the existing template record
-     *
-     * @param int $pageId
-     * @param string $typoScript
-     */
-    protected function addTypoScriptToTemplateRecord($pageId, $typoScript)
-    {
-        $connection = $this->getDatabaseConnection();
-
-        $template = $connection->exec_SELECTgetSingleRow('*', 'sys_template', 'pid = '. $pageId . ' AND root = 1');
-        if (empty($template)) {
-            $this->fail('Cannot find root template on page with id: "' . $pageId . '"');
-        }
-        $updateFields['config'] = $template['config'] . LF . $typoScript;
-        $connection->exec_UPDATEquery(
-            'sys_template',
-            'uid = ' . $template['uid'],
-            $updateFields
-        );
-    }
 }
index 7803961..fbccb00 100644 (file)
@@ -15,7 +15,6 @@ namespace TYPO3\CMS\Frontend\Tests\Functional\Rendering;
  * The TYPO3 project - inspiring people to share!
  */
 
-use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Core\Tests\Functional\Framework\Frontend\Response;
 
 /**
@@ -1224,26 +1223,4 @@ class LocalizedContentRenderingTest extends \TYPO3\CMS\Core\Tests\Functional\Dat
         ];
         return array_diff($allElements, $visibleTitles);
     }
-
-    /**
-     * Adds TypoScript setup snippet to the existing template record
-     *
-     * @param int $pageId
-     * @param string $typoScript
-     */
-    protected function addTypoScriptToTemplateRecord($pageId, $typoScript)
-    {
-        $connection = $this->getDatabaseConnection();
-
-        $template = $connection->exec_SELECTgetSingleRow('*', 'sys_template', 'pid = '. $pageId . ' AND root = 1');
-        if (empty($template)) {
-            $this->fail('Cannot find root template on page with id: "' . $pageId . '"');
-        }
-        $updateFields['config'] = $template['config'] . LF . $typoScript;
-        $connection->exec_UPDATEquery(
-            'sys_template',
-            'uid = ' . $template['uid'],
-            $updateFields
-        );
-    }
 }