Project

General

Profile

Actions

Task #83889

closed

Task #83888: Notice free unit tests

Handle E_NOTICE free unit tests

Added by Christian Kuhn almost 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2018-02-13
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The main patch raises typo3/testing-framework to 3.0.0 which comes with a version that no longer suppresses E_NOTICE PHP errors by default and lets tests fail that raises them.

The static property $suppressNotices in unit test files is set to true for all test cases that currently raise E_NOTICE errors to suppress them on a by-file basis.

Following patches should remove this property one-by-one again by fixing the system under test the unit test case handles.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Epic #84280: Make unit tests notice freeClosedJan Helke2018-03-15

Actions
Actions #1

Updated by Gerrit Code Review almost 7 years ago

  • Status changed from New to Under Review

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

Actions #2

Updated by Gerrit Code Review almost 7 years ago

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

Actions #3

Updated by Gerrit Code Review almost 7 years ago

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

Actions #4

Updated by Gerrit Code Review almost 7 years ago

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

Actions #5

Updated by Christian Kuhn almost 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #6

Updated by Christian Kuhn over 6 years ago

  • Related to Epic #84280: Make unit tests notice free added
Actions #7

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF