-
-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUGFIX: Repair behat tests and Neos9-Beta-10 compatibility #66
Conversation
# Conflicts: # Tests/Behavior/Features/Bootstrap/FeatureContext.php
If I'm not misstaken we now need to adjust the tests to the workspace command change: neos/neos-development-collection#4708 |
Yes and that phpstan crashes the ci is also suboptimal |
setting up the sync / async mode is not necessary anymore
Migrated in a similar fashion as the DocumentUriPathProjection here: https://github.com/neos/neos-development-collection/pull/4993/files#diff-2e3df51a3340b51e08cf9d2ba5aade329a8c0e073354c1d5a72e42c3aaf5a127R618 Solves: #76
$this->getNodeAddress($event->contentStreamId, $newLocation->coveredDimensionSpacePoint, $descendantOfNode->getNodeAggregateId()), | ||
$this->contentRepository->id | ||
), iterator_to_array($descendantsOfNode)); | ||
foreach ($event->succeedingSiblingsForCoverage as $interdimensionalSibling) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Migrated in a similar fashion as the DocumentUriPathProjection here:
https://github.com/neos/neos-development-collection/pull/4993/files#diff-2e3df51a3340b51e08cf9d2ba5aade329a8c0e073354c1d5a72e42c3aaf5a127R618
This was fixed with neos/behat#39
5fa05b3
to
5886d3d
Compare
1ecdaba
to
325a10c
Compare
The way how we write behat tests has undergone major changes via
neos/behat
and the runtime helpers in Neos as well.For example we use workspace names now and dont block anymore.
Resolves: #76