-
Notifications
You must be signed in to change notification settings - Fork 17
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
Remove TUIDs #131
Remove TUIDs #131
Conversation
I don't understand that failure. The build runs perfectly on my local machine with the exact same setup :-/ |
Does not look like it is an issue with your changes but rather a general thing. I get exactly the same error in #133. |
It has to do with the changes I merged in vitruv-tools/Vitruv-Domains-ComponentBasedSystems#90, so even the |
I get the error locally (macOS 11.2.1, Eclipse 12/20). However, I am currently short on time to investigate on it. Here is the stacktrace, maybe it helps. Stacktrackjava.lang.IllegalStateException: No EObject could be found for UUID: _5usbIHXaEeuu3YVhoPGQkA |
Then it seems to be an OS-specific issue. I'll check that in a VM. |
This is annoying. I even dont't have the failure on a Linux system... |
I've disabled the test for now and provide pull request #134 that reenables it. We can figure out what's going on there without preventing further development due to one failing test case. |
This PR adapts to the removal of TUIDs from the framework and domains. It removes all TUID-related logic and dependencies from the applications.
In addition, this PR disables the editor tests, as they functionality currently depends on TUIDs (see vitruv-tools/Vitruv-Domains-ComponentBasedSystems#91). We discuss why we temporarily disable these tests in #132.