Bug #64055: PHPUnit-Tests fail on Windows (2) - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #64055

closed

PHPUnit-Tests fail on Windows (2)

Added by Stephan Großberndt over 9 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Target version:

next-patchlevel

Start date:

2014-12-29

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

medium

Is Regression:

Sprint Focus:

Description

Another test failing on windows (similar to #63943)

Fatal error: Cannot redeclare class TYPO3\CMS\Backend\Tests\Unit\Utility\BackendUtility54a1a650122b7 in D:\typo3\master\typo3\sysext\backend\Tests\Unit\Utility\BackendUtilityTest.php(242) : eval()'d code on line 1

This is due to

    public function getProcessedValueForGroupWithMultipleAllowedTables() {
        // Disable getRecordWSOL and getRecordTitle dependency by returning stable results
        /** @var \PHPUnit_Framework_MockObject_MockObject|\TYPO3\CMS\Backend\Utility\BackendUtility $subject */
        $className = uniqid('BackendUtility');
        $subject = __NAMESPACE__ . '\\' . $className;

which has to be replaced with

$className = 'BackendUtility' . md5(uniqid(mt_rand(), TRUE));

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35665

Actions

Copy link

Updated by Stephan Großberndt over 9 years ago

Status changed from Under Review to Resolved
% Done changed from 0 to 100

Applied in changeset 61da9595156e23b849f302502f6013dd1bd31e55.

Actions

Copy link

#10

Updated by Gerrit Code Review over 9 years ago

Status changed from Resolved to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36110

Actions

Copy link

#11

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36133

Actions

Copy link

#12

Updated by Stephan Großberndt over 9 years ago

Complexity changed from no-brainer to medium

It turned out that there is a inconsistent usage of uniqid() all over the core. A single implementation of getUniqueId() in BaseTestCase will be called for all usages of uniqid() in tests.

Actions

Copy link

#13