-
Notifications
You must be signed in to change notification settings - Fork 28
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
Beginning to deepcopy workspace #520
Conversation
@erikssonpatrick This is what you asked for: deep copies of the workspace. I only tested it on some very simple stuff though, as I as of now have no use for this so no test case set up. |
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.
At least a minimal test cases should be included. Nothing fancy, but at least something that verifies the intended behaviour. Something like:
- Create a workspace
- Set one variable to a known value
- Make a deep-copy
- Change the variable value in the original workspace
- Verify that the variable in the copied workspace still has the correct, original value
Just a clarification. I asked about this, but that was under the assumption that the feature existed. I got that impression from some discussion. |
@olemke This now has the tests you requested. |
No description provided.