-
Notifications
You must be signed in to change notification settings - Fork 40
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
Suggestion: Minimal Program to run Specs #20
Comments
There is some golang implementation of velocypack already: I don't want to discourage your development by that, I justed wanted to provide some pointer to it. Regarding the original question, do you mean we should provide a binary for velocypack validation, which could be invoked like |
Nah, okay, I overlooked the golang implementation. Maybe I'll pick another language then, or I'll take a look at doing this with golang anyway, just as an exercise :) What you describe would probably also do the job. Like I said initially I'm not 100% sure on how this should work exactly. The goal would be to be able to compare generated velocypack data using an official tool that can run over JSON (e.g. from https://github.com/arangodb/velocypack/tree/master/tests/jsonSample) to data generated by a 3rd party tool. Maybe you could even use that as part of CI to test your own, non-C++ implemented libraries. |
Ok, got it. |
I was thinking about using your specification to improve my at best mediocre golang skills. It would be nice, if there was a test suite, like a bunch of json files with a validation tool. So that you can compare and verify 3rd party implementations.
I haven't thought this completely through, but I could imaging something like this:
Where
validator
would be the "official" binary using the library which does through aspec/*.json
folder, callsmybinary
with the path to the example, expects the result on stdout and then compares the result.This way it would be very easy – not only for implementors, but also for you to verify 3rd party implementation for correctness and completeness.
The text was updated successfully, but these errors were encountered: