Skip to content
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

mlGenerateUnitTestSuite sjs support #511

Closed
prestonmcgowan opened this issue Oct 9, 2019 · 5 comments
Closed

mlGenerateUnitTestSuite sjs support #511

prestonmcgowan opened this issue Oct 9, 2019 · 5 comments
Assignees
Milestone

Comments

@prestonmcgowan
Copy link

Allow the mlGenerateUnitTestSuite to take in an argument to create sjs or xqy unit test samples. The third sample could be a hybrid of sjs and xqy, but I am not sure of a good way to generate that.

Impacted file: ml-gradle/src/main/groovy/com/marklogic/gradle/task/test/GenerateUnitTestSuiteTask.groovy

Possible sample output from modifying the xqy sample files.

sjsSampleTestSuite.zip

@grtjn
Copy link

grtjn commented Apr 21, 2021

It would make a lot of sense if this task code would be moved to the ml-unit-test repo, hence this ticket should move there too?

@dmcassel
Copy link
Contributor

It would make a lot of sense if this task code would be moved to the ml-unit-test repo, hence this ticket should move there too?

I don't think marklogic-unit-test currently exports any tasks, so there would be no place for the code to land (currently).

@grtjn
Copy link

grtjn commented Apr 21, 2021

Not something that can't be fixed.. :)

Can't imagine it to be very hard. Wouldn't it be matter of somehow adding it to the ml-unit-test-client? Then again, not really an expert on exposing gradle tasks in jars.. :)

@dmcassel
Copy link
Contributor

Not something that can't be fixed.. :)

Can't imagine it to be very hard. Wouldn't it be matter of somehow adding it to the ml-unit-test-client? Then again, not really an expert on exposing gradle tasks in jars.. :)

Sounds like a question for @rjrudin

@dmcassel
Copy link
Contributor

dmcassel commented Jan 5, 2023

@rjrudin fyi I'm preparing a PR for this

@rjrudin rjrudin added this to the 4.5.0 milestone Jan 10, 2023
@rjrudin rjrudin closed this as completed Feb 6, 2023
rjrudin pushed a commit that referenced this issue Aug 23, 2024
Merge in changes for 4.8.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants