-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add new generate solution task #112
Conversation
src/integrationTest/groovy/wooga/gradle/unity/tasks/GenerateSolutionTaskIntegrationSpec.groovy
Show resolved
Hide resolved
src/integrationTest/groovy/wooga/gradle/unity/tasks/GenerateSolutionTaskIntegrationSpec.groovy
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.
The implementation looks good. Just write the test to check if the solution was created.
src/integrationTest/groovy/wooga/gradle/unity/tasks/GenerateSolutionTaskIntegrationSpec.groovy
Outdated
Show resolved
Hide resolved
@@ -339,4 +339,13 @@ class UnityPluginIntegrationSpec extends UnityIntegrationSpec { | |||
result.wasSkipped("returnUnityLicense") | |||
result.wasExecuted("activateUnity") | |||
} | |||
|
|||
@UnityPluginTestOptions(forceMockTaskRun = false, disableAutoActivateAndLicense = false) | |||
def "runs generateSolution task from cli"() { |
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.
I don't the task is necessarily invoked from the CLI. We could use a GUI to have them executed!
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.
Yeah, in this case, you are right.
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.
Related to the other conversation:
In this case, we are testing that the generateSolution
task is generated by the plugin.
3f72f23
to
ef6add4
Compare
ef6add4
to
b804038
Compare
Description
Created new generateSolution task that generates a solution file (if it doesn't exists) by executing Unity's
UnityEditor.SyncVS.SyncSolution
method. Its a pretty simple operation, with no parameters needed so there are no properties associated to this task.Changes