[TASK] Fix duplicated array keys in tests 41/28641/2
authorTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 22 Mar 2014 13:47:24 +0000 (14:47 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sat, 22 Mar 2014 16:36:59 +0000 (17:36 +0100)
In few tests dataproviders have duplicated array keys,
which results in some cases not being tested.
This patch removes duplicated entries and changes array
key names where needed.

Resolves: #57180
Releases: 6.2
Change-Id: I68ea38f89ed7f304b57165ca1d225ffdbe2a7a15
Reviewed-on: https://review.typo3.org/28641
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
typo3/sysext/core/Tests/Unit/DataHandling/DataHandlerTest.php
typo3/sysext/core/Tests/Unit/Html/RteHtmlParserTest.php
typo3/sysext/core/Tests/Unit/Resource/Collection/FileCollectionRegistryTest.php
typo3/sysext/extbase/Tests/Unit/Persistence/Generic/Storage/Typo3DbQueryParserTest.php
typo3/sysext/frontend/Tests/Unit/Page/CacheHashCalculatorTest.php
typo3/sysext/scheduler/Tests/Unit/CronCommand/NormalizeCommandTest.php

index 3f94eb6..4ef0ff1 100644 (file)
@@ -568,10 +568,6 @@ class DataHandlerTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                                FALSE,
                                '0', NULL, 'int', TRUE
                        ),
-                       'integer with allowed null value null vs. null' => array(
-                               TRUE,
-                               NULL, NULL, 'int', TRUE
-                       ),
                );
        }
 
index a074a4d..3f8e7ed 100644 (file)
@@ -271,9 +271,9 @@ class RteHtmlParserTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                                '<p>paragraph1</p>' . CRLF . '<p>paragraph2</p>',
                                'paragraph1' . CRLF . 'paragraph2',
                        ),
-                       'Double spacing paragraph' => array(
+                       'Double spacing paragraph with text' => array(
                                '<p>&nbsp;</p><p>&nbsp;</p><p>paragraph1</p>',
-                               CRLF . CRLF . paragraph1,
+                               CRLF . CRLF . 'paragraph1',
                        ),
                        'Paragraph followed by linebreak' => array(
                                '<p>paragraph</p>' . CRLF,
@@ -295,14 +295,6 @@ class RteHtmlParserTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                                '<p>paragraph</p>' . CRLF . '<p>&nbsp;</p>' . CRLF . '<p>&nbsp;</p>',
                                'paragraph' . CRLF . CRLF . CRLF,
                        ),
-                       'Paragraph followed by paragraph' => array(
-                               '<p>paragraph1</p>' . '<p>paragraph2</p>',
-                               'paragraph1' . CRLF . 'paragraph2',
-                       ),
-                       'Paragraph followed by paragraph, linebreak-separated' => array(
-                               '<p>paragraph1</p>' . CRLF . '<p>paragraph2</p>',
-                               'paragraph1' . CRLF . 'paragraph2',
-                       ),
                        'Paragraph followed by spacing paragraph and by paragraph' => array(
                                '<p>paragraph1</p>' . '<p>&nbsp;</p>' . '<p>paragraph2</p>',
                                'paragraph1' . CRLF . CRLF . 'paragraph2',
index 7ccee56..734bfd4 100644 (file)
@@ -164,7 +164,6 @@ class FileCollectionRegistryTest extends \TYPO3\CMS\Core\Tests\Unit\Resource\Bas
                        'columns' => array(
                                'type' => array(
                                        'config' => array(
-                                               'items' => array('Type A', 'typeA'),
                                                'items' => array('Type B', 'typeB')
                                        )
                                )
index a5114c2..7bccefe 100644 (file)
@@ -292,8 +292,8 @@ class Typo3DbQueryParserTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                return array(
                        'in be: respectEnableFields=false' => array('BE', FALSE, NULL),
                        'in be: respectEnableFields=true' => array('BE', TRUE, array('tx_foo_table.disabled_column=0 AND (tx_foo_table.starttime_column<=123456789) AND tx_foo_table.deleted_column=0')),
-                       'in be: respectEnableFields=false' => array('FE', FALSE, NULL),
-                       'in be: respectEnableFields=true' => array('FE', TRUE, array('tx_foo_table.deleted_column=0 AND tx_foo_table.disabled_column=0 AND tx_foo_table.starttime_column<=123456789'))
+                       'in FE: respectEnableFields=false' => array('FE', FALSE, NULL),
+                       'in FE: respectEnableFields=true' => array('FE', TRUE, array('tx_foo_table.deleted_column=0 AND tx_foo_table.disabled_column=0 AND tx_foo_table.starttime_column<=123456789'))
                );
        }
 
index 64c1bec..f43f6c6 100644 (file)
@@ -118,7 +118,7 @@ class CacheHashCalculatorTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                                'id=1&type=3&exclude1=x&no_cache=1',
                                array()
                        ),
-                       'System and exclude parameters should be omitted' => array(
+                       'System and exclude parameters should be omitted, others should stay' => array(
                                'id=1&type=3&key=x&no_cache=1',
                                array('encryptionKey', 'key')
                        )
@@ -144,7 +144,7 @@ class CacheHashCalculatorTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                        'Querystring has only system parameters so we should not have a cacheHash' => array('id=1&type=val', ''),
                        'Trivial key value combination should generate hash' => array('&key=value', $knowHash),
                        'Only the relevant parts should be taken into account' => array('&key=value&exclude1=val', $knowHash),
-                       'Only the relevant parts should be taken into account' => array('&exclude2=val&key=value', $knowHash),
+                       'Only the relevant parts should be taken into account(exclude2 before key)' => array('&exclude2=val&key=value', $knowHash),
                        'System parameters should not be taken into account' => array('&id=1&key=value', $knowHash),
                        'Admin panel parameters should not be taken into account' => array('&TSFE_ADMIN_PANEL[display]=7&key=value', $knowHash),
                        'Trivial hash for sorted parameters should be right' => array('a=v&b=v', '0f40b089cdad149aea99e9bf4badaa93'),
@@ -168,8 +168,8 @@ class CacheHashCalculatorTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                        'Empty parameter strings should not require anything.' => array('', FALSE),
                        'Normal parameters aren\'t required.' => array('key=value', FALSE),
                        'Configured "req1" to be required.' => array('req1=value', TRUE),
-                       'Configured "req1" to be requiredm, should also work in combined context' => array('&key=value&req1=value', TRUE),
-                       'Configured "req1" to be requiredm, should also work in combined context' => array('req1=value&key=value', TRUE)
+                       'Configured "req1" to be required, should also work in combined context' => array('&key=value&req1=value', TRUE),
+                       'Configured "req1" to be required, should also work in combined context (key at the end)' => array('req1=value&key=value', TRUE)
                );
        }
 
@@ -222,12 +222,12 @@ class CacheHashCalculatorTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                                array('excludedParametersIfEmpty' => array(), 'excludeAllEmptyParameters' => FALSE),
                                array('encryptionKey', 'key1', 'key2', 'key3')
                        ),
-                       'Due to the empty value, "key2" should be skipped' => array(
+                       'Due to the empty value, "key2" should be skipped(with equals sign' => array(
                                'key1=v&key2=&key3=',
                                array('excludedParametersIfEmpty' => array('key2'), 'excludeAllEmptyParameters' => FALSE),
                                array('encryptionKey', 'key1', 'key3')
                        ),
-                       'Due to the empty value, "key2" should be skipped' => array(
+                       'Due to the empty value, "key2" should be skipped(without equals sign)' => array(
                                'key1=v&key2&key3',
                                array('excludedParametersIfEmpty' => array('key2'), 'excludeAllEmptyParameters' => FALSE),
                                array('encryptionKey', 'key1', 'key3')
index d6c459b..89c3706 100644 (file)
@@ -184,23 +184,23 @@ class NormalizeCommandTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
         */
        static public function normalizeMonthAndWeekdayFieldValidDataProvider() {
                return array(
-                       '*' => array('*', TRUE, '*'),
-                       'string 1' => array('1', TRUE, '1'),
+                       '* monthField' => array('*', TRUE, '*'),
+                       'string 1 monthField' => array('1', TRUE, '1'),
                        'jan' => array('jan', TRUE, '1'),
                        'feb/2' => array('feb/2', TRUE, '2'),
                        'jan-feb/2' => array('jan-feb/2', TRUE, '1'),
-                       '1-2' => array('1-2', TRUE, '1,2'),
+                       '1-2 monthField' => array('1-2', TRUE, '1,2'),
                        '1-3/2,feb,may,6' => array('1-3/2,feb,may,6', TRUE, '1,2,3,5,6'),
                        '*/4' => array('*/4', TRUE, '1,5,9'),
-                       '*' => array('*', FALSE, '*'),
-                       'string 1' => array('1', FALSE, '1'),
+                       '* !monthField' => array('*', FALSE, '*'),
+                       'string 1, !monthField' => array('1', FALSE, '1'),
                        'fri' => array('fri', FALSE, '5'),
                        'sun' => array('sun', FALSE, '7'),
                        'string 0 for sunday' => array('0', FALSE, '7'),
                        '0,1' => array('0,1', FALSE, '1,7'),
                        '*/3' => array('*/3', FALSE, '1,4,7'),
                        'tue/2' => array('tue/2', FALSE, '2'),
-                       '1-2' => array('1-2', FALSE, '1,2'),
+                       '1-2 !monthField' => array('1-2', FALSE, '1,2'),
                        'tue-fri/2' => array('tue-fri/2', FALSE, '2,4'),
                        '1-3/2,tue,fri,6' => array('1-3/2,tue,fri,6', FALSE, '1,2,3,5,6')
                );
@@ -288,7 +288,6 @@ class NormalizeCommandTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                        '4-3' => array('4-3', 0, 59),
                        '/2' => array('/2', 0, 59),
                        '/' => array('/', 0, 59),
-                       'string foo' => array('foo', 0, 59),
                        'left bound too low' => array('2-4', 3, 4),
                        'right bound too high' => array('2-4', 2, 3),
                        'left and right bound' => array('2-5', 2, 4),