Bug #44765
Functional test broken
Status: | Resolved | Start date: | 2013-01-23 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Karsten Dambekalns | % Done: | 100% |
|
Category: | Security | |||
Target version: | TYPO3 Flow Base Distribution - 2.0 | |||
PHP Version: | Complexity: | |||
Has patch: | No | Affected Flow version: | Flow 2.0.0 beta 1 |
Description
If you run the functional tests, you'll get the following exception thrown.
TYPO3\Flow\Security\Exception: The security Context cannot be initialized yet. Please check if it can be initialized with $securityContext->canBeInitialized() before trying to do so.
This is a side effect of the following commit:
commit ce08c301e116f3e775657a977c3f1bc3ee4fcad3 Author: Sebastian Kurfuerst <sebastian@typo3.org> Date: Thu Jan 17 17:57:44 2013 +0100 [BUGFIX]Â The security context is only allowed to be initialized after routing took place ... This is a follow-up to issue #42601; where the according code has been implemented. Change-Id: I724c1b352dd1807ba53b1e336f2d90e90360ff4d Releases: master, 2.0
Related issues
Associated revisions
[BUGFIX] Fix security-related functional test failures
The change I724c1b352dd1807ba53b1e336f2d90e90360ff4d introduced some
test failures. This change takes care of the failing functional tests.
It does that by:
- changing the order in which security is set up in the
FunctionalTestCase provided by Flow. - setting the "current request" again after a call to clearContext() in
ContentSecurityTest - adjusting the expected exception in MethodSecurityTest in two places
Change-Id: I353e2cba11473cf9ddef82f96b6a79d9d6fefbba
Fixes: #44765
Related: #42601
Releases: master
[BUGFIX] Fix security-related functional test failures, part 2
The change I724c1b352dd1807ba53b1e336f2d90e90360ff4d introduced some
test failures. This change takes care of the failing functional tests.
It does that by:
- setting the "current request" again after a call to clearContext() in
ContentSecurityTest - adjusting the expected exception in MethodSecurityTest in two places
This is a followup to I353e2cba11473cf9ddef82f96b6a79d9d6fefbba which
was broken after having fixed those already.
Change-Id: I09182972baf668abbb2cf583e8deebdc31e90205
Related: #42601
Fixes: #44765
Releases: master
[BUGFIX] Fix security-related functional test failures
The change I724c1b352dd1807ba53b1e336f2d90e90360ff4d introduced some
test failures. This change takes care of the failing functional tests.
It does that by:
- changing the order in which security is set up in the
FunctionalTestCase provided by Flow. - setting the "current request" again after a call to clearContext() in
ContentSecurityTest - adjusting the expected exception in MethodSecurityTest in two places
Change-Id: I353e2cba11473cf9ddef82f96b6a79d9d6fefbba
Fixes: #44765
Related: #42601
Releases: master, 2.0
[BUGFIX] Fix security-related functional test failures, part 2
The change I724c1b352dd1807ba53b1e336f2d90e90360ff4d introduced some
test failures. This change takes care of the failing functional tests.
It does that by:
- setting the "current request" again after a call to clearContext() in
ContentSecurityTest - adjusting the expected exception in MethodSecurityTest in two places
This is a followup to I353e2cba11473cf9ddef82f96b6a79d9d6fefbba which
was broken after having fixed those already.
Change-Id: I09182972baf668abbb2cf583e8deebdc31e90205
Related: #42601
Fixes: #44765
Releases: master, 2.0
History
#1 Updated by Karsten Dambekalns over 2 years ago
- Status changed from New to Under Review
- Assigned To set to Karsten Dambekalns
- Target version set to 2.0
#2 Updated by Gerrit Code Review over 2 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17669
#3 Updated by Gerrit Code Review over 2 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17669
#4 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 55a312f28997fc4a06914ba0279db4ae57219de4.
#5 Updated by Gerrit Code Review over 2 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17764
#6 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
Applied in changeset 56c6d852cdc7bc3faea8070708be6cd68bfc5cb9.
#7 Updated by Gerrit Code Review over 2 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19103
#8 Updated by Gerrit Code Review over 2 years ago
Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19104
#9 Updated by Gerrit Code Review over 2 years ago
Patch set 2 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19103
#10 Updated by Gerrit Code Review over 2 years ago
Patch set 2 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19104
#11 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
Applied in changeset 5612a5619107d5e40728264a19a0fb02bad460ec.