Epic #83888

Notice free unit tests

Added by Christian Kuhn 9 days ago. Updated 2 days ago.

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

100%

Sprint Focus:

Subtasks

Task #83889: Handle E_NOTICE free unit testsResolved

Task #83908: Notice free InlineStackProcessor testingResolved

Task #83975: Notice free TcaSelectItems testingResolved

Bug #83981: Notice free FormEngine testingResolved

Associated revisions

Revision e514f439 (diff)
Added by Christian Kuhn 9 days ago

[!!!][TASK] Handle E_NOTICE free unit tests

Raise typo3/testing-framework from 2.0.4 to 3.0.0:
composer require --dev typo3/testing-framework ^3.0

The new typo3/testing-framework version comes with a change
that no longer suppresses E_NOTICE PHP errors by default and lets
tests fail that raise 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.

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

Change-Id: I7d44fe3723ce5820a37051ae31f16b2f142ffba6
Resolves: #83889
Related: #83888
Releases: master
Reviewed-on: https://review.typo3.org/55697
Reviewed-by: Tymoteusz Motylewski <>
Tested-by: Tymoteusz Motylewski <>
Tested-by: TYPO3com <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review 2 days 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 https://review.typo3.org/55834

Also available in: Atom PDF