-
Notifications
You must be signed in to change notification settings - Fork 5
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
Rewrote SystemComponentSpec from groovy to scala #840
Rewrote SystemComponentSpec from groovy to scala #840
Conversation
…functional style.
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.
Looks good so far. Just minor things and please check if this can be included into scala/edu/ie3/simona/model/grid/SystemComponentSpec.scala
…a. Delete Model/SystemComponentSpec.scala.
Is the simulation start and end time correctly implemented? I'm not sure. The test is now integrated into grid/systemcomponentspec.scala. |
src/test/scala/edu/ie3/simona/model/grid/SystemComponentSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/edu/ie3/simona/model/grid/SystemComponentSpec.scala
Outdated
Show resolved
Hide resolved
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.
LGTM!
SystemComponentTest convert from groovy to scala.
This PR is related to Issue #646