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

[question]: add mocha xqsuite script from generator #124

Open
duncdrum opened this issue Oct 26, 2020 · 3 comments
Open

[question]: add mocha xqsuite script from generator #124

duncdrum opened this issue Oct 26, 2020 · 3 comments

Comments

@duncdrum
Copy link
Contributor

duncdrum commented Oct 26, 2020

I now have a pretty robust and dynamic test generator for xqsuite tests executed on the server and then "parsed" by Mocha. see generators/app/templates/tests/xqs/xqSuite.js. Should we include that here?

The only thing that we would need to adjust is adding a method to provide the location of the xqsuite runner (and or suite) file.

The downside it would add Mocha as a dependency.

The upside you could run xqsuite tests from inside node-exist.

@line-o what do you think?

@line-o
Copy link
Member

line-o commented Oct 26, 2020

I do like your approach - it is also very similar to https://github.com/line-o/xbow/blob/master/test/mocha/xqSuite.js. This would add mocha as a dependency, too.
I do like your approach to directly run the test-runner instead of uploading it. My version does test more metrics though. I think we should pick the best of both.

@duncdrum
Copy link
Contributor Author

Yes great minds thinking alike … i have these three fixtures of different junit-reports i encountered (and slimmed down) in the wild. Most of my work was making sure that all three kind of runs are reported properly, and that repeated runs stay consistent.

How does your code deal with repeated runs, and with those variations in report format?

@line-o
Copy link
Member

line-o commented Feb 9, 2022

I would like to pick this up after a while of hibernation. How about a paired programming session to get this included?
What I would like to achieve:

  • the level of reporting your script returns which looks much better than what I currently have
  • upload and run test runner using the database connection that node-exist provides
  • inspect result with tape instead of mocha to not add a new dependency

How does that sound @duncdrum?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants