-
-
Notifications
You must be signed in to change notification settings - Fork 224
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
TASK: Remove workspaceTitle
, workspaceDescription
and workspaceOwner
use from behat core tests
#5271
TASK: Remove workspaceTitle
, workspaceDescription
and workspaceOwner
use from behat core tests
#5271
Conversation
d42c041
to
adce185
Compare
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.
Marked the places that are different. Bastian and me made these changes together and i just re-reviewed them fully by diffing
git diff -U0 HEAD^ > ../myCurrent.diff
and then replacing the noise from the diff and see whats left.
@@ -73,7 +73,6 @@ Feature: Behavior of Node timestamp properties "created", "originalCreated", "la | |||
| workspaceName | "user-test" | | |||
| baseWorkspaceName | "review" | | |||
| newContentStreamId | "cs-user" | | |||
| workspaceOwner | "some-user" | |
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.
fyi workspaceOwner used ... but no effect
@@ -44,7 +44,6 @@ Feature: Workspace rebasing - conflicting changes | |||
| workspaceName | "user-test" | | |||
| baseWorkspaceName | "live" | | |||
| newContentStreamId | "user-cs-identifier" | | |||
| workspaceOwner | "owner-identifier" | |
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.
fyi workspaceOwner used ... but no effect
@@ -44,7 +44,6 @@ Feature: Workspace based content publishing | |||
| workspaceName | "user-test" | | |||
| baseWorkspaceName | "live" | | |||
| newContentStreamId | "user-cs-identifier" | | |||
| workspaceOwner | "owner-identifier" | |
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.
.
@@ -44,7 +44,6 @@ Feature: Workspace discarding - basic functionality | |||
| workspaceName | "user-test" | | |||
| baseWorkspaceName | "live" | | |||
| newContentStreamId | "user-cs-identifier" | | |||
| workspaceOwner | "owner-identifier" | |
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.
.
workspaceTitle
, workspaceDescription
and workspaceOwner
use from behat core tests
title, description and owner relation are now part of the Neos
WorkspaceMetadata
model (see #5146)Related: #4726